diff options
| author | Paolo Bonzini <pbonzini@redhat.com> | 2012-05-02 13:30:59 +0200 |
|---|---|---|
| committer | Anthony Liguori <aliguori@us.ibm.com> | 2012-05-14 10:06:21 -0500 |
| commit | 90ca64a970b1d10352df3b2c68099341e900abe9 (patch) | |
| tree | 35bc32b70fe643900676551400c1f64a10290cff | |
| parent | 77f4c9a68aedb9b0d290a1def1bfc8714be862ec (diff) | |
qdev: fix -device foo,?
Since most property types do not have a parse property now, this was
broken. Fix it by looking at the setter instead.
Reviewed-by: Anthony Liguori <aliguori@us.ibm.com>
Acked-by: Andreas F=E4rber <afaerber@suse.de>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
| -rw-r--r-- | hw/qdev-monitor.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/hw/qdev-monitor.c b/hw/qdev-monitor.c index dc4e4e1b84..41b9e2c82f 100644 --- a/hw/qdev-monitor.c +++ b/hw/qdev-monitor.c @@ -158,7 +158,7 @@ int qdev_device_help(QemuOpts *opts) * for removal. This conditional should be removed along with * it. */ - if (!prop->info->parse) { + if (!prop->info->set) { continue; /* no way to set it, don't show */ } error_printf("%s.%s=%s\n", driver, prop->name, @@ -166,7 +166,7 @@ int qdev_device_help(QemuOpts *opts) } if (info->bus_info) { for (prop = info->bus_info->props; prop && prop->name; prop++) { - if (!prop->info->parse) { + if (!prop->info->set) { continue; /* no way to set it, don't show */ } error_printf("%s.%s=%s\n", driver, prop->name, |