[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [v10][PATCH 11/16] tools/libxl: detect and avoid conflicts with RDM

But d_config is a libxl_domain_config which is supplied by libxl's
caller.  It might contain some rdms.

I guess this line make you or other guys confused so lets delete this
line directly.

I don't think I am very confused.

And if you still worry about something, I can add assert() at the
beginning of this function like this,

assert(!d_config->num_rdms && !d_config->rdms).

If you are sure that this assertion is correct, then that would be

But as I say above, I don't think it is.

Based on Campbell' explanation I think you guys are raising a reasonable concern. We shouldn't clear that over there arbitrarily.


Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.