GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

ACPI S3: Clean up resume path 

Remove the obscure path in source code, where ACPI S3 resume
was prohibited and acpi_resume() would return and continue
to BS_WRITE_TABLES.

The condition when ACPI S3 would be prohibited needs to be
checked early in romstage already. For the time being, there
has been little interest to have CMOS option to disable
ACPI S3 resume feature.

Change-Id: If5105912759427f94f84d46d1a3141aa75cbd6ef
Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/42498
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
This commit is contained in:
Kyösti Mälkki 2020-06-18 08:28:12 +03:00 committed by Angel Pons
parent 470f319b9b
commit a4c0e1a51f
3 changed files with 6 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/arch/x86/acpi_s3.c
View File

@ -57,11 +57,6 @@ extern unsigned int __wakeup_size;
static void acpi_jump_to_wakeup(void *vector)
{
if (!acpi_s3_resume_allowed()) {
printk(BIOS_WARNING, "ACPI: S3 resume not allowed.\n");
return;
}
/* Copy wakeup trampoline in place. */
memcpy((void *)WAKEUP_BASE, &__wakeup, __wakeup_size);
@ -76,7 +71,7 @@ void __weak mainboard_suspend_resume(void)
{
}
void acpi_resume(void *wake_vec)
void __noreturn acpi_resume(void *wake_vec)
{
/* Restore GNVS pointer in SMM if found. */
apm_control(APM_CNT_GNVS_UPDATE);
@ -86,4 +81,6 @@ void acpi_resume(void *wake_vec)
post_code(POST_OS_RESUME);
acpi_jump_to_wakeup(wake_vec);
die("Failed the jump to wakeup vector\n");
}

2
src/include/acpi/acpi.h
View File

@ -987,7 +987,7 @@ unsigned long acpi_create_hest_error_source(acpi_hest_t *hest,
acpi_hest_esd_t *esd, u16 type, void *data, u16 len);
/* For ACPI S3 support. */
void acpi_resume(void *wake_vec);
void __noreturn acpi_resume(void *wake_vec);
void mainboard_suspend_resume(void);
void *acpi_find_wakeup_vector(void);

4
src/lib/hardwaremain.c
View File

@ -169,9 +169,9 @@ static boot_state_t bs_os_resume(void *wake_vector)
if (CONFIG(HAVE_ACPI_RESUME)) {
arch_bootstate_coreboot_exit();
acpi_resume(wake_vector);
/* We will not come back. */
}
return BS_WRITE_TABLES;
die("Failed OS resume\n");
}
static boot_state_t bs_write_tables(void *arg)