On Mon, Dec 03, 2018 at 09:28:36AM -0700, Alex Williamson wrote: > Including all machine types that might have a pcie-root-port. > > Cc: Peter Maydell <peter.mayd...@linaro.org> > Cc: Michael S. Tsirkin <m...@redhat.com> > Cc: Marcel Apfelbaum <marcel.apfelb...@gmail.com> > Cc: Paolo Bonzini <pbonz...@redhat.com> > Cc: Richard Henderson <r...@twiddle.net> > Cc: Eduardo Habkost <ehabk...@redhat.com> > Cc: David Gibson <da...@gibson.dropbear.id.au> > Signed-off-by: Alex Williamson <alex.william...@redhat.com> > --- [...] > +static void virt_3_1_instance_init(Object *obj) > +{ > + virt_4_0_instance_init(obj); > +} [...] > +/* > + * pseries-4.0 > + */ > +static void spapr_machine_4_0_instance_options(MachineState *machine) > +{ > +}
Not something specific about patch, but: Why exactly virt and pseries machines need separate instance initialization functions? I'd like to eventually make all machine-type-version-specific behavior instrospectable, so it needs to be available at class_init time. -- Eduardo