qom/object properties cleanups

[PATCH 0/6] qom/object properties cleanups

marcandre.lureau@redhat.com posted 6 patches 6 months, 2 weeks ago

Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20250429140306.190384-1-marcandre.lureau@redhat.com

Maintainers: Stefan Berger <stefanb@linux.vnet.ibm.com>, Paolo Bonzini <pbonzini@redhat.com>, "Daniel P. Berrangé" <berrange@redhat.com>, Eduardo Habkost <eduardo@habkost.net>

backends/tpm/tpm_util.c          |  1 +
hw/core/qdev-properties-system.c |  1 +
hw/core/qdev-properties.c        | 18 +++---
qom/object.c                     | 18 ++----
tests/unit/test-qdev.c           | 96 ++++++++++++++++++++++++++++++++
tests/unit/meson.build           |  1 +
6 files changed, 114 insertions(+), 21 deletions(-)
create mode 100644 tests/unit/test-qdev.c

Expand all Fold all

[PATCH 0/6] qom/object properties cleanups

Posted by marcandre.lureau@redhat.com 6 months, 2 weeks ago

From: Marc-André Lureau <marcandre.lureau@redhat.com>

Hi,

While reviewing some related code, I noticed array properties are leaking.

I also wonder why I refactored object_property_del_all() with a HashTable, and
whether it's really necessary. Here is also a RFC to simplify that code.

Marc-André Lureau (6):
  qdev: make release_string() idempotent
  qdev: make release_drive() idempotent
  qdev: make release_tpm() idempotent
  qdev: make release_prop_array() free array and idempotent
  tests: add /qdev/free-properties test
  RFC: qom/object: simplify object_property_del_all()

 backends/tpm/tpm_util.c          |  1 +
 hw/core/qdev-properties-system.c |  1 +
 hw/core/qdev-properties.c        | 18 +++---
 qom/object.c                     | 18 ++----
 tests/unit/test-qdev.c           | 96 ++++++++++++++++++++++++++++++++
 tests/unit/meson.build           |  1 +
 6 files changed, 114 insertions(+), 21 deletions(-)
 create mode 100644 tests/unit/test-qdev.c

-- 
2.49.0