Re: [PATCH v2] qemu: process: Improve update of maximum baloon state at startup

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

 



On Tue, Jul 14, 2015 at 13:50:24 +0200, Ján Tomko wrote:
> s/baloon/balloon/ in the subject
> 
> On Wed, Jul 01, 2015 at 10:13:27AM +0200, Peter Krempa wrote:
> > In commit 641a145d73fdc3dd9350fd57b3d3247abf101c05 I've added code that
> > resets the ballon memory value to full size prior to resuming the vCPUs
> 
> s/ballon/balloon/
> 
> > since the size certainly was not reduced at that point.
> > 
> > Since qemuProcess start is used also in code paths with already booted
> 
> qemuProcessStart
> 
> > up guests (migration, save/restore) the assumption is not entirely true
> > since the guest might already been running before.
> > 
> > This patch adds a function that queries the monitor rather than using
> > the full size since a balloon event would not be reissued in case we are
> 
> This one is fine.
> 
> > recovering a saved migration state.
> > 
> > Additionally the new function is used also when reconnecting to a VM
> > after libvirtd restart since we might have missed a few ballon events
> 
> s/ballon/balloon/
> 
> > while libvirtd was not running.
> > ---
> >  src/qemu/qemu_process.c | 38 +++++++++++++++++++++++++++++++++++++-
> >  1 file changed, 37 insertions(+), 1 deletion(-)
> > 
> 
> 
> > diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c
> > index ba84182..13b1696 100644
> > --- a/src/qemu/qemu_process.c
> > +++ b/src/qemu/qemu_process.c
> > @@ -2065,6 +2065,38 @@ qemuProcessReconnectRefreshChannelVirtioState(virQEMUDriverPtr driver,
> > 
> > 
> >  static int
> > +qemuProcessRefreshBallonState(virQEMUDriverPtr driver,
> 
> s/Ballon/Balloon/
> 
> > +                              virDomainObjPtr vm,
> > +                              int asyncJob)
> > +{
> > +    unsigned long long balloon;
> > +    int rc;
> > +
> > +    /* if no balloning is available, the current size equals to the current
> 
> s/ballon/balloon/
> 
> ACK with the subject and the function name fixed.

I've fixed all of them and pushed this patch.

Thanks.

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]