[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PULL 1/5] vl.c: Fix regression in machine error message
From: |
Markus Armbruster |
Subject: |
[Qemu-devel] [PULL 1/5] vl.c: Fix regression in machine error message |
Date: |
Fri, 19 Feb 2016 14:27:45 +0100 |
From: Marcel Apfelbaum <address@hidden>
Commit e1ce0c3cb (vl.c: fix regression when reading machine type
from config file) fixed the error message when the machine type
was supplied inside the config file. However now the option name
is not displayed correctly if the error happens when the machine
is specified at command line.
Running
./x86_64-softmmu/qemu-system-x86_64 -M q35-1.5 -redir tcp:8022::22
will result in the error message:
qemu-system-x86_64: -redir tcp:8022::22: unsupported machine type
Use -machine help to list supported machines
Fixed it by restoring the error location and also extracted the code
dealing with machine options into a separate function.
Reported-by: Michael S. Tsirkin <address@hidden>
Reviewed-by: Laszlo Ersek <address@hidden>
Signed-off-by: Marcel Apfelbaum <address@hidden>
Reviewed-by: Eduardo Habkost <address@hidden>
Signed-off-by: Eduardo Habkost <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: Markus Armbruster <address@hidden>
---
vl.c | 37 ++++++++++++++++++++++++++-----------
1 file changed, 26 insertions(+), 11 deletions(-)
diff --git a/vl.c b/vl.c
index 18e6086..226bdeb 100644
--- a/vl.c
+++ b/vl.c
@@ -2762,6 +2762,31 @@ static const QEMUOption *lookup_opt(int argc, char
**argv,
return popt;
}
+static void set_machine_options(MachineClass **machine_class)
+{
+ const char *optarg;
+ QemuOpts *opts;
+ Location loc;
+
+ loc_push_none(&loc);
+
+ opts = qemu_get_machine_opts();
+ qemu_opts_loc_restore(opts);
+
+ optarg = qemu_opt_get(opts, "type");
+ if (optarg) {
+ *machine_class = machine_parse(optarg);
+ }
+
+ if (*machine_class == NULL) {
+ error_report("No machine specified, and there is no default");
+ error_printf("Use -machine help to list supported machines\n");
+ exit(1);
+ }
+
+ loc_pop(&loc);
+}
+
static int machine_set_property(void *opaque,
const char *name, const char *value,
Error **errp)
@@ -3986,17 +4011,7 @@ int main(int argc, char **argv, char **envp)
replay_configure(icount_opts);
- opts = qemu_get_machine_opts();
- optarg = qemu_opt_get(opts, "type");
- if (optarg) {
- machine_class = machine_parse(optarg);
- }
-
- if (machine_class == NULL) {
- error_report("No machine specified, and there is no default");
- error_printf("Use -machine help to list supported machines\n");
- exit(1);
- }
+ set_machine_options(&machine_class);
set_memory_options(&ram_slots, &maxram_size, machine_class);
--
2.4.3
- [Qemu-devel] [PULL 0/5] Error reporting patches for 2016-02-19, Markus Armbruster, 2016/02/19
- [Qemu-devel] [PULL 5/5] vl: Clean up machine selection in main()., Markus Armbruster, 2016/02/19
- [Qemu-devel] [PULL 4/5] vl: Set error location when parsing memory options, Markus Armbruster, 2016/02/19
- [Qemu-devel] [PULL 2/5] vl: Reset location after handling command-line arguments, Markus Armbruster, 2016/02/19
- [Qemu-devel] [PULL 3/5] replay: Set error location properly when parsing options, Markus Armbruster, 2016/02/19
- [Qemu-devel] [PULL 1/5] vl.c: Fix regression in machine error message,
Markus Armbruster <=
- Re: [Qemu-devel] [PULL 0/5] Error reporting patches for 2016-02-19, Peter Maydell, 2016/02/19