Change in coreboot[master]: coreboot_table: Measure time of `write_coreboot_table()`

[Paul Menzel (Code Review)]

Paul Menzel has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/38315 )

Change subject: coreboot_table: Measure time of `write_coreboot_table()`
......................................................................

coreboot_table: Measure time of `write_coreboot_table()`

Change-Id: Ibd3e1595854ba15253d86cd3a09671a17f10c993
Signed-off-by: Paul Menzel 
---
M src/lib/coreboot_table.c
1 file changed, 5 insertions(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/15/38315/1

diff --git a/src/lib/coreboot_table.c b/src/lib/coreboot_table.c
index e42cb3b..c081df0 100644
--- a/src/lib/coreboot_table.c
+++ b/src/lib/coreboot_table.c
@@ -35,6 +35,7 @@
 #include <spi_flash.h>
 #include <security/vboot/misc.h>
 #include <security/vboot/vbnv_layout.h>
+#include <timer.h>
 #if CONFIG(USE_OPTION_TABLE)
 #include <option_table.h>
 #endif
@@ -581,6 +582,7 @@
    uintptr_t cbtable_end;
    size_t cbtable_size;
    const size_t max_table_size = COREBOOT_TABLE_SIZE;
+	struct stopwatch sw;

cbtable_start = (uintptr_t)cbmem_add(CBMEM_ID_CBTABLE, max_table_size);

@@ -593,6 +595,7 @@
    arch_write_tables(cbtable_start);

/* Write the coreboot table. */
+	stopwatch_init_msecs(&sw);
    cbtable_end = write_coreboot_table(cbtable_start);
    cbtable_size = cbtable_end - cbtable_start;

@@ -601,7 +604,8 @@
    		__func__, cbtable_size, max_table_size);
    }

-	printk(BIOS_DEBUG, "coreboot table: %zd bytes.\n", cbtable_size);
+	printk(BIOS_DEBUG, "coreboot table: %zd bytes (took %lu ms).\n",
+		cbtable_size, stopwatch_duration_msecs(&sw));

/* Print CBMEM sections */
    cbmem_list();

[Paul Menzel (Code Review)]

Hello build bot (Jenkins),

I'd like you to reexamine a change. Please visit

https://review.coreboot.org/c/coreboot/+/38315

to look at the new patch set (#2).

Change subject: coreboot_table: Measure time of `write_coreboot_table()`
......................................................................

coreboot_table: Measure time of `write_coreboot_table()`

Change-Id: Ibd3e1595854ba15253d86cd3a09671a17f10c993
Signed-off-by: Paul Menzel 
---
M src/lib/coreboot_table.c
1 file changed, 5 insertions(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/15/38315/2

[XS] Change in coreboot[master]: coreboot_table: Measure time of `write_coreboot_table()`

[Stefan Reinauer (Code Review)]

Stefan Reinauer has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/38315?usp=email )

Change subject: coreboot_table: Measure time of `write_coreboot_table()`
......................................................................

Abandoned