[coreboot-gerrit] New patch to review for coreboot: arch/x86/tables: Add time stamps for each table type

Paul Menzel (paulepanter@users.sourceforge.net) gerrit at coreboot.org
Tue Dec 6 21:52:01 CET 2016


Paul Menzel (paulepanter at users.sourceforge.net) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/17739

-gerrit

commit e8e878deb27bf429dcb43f511e6d3a66ba7e5a79
Author: Paul Menzel <paulepanter at users.sourceforge.net>
Date:   Tue Dec 6 21:41:08 2016 +0100

    arch/x86/tables: Add time stamps for each table type
    
    Change-Id: I9ec6a8d35dfdb73240a0d1b5038f105655a071a0
    Signed-off-by: Paul Menzel <paulepanter at users.sourceforge.net>
---
 src/arch/x86/tables.c                                  | 17 +++++++++++++----
 src/commonlib/include/commonlib/timestamp_serialized.h | 14 +++++++++++---
 2 files changed, 24 insertions(+), 7 deletions(-)

diff --git a/src/arch/x86/tables.c b/src/arch/x86/tables.c
index edcb717..e92928d 100644
--- a/src/arch/x86/tables.c
+++ b/src/arch/x86/tables.c
@@ -24,6 +24,7 @@
 #include <arch/smp/mpspec.h>
 #include <arch/acpi.h>
 #include <string.h>
+#include <timestamp.h>
 #include <cbmem.h>
 #include <smbios.h>
 
@@ -191,18 +192,26 @@ void arch_write_tables(uintptr_t coreboot_table)
 	unsigned long rom_table_end = 0xf0000;
 
 	/* This table must be between 0x0f0000 and 0x100000 */
-	if (IS_ENABLED(CONFIG_GENERATE_PIRQ_TABLE))
+	if (IS_ENABLED(CONFIG_GENERATE_PIRQ_TABLE)) {
+		timestamp_add_now(TS_PIRQ_TABLE);
 		rom_table_end = write_pirq_table(rom_table_end);
+	}
 
 	/* The smp table must be in 0-1K, 639K-640K, or 960K-1M */
-	if (IS_ENABLED(CONFIG_GENERATE_MP_TABLE))
+	if (IS_ENABLED(CONFIG_GENERATE_MP_TABLE)) {
+		timestamp_add_now(TS_MP_TABLE);
 		rom_table_end = write_mptable(rom_table_end);
+	}
 
-	if (IS_ENABLED(CONFIG_HAVE_ACPI_TABLES))
+	if (IS_ENABLED(CONFIG_HAVE_ACPI_TABLES)) {
+		timestamp_add_now(TS_ACPI_TABLE);
 		rom_table_end = write_acpi_table(rom_table_end);
+	}
 
-	if (IS_ENABLED(CONFIG_GENERATE_SMBIOS_TABLES))
+	if (IS_ENABLED(CONFIG_GENERATE_SMBIOS_TABLES)) {
+		timestamp_add_now(TS_SMBIOS_TABLE);
 		rom_table_end = write_smbios_table(rom_table_end);
+	}
 
 	sz = write_coreboot_forwarding_table(forwarding_table, coreboot_table);
 
diff --git a/src/commonlib/include/commonlib/timestamp_serialized.h b/src/commonlib/include/commonlib/timestamp_serialized.h
index 4f109a7..a707c73 100644
--- a/src/commonlib/include/commonlib/timestamp_serialized.h
+++ b/src/commonlib/include/commonlib/timestamp_serialized.h
@@ -56,9 +56,13 @@ enum timestamp_id {
 	TS_DEVICE_DONE = 70,
 	TS_CBMEM_POST = 75,
 	TS_WRITE_TABLES = 80,
-	TS_COREBOOT_TABLE = 82,
-	TS_OPTION_TABLE = 84,
-	TS_BOOTMEM_INIT = 86,
+	TS_PIRQ_TABLE = 81,
+	TS_MP_TABLE = 82,
+	TS_ACPI_TABLE = 83,
+	TS_SMBIOS_TABLE = 84,
+	TS_COREBOOT_TABLE = 85,
+	TS_OPTION_TABLE = 86,
+	TS_BOOTMEM_INIT = 87,
 	TS_FRAMEBUFFER = 88,
 	TS_LOAD_PAYLOAD = 90,
 	TS_ACPI_WAKE_JUMP = 98,
@@ -145,6 +149,10 @@ static const struct timestamp_id_to_name {
 	{ TS_DEVICE_DONE,	"device setup done" },
 	{ TS_CBMEM_POST,	"cbmem post" },
 	{ TS_WRITE_TABLES,	"write tables" },
+	{ TS_PIRQ_TABLE,	"write PIRQ table" },
+	{ TS_MP_TABLE,		"write MP table" },
+	{ TS_ACPI_TABLE,	"write ACPI table" },
+	{ TS_SMBIOS_TABLE,	"write SMBIOS table" },
 	{ TS_COREBOOT_TABLE,	"write coreboot table" },
 	{ TS_OPTION_TABLE,	"write option table" },
 	{ TS_BOOTMEM_INIT,	"initialize memory map at boot time" },



More information about the coreboot-gerrit mailing list