From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:40995) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Z7Rgz-0002xU-Tt for qemu-devel@nongnu.org; Tue, 23 Jun 2015 13:09:05 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Z7Rgw-0003BV-Q2 for qemu-devel@nongnu.org; Tue, 23 Jun 2015 13:09:01 -0400 Received: from mx1.redhat.com ([209.132.183.28]:43038) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Z7Rgw-0003BH-Kw for qemu-devel@nongnu.org; Tue, 23 Jun 2015 13:08:58 -0400 Received: from int-mx10.intmail.prod.int.phx2.redhat.com (int-mx10.intmail.prod.int.phx2.redhat.com [10.5.11.23]) by mx1.redhat.com (Postfix) with ESMTPS id 5279C3672A1 for ; Tue, 23 Jun 2015 17:08:58 +0000 (UTC) Message-ID: <558992A7.7090005@redhat.com> Date: Tue, 23 Jun 2015 19:08:55 +0200 From: Laszlo Ersek MIME-Version: 1.0 References: <1435078851-10380-1-git-send-email-ehabkost@redhat.com> In-Reply-To: <1435078851-10380-1-git-send-email-ehabkost@redhat.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH] pc: Fix resue of pc-i440fx-2.4 on pc-i440fx-2.3 machine_options List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Eduardo Habkost , qemu-devel@nongnu.org, "Michael S. Tsirkin" Cc: "Dr. David Alan Gilbert" On 06/23/15 19:00, Eduardo Habkost wrote: > On commit fddd179ab962f6f78a8493742e1068d6a620e059, I broke the chaining of > *_machine_options() functions on pc-i440fx-2.3, at: > > -#define PC_I440FX_2_3_MACHINE_OPTIONS \ > - PC_I440FX_2_4_MACHINE_OPTIONS, \ > - .alias = NULL, \ > - .is_default = 0 > +static void pc_i440fx_2_3_machine_options(QEMUMachine *m) > +{ > + pc_i440fx_machine_options(m); > + m->alias = NULL; > + m->is_default = 0; > +} > > I have replaced PC_I440FX_2_4_MACHINE_OPTIONS with a > pc_i440fx_machine_options() call, instead of calling > pc_i440fx_2_4_machine_options(). This broke the setting of default_machine_opts > and default_display on pc-i440fx-{2.0,2,1,2.2,2.3}. > > Fix this by making pc_i440fx_2_3_machine_options() reuse > pc_i440fx_2_4_machine_options(). > > Reported-by: "Dr. David Alan Gilbert" > Signed-off-by: Eduardo Habkost > --- > hw/i386/pc_piix.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/hw/i386/pc_piix.c b/hw/i386/pc_piix.c > index e142f75..b335856 100644 > --- a/hw/i386/pc_piix.c > +++ b/hw/i386/pc_piix.c > @@ -486,7 +486,7 @@ DEFINE_I440FX_MACHINE(v2_4, "pc-i440fx-2.4", NULL, > > static void pc_i440fx_2_3_machine_options(MachineClass *m) > { > - pc_i440fx_machine_options(m); > + pc_i440fx_2_4_machine_options(m); > m->alias = NULL; > m->is_default = 0; > SET_MACHINE_COMPAT(m, PC_COMPAT_2_3); > looks good to me Reviewed-by: Laszlo Ersek