Re: [Xen-devel] [libvirt] [PATCH RESENT 04/12] libxl: populate xenstore memory entries at startup

[Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>]

On Fri, 19 Apr 2013, Marek Marczykowski wrote:
> >> And to answer you question - libvirt rely on libxl autoballoon.
> > 
> > Could we introduce something similar to autoballoon=auto to libvirt?
> > 
> > Maybe we should push down the autoballoon option to libxl: we should
> > probably rename autoballoon to libxl_memory_management, enable it
> > automatically during initialization (and call
> > libxl__fill_dom0_memory_info) unless dom0_mem has been passed as an
> > option, in which case we would disable it. If we do it at the libxl
> > level we would cover xl as well as libvirt and also fix the "xl set-mem
> > 0" case.
> 
> Looks good for me, but IIUC it's too late for such change in 4.3 and it
> doesn't qualify for later backport, right? If so, some approach still is
> needed in libvirt for xen 4.2 and 4.3 (like my simple patch). This still
> doesn't make libvirt compatible with dom0_mem, but at least cover one
> (independent) case.
> Perhaps additionally autoballoon=auto code should be
> copied from xl to libvirt to cover dom0_mem case with xen 4.2 and 4.3?

Might be a decent solution for the moment.

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