On Tue, Feb 04, 2014 at 12:18:34PM -0500, Marcelo Tosatti wrote:
On Tue, Feb 04, 2014 at 05:10:13PM +0000, Daniel P. Berrange wrote:
> > Because there is no guarantee with -mem-prealloc. For instance, if the
> > hugepage path is not actually hugetlbfs backed, QEMU falls back to
> > malloc().
>
> Well if you can't fix -mem-prealloc to properly report errors for reasons
> of back compat, then it is certainly possible to add a further
'strict=yes|no'
> option to the CLI arg request that it verify this. I don't see any reason
> why this checking code should be in libvirt rather than QEMU.
>
> Regards,
> Daniel
OK, do you want the StrictHugepage element to exist, and default
libvirt behaviour to remain as it is today (fallback to malloc),
or
do you want libvirt to pass strict mode to -mem-prealloc as default
when using hugepages element? (which would change behaviour of libvirt
for existing guests)
I'd say the latter - if the user is requesting hugepages they should
be guaranteed to get them. Libvirt doesn't ever use -mem-path with
a non-hugetlbfs filesystem path AFAIR, so the fallback to malloc
doesn't really make any sense for our needs.
Daniel
--
|:
http://berrange.com -o-
http://www.flickr.com/photos/dberrange/ :|
|:
http://libvirt.org -o-
http://virt-manager.org :|
|:
http://autobuild.org -o-
http://search.cpan.org/~danberr/ :|
|:
http://entangle-photo.org -o-
http://live.gnome.org/gtk-vnc :|