On (Thu) 28 Jul 2011 [09:45:44], Markus Armbruster wrote:
> Amit Shah <amit.s...@redhat.com> 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 <amit.s...@redhat.com>
> > ---
> > 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
