Stefan Reinauer ([email protected]) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/1781
-gerrit commit 253173c46412c27a1a11e5ecb7ed50851dcb074d Author: Stefan Reinauer <[email protected]> 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 <[email protected]> --- 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 -- coreboot mailing list: [email protected] http://www.coreboot.org/mailman/listinfo/coreboot
