[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH for-1.4 v3 00/12] qdev: correct reference counting
From: |
Paolo Bonzini |
Subject: |
[Qemu-devel] [PATCH for-1.4 v3 00/12] qdev: correct reference counting |
Date: |
Fri, 25 Jan 2013 12:46:48 +0100 |
This series makes the ref_count field of device and bus objects actually
match the number of references that the objects have. Once this is
done, the question "how do I delete an object?" has a simple answer:
use object_unparent if the object is reachable from the QOM tree, else use
object_unref. qdev_free and qbus_free become equivalent to simply
object_unparent. object_delete instead is replaced by object_unref.
Patches 1-3 fix some warts in the last minute patches that went in 1.3.
Patches 4-10 are the bulk of the series. Patches 11-12 touches the CPU
classes instead.
Paolo
v1->v2:
There is a new patch (#4) that documents how link properties are
reference counted, and the transfer of ownership when the property
is created/deleted. I renamed qbus_remove_children to bus_unparent,
and rebased for the conversion of ->state to ->realized. PPC was also
using object_delete for its CPU, too. Otherwise, there are no changes.
v2->v3:
Patches 1 and 2 reorganized to keep qbus_realize static. sysbus
still allocated with g_malloc.
Paolo Bonzini (12):
qdev: remove duplication between qbus_create and qbus_create_inplace
pci: use qbus_create in pci_bus_new
qom: preserve object while unparenting it
qom: document reference counting of link properties
qdev: add reference count to a device for the BusChild
qdev: move deletion of children from finalize to unparent
qdev: move unrealization of devices from finalize to unparent
qdev: add reference for the bus while it is referred to by the
DeviceState
qdev: inline object_delete into qbus_free/qdev_free
qdev: drop extra references at creation time
cpu: do not use object_delete
qom: remove object_delete
hw/pci/pci.c | 21 ++++++---
hw/qdev-monitor.c | 5 ++-
hw/qdev.c | 105 +++++++++++++++++++++++++-------------------
include/qom/object.h | 22 +++-------
linux-user/syscall.c | 2 +-
qom/object.c | 9 +---
target-i386/helper.c | 4 +-
target-ppc/translate_init.c | 2 +-
target-sparc/cpu.c | 2 +-
vl.c | 1 +
10 files changed, 95 insertions(+), 78 deletions(-)
--
1.8.1
- [Qemu-devel] [PATCH for-1.4 v3 00/12] qdev: correct reference counting,
Paolo Bonzini <=
- [Qemu-devel] [PATCH v3 11/12] cpu: do not use object_delete, Paolo Bonzini, 2013/01/25
- [Qemu-devel] [PATCH v3 01/12] qdev: remove duplication between qbus_create and qbus_create_inplace, Paolo Bonzini, 2013/01/25
- [Qemu-devel] [PATCH v3 08/12] qdev: add reference for the bus while it is referred to by the DeviceState, Paolo Bonzini, 2013/01/25
- [Qemu-devel] [PATCH v3 03/12] qom: preserve object while unparenting it, Paolo Bonzini, 2013/01/25
- [Qemu-devel] [PATCH v3 02/12] pci: use qbus_create in pci_bus_new, Paolo Bonzini, 2013/01/25
- [Qemu-devel] [PATCH v3 04/12] qom: document reference counting of link properties, Paolo Bonzini, 2013/01/25
- [Qemu-devel] [PATCH v3 05/12] qdev: add reference count to a device for the BusChild, Paolo Bonzini, 2013/01/25