Re: [Qemu-devel] [PATCH 01/16] qdev: fix hot-unplug

2012-02-04 Thread Anthony Liguori
On 02/04/2012 12:51 AM, Paolo Bonzini wrote: On 02/04/2012 04:03 AM, Anthony Liguori wrote: There's no object_ref() in qdev_device_add(). The 2 references come from adding a child link to /peripheral and via object_new(). Sure, but there's when the object_new() reference becomes unreachable.

Re: [Qemu-devel] [PATCH 01/16] qdev: fix hot-unplug

2012-02-03 Thread Anthony Liguori
On 02/02/2012 10:45 AM, Paolo Bonzini wrote: The reference that is returned by qdev_device_add is never given back, so that device_del does not cause the refcount to go to zero (and thus does nothing). Signed-off-by: Paolo Bonzinipbonz...@redhat.com This isn't needed in qom-upstream.14.

Re: [Qemu-devel] [PATCH 01/16] qdev: fix hot-unplug

2012-02-03 Thread Anthony Liguori
On 02/02/2012 01:07 PM, Alexander Graf wrote: On 02.02.2012, at 20:01, Anthony Liguori wrote: On 02/02/2012 11:29 AM, Paolo Bonzini wrote: On 02/02/2012 06:03 PM, Anthony Liguori wrote: Is this still needed with qom-upstream.14? I fixed a bug on .14 that involved child properties that

Re: [Qemu-devel] [PATCH 01/16] qdev: fix hot-unplug

2012-02-03 Thread Alexander Graf
On 03.02.2012, at 17:37, Anthony Liguori anth...@codemonkey.ws wrote: On 02/02/2012 01:07 PM, Alexander Graf wrote: On 02.02.2012, at 20:01, Anthony Liguori wrote: On 02/02/2012 11:29 AM, Paolo Bonzini wrote: On 02/02/2012 06:03 PM, Anthony Liguori wrote: Is this still needed with

Re: [Qemu-devel] [PATCH 01/16] qdev: fix hot-unplug

2012-02-03 Thread Anthony Liguori
On 02/03/2012 10:57 AM, Alexander Graf wrote: On 03.02.2012, at 17:37, Anthony Liguorianth...@codemonkey.ws wrote: On 02/02/2012 01:07 PM, Alexander Graf wrote: On 02.02.2012, at 20:01, Anthony Liguori wrote: On 02/02/2012 11:29 AM, Paolo Bonzini wrote: On 02/02/2012 06:03 PM, Anthony

Re: [Qemu-devel] [PATCH 01/16] qdev: fix hot-unplug

2012-02-03 Thread Paolo Bonzini
On 02/03/2012 03:27 PM, Anthony Liguori wrote: On 02/02/2012 10:45 AM, Paolo Bonzini wrote: The reference that is returned by qdev_device_add is never given back, so that device_del does not cause the refcount to go to zero (and thus does nothing). Signed-off-by: Paolo

Re: [Qemu-devel] [PATCH 01/16] qdev: fix hot-unplug

2012-02-03 Thread Anthony Liguori
On 02/03/2012 06:27 PM, Paolo Bonzini wrote: On 02/03/2012 03:27 PM, Anthony Liguori wrote: On 02/02/2012 10:45 AM, Paolo Bonzini wrote: The reference that is returned by qdev_device_add is never given back, so that device_del does not cause the refcount to go to zero (and thus does nothing).

Re: [Qemu-devel] [PATCH 01/16] qdev: fix hot-unplug

2012-02-03 Thread Paolo Bonzini
On 02/04/2012 04:03 AM, Anthony Liguori wrote: There's no object_ref() in qdev_device_add(). The 2 references come from adding a child link to /peripheral and via object_new(). Sure, but there's when the object_new() reference becomes unreachable. At this point, if it weren't for /peripheral

[Qemu-devel] [PATCH 01/16] qdev: fix hot-unplug

2012-02-02 Thread Paolo Bonzini
The reference that is returned by qdev_device_add is never given back, so that device_del does not cause the refcount to go to zero (and thus does nothing). Signed-off-by: Paolo Bonzini pbonz...@redhat.com --- vl.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/vl.c

Re: [Qemu-devel] [PATCH 01/16] qdev: fix hot-unplug

2012-02-02 Thread Anthony Liguori
On 02/02/2012 10:45 AM, Paolo Bonzini wrote: The reference that is returned by qdev_device_add is never given back, so that device_del does not cause the refcount to go to zero (and thus does nothing). Signed-off-by: Paolo Bonzinipbonz...@redhat.com --- vl.c |1 + 1 files changed, 1

Re: [Qemu-devel] [PATCH 01/16] qdev: fix hot-unplug

2012-02-02 Thread Paolo Bonzini
On 02/02/2012 06:03 PM, Anthony Liguori wrote: Is this still needed with qom-upstream.14? I fixed a bug on .14 that involved child properties that was making device-del sometimes fail. Not sure, I tried with .13 but, from the look of it, it should still be there. Regarding the .13-.14

Re: [Qemu-devel] [PATCH 01/16] qdev: fix hot-unplug

2012-02-02 Thread Anthony Liguori
On 02/02/2012 11:29 AM, Paolo Bonzini wrote: On 02/02/2012 06:03 PM, Anthony Liguori wrote: Is this still needed with qom-upstream.14? I fixed a bug on .14 that involved child properties that was making device-del sometimes fail. Not sure, I tried with .13 but, from the look of it, it

Re: [Qemu-devel] [PATCH 01/16] qdev: fix hot-unplug

2012-02-02 Thread Alexander Graf
On 02.02.2012, at 20:01, Anthony Liguori wrote: On 02/02/2012 11:29 AM, Paolo Bonzini wrote: On 02/02/2012 06:03 PM, Anthony Liguori wrote: Is this still needed with qom-upstream.14? I fixed a bug on .14 that involved child properties that was making device-del sometimes fail. Not

Re: [Qemu-devel] [PATCH 01/16] qdev: fix hot-unplug

2012-02-02 Thread Anthony Liguori
On 02/02/2012 01:07 PM, Alexander Graf wrote: On 02.02.2012, at 20:01, Anthony Liguori wrote: On 02/02/2012 11:29 AM, Paolo Bonzini wrote: On 02/02/2012 06:03 PM, Anthony Liguori wrote: Is this still needed with qom-upstream.14? I fixed a bug on .14 that involved child properties that

Re: [Qemu-devel] [PATCH 01/16] qdev: fix hot-unplug

2012-02-02 Thread Alexander Graf
On 02.02.2012, at 21:03, Anthony Liguori wrote: On 02/02/2012 01:07 PM, Alexander Graf wrote: On 02.02.2012, at 20:01, Anthony Liguori wrote: On 02/02/2012 11:29 AM, Paolo Bonzini wrote: On 02/02/2012 06:03 PM, Anthony Liguori wrote: Is this still needed with qom-upstream.14? I