[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PULL v2 08/12] qom: Add helper method for getting user obj
From: |
Andreas Färber |
Subject: |
[Qemu-devel] [PULL v2 08/12] qom: Add helper method for getting user objects root |
Date: |
Wed, 27 May 2015 20:20:19 +0200 |
From: "Daniel P. Berrange" <address@hidden>
Add object_get_objects_root() method which is a convience for
obtaining the Object * located at /objects in the object
composition tree. Convert existing code over to use the new
API where appropriate.
Signed-off-by: Daniel P. Berrange <address@hidden>
Signed-off-by: Andreas Färber <address@hidden>
---
include/qom/object.h | 12 ++++++++++++
iothread.c | 4 +---
numa.c | 2 +-
qmp.c | 6 +++---
qom/object.c | 5 +++++
5 files changed, 22 insertions(+), 7 deletions(-)
diff --git a/include/qom/object.h b/include/qom/object.h
index d2d7748..d30c68c 100644
--- a/include/qom/object.h
+++ b/include/qom/object.h
@@ -1026,6 +1026,18 @@ const char *object_property_get_type(Object *obj, const
char *name,
*/
Object *object_get_root(void);
+
+/**
+ * object_get_objects_root:
+ *
+ * Get the container object that holds user created
+ * object instances. This is the object at path
+ * "/objects"
+ *
+ * Returns: the user object container
+ */
+Object *object_get_objects_root(void);
+
/**
* object_get_canonical_path_component:
*
diff --git a/iothread.c b/iothread.c
index 0416fc4..98dcdf9 100644
--- a/iothread.c
+++ b/iothread.c
@@ -19,8 +19,6 @@
#include "qmp-commands.h"
#include "qemu/error-report.h"
-#define IOTHREADS_PATH "/objects"
-
typedef ObjectClass IOThreadClass;
#define IOTHREAD_GET_CLASS(obj) \
@@ -153,7 +151,7 @@ IOThreadInfoList *qmp_query_iothreads(Error **errp)
{
IOThreadInfoList *head = NULL;
IOThreadInfoList **prev = &head;
- Object *container = container_get(object_get_root(), IOTHREADS_PATH);
+ Object *container = object_get_objects_root();
object_child_foreach(container, query_one_iothread, &prev);
return head;
diff --git a/numa.c b/numa.c
index c975fb2..b14a5c1 100644
--- a/numa.c
+++ b/numa.c
@@ -486,7 +486,7 @@ MemdevList *qmp_query_memdev(Error **errp)
Object *obj;
MemdevList *list = NULL;
- obj = object_resolve_path("/objects", NULL);
+ obj = object_get_objects_root();
if (obj == NULL) {
return NULL;
}
diff --git a/qmp.c b/qmp.c
index 3f5dfe3..fa013e3 100644
--- a/qmp.c
+++ b/qmp.c
@@ -651,7 +651,7 @@ void object_add(const char *type, const char *id, const
QDict *qdict,
}
}
- object_property_add_child(container_get(object_get_root(), "/objects"),
+ object_property_add_child(object_get_objects_root(),
id, obj, &local_err);
if (local_err) {
goto out;
@@ -659,7 +659,7 @@ void object_add(const char *type, const char *id, const
QDict *qdict,
user_creatable_complete(obj, &local_err);
if (local_err) {
- object_property_del(container_get(object_get_root(), "/objects"),
+ object_property_del(object_get_objects_root(),
id, &error_abort);
goto out;
}
@@ -706,7 +706,7 @@ void qmp_object_del(const char *id, Error **errp)
Object *container;
Object *obj;
- container = container_get(object_get_root(), "/objects");
+ container = object_get_objects_root();
obj = object_resolve_path_component(container, id);
if (!obj) {
error_setg(errp, "object id not found");
diff --git a/qom/object.c b/qom/object.c
index 550dd48..63bd300 100644
--- a/qom/object.c
+++ b/qom/object.c
@@ -1054,6 +1054,11 @@ Object *object_get_root(void)
return root;
}
+Object *object_get_objects_root(void)
+{
+ return container_get(object_get_root(), "/objects");
+}
+
static void object_get_child_property(Object *obj, Visitor *v, void *opaque,
const char *name, Error **errp)
{
--
2.1.4
- [Qemu-devel] [PULL v2 00/12] QOM devices patch queue 2015-05-27, Andreas Färber, 2015/05/27
- [Qemu-devel] [PULL v2 01/12] qom: strdup() target property name on object_property_add_alias(), Andreas Färber, 2015/05/27
- [Qemu-devel] [PULL v2 04/12] scripts: Add support for path as argument of qom-tree, Andreas Färber, 2015/05/27
- [Qemu-devel] [PULL v2 05/12] backends: Fix typename of 'policy' enum property in hostmem obj, Andreas Färber, 2015/05/27
- [Qemu-devel] [PULL v2 02/12] qdev: Free property names after registering gpio aliases, Andreas Färber, 2015/05/27
- [Qemu-devel] [PULL v2 08/12] qom: Add helper method for getting user objects root,
Andreas Färber <=
- [Qemu-devel] [PULL v2 12/12] qom: Add object_property_add_const_link(), Andreas Färber, 2015/05/27
- [Qemu-devel] [PULL v2 06/12] doc: Document user creatable object types in help text, Andreas Färber, 2015/05/27
- [Qemu-devel] [PULL v2 07/12] vl: Create (most) objects before creating chardev backends, Andreas Färber, 2015/05/27
- [Qemu-devel] [PULL v2 11/12] qom: Add an object_property_add_enum() helper function, Andreas Färber, 2015/05/27
- [Qemu-devel] [PULL v2 10/12] qom: Make enum string tables const-correct, Andreas Färber, 2015/05/27
- [Qemu-devel] [PULL v2 09/12] qom: Add object_new_with_props() / object_new_withpropv() helpers, Andreas Färber, 2015/05/27
- [Qemu-devel] [PULL v2 03/12] tests: Use qtest_add_data_func() consistently, Andreas Färber, 2015/05/27
- Re: [Qemu-devel] [PULL v2 00/12] QOM devices patch queue 2015-05-27, Peter Maydell, 2015/05/28