On Tue, Jul 25, 2017 at 07:10:21PM +0200, Paolo Bonzini wrote:
On 25/07/2017 18:23, Paolo Bonzini wrote:
On 25/07/2017 18:14, Laszlo Ersek wrote:
"No regressions became apparent in tests with a range of Windows (XP-10)"
In theory, w2k falls within that range.
Nope, Windows 2000 is like NT 5.0, XP is like NT 5.1. :(
One possibility is to fix it in SeaBIOS instead: if you get a 2.0 FADT and an XSDT and no RSDT, it can build an RSDT and a 1.0 FADT itself, patching the RSDT to point to it.
It's a hack, but it's the only place I see to make it "just work". And it could be extended nicely in the future.
It's an impressive hack!
All QEMU would have to do is to provide an XSDT _instead_ of an RSDT.
[...]
SeaBIOS:
From 73b0facdb7349f5dbf24dc675647b68eeeec0ff4 Mon Sep 17 00:00:00 2001 From: Paolo Bonzini pbonzini@redhat.com Date: Tue, 25 Jul 2017 18:50:19 +0200 Subject: [PATCH 1/2] seabios: build RSDT from XSDT
Old operating systems would like to have a v1 FADT, but new operating systems would like to have v3.
Since old operating systems do not know about XSDTs, the solution is to point the RSDT to a v1 FADT and the XSDT to a v3 FADT.
But, OVMF is not able to do that and barfs when it sees the second FADT---plus really it's only BIOS operating systems such as win2k that complain. So instead: 1) make QEMU provide an XSDT only; 2) build the RSDT and v1 FADT in SeaBIOS.
This patch makes SeaBIOS build an RSDT out of an existing XSDT.
I'd really prefer not to have SeaBIOS go back to producing ACPI tables.
As an alternative, how about some other possible hacks:
1 - ovmf filters out the extra tables that it barfs on.
2 - change ovmf to read the fw_cfg linker script 'etc/table-loader-ovmf' instead of '/etc/table-loader' and change qemu to generate two linker scripts - one for seabios and one for ovmf.
3 - same as 2, but change seabios to use 'etc/table-loader-seabios' and leave ovmf unchanged.
4 - change seabios to read both the linker script 'etc/table-loader' _and_ some new linker script '/etc/table-loader-legacy'. Have qemu introduce the RSDT/FADTv1 in the "legacy" linker script.
-Kevin