Re: [PATCHv2 07/10] qemu: add support for memory devices

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

 



On Thu, Feb 26, 2015 at 17:58:36 +0800, Zhu Guihua wrote:
> 
> On 02/25/2015 10:44 PM, Peter Krempa wrote:
> > Add support to start qemu instance with 'pc-dimm' device. Thanks to the
> > refactors we are able to reuse the existing function to determine the
> > parameters.
> > ---
> >
> > Notes:
> >      Version 2:
> >      - dropped the ACPI naming
> >
> >   src/qemu/qemu_command.c                            | 130 ++++++++++++++++++++-
> >   src/qemu/qemu_domain.c                             |  26 ++++-
> >   src/qemu/qemu_domain.h                             |   1 +
> >   .../qemuxml2argv-memory-hotplug-dimm.args          |  11 ++
> >   tests/qemuxml2argvtest.c                           |   2 +
> >   tests/qemuxml2xmltest.c                            |   1 +
> >   6 files changed, 167 insertions(+), 4 deletions(-)
> >   create mode 100644 tests/qemuxml2argvdata/qemuxml2argv-memory-hotplug-dimm.args

...

> > +/**
> > + * qemuDomainMemoryDeviceAlignSize:
> > + * @mem: memory device definition object
> > + *
> > + * Aligns the size of the memory module as qemu enforces it. The size is updated
> > + * inplace. Default rounding is now to 1 MiB (qemu requires rouding to page,
> > + * size so this should be safe).
> > + */
> > +void
> > +qemuDomainMemoryDeviceAlignSize(virDomainMemoryDefPtr mem)
> > +{
> > +    mem->size = VIR_ROUND_UP(mem->size, 1024);
> > +}
> 
> Because of linux limitation, the size of the memory module of linux 
> guests must be multiples of 128MiB,
> so could we add this limitation by libvirt which knows type of guest it 
> installs/runs?

Libvirt doesn't know the type of the guest that it's running inside. I
also remember reading somewhere that the limit for Windows is 2MiB, so
we can only agree on something minimal.

Anything above is should be limited by the management using libvirt.

> 
> Regards,
> Zhu

Peter

Attachment: signature.asc
Description: Digital 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]