Re: [Qemu-devel] [PATCH v3 0/5] qom: link property fixes

[Stefan Hajnoczi]

On Wed, Mar 05, 2014 at 09:51:19PM +0100, Stefan Hajnoczi wrote:
> v3:
>  * Split first commit for easier reviewing [afaerber]
>  * Turn bool argument into flags bitmask [pbonzini]
>  * Rename set() callback to check() [pbonzini]
>  * Rename object_property_default_set_link() to
>    object_property_allow_set_link()
>  * Rename qdev_prop_default_set_link() to
>    qdev_prop_allow_set_link_before_realize()
> 
> v2:
>  * A NULL set() callback should make the property read-only
> 
> There are two problems with QOM link properties:
> 
> 1. There are refcount leaks in many object_property_add_link() callers.
> 2. There is no way to prevent link properties from being set after a device is
>    realized.
> 
> This patch series fixes these issues by extending the
> object_property_add_link() function.
> 
> Stefan Hajnoczi (5):
>   qom: split object_property_set_link()
>   qom: don't make link NULL on object_property_set_link() failure
>   qom: make QOM link property unref optional
>   qom: add check() argument to object_property_add_link()
>   virtio-rng: avoid default_backend refcount leak
> 
>  hw/core/qdev-properties.c    |  12 ++++
>  hw/core/qdev.c               |  10 +++-
>  hw/dma/xilinx_axidma.c       |  20 +++++--
>  hw/net/xilinx_axienet.c      |  20 +++++--
>  hw/pcmcia/pxa2xx.c           |   4 +-
>  hw/s390x/s390-virtio-bus.c   |   4 +-
>  hw/s390x/virtio-ccw.c        |   4 +-
>  hw/virtio/virtio-pci.c       |   4 +-
>  hw/virtio/virtio-rng.c       |   7 ++-
>  include/hw/qdev-properties.h |  11 ++++
>  include/qom/object.h         |  29 +++++++++-
>  qom/object.c                 | 130 
> ++++++++++++++++++++++++++++++++-----------
>  ui/console.c                 |   5 +-
>  13 files changed, 211 insertions(+), 49 deletions(-)

Ping?