[coreboot-gerrit] [M] Change in coreboot[main]: acpi/acpi.c: fix XSDT handling on QEMU

[Arthur Heymans (Code Review)]

Attention is currently required from: Cliff Huang, Krystian Hebel, Lance Zhao, Paul Menzel, Tim Wawrzynczak.

Arthur Heymans has posted comments on this change by Krystian Hebel. ( https://review.coreboot.org/c/coreboot/+/82698?usp=email )

Change subject: acpi/acpi.c: fix XSDT handling on QEMU
......................................................................

Patch Set 2:

(1 comment)

File src/acpi/acpi.c:

https://review.coreboot.org/c/coreboot/+/82698/comment/cccff3b0_766a8de2?usp... :
PS2, Line 1491: if (rsdp->xsdt_address == 0) {
              : 			xsdt = (acpi_xsdt_t *)current;
I think it's better to be explicitly look whether xsdt was added on first boot behind the current pointer?

if (!rsdp->xsdt_address && rsdp->xsdt_address + sizeof(acpi_xsdt_t) => current) {
                current = rsdp->xsdt_address + sizeof(acpi_xsdt_t);
                current = acpi_align_current(current);
          }