Attention is currently required from: Felix Held, Lance Zhao, Nico Huber, Tim Wawrzynczak.
Hello Lance Zhao, Nico Huber, Tim Wawrzynczak, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/75613?usp=email
to look at the new patch set (#2).
The following approvals got outdated and were removed: Code-Review+1 by Nico Huber, Verified+1 by build bot (Jenkins)
Change subject: [WIP] acpi/acpigen: avoid truncating length in acpigen_resource_producer_io ......................................................................
[WIP] acpi/acpigen: avoid truncating length in acpigen_resource_producer_io
When an IO resource producer is generated that covers the whole IO space from 0 to 0xffff, the length field in the word resource ACPI type would overflow and be truncated which results in Linux not finding any usable IO space to use for the PCI IO BARs. Instead generate a double word IO resource producer in this case.
TEST=Windows is ok with this while Linux complains a bit, but still ends up doing what it's expected to do in that case:
acpi PNP0A08:00: host bridge window expanded to [io 0x0000-0xffff]; [io 0x0000-0xffff window] ignored
Signed-off-by: Felix Held felix-coreboot@felixheld.de Change-Id: I8a59cdfcfa30a8fdd13f8db3dc1447994c266c8b --- M src/acpi/acpigen.c 1 file changed, 25 insertions(+), 11 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/13/75613/2