[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 5/5] virtio-balloon: Unregister savevm sectio
From: |
Amit Shah |
Subject: |
Re: [Qemu-devel] [PATCH v2 5/5] virtio-balloon: Unregister savevm section on device unplug |
Date: |
Thu, 28 Jul 2011 15:07:52 +0530 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On (Thu) 28 Jul 2011 [09:45:44], Markus Armbruster wrote:
> Amit Shah <address@hidden> writes:
>
> > Migrating after unplugging a virtio-balloon device resulted in an error
> > message on the destination:
> >
> > Unknown savevm section or instance '0000:00:04.0/virtio-balloon' 0
> > load of migration failed
> >
> > Fix this by unregistering the section on device unplug.
> >
> > Signed-off-by: Amit Shah <address@hidden>
> > ---
> > hw/virtio-balloon.c | 4 ++++
> > 1 files changed, 4 insertions(+), 0 deletions(-)
> >
> > diff --git a/hw/virtio-balloon.c b/hw/virtio-balloon.c
> > index 0ce0049..072a88a 100644
> > --- a/hw/virtio-balloon.c
> > +++ b/hw/virtio-balloon.c
> > @@ -45,6 +45,7 @@ typedef struct VirtIOBalloon
> > size_t stats_vq_offset;
> > MonitorCompletion *stats_callback;
> > void *stats_opaque_callback_data;
> > + DeviceState *qdev;
> > } VirtIOBalloon;
>
> All the other virtio device structs already have such a pointer back to
> the proxy. Suggests that it should live in VirtIODevice, and be set up
> in generic code. Again, outside the scope of this series.
>
> I hate the virtio pointer thicket.
Yep :-(
Amit
Re: [Qemu-devel] [PATCH v2 0/5] balloon: fix memleaks, invalid arguments, unplug, Markus Armbruster, 2011/07/28
Re: [Qemu-devel] [PATCH v2 0/5] balloon: fix memleaks, invalid arguments, unplug, Michael S. Tsirkin, 2011/07/28