From: Anthony Liguori <aliguori@us.ibm.com>
To: qemu-devel@nongnu.org
Cc: Anthony Liguori <aliguori@us.ibm.com>,
Eric Blake <eblake@redhat.com>,
Eduardo Habkost <ehabkost@redhat.com>,
Gerd Hoffman <kraxel@redhat.com>
Subject: [Qemu-devel] [PATCH 5/9] vl: enable system configuration to be used
Date: Mon, 19 Mar 2012 10:09:19 -0500 [thread overview]
Message-ID: <1332169763-30665-6-git-send-email-aliguori@us.ibm.com> (raw)
In-Reply-To: <1332169763-30665-1-git-send-email-aliguori@us.ibm.com>
Now that we have options grouped, allow the [system] section to be used to set
these options.
Signed-off-by: Anthony Liguori <aliguori@us.ibm.com>
---
vl.c | 62 +++++++++++++++++++++++++++++++++++++++++++++++++++++---------
1 files changed, 53 insertions(+), 9 deletions(-)
diff --git a/vl.c b/vl.c
index 9a7ad40..5e86311 100644
--- a/vl.c
+++ b/vl.c
@@ -2187,6 +2187,24 @@ static void qemu_run_machine_init_done_notifiers(void)
notifier_list_notify(&machine_init_done_notifiers, NULL);
}
+static const QEMUOption *find_opt_by_name(const char *name)
+{
+ const QEMUOption *popt;
+
+ popt = qemu_options;
+ for(;;) {
+ if (!popt->name) {
+ break;
+ }
+ if (!strcmp(popt->name, name)) {
+ return popt;
+ }
+ popt++;
+ }
+
+ return NULL;
+}
+
static const QEMUOption *lookup_opt(int argc, char **argv,
const char **poptarg, int *poptind)
{
@@ -2200,15 +2218,10 @@ static const QEMUOption *lookup_opt(int argc, char **argv,
/* Treat --foo the same as -foo. */
if (r[1] == '-')
r++;
- popt = qemu_options;
- for(;;) {
- if (!popt->name) {
- error_report("invalid option");
- exit(1);
- }
- if (!strcmp(popt->name, r + 1))
- break;
- popt++;
+ popt = find_opt_by_name(r + 1);
+ if (popt == NULL) {
+ error_report("invalid option");
+ exit(1);
}
if (popt->flags & HAS_ARG) {
if (optind >= argc) {
@@ -2273,6 +2286,35 @@ typedef struct QemuOptions
int defconfig;
} QemuOptions;
+static void qemu_parse_option(int index, const char *optarg, QemuOptions *options);
+
+static int qemu_parse_system_option(const char *name, const char *value, void *opaque)
+{
+ QemuOptions *options = opaque;
+ const QEMUOption *popt;
+
+ popt = find_opt_by_name(name);
+ if (popt == NULL) {
+ return -1;
+ }
+
+ qemu_parse_option(popt->index, value, options);
+
+ return 0;
+}
+
+static int qemu_parse_system_section(QemuOpts *opts, void *opaque)
+{
+ QemuOptions *options = opaque;
+
+ return qemu_opt_foreach(opts, qemu_parse_system_option, options, 0);
+}
+
+static void qemu_parse_system_sections(QemuOptions *options)
+{
+ qemu_opts_foreach(qemu_find_opts("system"), qemu_parse_system_section, options, 1);
+}
+
static void qemu_parse_option(int index, const char *optarg, QemuOptions *options)
{
QemuOpts *opts;
@@ -3143,6 +3185,8 @@ static void qemu_parse_options(int argc, char **argv, QemuOptions *options)
qemu_parse_option(popt->index, optarg, options);
}
}
+
+ qemu_parse_system_sections(options);
}
int main(int argc, char **argv, char **envp)
--
1.7.5.4
next prev parent reply other threads:[~2012-03-19 15:09 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-03-19 15:09 [Qemu-devel] [RFC PATCH 0/9] qemu capabilities reporting and config changes Anthony Liguori
2012-03-19 15:09 ` [Qemu-devel] [PATCH 1/9] qemu-config: fix -writeconfig when using qemu_opt_set_bool Anthony Liguori
2012-03-19 15:09 ` [Qemu-devel] [PATCH 2/9] qemu-config: friends don't let friends use sscanf Anthony Liguori
2012-03-19 15:09 ` [Qemu-devel] [PATCH 3/9] vl: refactor command line parsing to allow options to be set via config Anthony Liguori
2012-03-19 15:09 ` [Qemu-devel] [PATCH 4/9] vl: mark system configuration options in qemu-options.hx Anthony Liguori
2012-03-19 15:09 ` Anthony Liguori [this message]
2012-03-19 15:09 ` [Qemu-devel] [PATCH 6/9] vl: parse all options via QemuOpts Anthony Liguori
2012-03-19 15:09 ` [Qemu-devel] [PATCH 7/9] qmp: expose a command to query capabilities of config parser Anthony Liguori
2012-03-19 20:10 ` Eric Blake
2012-03-19 20:19 ` Anthony Liguori
2012-03-19 20:31 ` Eric Blake
2012-03-19 20:41 ` Anthony Liguori
2012-03-19 15:09 ` [Qemu-devel] [PATCH 8/9] vl: add -query-capabilities Anthony Liguori
2012-03-20 7:49 ` Gerd Hoffmann
2012-03-20 10:33 ` Daniel P. Berrange
2012-03-20 19:39 ` Eduardo Habkost
2012-03-19 15:09 ` [Qemu-devel] [PATCH 9/9] Add a management tool writer's guide Anthony Liguori
2012-03-19 16:09 ` [Qemu-devel] [RFC PATCH 0/9] qemu capabilities reporting and config changes Paolo Bonzini
2012-03-19 16:31 ` Anthony Liguori
2012-03-19 16:33 ` Paolo Bonzini
2012-03-19 16:41 ` Anthony Liguori
2012-03-19 16:45 ` Paolo Bonzini
2012-03-19 16:53 ` Anthony Liguori
2012-03-19 17:03 ` Paolo Bonzini
2012-03-20 7:54 ` Gerd Hoffmann
2012-03-20 19:47 ` Eduardo Habkost
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=1332169763-30665-6-git-send-email-aliguori@us.ibm.com \
--to=aliguori@us.ibm.com \
--cc=eblake@redhat.com \
--cc=ehabkost@redhat.com \
--cc=kraxel@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).