All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Daniel P. Berrangé" <berrange@redhat.com>
To: qemu-devel@nongnu.org
Cc: devel@lists.libvirt.org, "Alex Bennée" <alex.bennee@linaro.org>,
	"Christian Brauner" <brauner@kernel.org>,
	"Markus Armbruster" <armbru@redhat.com>,
	"Marc-André Lureau" <marcandre.lureau@redhat.com>,
	"Dr. David Alan Gilbert" <dave@treblig.org>,
	"Paolo Bonzini" <pbonzini@redhat.com>,
	"Philippe Mathieu-Daudé" <philmd@mailo.com>,
	"Daniel P. Berrangé" <berrange@redhat.com>
Subject: [PATCH v3 30/35] qom: add method for getting the "id" of a QOM object
Date: Thu, 18 Jun 2026 11:58:31 +0100	[thread overview]
Message-ID: <20260618105836.991609-31-berrange@redhat.com> (raw)
In-Reply-To: <20260618105836.991609-1-berrange@redhat.com>

The 'user_creatable_del' method doesn't want the 'Object *', but
instead the "id" of the object to be deleted. While most of its
work could be done with the "Object *" alone, purging the QemuOpts
values needs the "id".

The "id" is not recorded against an Object though, it is indirectly
stored as a property linking the parent and child objects together.
Add an accessor that can fetch this property name.

Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
---
 include/qom/object.h | 10 ++++++++++
 qom/object.c         | 17 +++++++++++++++++
 2 files changed, 27 insertions(+)

