[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 3/4] add cpu-model option to -machine
From: |
Eduardo Habkost |
Subject: |
Re: [Qemu-devel] [PATCH 3/4] add cpu-model option to -machine |
Date: |
Tue, 30 Apr 2013 09:09:05 -0300 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Tue, Apr 30, 2013 at 08:34:02AM +0200, Igor Mammedov wrote:
> Provides globally accessible cpu-model via machine opts.
>
> Signed-off-by: Igor Mammedov <address@hidden>
Reviewed-by: Eduardo Habkost <address@hidden>
It looks like most data on QEMUMachineInitArgs is already present on
machine_opts. Maybe we could add machine_opts to QEMUMachineInitArgs and
gradually remove existing QEMUMachineInitArgs fields?
> ---
> Note:
> - it will be used in cpu-add hook on tartget-i386.
> ---
> vl.c | 12 +++++++++++-
> 1 file changed, 11 insertions(+), 1 deletion(-)
>
> diff --git a/vl.c b/vl.c
> index 1e7d474..37a0f81 100644
> --- a/vl.c
> +++ b/vl.c
> @@ -429,6 +429,10 @@ static QemuOptsList qemu_machine_opts = {
> .name = "usb",
> .type = QEMU_OPT_BOOL,
> .help = "Set on/off to enable/disable usb",
> + }, {
> + .name = "cpu-model",
> + .type = QEMU_OPT_STRING,
> + .help = "alias for \"-cpu\" CPU model definition",
> },
> { /* End of list */ }
> },
> @@ -2979,7 +2983,7 @@ int main(int argc, char **argv, char **envp)
> }
> case QEMU_OPTION_cpu:
> /* hw initialization will check this */
> - cpu_model = optarg;
> + qemu_opts_set(qemu_find_opts("machine"), 0, "cpu-model",
> optarg);
> break;
> case QEMU_OPTION_hda:
> {
> @@ -3919,6 +3923,11 @@ int main(int argc, char **argv, char **envp)
> */
> cpudef_init();
>
> + machine_opts = qemu_opts_find(qemu_find_opts("machine"), 0);
> + if (machine_opts) {
> + cpu_model = qemu_opt_get(machine_opts, "cpu-model");
> + }
> +
> if (cpu_model && is_help_option(cpu_model)) {
> list_cpus(stdout, &fprintf, cpu_model);
> exit(0);
> @@ -4124,6 +4133,7 @@ int main(int argc, char **argv, char **envp)
> kernel_filename = qemu_opt_get(machine_opts, "kernel");
> initrd_filename = qemu_opt_get(machine_opts, "initrd");
> kernel_cmdline = qemu_opt_get(machine_opts, "append");
> + cpu_model = qemu_opt_get(machine_opts, "cpu-model");
> } else {
> kernel_filename = initrd_filename = kernel_cmdline = NULL;
> }
> --
> 1.8.2.1
>
--
Eduardo