[coreboot-gerrit] New patch to review for coreboot: acpi: Align FACS to 64 bytes

Patrick Georgi (pgeorgi@google.com) gerrit at coreboot.org
Sat Aug 8 23:22:12 CEST 2015


Patrick Georgi (pgeorgi at google.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/11141

-gerrit

commit 7736e4e87643954e4344ab7026852819875e19b5
Author: Patrick Georgi <patrick at georgi-clan.de>
Date:   Sat Aug 8 23:20:58 2015 +0200

    acpi: Align FACS to 64 bytes
    
    The spec states (5.2.10): "The BIOS aligns the FACS on a 64-byte boundary
    anywhere within the system's memory address space."
    
    Change-Id: Ie9415e505525dbdd418028d4954018c829921a18
    Signed-off-by: Patrick Georgi <patrick at georgi-clan.de>
---
 src/arch/x86/acpi.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/arch/x86/acpi.c b/src/arch/x86/acpi.c
index 134e437..0439ab5 100644
--- a/src/arch/x86/acpi.c
+++ b/src/arch/x86/acpi.c
@@ -815,6 +815,7 @@ unsigned long write_acpi_tables(unsigned long start)
 	acpi_write_xsdt(xsdt, oem_id, oem_table_id);
 
 	printk(BIOS_DEBUG, "ACPI:    * FACS\n");
+	current = (ALIGN(current, 64));
 	facs = (acpi_facs_t *) current;
 	current += sizeof(acpi_facs_t);
 	ALIGN_CURRENT;



More information about the coreboot-gerrit mailing list