From: BALATON Zoltan <balaton@eik.bme.hu>
To: qemu-devel@nongnu.org
Cc: Paolo Bonzini <pbonzini@redhat.com>,
Eduardo Habkost <eduardo@habkost.net>,
Dr. David Alan Gilbert <dave@treblig.org>,
berrange@redhat.com
Subject: [PATCH v2] hmp: Add option to info qtree to omit details
Date: Fri, 1 Mar 2024 19:01:10 +0100 [thread overview]
Message-ID: <20240307183812.0105D4E6004@zero.eik.bme.hu> (raw)
The output of info qtree monitor command is very long. Add an option
to print a brief overview omitting all the details.
Signed-off-by: BALATON Zoltan <balaton@eik.bme.hu>
Reviewed-by: Dr. David Alan Gilbert <dave@treblig.org>
---
v2:
- Change the variable name to deails too
- Add braces to if (checkpatch did not warn for this so just noticed)
hmp-commands-info.hx | 6 +++---
system/qdev-monitor.c | 27 +++++++++++++++------------
2 files changed, 18 insertions(+), 15 deletions(-)
diff --git a/hmp-commands-info.hx b/hmp-commands-info.hx
index da120f82a3..ad1b1306e3 100644
--- a/hmp-commands-info.hx
+++ b/hmp-commands-info.hx
@@ -540,9 +540,9 @@ ERST
{
.name = "qtree",
- .args_type = "",
- .params = "",
- .help = "show device tree",
+ .args_type = "brief:-b",
+ .params = "[-b]",
+ .help = "show device tree (-b: brief, omit properties)",
.cmd = hmp_info_qtree,
},
diff --git a/system/qdev-monitor.c b/system/qdev-monitor.c
index a13db763e5..ad91e74181 100644
--- a/system/qdev-monitor.c
+++ b/system/qdev-monitor.c
@@ -744,7 +744,6 @@ DeviceState *qdev_device_add(QemuOpts *opts, Error **errp)
}
#define qdev_printf(fmt, ...) monitor_printf(mon, "%*s" fmt, indent, "", ## __VA_ARGS__)
-static void qbus_print(Monitor *mon, BusState *bus, int indent);
static void qdev_print_props(Monitor *mon, DeviceState *dev, Property *props,
int indent)
@@ -784,13 +783,9 @@ static void bus_print_dev(BusState *bus, Monitor *mon, DeviceState *dev, int ind
static void qdev_print(Monitor *mon, DeviceState *dev, int indent)
{
ObjectClass *class;
- BusState *child;
NamedGPIOList *ngl;
NamedClockList *ncl;
- qdev_printf("dev: %s, id \"%s\"\n", object_get_typename(OBJECT(dev)),
- dev->id ? dev->id : "");
- indent += 2;
QLIST_FOREACH(ngl, &dev->gpios, node) {
if (ngl->num_in) {
qdev_printf("gpio-in \"%s\" %d\n", ngl->name ? ngl->name : "",
@@ -814,12 +809,9 @@ static void qdev_print(Monitor *mon, DeviceState *dev, int indent)
class = object_class_get_parent(class);
} while (class != object_class_by_name(TYPE_DEVICE));
bus_print_dev(dev->parent_bus, mon, dev, indent);
- QLIST_FOREACH(child, &dev->child_bus, sibling) {
- qbus_print(mon, child, indent);
- }
}
-static void qbus_print(Monitor *mon, BusState *bus, int indent)
+static void qbus_print(Monitor *mon, BusState *bus, int indent, bool details)
{
BusChild *kid;
@@ -827,16 +819,27 @@ static void qbus_print(Monitor *mon, BusState *bus, int indent)
indent += 2;
qdev_printf("type %s\n", object_get_typename(OBJECT(bus)));
QTAILQ_FOREACH(kid, &bus->children, sibling) {
+ BusState *child_bus;
DeviceState *dev = kid->child;
- qdev_print(mon, dev, indent);
+ qdev_printf("dev: %s, id \"%s\"\n", object_get_typename(OBJECT(dev)),
+ dev->id ? dev->id : "");
+ if (details) {
+ qdev_print(mon, dev, indent + 2);
+ }
+ QLIST_FOREACH(child_bus, &dev->child_bus, sibling) {
+ qbus_print(mon, child_bus, indent + 2, details);
+ }
}
}
#undef qdev_printf
void hmp_info_qtree(Monitor *mon, const QDict *qdict)
{
- if (sysbus_get_default())
- qbus_print(mon, sysbus_get_default(), 0);
+ bool details = !qdict_get_try_bool(qdict, "brief", false);
+
+ if (sysbus_get_default()) {
+ qbus_print(mon, sysbus_get_default(), 0, details);
+ }
}
void hmp_info_qdm(Monitor *mon, const QDict *qdict)
--
2.30.9
next reply other threads:[~2024-03-07 18:39 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-03-01 18:01 BALATON Zoltan [this message]
2024-03-08 8:16 ` [PATCH v2] hmp: Add option to info qtree to omit details Philippe Mathieu-Daudé
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=20240307183812.0105D4E6004@zero.eik.bme.hu \
--to=balaton@eik.bme.hu \
--cc=berrange@redhat.com \
--cc=dave@treblig.org \
--cc=eduardo@habkost.net \
--cc=pbonzini@redhat.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.