[Libvir] [PATCH] Implement memory operations for qemu driver
Cole Robinson
crobinso at redhat.com
Tue Mar 18 18:28:36 UTC 2008
Cole Robinson wrote:
> The attached patch implements the following operations for the qemu driver:
>
> virDomainGetMaxMemory
> virDomainSetMaxMemory
> virDomainSetMemory
>
> A few questions/comments:
>
> 1) I changed maxmem and memory in the qemu_vm_def struct to unsigned long
> to match the public api for memory values. Seems to work, but not sure
> if there are any undesirable side effects to this.
>
> 2) Should SetMaxMem be able to be called on a running guest? This code
> allows it, since maxmem is basically a metavalue that doesn't directly
> affect a guest.
>
> 3) Should maxmem be able to be set lower than the currently allocated mem?
> This code does not allow this. If this changed, would also need to take
> into account how we would handle this if we can change the maxmem while
> the guest is running. After rethinking, we probably should be able to
> do this, but I haven't changed the code.
>
> Thanks,
> Cole
>
In testing this with virt-manager I found a bug: I was using comparing maxmem
against the wrong memory value. Fixed this and updated the parameter names
to be a bit more clear. Seems to work as excepted now.
Thanks,
Cole
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: libvirt-qemu-mem-commands-3-patch
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20080318/f5b28830/attachment-0001.ksh>
More information about the libvir-list
mailing list