[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 10/13] qdev: add user-specified identifier to device
From: |
Gerd Hoffmann |
Subject: |
[Qemu-devel] [PATCH 10/13] qdev: add user-specified identifier to devices. |
Date: |
Fri, 3 Jul 2009 12:22:55 +0200 |
Add id field to DeviceState. Make qdev_device_add() fill it if supplied
on the command line (i.e. -device foo,id=bar). Make "info qtree" print it.
This helps users and management apps identifying devices in monitor
output, which is especially useful with otherwise identical devices
such as two virtio disks.
Signed-off-by: Gerd Hoffmann <address@hidden>
---
hw/qdev.c | 5 ++++-
hw/qdev.h | 1 +
2 files changed, 5 insertions(+), 1 deletions(-)
diff --git a/hw/qdev.c b/hw/qdev.c
index d72a8ef..979ed4f 100644
--- a/hw/qdev.c
+++ b/hw/qdev.c
@@ -147,6 +147,7 @@ DeviceState *qdev_device_add(const char *cmdline)
qdev = info->bus_info->add(driver, strlen(addr) ? addr : NULL);
if (params) {
+ get_param_value(qdev->id, sizeof(qdev->id), "id", params);
while (params[0]) {
if (2 != sscanf(params, "%31[^=]=%255[^,]%n", tag, value, &n)) {
fprintf(stderr, "parse error at \"%s\"\n", params);
@@ -155,6 +156,8 @@ DeviceState *qdev_device_add(const char *cmdline)
params += n;
if (strcmp(tag, "addr") == 0)
continue;
+ if (strcmp(tag, "id") == 0)
+ continue;
if (-1 == qdev_prop_parse(qdev, tag, value)) {
fprintf(stderr, "can't set property \"%s\" to \"%s\" for
\"%s\"\n",
tag, value, driver);
@@ -329,7 +332,7 @@ static void qdev_print_props(Monitor *mon, DeviceState
*dev, Property *props,
static void qdev_print(Monitor *mon, DeviceState *dev, int indent)
{
BusState *child;
- qdev_printf("dev: %s\n", dev->info->name);
+ qdev_printf("dev: %s, id \"%s\"\n", dev->info->name, dev->id);
indent += 2;
if (dev->num_gpio_in) {
qdev_printf("gpio-in %d\n", dev->num_gpio_in);
diff --git a/hw/qdev.h b/hw/qdev.h
index 03413b1..cf6083f 100644
--- a/hw/qdev.h
+++ b/hw/qdev.h
@@ -17,6 +17,7 @@ typedef struct BusInfo BusInfo;
/* This structure should not be accessed directly. We declare it here
so that it can be embedded in individual device state structures. */
struct DeviceState {
+ char id[32];
DeviceInfo *info;
BusState *parent_bus;
int num_gpio_out;
--
1.6.2.5
- [Qemu-devel] [PATCH 0/13] qdev patches: properties, -device switch, id=<tag> & more., Gerd Hoffmann, 2009/07/03
- [Qemu-devel] [PATCH 02/13] qdev: factor out driver search to qdev_find_info(), Gerd Hoffmann, 2009/07/03
- [Qemu-devel] [PATCH 03/13] qdev/pci: make pci_create return DeviceState instead of PCIDevice., Gerd Hoffmann, 2009/07/03
- [Qemu-devel] [PATCH 04/13] qdev: add generic qdev_device_add(), Gerd Hoffmann, 2009/07/03
- [Qemu-devel] [PATCH 07/13] qdev: es1370 description, Gerd Hoffmann, 2009/07/03
- [Qemu-devel] [PATCH 06/13] qdev: add no_user, alias and desc, Gerd Hoffmann, 2009/07/03
- [Qemu-devel] [PATCH 05/13] qdev: add -device command line option., Gerd Hoffmann, 2009/07/03
- [Qemu-devel] [PATCH 10/13] qdev: add user-specified identifier to devices.,
Gerd Hoffmann <=
- [Qemu-devel] [PATCH 08/13] qdev: convert all vga, Gerd Hoffmann, 2009/07/03
- [Qemu-devel] [PATCH 13/13] qdev: print device id in "info pci"., Gerd Hoffmann, 2009/07/03
- [Qemu-devel] [PATCH 09/13] qdev/pci: hook up i440fx., Gerd Hoffmann, 2009/07/03
- [Qemu-devel] [PATCH 01/13] qdev: rework device properties., Gerd Hoffmann, 2009/07/03
- [Qemu-devel] [PATCH 11/13] switch balloon initialization to -device., Gerd Hoffmann, 2009/07/03
- [Qemu-devel] [PATCH 12/13] qdev: add id= support for pci nics., Gerd Hoffmann, 2009/07/03