[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCHv2 03/10] pseries: Remove redundant calls to spap
From: |
Thomas Huth |
Subject: |
Re: [Qemu-devel] [PATCHv2 03/10] pseries: Remove redundant calls to spapr_machine_initfn() |
Date: |
Mon, 7 Dec 2015 11:25:38 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 |
On 07/12/15 04:34, David Gibson wrote:
> The instance_init() functions for several of the pseries-x.y versioned
> machine types explicitly call spapr_machine_initfn(). But that's the
> instance_init function for the common parent of all those machine types,
> so will already have been called beforehand by the QOM infrastructure.
>
> Remove the redundant calls.
>
> Signed-off-by: David Gibson <address@hidden>
> ---
> hw/ppc/spapr.c | 3 ---
> 1 file changed, 3 deletions(-)
>
> diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c
> index c126e10..c7c6517 100644
> --- a/hw/ppc/spapr.c
> +++ b/hw/ppc/spapr.c
> @@ -2366,7 +2366,6 @@ static void spapr_compat_2_3(Object *obj)
> static void spapr_machine_2_3_instance_init(Object *obj)
> {
> spapr_compat_2_3(obj);
> - spapr_machine_initfn(obj);
> }
>
> static void spapr_machine_2_3_class_init(ObjectClass *oc, void *data)
> @@ -2409,7 +2408,6 @@ static void spapr_compat_2_2(Object *obj)
> static void spapr_machine_2_2_instance_init(Object *obj)
> {
> spapr_compat_2_2(obj);
> - spapr_machine_initfn(obj);
> }
>
> static void spapr_machine_2_2_class_init(ObjectClass *oc, void *data)
> @@ -2446,7 +2444,6 @@ static void spapr_compat_2_1(Object *obj)
> static void spapr_machine_2_1_instance_init(Object *obj)
> {
> spapr_compat_2_1(obj);
> - spapr_machine_initfn(obj);
> }
>
> static void spapr_machine_2_1_class_init(ObjectClass *oc, void *data)
Right, I just checked, and without this patch, spapr_machine_initfn() is
indeed called twice.
Reviewed-by: Thomas Huth <address@hidden>
- [Qemu-devel] [PATCHv2 00/10] Clean up pseries machine versioning, David Gibson, 2015/12/06
- [Qemu-devel] [PATCHv2 01/10] pseries: Remove redundant setting of mc->name for pseries-2.5 machine, David Gibson, 2015/12/06
- [Qemu-devel] [PATCHv2 03/10] pseries: Remove redundant calls to spapr_machine_initfn(), David Gibson, 2015/12/06
- [Qemu-devel] [PATCHv2 04/10] pseries: Remove versions from mc->desc, David Gibson, 2015/12/06
- [Qemu-devel] [PATCHv2 06/10] pseries: Use SET_MACHINE_COMPAT, David Gibson, 2015/12/06
- [Qemu-devel] [PATCHv2 09/10] pseries: Improve setting of default machine version, David Gibson, 2015/12/06
- [Qemu-devel] [PATCHv2 05/10] Move SET_MACHINE_COMPAT macro to boards.h, David Gibson, 2015/12/06