summary refs log tree commit diff stats
path: root/hw/qdev.c
diff options
context:
space:
mode:
authorMarkus Armbruster <armbru@redhat.com>2010-02-19 13:31:49 +0100
committerMarkus Armbruster <armbru@redhat.com>2010-03-16 16:58:32 +0100
commitc64eafaf0c2f080d81f4c51a1eb2a98ceb40d2c8 (patch)
tree1b1b6a2941f57053763d8cb680e7c43bed92a1d4 /hw/qdev.c
parent0c17542d90a3863048ad6daff5de31c5c4d367d4 (diff)
downloadfocaccia-qemu-c64eafaf0c2f080d81f4c51a1eb2a98ceb40d2c8.tar.gz
focaccia-qemu-c64eafaf0c2f080d81f4c51a1eb2a98ceb40d2c8.zip
qdev: Hide "no_user" devices from users
Users can't create them, so qdev_device_help() shouldn't list them.
Fix that.

Also make qdev_device_add() pretend they don't exist.  Before, it
rejected them with a "can't be added via command line" message, which
wasn't quite right for monitor command device_add.
Diffstat (limited to 'hw/qdev.c')
-rw-r--r--hw/qdev.c10
1 files changed, 4 insertions, 6 deletions
diff --git a/hw/qdev.c b/hw/qdev.c
index de60108b1a..233480e12d 100644
--- a/hw/qdev.c
+++ b/hw/qdev.c
@@ -162,6 +162,9 @@ int qdev_device_help(QemuOpts *opts)
     driver = qemu_opt_get(opts, "driver");
     if (driver && !strcmp(driver, "?")) {
         for (info = device_info_list; info != NULL; info = info->next) {
+            if (info->no_user) {
+                continue;       /* not available, don't show */
+            }
             qdev_print_devinfo(info);
         }
         return 1;
@@ -197,15 +200,10 @@ DeviceState *qdev_device_add(QemuOpts *opts)
 
     /* find driver */
     info = qdev_find_info(NULL, driver);
-    if (!info) {
+    if (!info || info->no_user) {
         qerror_report(QERR_DEVICE_NOT_FOUND, driver);
         return NULL;
     }
-    if (info->no_user) {
-        error_report("device \"%s\" can't be added via command line",
-                     info->name);
-        return NULL;
-    }
 
     /* find bus */
     path = qemu_opt_get(opts, "bus");