[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 4/5] vl: Document dependencies hiding in glob
From: |
Igor Mammedov |
Subject: |
Re: [Qemu-devel] [PATCH v2 4/5] vl: Document dependencies hiding in global and compat props |
Date: |
Tue, 2 Apr 2019 12:53:41 +0200 |
On Mon, 1 Apr 2019 11:08:26 +0200
Markus Armbruster <address@hidden> wrote:
> Signed-off-by: Markus Armbruster <address@hidden>
Reviewed-by: Igor Mammedov <address@hidden>
> ---
> vl.c | 24 ++++++++++++++++++++++--
> 1 file changed, 22 insertions(+), 2 deletions(-)
>
> diff --git a/vl.c b/vl.c
> index e4d7ad6b85..8c6d06b01d 100644
> --- a/vl.c
> +++ b/vl.c
> @@ -4285,16 +4285,36 @@ int main(int argc, char **argv, char **envp)
> current_machine->maxram_size = maxram_size;
> current_machine->ram_slots = ram_slots;
>
> + /*
> + * Note: uses machine properties such as kernel-irqchip, must run
> + * after machine_set_property().
> + */
> configure_accelerator(current_machine, argv[0]);
>
> + /*
> + * Beware, QOM objects created before this point miss global and
> + * compat properties.
> + *
> + * Global properties get set up by qdev_prop_register_global(),
> + * called from user_register_global_props(), and certain option
> + * desugaring. Also in CPU feature desugaring (buried in
> + * parse_cpu_model()), which happens below this point, but may
> + * only target the CPU type, which can only be created after
> + * parse_cpu_model() returned the type.
> + *
> + * Machine compat properties: object_set_machine_compat_props().
> + * Accelerator compat props: object_set_accelerator_compat_props(),
> + * called from configure_accelerator().
> + */
> +
> if (!qtest_enabled() && machine_class->deprecation_reason) {
> error_report("Machine type '%s' is deprecated: %s",
> machine_class->name, machine_class->deprecation_reason);
> }
>
> /*
> - * Migration object can only be created after global properties
> - * are applied correctly.
> + * Note: creates a QOM object, must run only after global and
> + * compat properties have been set up.
> */
> migration_object_init();
>
Re: [Qemu-devel] [PATCH v2 2/5] Revert "migration: move only_migratable to MigrationState", Igor Mammedov, 2019/04/02