From: "Daniel P. Berrangé" <berrange@redhat.com>
To: qemu-devel@nongnu.org
Cc: "Dr. David Alan Gilbert" <dave@treblig.org>,
"Marc-André Lureau" <marcandre.lureau@redhat.com>,
"Christian Brauner" <brauner@kernel.org>,
devel@lists.libvirt.org, "Markus Armbruster" <armbru@redhat.com>,
"Philippe Mathieu-Daudé" <philmd@mailo.com>,
"Paolo Bonzini" <pbonzini@redhat.com>,
"Alex Bennée" <alex.bennee@linaro.org>,
"Daniel P. Berrangé" <berrange@redhat.com>
Subject: [PATCH v4 30/35] qom: add method for getting the "id" of a QOM object
Date: Tue, 23 Jun 2026 10:55:46 +0100 [thread overview]
Message-ID: <20260623095551.2106712-31-berrange@redhat.com> (raw)
In-Reply-To: <20260623095551.2106712-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 f79b2cf361..66bbd1488b 100644
--- a/qom/object.c
+++ b/qom/object.c
@@ -1952,6 +1952,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
next prev parent reply other threads:[~2026-06-23 9:58 UTC|newest]
Thread overview: 45+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-06-23 9:55 [PATCH v4 00/35] monitor: turn QMP and HMP into QOM objects Daniel P. Berrangé
2026-06-23 9:55 ` [PATCH v4 01/35] qom: replace 'can_be_deleted' with 'prepare_delete' Daniel P. Berrangé
2026-06-23 9:55 ` [PATCH v4 02/35] monitor: replace 'common' with 'parent_obj' in MonitorHMP Daniel P. Berrangé
2026-06-23 9:55 ` [PATCH v4 03/35] monitor: replace 'common' with 'parent_obj' in MonitorQMP Daniel P. Berrangé
2026-06-23 9:55 ` [PATCH v4 04/35] monitor: rename monitor_init* to monitor_new* Daniel P. Berrangé
2026-06-23 9:55 ` [PATCH v4 05/35] monitor: minimal conversion of monitors to QOM Daniel P. Berrangé
2026-06-23 9:55 ` [PATCH v4 06/35] monitor: add 'chardev' property to Monitor base class Daniel P. Berrangé
2026-06-23 9:55 ` [PATCH v4 07/35] monitor: add 'readline' property to HMP Monitor class Daniel P. Berrangé
2026-06-23 9:55 ` [PATCH v4 08/35] monitor: add 'pretty' property to QMP " Daniel P. Berrangé
2026-06-23 9:55 ` [PATCH v4 09/35] monitor: remove 'skip_flush' field Daniel P. Berrangé
2026-06-23 9:55 ` [PATCH v4 10/35] monitor: move monitor_data_(init|destroy) into QOM init/finalize Daniel P. Berrangé
2026-06-23 9:55 ` [PATCH v4 11/35] monitor: use class methods for monitor_vprintf Daniel P. Berrangé
2026-06-23 9:55 ` [PATCH v4 12/35] monitor: use class methods for monitor_qapi_event_emit Daniel P. Berrangé
2026-06-23 9:55 ` [PATCH v4 13/35] monitor: use class methods for monitor_accept_input Daniel P. Berrangé
2026-06-23 9:55 ` [PATCH v4 14/35] monitor: use class method for I/O thread request Daniel P. Berrangé
2026-06-23 9:55 ` [PATCH v4 15/35] monitor: use dynamic cast in monitor_qmp_requests_pop_any_with_lock Daniel P. Berrangé
2026-06-23 9:55 ` [PATCH v4 16/35] util: use dynamic cast in error vreport Daniel P. Berrangé
2026-06-23 9:55 ` [PATCH v4 17/35] monitor: drop unused monitor_cur_is_qmp Daniel P. Berrangé
2026-06-23 9:55 ` [PATCH v4 18/35] monitor: use dynamic cast in QMP commands Daniel P. Berrangé
2026-06-23 9:55 ` [PATCH v4 19/35] monitor: use dynamic cast in monitor_is_hmp_non_interactive Daniel P. Berrangé
2026-06-23 9:55 ` [PATCH v4 20/35] monitor: drop unused monitor_is_qmp method Daniel P. Berrangé
2026-06-23 9:55 ` [PATCH v4 21/35] monitor: eliminate monitor_is_hmp_non_interactive method Daniel P. Berrangé
2026-06-23 9:55 ` [PATCH v4 22/35] monitor: implement "user creatable" interface for adding monitors Daniel P. Berrangé
2026-06-23 9:55 ` [PATCH v4 23/35] monitor: convert from oneshot BH to persistent BH Daniel P. Berrangé
2026-06-23 9:55 ` [PATCH v4 24/35] monitor: reject attempts to delete the current monitor Daniel P. Berrangé
2026-06-23 9:55 ` [PATCH v4 25/35] monitor: protect qemu_chr_fe_accept_input with monitor lock Daniel P. Berrangé
2026-06-23 9:55 ` [PATCH v4 26/35] monitor: implement support for deleting QMP objects Daniel P. Berrangé
2026-06-23 9:55 ` [PATCH v4 27/35] tests/qtest: add tests for dynamic monitor add/remove Daniel P. Berrangé
2026-06-23 9:55 ` [PATCH v4 28/35] tests/functional: add e2e test for dynamic QMP monitor hotplug Daniel P. Berrangé
2026-06-23 9:55 ` [PATCH v4 29/35] tests/functional: add a stress test for monitor hot unplug Daniel P. Berrangé
2026-06-23 9:55 ` Daniel P. Berrangé [this message]
2026-06-23 9:55 ` [PATCH v4 31/35] qom: add trace events for user creatable create/delete APIs Daniel P. Berrangé
2026-06-23 9:55 ` [PATCH v4 32/35] monitor: add support for auto-deleting monitors upon close Daniel P. Berrangé
2026-06-23 11:53 ` marcandre.lureau
2026-06-23 11:55 ` Daniel P. Berrangé
2026-06-23 9:55 ` [PATCH v4 33/35] tests: switch from -mon to -object monitor-qmp Daniel P. Berrangé
2026-06-23 9:55 ` [PATCH v4 34/35] qemu-options: document new monitor-hmp and monitor-qmp objects Daniel P. Berrangé
2026-06-23 9:55 ` [PATCH v4 35/35] docs: mark '-mon' as deprecated in favour of -object Daniel P. Berrangé
2026-06-23 11:03 ` [PATCH v4 00/35] monitor: turn QMP and HMP into QOM objects Peter Krempa
2026-06-23 11:07 ` Peter Krempa
2026-06-23 11:38 ` Daniel P. Berrangé
2026-06-23 11:52 ` Daniel P. Berrangé
2026-06-23 11:54 ` Marc-André Lureau
2026-06-23 12:01 ` Daniel P. Berrangé
2026-06-23 12:18 ` Peter Krempa
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=20260623095551.2106712-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.