Re: [libvirt] [PATCH] add the check of memory size in xenXMDriver forsetmem of Xen domain

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

 



Hi, Daniel,

This patch itself is OK.

But I also think MIN_XEN_GUEST_SIZE should be defined in xen_unified.h 
not src/internal.h. Since this variable is Xen specific.
How do you think?

Thanks
Atsushi SAKAI


"S.Sakamoto" <fj0588di@xxxxxxxxxxxxxxxxx> wrote:

> 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@xxxxxxxxxxxxxxxxx> 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@xxxxxxxxxx
> > > https://www.redhat.com/mailman/listinfo/libvir-list
> > 
> > 
> > 
> 
> --
> Libvir-list mailing list
> Libvir-list@xxxxxxxxxx
> https://www.redhat.com/mailman/listinfo/libvir-list


--
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]