On Mon, Jul 12, 2021 at 10:15:52PM -0400, Jason Andryuk wrote: > qemu can't start a xen vm after commit d8fb7d0969d5 > "vl: switch -M parsing to keyval" with: > > $ ./qemu-system-i386 -M xenfv > Unexpected error in object_property_find_err() at ../qom/object.c:1298: > qemu-system-i386: Property 'xenfv-3.1-machine.accel' not found > Aborted (core dumped) > > The default_machine_opts handling doesn't process the legacy machine > options like "accel". Call qemu_apply_legacy_machine_options to provide > the legacy handling. > > Signed-off-by: Jason Andryuk <jandr...@gmail.com>
Reviewed-by: Anthony PERARD <anthony.per...@citrix.com> I can't find a different way to set a default "accelerator" to a machine, so this patch seems necessary. Thanks, > --- > softmmu/vl.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/softmmu/vl.c b/softmmu/vl.c > index 4df1496101..f4d8630fc6 100644 > --- a/softmmu/vl.c > +++ b/softmmu/vl.c > @@ -2126,6 +2126,7 @@ static void qemu_create_machine(QDict *qdict) > QDict *default_opts = > keyval_parse(machine_class->default_machine_opts, NULL, NULL, > &error_abort); > + qemu_apply_legacy_machine_options(default_opts); > object_set_properties_from_keyval(OBJECT(current_machine), > default_opts, > false, &error_abort); > qobject_unref(default_opts); -- Anthony PERARD
