Re: [Qemu-devel] Re: [Bochs-developers] [PATCH] Read additional ACPI tables from a VM

[Marcelo Tosatti]

On Tue, Feb 10, 2009 at 09:20:06AM +0200, Gleb Natapov wrote:
> On Mon, Feb 09, 2009 at 03:28:52PM -0600, Anthony Liguori wrote:
> > Gleb Natapov wrote:
> >>>> +
> >>>> +static int acpi_load_table(int i, uint32_t addr, uint16_t *len)
> >>>> +{
> >>>> +    qemu_cfg_read((uint8_t*)len, sizeof(*len));
> >>>> +
> >>>> +    if (!*len)
> >>>> +        return -1;
> >>>> +
> >>>> +    qemu_cfg_read((uint8_t*)addr, *len);
> >>>> +    return 0;
> >>>> +}
> >>>> #endif
> >>>>
> >>>> void init_smp_msrs(void)
> >>>>       
> >>> Can you please diff against bochs cvs?
> >>>
> >>>     
> >> Will do. This patch was initially intended to go to qemu patch series so I
> >> did it against patched bios source.
> >
> > I'm happy to update to a more recent BIOS snapshot...
> >
> >>  And unpatched bochs bios does not
> >> boot linux with qemu. IRQ routing problem or something.
> >>   
> >
> > Unless upstream Bochs has regressed.  Did you apply the other patches in  
> > the QEMU series?  Some are no longer needed I think.
> >
> With all qemu patches IRQ problem goes away. (One qemu patch conflicts with
> the last bochs commit so I reverted it before applying qemu series).

Ugh, I was getting "GRUB Error 16: Inconsistent filesystem structure"
which appeared to be caused by 864034d9a3190e0c2c10382ae015b5b5fd16718a.

But now its magically working again. Or not? 

Anthony, attached are updated patches + series to sync with Bochs.
Please give it a try.

0001_bx-qemu.patch
Description: Text document

0002_kvm-bios-update-smbios-table-to-report-memory-above-4g.patch
Description: Text document

0003_kvm-bios-generate-mptable-unconditionally.patch
Description: Text document

0004_kvm-bios-resolve-memory-device-roll-over-reporting--issues-with-32g-guests.patch
Description: Text document

0005_kvm-bios-fix-smbios-memory-device-length-boundary--condition.patch
Description: Text document

series
Description: Text document