Stefan Reinauer (stefan.reinauer@coreboot.org) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/1781
-gerrit
commit 46a3788ce2543d47c02a2ed721d0beec9202ff4d Author: Stefan Reinauer reinauer@chromium.org Date: Mon Oct 15 13:41:56 2012 -0700
Store timestamps before resuming from suspend
in the resume case, timestamps were collected in RAM stage but not stored in CBMEM. This leads to only a single time stamp covering 200ms being available for all of ram stage.
Change-Id: Ibf0bb92caf5e032c12fe4e1b9b84b3624d499511 Signed-off-by: Stefan Reinauer reinauer@google.com --- src/boot/hardwaremain.c | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/src/boot/hardwaremain.c b/src/boot/hardwaremain.c index 8348493..b08fe79 100644 --- a/src/boot/hardwaremain.c +++ b/src/boot/hardwaremain.c @@ -121,10 +121,6 @@ void hardwaremain(int boot_complete) cbmemc_reinit(); #endif #endif -#if CONFIG_HAVE_ACPI_RESUME - suspend_resume(); - post_code(0x8a); -#endif
timestamp_add(TS_START_RAMSTAGE, timestamps[0]); timestamp_add(TS_DEVICE_ENUMERATE, timestamps[1]); @@ -132,6 +128,12 @@ void hardwaremain(int boot_complete) timestamp_add(TS_DEVICE_ENABLE, timestamps[3]); timestamp_add(TS_DEVICE_INITIALIZE, timestamps[4]); timestamp_add(TS_DEVICE_DONE, timestamps[5]); + +#if CONFIG_HAVE_ACPI_RESUME + suspend_resume(); + post_code(0x8a); +#endif + timestamp_add_now(TS_CBMEM_POST);
#if CONFIG_WRITE_HIGH_TABLES