[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH V2 5/5] vl.c: Remove QEMUMachine usage
From: |
Andreas Färber |
Subject: |
Re: [Qemu-devel] [PATCH V2 5/5] vl.c: Remove QEMUMachine usage |
Date: |
Thu, 03 Apr 2014 19:09:37 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.3.0 |
Am 31.03.2014 11:26, schrieb Marcel Apfelbaum:
> All the references of QEMUMachine are already
> replaced by MachineClass.
>
> Signed-off-by: Marcel Apfelbaum <address@hidden>
> ---
> include/hw/boards.h | 7 +++----
> vl.c | 3 +--
> 2 files changed, 4 insertions(+), 6 deletions(-)
>
> diff --git a/include/hw/boards.h b/include/hw/boards.h
> index 7cf1f07..66ee98a 100644
> --- a/include/hw/boards.h
> +++ b/include/hw/boards.h
> @@ -7,8 +7,10 @@
> #include "hw/qdev.h"
> #include "qom/object.h"
>
> +typedef struct MachineClass MachineClass;
> +
> typedef struct QEMUMachineInitArgs {
> - const QEMUMachine *machine;
> + const MachineClass *machine;
> ram_addr_t ram_size;
> const char *boot_order;
> const char *kernel_filename;
> @@ -46,7 +48,6 @@ struct QEMUMachine {
> const char *default_machine_opts;
> const char *default_boot_order;
> GlobalProperty *compat_props;
> - struct QEMUMachine *next;
This field shouldn't be used in qemu.git any more since you adopted the
QOM way of iterating over the types. Could you drop this field as a very
first patch, pointing to the commit which obsoleted it?
Regards,
Andreas
> const char *hw_version;
> };
>
> @@ -63,7 +64,6 @@ int qemu_register_machine(QEMUMachine *m);
> OBJECT_CLASS_CHECK(MachineClass, (klass), TYPE_MACHINE)
>
> typedef struct MachineState MachineState;
> -typedef struct MachineClass MachineClass;
>
> MachineClass *find_default_machine(void);
> extern MachineState *current_machine;
> @@ -77,7 +77,6 @@ struct MachineClass {
> ObjectClass parent_class;
> /*< public >*/
>
> - QEMUMachine *qemu_machine;
> const char *name;
> const char *alias;
> const char *desc;
> diff --git a/vl.c b/vl.c
> index a4f8a6b..190bd5d 100644
> --- a/vl.c
> +++ b/vl.c
> @@ -2961,7 +2961,6 @@ int main(int argc, char **argv, char **envp)
> const char *optarg;
> const char *loadvm = NULL;
> MachineClass *machine_class;
> - QEMUMachine *machine = NULL;
> const char *cpu_model;
> const char *vga_model = NULL;
> const char *qtest_chrdev = NULL;
> @@ -4388,7 +4387,7 @@ int main(int argc, char **argv, char **envp)
>
> qdev_machine_init();
>
> - QEMUMachineInitArgs args = { .machine = machine,
> + QEMUMachineInitArgs args = { .machine = machine_class,
> .ram_size = ram_size,
> .boot_order = boot_order,
> .kernel_filename = kernel_filename,
>
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
- Re: [Qemu-devel] [PATCH V2 5/5] vl.c: Remove QEMUMachine usage,
Andreas Färber <=