[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [RFC PATCH 2/5] qmp: further consolidate listing of device
From: |
Roman Kagan |
Subject: |
[Qemu-devel] [RFC PATCH 2/5] qmp: further consolidate listing of device and object properties |
Date: |
Fri, 14 Dec 2018 16:57:07 +0000 |
Take the approach of commit 35f63767dc77d85bebff6c6565aceaf74023776a
"qmp: Merge ObjectPropertyInfo and DevicePropertyInfo" one step further:
drop device property-specific code from qmp_device_list_properties and
consolidate the resulting common part with qmp_qom_list_properties.
Signed-off-by: Roman Kagan <address@hidden>
---
qmp.c | 92 ++++++++++++++---------------------------------------------
1 file changed, 22 insertions(+), 70 deletions(-)
diff --git a/qmp.c b/qmp.c
index e7c0a2fd60..673dfa72ce 100644
--- a/qmp.c
+++ b/qmp.c
@@ -430,56 +430,22 @@ ObjectTypeInfoList *qmp_qom_list_types(bool
has_implements,
return ret;
}
-/* Return a DevicePropertyInfo for a qdev property.
- *
- * If a qdev property with the given name does not exist, use the given default
- * type. If the qdev property info should not be shown, return NULL.
- *
- * The caller must free the return value.
- */
-static ObjectPropertyInfo *make_device_property_info(ObjectClass *klass,
- const char *name,
- const char *default_type,
- const char *description)
+static void push_property_info(ObjectPropertyInfoList **prop_list,
+ ObjectProperty *prop)
{
ObjectPropertyInfo *info;
- Property *prop;
-
- do {
- for (prop = DEVICE_CLASS(klass)->props; prop && prop->name; prop++) {
- if (strcmp(name, prop->name) != 0) {
- continue;
- }
-
- /*
- * TODO Properties without a parser are just for dirty hacks.
- * qdev_prop_ptr is the only such PropertyInfo. It's marked
- * for removal. This conditional should be removed along with
- * it.
- */
- if (!prop->info->set && !prop->info->create) {
- return NULL; /* no way to set it, don't show */
- }
-
- info = g_malloc0(sizeof(*info));
- info->name = g_strdup(prop->name);
- info->type = default_type ? g_strdup(default_type)
- : g_strdup(prop->info->name);
- info->has_description = !!prop->info->description;
- info->description = g_strdup(prop->info->description);
- return info;
- }
- klass = object_class_get_parent(klass);
- } while (klass != object_class_by_name(TYPE_DEVICE));
-
- /* Not a qdev property, use the default type */
- info = g_malloc0(sizeof(*info));
- info->name = g_strdup(name);
- info->type = g_strdup(default_type);
- info->has_description = !!description;
- info->description = g_strdup(description);
-
- return info;
+ ObjectPropertyInfoList *entry;
+
+ info = g_new0(ObjectPropertyInfo, 1);
+ info->name = g_strdup(prop->name);
+ info->type = g_strdup(prop->type);
+ info->has_description = !!prop->description;
+ info->description = g_strdup(prop->description);
+
+ entry = g_new0(ObjectPropertyInfoList, 1);
+ entry->value = info;
+ entry->next = *prop_list;
+ *prop_list = entry;
}
ObjectPropertyInfoList *qmp_device_list_properties(const char *typename,
@@ -514,9 +480,6 @@ ObjectPropertyInfoList *qmp_device_list_properties(const
char *typename,
object_property_iter_init(&iter, obj);
while ((prop = object_property_iter_next(&iter))) {
- ObjectPropertyInfo *info;
- ObjectPropertyInfoList *entry;
-
/* Skip Object and DeviceState properties */
if (strcmp(prop->name, "type") == 0 ||
strcmp(prop->name, "realized") == 0 ||
@@ -533,16 +496,12 @@ ObjectPropertyInfoList *qmp_device_list_properties(const
char *typename,
continue;
}
- info = make_device_property_info(klass, prop->name, prop->type,
- prop->description);
- if (!info) {
+ /* Skip readonly properties. */
+ if (!prop->set) {
continue;
}
- entry = g_malloc0(sizeof(*entry));
- entry->value = info;
- entry->next = prop_list;
- prop_list = entry;
+ push_property_info(&prop_list, prop);
}
object_unref(obj);
@@ -579,19 +538,12 @@ ObjectPropertyInfoList *qmp_qom_list_properties(const
char *typename,
object_property_iter_init(&iter, obj);
}
while ((prop = object_property_iter_next(&iter))) {
- ObjectPropertyInfo *info;
- ObjectPropertyInfoList *entry;
+ /* Skip readonly properties. */
+ if (!prop->set) {
+ continue;
+ }
- info = g_malloc0(sizeof(*info));
- info->name = g_strdup(prop->name);
- info->type = g_strdup(prop->type);
- info->has_description = !!prop->description;
- info->description = g_strdup(prop->description);
-
- entry = g_malloc0(sizeof(*entry));
- entry->value = info;
- entry->next = prop_list;
- prop_list = entry;
+ push_property_info(&prop_list, prop);
}
object_unref(obj);
--
2.19.2
- [Qemu-devel] [RFC PATCH 0/5] expose negotiated virtio features in r/o properties, Roman Kagan, 2018/12/14
- [Qemu-devel] [RFC PATCH 3/5] qdev-properties: add r/o 64bit bitfield property, Roman Kagan, 2018/12/14
- [Qemu-devel] [RFC PATCH 4/5] virtio: drop DEFINE_VIRTIO_COMMON_FEATURES, Roman Kagan, 2018/12/14
- [Qemu-devel] [RFC PATCH 1/5] qom: preserve get/set presence in aliased properties, Roman Kagan, 2018/12/14
- [Qemu-devel] [RFC PATCH 2/5] qmp: further consolidate listing of device and object properties,
Roman Kagan <=
- [Qemu-devel] [RFC PATCH 5/5] virtio: expose negotiated features in r/o properties, Roman Kagan, 2018/12/14
- Re: [Qemu-devel] [RFC PATCH 0/5] expose negotiated virtio features in r/o properties, no-reply, 2018/12/23