summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAaron Durbin <adurbin@chromium.org>2016-11-04 10:07:14 -0500
committerFurquan Shaikh <furquan@google.com>2016-11-06 18:14:29 +0100
commit7d9068fe0b5d87fe3fe075d5d9a5f7493f5b1eac (patch)
treeae9b6a540a055222398a830e9b84dda9b832d652
parentd5be4e3d7e7d092d3ea9dc98738c39ea43d3f738 (diff)
downloadcoreboot-7d9068fe0b5d87fe3fe075d5d9a5f7493f5b1eac.tar.xz
soc/intel/common: log event when MRC cache is updated
Log when the MRC cache is attempted to be updated with status of success or failure. Just one slot is supported currently which is deemed 'normal'. This is because there are more slots anticipated in the future. BUG=chrome-os-partner:59395 Change-Id: I0f81458325697aff9924cc359a4173e0d35da5da Signed-off-by: Aaron Durbin <adurbin@chromium.org> Reviewed-on: https://review.coreboot.org/17231 Reviewed-by: Furquan Shaikh <furquan@google.com> Tested-by: build bot (Jenkins)
-rw-r--r--src/include/elog.h10
-rw-r--r--src/soc/intel/common/mrc_cache.c19
2 files changed, 29 insertions, 0 deletions
diff --git a/src/include/elog.h b/src/include/elog.h
index bdbafc2b19..7fdcda71a7 100644
--- a/src/include/elog.h
+++ b/src/include/elog.h
@@ -149,6 +149,16 @@ struct elog_event_data_me_extended {
#define ELOG_TYPE_WAKE 0xa8
#define ELOG_TYPE_FW_WAKE 0xa9
+/* Memory Cache Update */
+#define ELOG_TYPE_MEM_CACHE_UPDATE 0xaa
+#define ELOG_MEM_CACHE_UPDATE_SLOT_NORMAL 0
+#define ELOG_MEM_CACHE_UPDATE_STATUS_SUCCESS 0
+#define ELOG_MEM_CACHE_UPDATE_STATUS_FAIL 1
+struct elog_event_mem_cache_update {
+ u8 slot;
+ u8 status;
+} __attribute__ ((packed));
+
#if CONFIG_ELOG
/* Eventlog backing storage must be initialized before calling elog_init(). */
extern int elog_init(void);
diff --git a/src/soc/intel/common/mrc_cache.c b/src/soc/intel/common/mrc_cache.c
index c123da9dc7..bef3ba204c 100644
--- a/src/soc/intel/common/mrc_cache.c
+++ b/src/soc/intel/common/mrc_cache.c
@@ -17,6 +17,7 @@
#include <bootstate.h>
#include <console/console.h>
#include <cbmem.h>
+#include <elog.h>
#include <fmap.h>
#include <ip_checksum.h>
#include <vboot/vboot_common.h>
@@ -281,6 +282,18 @@ static int protect_mrc_cache(const struct mrc_data_region *region)
return 0;
}
+static void log_event_cache_update(uint8_t slot, uint8_t status)
+{
+ const int type = ELOG_TYPE_MEM_CACHE_UPDATE;
+ struct elog_event_mem_cache_update event = {
+ .slot = slot,
+ .status = status,
+ };
+
+ if (elog_add_event_raw(type, &event, sizeof(event)) < 0)
+ printk(BIOS_ERR, "Failed to log mem cache update event.\n");
+}
+
static void update_mrc_cache(void *unused)
{
const struct mrc_saved_data *current_boot;
@@ -336,7 +349,13 @@ static void update_mrc_cache(void *unused)
current_boot->size + sizeof(*current_boot))) {
printk(BIOS_DEBUG, "Failure writing MRC cache to %p.\n",
next_slot);
+ log_event_cache_update(ELOG_MEM_CACHE_UPDATE_SLOT_NORMAL,
+ ELOG_MEM_CACHE_UPDATE_STATUS_FAIL);
+ } else {
+ log_event_cache_update(ELOG_MEM_CACHE_UPDATE_SLOT_NORMAL,
+ ELOG_MEM_CACHE_UPDATE_STATUS_SUCCESS);
}
+
protect_mrc_cache(&region);
}