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

Re: [Xen-devel] [PATCH v5 08/24] libxl: introduce libxl__vnuma_config_check

On Fri, Feb 13, 2015 at 02:15:47PM +0000, Ian Jackson wrote:
> Wei Liu writes ("[PATCH v5 08/24] libxl: introduce 
> libxl__vnuma_config_check"):
> > This function is used to check whether vNUMA configuration (be it
> > auto-generated or supplied by user) is valid.
> This looks plausible, but I think you should explain what the impact
> of this patch is.  Presumably the intent is to replace various later
> failures with ERROR_FAIL with something more useful and more
> specific ?

Yes, providing more useful error message is on aspect. Another aspect is
just to do sanity check -- passing an invalid layout to guest doesn't
make much sense.

> Are there any cases which this new check forbids but which are
> currently accepted by libxl ?  If so then we have to think about
> compatibility.

First thing is there is no previous supported vNUMA interface in
toolstack so there won't be a situation where previous good config
doesn't pass this check.

Second thing is if user supplies a config without vNUMA configuration
this function will not get called, so it won't have any effect.

> Also I would like to see an ack from the authors of the vnuma support,
> as I'm not familiar enough with vnuma to fully understand the
> semantics of the new checks.

Elena and Dario, what do you think?


> Thanks,
> Ian.

Xen-devel mailing list



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