[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 05/10] QemuOpts: add -set option
From: |
Gerd Hoffmann |
Subject: |
[Qemu-devel] [PATCH 05/10] QemuOpts: add -set option |
Date: |
Fri, 31 Jul 2009 12:25:36 +0200 |
One use case will be file for drives (no filename quoting issues), i.e.
-drive id=test,if=virtio
-set drive.test.file=/vmdisk/test-virtio.img
It will work for any other option (assuming handled by QemuOpts) though.
Except for id= for obvious reasons ;).
Signed-off-by: Gerd Hoffmann <address@hidden>
---
qemu-config.c | 41 +++++++++++++++++++++++++++++++++++++++++
qemu-config.h | 2 ++
qemu-options.hx | 6 +++++-
vl.c | 4 ++++
4 files changed, 52 insertions(+), 1 deletions(-)
diff --git a/qemu-config.c b/qemu-config.c
index 786f055..bcfb6eb 100644
--- a/qemu-config.c
+++ b/qemu-config.c
@@ -71,3 +71,44 @@ QemuOptsList qemu_drive_opts = {
},
};
+static QemuOptsList *lists[] = {
+ &qemu_drive_opts,
+ NULL,
+};
+
+int qemu_set_option(const char *str)
+{
+ char group[64], id[64], arg[64];
+ QemuOpts *opts;
+ int i, rc, offset;
+
+ rc = sscanf(str, "%63[^.].%63[^.].%63[^=]%n", group, id, arg, &offset);
+ if (rc < 3 || str[offset] != '=') {
+ fprintf(stderr, "can't parse: \"%s\"\n", str);
+ return -1;
+ }
+
+ for (i = 0; lists[i] != NULL; i++) {
+ if (strcmp(lists[i]->name, group) == 0)
+ break;
+ }
+ if (lists[i] == NULL) {
+ fprintf(stderr, "there is no option group \"%s\"\n", group);
+ return -1;
+ }
+
+ opts = qemu_opts_find(lists[i], id);
+ if (!opts) {
+ fprintf(stderr, "there is no %s \"%s\" defined\n",
+ lists[i]->name, id);
+ return -1;
+ }
+
+ if (-1 == qemu_opt_set(opts, arg, str+offset+1)) {
+ fprintf(stderr, "failed to set \"%s\" for %s \"%s\"\n",
+ arg, lists[i]->name, id);
+ return -1;
+ }
+ return 0;
+}
+
diff --git a/qemu-config.h b/qemu-config.h
index 3ada418..7faec9b 100644
--- a/qemu-config.h
+++ b/qemu-config.h
@@ -1 +1,3 @@
extern QemuOptsList qemu_drive_opts;
+
+int qemu_set_option(const char *str);
diff --git a/qemu-options.hx b/qemu-options.hx
index 1b420a3..38989f1 100644
--- a/qemu-options.hx
+++ b/qemu-options.hx
@@ -95,8 +95,12 @@ DEF("drive", HAS_ARG, QEMU_OPTION_drive,
"-drive [file=file][,if=type][,bus=n][,unit=m][,media=d][,index=i]\n"
" [,cyls=c,heads=h,secs=s[,trans=t]][,snapshot=on|off]\n"
" [,cache=writethrough|writeback|none][,format=f][,serial=s]\n"
- " [,addr=A]\n"
+ " [,addr=A][,id=name]\n"
" use 'file' as a drive image\n")
+DEF("set", HAS_ARG, QEMU_OPTION_set,
+ "-set group.id.arg=value\n"
+ " set <arg> parameter for item <id> of type <group>\n"
+ " i.e. -set drive.$id.file=/path/to/image\n")
STEXI
@item -drive @var{option}[,@var{option}[,@var{option}[,...]]]
diff --git a/vl.c b/vl.c
index 5a9ff70..698182e 100644
--- a/vl.c
+++ b/vl.c
@@ -4963,6 +4963,10 @@ int main(int argc, char **argv, char **envp)
case QEMU_OPTION_drive:
drive_add(NULL, "%s", optarg);
break;
+ case QEMU_OPTION_set:
+ if (qemu_set_option(optarg) != 0)
+ exit(1);
+ break;
case QEMU_OPTION_mtdblock:
drive_add(optarg, MTD_ALIAS);
break;
--
1.6.2.5
- [Qemu-devel] [PATCH 0/10] QemuOpts+qdev patches., Gerd Hoffmann, 2009/07/31
- [Qemu-devel] [PATCH 01/10] QemuOpts: add some functions, Gerd Hoffmann, 2009/07/31
- [Qemu-devel] [PATCH 02/10] QemuOpts: qemu_opts_parse: fix id= parsing, Gerd Hoffmann, 2009/07/31
- [Qemu-devel] [PATCH 05/10] QemuOpts: add -set option,
Gerd Hoffmann <=
- [Qemu-devel] [PATCH 03/10] QemuOpts: make the drive id actually show up in "info block"., Gerd Hoffmann, 2009/07/31
- [Qemu-devel] [PATCH 09/10] qdev/prop: add drive property., Gerd Hoffmann, 2009/07/31
- [Qemu-devel] [PATCH 07/10] constify drive_get_by_id arg, Gerd Hoffmann, 2009/07/31
- [Qemu-devel] [PATCH 08/10] add -drive if=none, Gerd Hoffmann, 2009/07/31
- [Qemu-devel] [PATCH 04/10] QemuOpts: create qemu-config.h, Gerd Hoffmann, 2009/07/31
- [Qemu-devel] [PATCH 10/10] qdev-ify virtio-blk., Gerd Hoffmann, 2009/07/31
- [Qemu-devel] [PATCH 06/10] QemuOpts: switch over -device., Gerd Hoffmann, 2009/07/31