Ian Jackson writes ("Re: [PATCH-for-4.13 v2 0/2] libxl: fix assertion failure"):
> I am continuing to look at the defaulting and config management here
> with a view to getting rid of some of the duplicated code and moving
> it all into libxl.

Specifically, Paul Durrant writes:
> Ok. Specifically libxl_domain_need_memory() only has access to the
> build info, but libxl__domain_build_info_setdefault() does not have
> access to the create info, so it can't choose a sensible default. To
> avoid re-writing lots of code I went with having xl calulate a
> default. (Prior to my patch there was no calculated overhead anyway so
> a host without shared EPT and autoballooning on was already playing
> russian roulette).

So some restructuring may be needed.  I'll see how invasive it looks.

Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel

Reply via email to