[libvirt] [PATCH] add the check of memory size in xenXMDriver for setmem of Xen domain

[S.Sakamoto]

Hi,

I make the patch.
This patch adds the check of memory size in xenXMDriver,
to handle the xenStoreDriver error when the set value
by setmem(setmaxmem) is 4096-65535 definitely.


Thanks,
Shigeki Sakamoto.


Index: src/xm_internal.c
===================================================================
RCS file: /data/cvs/libvirt/src/xm_internal.c,v
retrieving revision 1.95
diff -u -p -r1.95 xm_internal.c
--- src/xm_internal.c	24 Oct 2008 11:20:08 -0000	1.95
+++ src/xm_internal.c	31 Oct 2008 05:52:27 -0000
@@ -1273,6 +1273,8 @@ int xenXMDomainSetMemory(virDomainPtr do
         return (-1);
     if (domain->id != -1)
         return (-1);
+    if (memory < 1024 * MIN_XEN_GUEST_SIZE);
+        return (-1);
 
     if (!(filename = virHashLookup(nameConfigMap, domain->name)))
         return (-1);



> At first, libvirt.c should not check VMM-arch specific value.
> In this meaning, xs_internal.c do the right thing.
> The problem is error handling on libvirt.c(Xen Store error is not handled properly).
> Do you investigate it?
> 
> Thanks
> Atsushi SAKAI
> 
> 
> 
> "S.Sakamoto" <fj0588di at aa.jp.fujitsu.com> wrote:
> 
> > Hi,
> > 
> > I have a question.
> > Why does libvirt check size of memory in xenStoreDomainSetMemory of xs_internal.c?
> > 
> > In the xen not exclude lifecycle,
> > when I do the following command for the domain that is shutoff,
> > error message is output and return value is 0 and the definition file(=/etc/xen/XXX) is updated.
> > 
> >   --------------------
> >   # virsh setmem(or setmaxmem) guestdom 65535(4096-65535)
> >   libvir: Xen Store error : invalid argument in xenStoreDomainSetMemory
> >   
> >   # echo $?
> >   0
> >   --------------------
> > 
> > As a result that I research this,
> > this error message is output by the check of the memory size in xenStoreDriver.
> > Because libvirt checks the memory size of under 4096 in libvirt.c,
> > I think that libvirt should check memory size range of 4096-65535 in libvirt.c.
> > 
> > 
> > Thanks,
> > Shigeki Sakamoto.
> > 
> > --
> > Libvir-list mailing list
> > Libvir-list at redhat.com
> > https://www.redhat.com/mailman/listinfo/libvir-list
> 
> 
>