Arthur Heymans has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/30789
Change subject: arch/x86/ebda: Don't trash the EBDA on the resume path ......................................................................
arch/x86/ebda: Don't trash the EBDA on the resume path
With some payloads the RSDP can end up in an area this function is trashing hence breaking S3 resume.
Change-Id: I9c54156bd8247e8a34dec6edc27cfc2d33cde595 Signed-off-by: Arthur Heymans arthur@aheymans.xyz --- M src/arch/x86/ebda.c 1 file changed, 3 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/89/30789/1
diff --git a/src/arch/x86/ebda.c b/src/arch/x86/ebda.c index f6726cf..62c81bd 100644 --- a/src/arch/x86/ebda.c +++ b/src/arch/x86/ebda.c @@ -18,6 +18,7 @@ #include <string.h> #include <arch/io.h> #include <arch/ebda.h> +#include <arch/acpi.h> #include <commonlib/endian.h> #include <console/console.h>
@@ -104,6 +105,8 @@
void setup_default_ebda(void) { + if (acpi_is_wakeup_s3()) + return; setup_ebda(DEFAULT_EBDA_LOWMEM, DEFAULT_EBDA_SEGMENT, DEFAULT_EBDA_SIZE);