On Mon, Aug 26, 2013 at 05:17:07PM +0200, Andreas Färber wrote:
> Am 26.08.2013 17:01, schrieb Paolo Bonzini:
> > Il 26/08/2013 16:34, Andreas Färber ha scritto:
> >> Am 26.08.2013 14:21, schrieb Paolo Bonzini:
> >>> Il 13/08/2013 00:02, Paolo Bonzini ha scritto:
> >>>> vapic requires the VAPIC ROM to be mapped into RAM. This is not
> >>>> possible without PAM hardware. This fixes a segmentation fault
> >>>> running with -M isapc.
> >>>>
> >>>> Cc: qemu-sta...@nongnu.org
> >>>> Signed-off-by: Paolo Bonzini <pbonz...@redhat.com>
> >>>> ---
> >>>>
> >>>> diff --git a/hw/i386/pc_piix.c b/hw/i386/pc_piix.c
> >>>> index a19e172..3aa244a 100644
> >>>> --- a/hw/i386/pc_piix.c
> >>>> +++ b/hw/i386/pc_piix.c
> >>>> @@ -743,6 +743,11 @@ static QEMUMachine isapc_machine = {
> >>>> .init = pc_init_isa,
> >>>> .max_cpus = 1,
> >>>> .compat_props = (GlobalProperty[]) {
> >>>> + {
> >>>> + .driver = "apic-common",
> >>>> + .property = "kvmvapic",
> >>>> + .value = "off",
> >>>> + },
> >>>> { /* end of list */ }
> >>>> },
> >>>> DEFAULT_MACHINE_OPTIONS,
> >>>>
> >>>
> >>> Ping.
> >>
> >> I had double-checked that my pending qom-test covers isapc and succeeds
> >> with and without your patch, so apparently this is visible at runtime only.
> >
> > Yes, it only happens after kvmvapic is actually enabled by the option ROM.
>
> Maybe Michael can pick it up along with the compat refactorings?
>
> Andreas
Sure, I'll put it on my tree but I already prepared a pull request
for today, it will go into the next one.
> --
> SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
> GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
