diff options
author | Stefan Reinauer <reinauer@chromium.org> | 2012-10-15 15:23:20 -0700 |
---|---|---|
committer | Ronald G. Minnich <rminnich@gmail.com> | 2012-11-13 18:24:53 +0100 |
commit | 4221a195745837b05725d7ffeda415516ac44a7f (patch) | |
tree | 6d0c641d636d3cb576daf49ba57c3c94cea081f5 /src/lib | |
parent | 7004b7c9e61640f1e7e7bf9043bf7b2a8603d956 (diff) |
Add method for delaying adding of timestamps
In hardwaremain() we can't add timestamps before we actually
reinitialized the cbmem area. Hence we kept the timestamps in
an array and added them later. This is ugly and intrusive and
helped hiding a bug that prevented any timestamps to be logged
in hardwaremain() when coming out of an S3 resume.
The problem is solved by moving the logic to keep a few timestamps
around into the timestamp code. This also gets rid of a lot of ugly
ifdefs in hardwaremain.c
Change-Id: I945fc4c77e990f620c18cbd054ccd87e746706ef
Signed-off-by: Stefan Reinauer <reinauer@google.com>
Reviewed-on: http://review.coreboot.org/1785
Tested-by: build bot (Jenkins)
Reviewed-by: Ronald G. Minnich <rminnich@gmail.com>
Diffstat (limited to 'src/lib')
-rw-r--r-- | src/lib/timestamp.c | 43 |
1 files changed, 43 insertions, 0 deletions
diff --git a/src/lib/timestamp.c b/src/lib/timestamp.c index bbb8197dca..2161d445db 100644 --- a/src/lib/timestamp.c +++ b/src/lib/timestamp.c @@ -21,6 +21,10 @@ #include <console/console.h> #include <cbmem.h> #include <timestamp.h> +#ifndef __PRE_RAM__ +/* For CAR_GLOBAL... This should move out of x86 specific code */ +#include <cpu/x86/car.h> +#endif #define MAX_TIMESTAMPS 30 @@ -72,3 +76,42 @@ void timestamp_add_now(enum timestamp_id id) { timestamp_add(id, rdtsc()); } + +#ifndef __PRE_RAM__ + +#define MAX_TIMESTAMP_CACHE 8 +struct timestamp_cache { + enum timestamp_id id; + tsc_t time; +} timestamp_cache[MAX_TIMESTAMP_CACHE] CAR_GLOBAL; + +static int timestamp_entries CAR_GLOBAL = 0; + +/** + * timestamp_stash() allows to temporarily cache time stamps. + * This is needed when time stamping before the CBMEM area + * is initialized. The function timestamp_sync() is used to + * write the time stamps to the CBMEM area. This is done in + * hardwaremain() + */ + +void timestamp_stash(enum timestamp_id id) +{ + if (timestamp_entries >= MAX_TIMESTAMP_CACHE) { + printk(BIOS_ERR, "ERROR: failed to add timestamp to cache\n"); + return; + } + timestamp_cache[timestamp_entries].id = id; + timestamp_cache[timestamp_entries].time = rdtsc(); + timestamp_entries++; +} + +void timestamp_sync(void) +{ + int i; + for (i = 0; i < timestamp_entries; i++) + timestamp_add(timestamp_cache[i].id, timestamp_cache[i].time); + timestamp_entries = 0; +} + +#endif |