[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 00/12] virtio: fix virtio child recount in tr
From: |
Gonglei (Arei) |
Subject: |
Re: [Qemu-devel] [PATCH v2 00/12] virtio: fix virtio child recount in transports |
Date: |
Tue, 30 Sep 2014 08:06:54 +0000 |
> Subject: Re: [PATCH v2 00/12] virtio: fix virtio child recount in transports
>
> On Tue, 30 Sep 2014 14:10:26 +0800
> <address@hidden> wrote:
>
> > From: Gonglei <address@hidden>
> >
> > virtio-$device-{pci, s390, ccw} all duplicate the
> > qdev properties of their virtio child. This approach does
> > not work well with string or pointer properties since we
> > must be careful about leaking or double-freeing them.
> >
> > Use the QOM alias property to forward property accesses to the
> > VirtIORNG child. This way no duplication is necessary.
>
> VirtIO$devtype?
>
Yes, sorry for my wrong coping.
> >
> > For their child, object_initialize() leaves the object with a refcount of 1.
> > object_property_add_child() adds its own reference which is dropped
> > again when the property is deleted.
> >
> > The upshot of this is that we always have a refcount >= 1. Upon hot
> > unplug the virtio-$device child is not finalized!
> >
> > Drop our reference after the child property has been added to the
> > parent.
> >
> > Changs since v1:
> > - add the same handling for virtio-9p-pci device in PATCH 10 and PATCH 11.
> > - add a wrapper function for better code sharing
> > in PATCH 12 (Cornelia/Michael/Paolo)
>
> This patchset, applied on current master, survives some light testing
> for s390-virtio and virtio-ccw (booting Linux, device_add/device_del
> for the various device types).
Thanks a lot! Cornelia. Please let us know the results :)
Best regards,
-Gonglei
[Qemu-devel] [PATCH v2 02/12] virtio-net: fix virtio-net child refcount in transports, arei.gonglei, 2014/09/30
[Qemu-devel] [PATCH v2 10/12] virtio-9p: use aliases instead of duplicate qdev properties, arei.gonglei, 2014/09/30
Re: [Qemu-devel] [PATCH v2 00/12] virtio: fix virtio child recount in transports, Cornelia Huck, 2014/09/30
- Re: [Qemu-devel] [PATCH v2 00/12] virtio: fix virtio child recount in transports,
Gonglei (Arei) <=
Re: [Qemu-devel] [PATCH v2 00/12] virtio: fix virtio child recount in transports, Paolo Bonzini, 2014/09/30