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
proper.

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.

Thanks
Tiejun


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Reply via email to