Re: RFC: <memory units=...> [was: libvirt doesn't boot kVM, hangs, qemu on 100% CPU]

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

 



Hey,

On Wed, Feb 22, 2012 at 04:42:59PM -0700, Eric Blake wrote:
> There ain't no way on earth you're going to boot a kernel in 2 megabytes
> of memory!
> 
> I propose enhancing the XML; on output, libvirt should produce:
> 
> <memory units='k'>2048</memory> => 2048 * kibibyte
> 
> the output unit must remain the same as it has always been, but the new
> attribute will make it easier for humans reading the XML to spot
> blunders like what spawned this thread.
> 
[...]
> Thoughts before I propose such a patch?

For what it's worth, there are other places with unitless sizes, there's at
least <video vram=...>, everything under <memtune> and <bandwidth> (not
something measured in bytes though), and these 3 <pool> children:
<allocation>, <capacity> and <available>.
(NB: looked at your patches now and saw you handle memtune)

Also, <volume><allocation> and <volume><capacity> already
support a "unit" attribute which accept the K, M, G, T, P, E values
(xxxbyte), for consistency it would be better for the new <memory> attribute
to use "unit" and not "units" as suggested by Mathias, and for both "unit"
attributes to accept the same values.

Christophe

Attachment: pgppy8EL642VP.pgp
Description: PGP signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]