Re: [Qemu-devel] [PATCH v2 0/3] fix bug about balloon working incorrectly when hotplug memeory

[Igor Mammedov]

On Wed, 19 Nov 2014 16:28:09 +0800
zhanghailiang <address@hidden> wrote:

> Hi,
> 
> Ping...?
> 
> Should this be fixed in 2.2?
I'd preffer it go after memory unplug is merged to see how balooning
will fare along with it.

Also ack from Luiz could be useful to make it sure that balloning will
work just fine with sparse memory and memory unplug.

> 
> Thanks,
> zhanghailiang
> On 2014/11/17 13:11, zhanghailiang wrote:
> > Hi,
> >
> > Patch 1 and 2 mainly fix bug about balloon not working correctly
> > when we do hotplug memory. It takes 'ram_size' as VM's real RAM
> > size which is wrong after we hotplug memory.
> >
> > This bug exists since we begin to support hotplug memory, and it is
> > better to fix it.
> >
> > Patch 3 add some trace events, it helps debugging balloon. If it is
> > unnecessary, pls feel free to remove it.
> >
> > Thanks,
> > zhanghailiang
> >
> > v2:
> > - fix compiling break for other targets that don't support pc-dimm
> >
> > zhanghailiang (3):
> >    pc-dimm: add a function to calculate VM's current RAM size
> >    virtio-balloon: Fix balloon not working correctly when hotplug
> > memory virtio-balloon: Add some trace events
> >
> >   hw/mem/pc-dimm.c                | 26 ++++++++++++++++++++++++++
> >   hw/virtio/virtio-balloon.c      | 21 +++++++++++++++------
> >   include/exec/cpu-common.h       |  1 +
> >   stubs/qmp_pc_dimm_device_list.c |  5 +++++
> >   trace-events                    |  4 ++++
> >   5 files changed, 51 insertions(+), 6 deletions(-)
> >
> 
>