diff --git a/include/qom/object.h b/include/qom/object.h
index 11f55613fc..216d2a30f6 100644
--- a/include/qom/object.h
+++ b/include/qom/object.h
@@ -1759,6 +1759,16 @@ ObjectProperty *object_property_try_add_child(Object *obj, const char *name,
 ObjectProperty *object_property_add_child(Object *obj, const char *name,
                                           Object *child);
 
+/**
+ * object_property_get_child_name:
+ * @obj: the object that owns the property
+ * @child: the object referenced by the child property
+ *
+ * Return the property name against which @child is registered
+ * with @obj, or NULL if @obj is not a parent of @child.
+ */
+char *object_property_get_child_name(Object *obj, Object *child);
+
 typedef enum {
     /* Unref the link pointer when the property is deleted */
     OBJ_PROP_LINK_STRONG = 0x1,
diff --git a/qom/object.c b/qom/object.c
index 0ac201de4c..956adfb050 100644
--- a/qom/object.c
+++ b/qom/object.c
@@ -1964,6 +1964,23 @@ object_property_add_child(Object *obj, const char *name,
     return object_property_try_add_child(obj, name, child, &error_abort);
 }
 
+char *object_property_get_child_name(Object *obj, Object *child)
+{
+    ObjectProperty *prop;
+    GHashTableIter iter;
+    gpointer key, value;
+
+    g_hash_table_iter_init(&iter, obj->properties);
+    while (g_hash_table_iter_next(&iter, &key, &value)) {
+        prop = value;
+        if (object_property_is_child(prop) && prop->opaque == child) {
+            return g_strdup(prop->name);
+        }
+    }
+    return NULL;
+}
+
+
 void object_property_allow_set_link(const Object *obj, const char *name,
                                     Object *val, Error **errp)
 {
-- 
2.54.0



  parent reply	other threads:[~2026-06-18 11:02 UTC|newest]

Thread overview: 43+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-06-18 10:58 [PATCH v3 00/35] monitor: turn QMP and HMP into QOM objects Daniel P. Berrangé
2026-06-18 10:58 ` [PATCH v3 01/35] qom: replace 'can_be_deleted' with 'prepare_delete' Daniel P. Berrangé
2026-06-18 10:58 ` [PATCH v3 02/35] monitor: replace 'common' with 'parent_obj' in MonitorHMP Daniel P. Berrangé
2026-06-18 10:58 ` [PATCH v3 03/35] monitor: replace 'common' with 'parent_obj' in MonitorQMP Daniel P. Berrangé
2026-06-18 10:58 ` [PATCH v3 04/35] monitor: rename monitor_init* to monitor_new* Daniel P. Berrangé
2026-06-18 10:58 ` [PATCH v3 05/35] monitor: minimal conversion of monitors to QOM Daniel P. Berrangé
2026-06-18 10:58 ` [PATCH v3 06/35] monitor: add 'chardev' property to Monitor base class Daniel P. Berrangé
2026-06-18 11:54   ` marcandre.lureau
2026-06-18 10:58 ` [PATCH v3 07/35] monitor: add 'readline' property to HMP Monitor class Daniel P. Berrangé
2026-06-18 10:58 ` [PATCH v3 08/35] monitor: add 'pretty' property to QMP " Daniel P. Berrangé
2026-06-18 10:58 ` [PATCH v3 09/35] monitor: remove 'skip_flush' field Daniel P. Berrangé
2026-06-18 10:58 ` [PATCH v3 10/35] monitor: move monitor_data_(init|destroy) into QOM init/finalize Daniel P. Berrangé
2026-06-18 10:58 ` [PATCH v3 11/35] monitor: use class methods for monitor_vprintf Daniel P. Berrangé
2026-06-18 10:58 ` [PATCH v3 12/35] monitor: use class methods for monitor_qapi_event_emit Daniel P. Berrangé
2026-06-18 10:58 ` [PATCH v3 13/35] monitor: use class methods for monitor_accept_input Daniel P. Berrangé
2026-06-18 10:58 ` [PATCH v3 14/35] monitor: use class method for I/O thread request Daniel P. Berrangé
2026-06-18 10:58 ` [PATCH v3 15/35] monitor: use dynamic cast in monitor_qmp_requests_pop_any_with_lock Daniel P. Berrangé
2026-06-18 10:58 ` [PATCH v3 16/35] util: use dynamic cast in error vreport Daniel P. Berrangé
2026-06-18 10:58 ` [PATCH v3 17/35] monitor: drop unused monitor_cur_is_qmp Daniel P. Berrangé
2026-06-18 10:58 ` [PATCH v3 18/35] monitor: use dynamic cast in QMP commands Daniel P. Berrangé
2026-06-18 10:58 ` [PATCH v3 19/35] monitor: use dynamic cast in monitor_is_hmp_non_interactive Daniel P. Berrangé
2026-06-18 10:58 ` [PATCH v3 20/35] monitor: drop unused monitor_is_qmp method Daniel P. Berrangé
2026-06-18 10:58 ` [PATCH v3 21/35] monitor: eliminate monitor_is_hmp_non_interactive method Daniel P. Berrangé
2026-06-18 10:58 ` [PATCH v3 22/35] monitor: implement "user creatable" interface for adding monitors Daniel P. Berrangé
2026-06-18 10:58 ` [PATCH v3 23/35] monitor: convert from oneshot BH to persistent BH Daniel P. Berrangé
2026-06-18 10:58 ` [PATCH v3 24/35] monitor: reject attempts to delete the current monitor Daniel P. Berrangé
2026-06-18 10:58 ` [PATCH v3 25/35] monitor: protect qemu_chr_fe_accept_input with monitor lock Daniel P. Berrangé
2026-06-18 10:58 ` [PATCH v3 26/35] monitor: implement support for deleting QMP objects Daniel P. Berrangé
2026-06-18 11:54   ` marcandre.lureau
2026-06-18 10:58 ` [PATCH v3 27/35] tests/qtest: add tests for dynamic monitor add/remove Daniel P. Berrangé
2026-06-18 10:58 ` [PATCH v3 28/35] tests/functional: add e2e test for dynamic QMP monitor hotplug Daniel P. Berrangé
2026-06-18 10:58 ` [PATCH v3 29/35] tests/functional: add a stress test for monitor hot unplug Daniel P. Berrangé
2026-06-18 11:54   ` marcandre.lureau
2026-06-18 10:58 ` Daniel P. Berrangé [this message]
2026-06-18 11:54   ` [PATCH v3 30/35] qom: add method for getting the "id" of a QOM object marcandre.lureau
2026-06-18 10:58 ` [PATCH v3 31/35] qom: add trace events for user creatable create/delete APIs Daniel P. Berrangé
2026-06-18 10:58 ` [PATCH v3 32/35] monitor: add support for auto-deleting monitors upon close Daniel P. Berrangé
2026-06-18 11:54   ` marcandre.lureau
2026-06-18 10:58 ` [PATCH v3 33/35] tests: switch from -mon to -object monitor-qmp Daniel P. Berrangé
2026-06-18 10:58 ` [PATCH v3 34/35] qemu-options: document new monitor-hmp and monitor-qmp objects Daniel P. Berrangé
2026-06-18 11:54   ` marcandre.lureau
2026-06-18 10:58 ` [PATCH v3 35/35] docs: mark '-mon' as deprecated in favour of -object Daniel P. Berrangé
2026-06-18 11:54   ` marcandre.lureau

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20260618105836.991609-31-berrange@redhat.com \
    --to=berrange@redhat.com \
    --cc=alex.bennee@linaro.org \
    --cc=armbru@redhat.com \
    --cc=brauner@kernel.org \
    --cc=dave@treblig.org \
    --cc=devel@lists.libvirt.org \
    --cc=marcandre.lureau@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=philmd@mailo.com \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.