GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Make elog_shrink not depend on having seperate memory/flash descriptors. 

The way elog_shrink currently works is that it completely clears the data in
the flash/flash descriptor and then recreates it using the part of the log
it's going to keep as stored in the memory descriptor. That scheme depends on
there being to independent copies of the log.

This change reworks elog_shrink so that it moves the data it wants to keep
within a single descriptor and then propogates it to the other and to flash
intact. This way, when one of the descriptors goes away, all we have to do is
remove the code that would update it.

Built and booted into ChromeOS on Link. Ran mosys eventlog list. Added
2000 events to the log and ran mosys eventlog list again. Echoed a 1 into
/sys/firmware/gsmi/clear_eventlog and ran mosys eventlog list.
BRANCH=None

Change-Id: I50d77a4f00ea3c6b3e0ec8996dab1a3b31580205
Signed-off-by: Gabe Black <gabeblack@google.com>
Reviewed-on: https://gerrit.chromium.org/gerrit/49305
Reviewed-by: Duncan Laurie <dlaurie@chromium.org>
Commit-Queue: Gabe Black <gabeblack@chromium.org>
Tested-by: Gabe Black <gabeblack@chromium.org>
Reviewed-on: http://review.coreboot.org/4240
Tested-by: build bot (Jenkins)
Reviewed-by: Ronald G. Minnich <rminnich@gmail.com>
This commit is contained in:
Gabe Black 2013-04-24 03:16:21 -07:00 committed by Alexandru Gagniuc
parent e62e0369d5
commit bfae4aa768
1 changed files with 12 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

25
src/drivers/elog/elog.c
View File

@ -621,9 +621,11 @@ static int elog_flash_area_bootstrap(void)
static int elog_shrink(void) static int elog_shrink(void)
{ {
struct elog_descriptor *mem = elog_get_mem(); struct elog_descriptor *mem = elog_get_mem();
struct elog_descriptor *flash = elog_get_flash();
struct event_header *event; struct event_header *event;
u16 discard_count = 0; u16 discard_count = 0;
u16 offset = 0; u16 offset = 0;
u16 new_size = 0;
elog_debug("elog_shrink()\n"); elog_debug("elog_shrink()\n");
@ -645,25 +647,22 @@ static int elog_shrink(void)
discard_count++; discard_count++;
} }
/* Erase flash area */ new_size = mem->next_event_offset - offset;
elog_flash_erase_area(); memmove(&mem->data[0], &mem->data[offset], new_size);
memset(&mem->data[new_size], ELOG_TYPE_EOL, mem->data_size - new_size);
elog_reinit_descriptor(mem);
elog_flash_erase(flash->backing_store, flash->total_size);
/* Ensure the area was successfully erased */ /* Ensure the area was successfully erased */
if (elog_get_flash()->next_event_offset >= CONFIG_ELOG_FULL_THRESHOLD) { if (mem->next_event_offset >= CONFIG_ELOG_FULL_THRESHOLD) {
printk(BIOS_ERR, "ELOG: Flash area was not erased!\n"); printk(BIOS_ERR, "ELOG: Flash area was not erased!\n");
return -1; return -1;
} }
/* Write new flash area */ elog_flash_write(flash->backing_store, mem->backing_store,
elog_prepare_empty(elog_get_flash(), mem->total_size);
(u8*)elog_get_event_base(mem, offset), elog_reinit_descriptor(flash);
mem->next_event_offset - offset);
/* Update memory area from flash */
if (elog_sync_flash_to_mem() < 0) {
printk(BIOS_ERR, "Unable to update memory area from flash\n");
return -1;
}
/* Add clear event */ /* Add clear event */
elog_add_event_word(ELOG_TYPE_LOG_CLEAR, offset); elog_add_event_word(ELOG_TYPE_LOG_CLEAR, offset);