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(a)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);
--
To view, visit https://review.coreboot.org/c/coreboot/+/30789
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I9c54156bd8247e8a34dec6edc27cfc2d33cde595
Gerrit-Change-Number: 30789
Gerrit-PatchSet: 1
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>
Gerrit-MessageType: newchange