Re: [Qemu-devel] [PATCH 1.3?] multiboot: fix e801 memory map

[Anthony Liguori <anthony@codemonkey.ws>]

Paolo Bonzini <pbonzini@redhat.com> writes:

> Il 29/11/2012 20:24, Alexander Graf ha scritto:
>>> > It is in hw/multiboot.c:
>>> > 
>>> >    stl_p(bootinfo + MBI_MEM_LOWER,   640);
>> You want to remove that one then.
>
> I wasn't sure of what happens if the multiboot option ROM is old.  Do we
> support that to any extent?

Option ROMs are migrated but reset after reboot.  I guess you could
migrate before the option ROM is executed but that seems extraordinarily
unlikely.

Regards,

Anthony Liguori

>
>> > Regarding the testcase, Xen will touch the EBDA without this patch and
>> > with http://permalink.gmane.org/gmane.comp.emulators.xen.devel/144855.
>> > However, SeaBIOS does not complain.
>>
>> I see :). Well, the patch is unintrusive enough for be ok for 1.3 IMHO.
>
> Actually it's relatively easy to see a failure with Xen and a need to
> PCI passthrough something with a bulky boot ROM; iSCSI or FC cards will
> do.  You'll need both the Xen patch above, and this patch to boot
> successfully.  Otherwise, it will fail to boot.
>
> Paolo