[coreboot-gerrit] New patch to review for coreboot: AGESA: Apply a threshold on event logging
Kyösti Mälkki (kyosti.malkki@gmail.com)
gerrit at coreboot.org
Thu Mar 2 15:30:38 CET 2017
Kyösti Mälkki (kyosti.malkki at gmail.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/18542
-gerrit
commit 80e33e007d735293257bc9944c3b6f2ae037c382
Author: Kyösti Mälkki <kyosti.malkki at gmail.com>
Date: Thu Mar 2 13:53:21 2017 +0200
AGESA: Apply a threshold on event logging
Implement as defined in AMD.h, do not add entries below
STATUS_LOG_LEVEL == AGESA_ALERT in the eventlog.
Change-Id: Ic9e45b1473b4fee46a1ad52d439e8682d961dc03
Signed-off-by: Kyösti Mälkki <kyosti.malkki at gmail.com>
---
src/vendorcode/amd/agesa/f10/Proc/CPU/cpuEventLog.c | 3 +++
src/vendorcode/amd/agesa/f12/Proc/CPU/cpuEventLog.c | 3 +++
src/vendorcode/amd/agesa/f14/Proc/CPU/cpuEventLog.c | 3 +++
src/vendorcode/amd/agesa/f15/Proc/CPU/cpuEventLog.c | 3 +++
src/vendorcode/amd/agesa/f15tn/Proc/CPU/cpuEventLog.c | 3 +++
src/vendorcode/amd/agesa/f16kb/Proc/CPU/cpuEventLog.c | 3 +++
6 files changed, 18 insertions(+)
diff --git a/src/vendorcode/amd/agesa/f10/Proc/CPU/cpuEventLog.c b/src/vendorcode/amd/agesa/f10/Proc/CPU/cpuEventLog.c
index 3c41f5d..6a0033f 100644
--- a/src/vendorcode/amd/agesa/f10/Proc/CPU/cpuEventLog.c
+++ b/src/vendorcode/amd/agesa/f10/Proc/CPU/cpuEventLog.c
@@ -199,6 +199,9 @@ PutEventLog (
UINT16 Index;
AGESA_STRUCT_BUFFER *AgesaEventAlloc;
+ if (EventClass < AGESA_STATUS_LOG_LEVEL)
+ return;
+
AgesaEventAlloc = NULL;
GetEventLogHeapPointer (&AgesaEventAlloc, StdHeader);
ASSERT (AgesaEventAlloc != NULL);
diff --git a/src/vendorcode/amd/agesa/f12/Proc/CPU/cpuEventLog.c b/src/vendorcode/amd/agesa/f12/Proc/CPU/cpuEventLog.c
index f802542..fcd23fb 100644
--- a/src/vendorcode/amd/agesa/f12/Proc/CPU/cpuEventLog.c
+++ b/src/vendorcode/amd/agesa/f12/Proc/CPU/cpuEventLog.c
@@ -213,6 +213,9 @@ PutEventLog (
(EventClass == AGESA_UNSUPPORTED) ? "UNSUPPORTED" :
"SUCCESS", EventInfo, DataParam1, DataParam2, DataParam3, DataParam4);
+ if (EventClass < AGESA_STATUS_LOG_LEVEL)
+ return;
+
AgesaEventAlloc = NULL;
GetEventLogHeapPointer (&AgesaEventAlloc, StdHeader);
ASSERT (AgesaEventAlloc != NULL);
diff --git a/src/vendorcode/amd/agesa/f14/Proc/CPU/cpuEventLog.c b/src/vendorcode/amd/agesa/f14/Proc/CPU/cpuEventLog.c
index e0ad3c6..b7c507f 100644
--- a/src/vendorcode/amd/agesa/f14/Proc/CPU/cpuEventLog.c
+++ b/src/vendorcode/amd/agesa/f14/Proc/CPU/cpuEventLog.c
@@ -216,6 +216,9 @@ PutEventLog (
(EventClass == AGESA_UNSUPPORTED) ? "UNSUPPORTED" :
"SUCCESS", EventInfo, DataParam1, DataParam2, DataParam3, DataParam4);
+ if (EventClass < AGESA_STATUS_LOG_LEVEL)
+ return;
+
AgesaEventAlloc = NULL;
GetEventLogHeapPointer (&AgesaEventAlloc, StdHeader);
ASSERT (AgesaEventAlloc != NULL);
diff --git a/src/vendorcode/amd/agesa/f15/Proc/CPU/cpuEventLog.c b/src/vendorcode/amd/agesa/f15/Proc/CPU/cpuEventLog.c
index 98d818c..a9a2821 100644
--- a/src/vendorcode/amd/agesa/f15/Proc/CPU/cpuEventLog.c
+++ b/src/vendorcode/amd/agesa/f15/Proc/CPU/cpuEventLog.c
@@ -214,6 +214,9 @@ PutEventLog (
(EventClass == AGESA_UNSUPPORTED) ? "UNSUPPORTED" :
"SUCCESS", EventInfo, DataParam1, DataParam2, DataParam3, DataParam4);
+ if (EventClass < AGESA_STATUS_LOG_LEVEL)
+ return;
+
AgesaEventAlloc = NULL;
GetEventLogHeapPointer (&AgesaEventAlloc, StdHeader);
ASSERT (AgesaEventAlloc != NULL);
diff --git a/src/vendorcode/amd/agesa/f15tn/Proc/CPU/cpuEventLog.c b/src/vendorcode/amd/agesa/f15tn/Proc/CPU/cpuEventLog.c
index f73f50b..c6f20bd 100644
--- a/src/vendorcode/amd/agesa/f15tn/Proc/CPU/cpuEventLog.c
+++ b/src/vendorcode/amd/agesa/f15tn/Proc/CPU/cpuEventLog.c
@@ -213,6 +213,9 @@ PutEventLog (
(EventClass == AGESA_UNSUPPORTED) ? "UNSUPPORTED" :
"SUCCESS", EventInfo, DataParam1, DataParam2, DataParam3, DataParam4);
+ if (EventClass < AGESA_STATUS_LOG_LEVEL)
+ return;
+
AgesaEventAlloc = NULL;
GetEventLogHeapPointer (&AgesaEventAlloc, StdHeader);
ASSERT (AgesaEventAlloc != NULL);
diff --git a/src/vendorcode/amd/agesa/f16kb/Proc/CPU/cpuEventLog.c b/src/vendorcode/amd/agesa/f16kb/Proc/CPU/cpuEventLog.c
index 4b6a9f9..5df76fb 100644
--- a/src/vendorcode/amd/agesa/f16kb/Proc/CPU/cpuEventLog.c
+++ b/src/vendorcode/amd/agesa/f16kb/Proc/CPU/cpuEventLog.c
@@ -213,6 +213,9 @@ PutEventLog (
(EventClass == AGESA_UNSUPPORTED) ? "UNSUPPORTED" :
"SUCCESS", EventInfo, DataParam1, DataParam2, DataParam3, DataParam4);
+ if (EventClass < AGESA_STATUS_LOG_LEVEL)
+ return;
+
AgesaEventAlloc = NULL;
GetEventLogHeapPointer (&AgesaEventAlloc, StdHeader);
ASSERT (AgesaEventAlloc != NULL);
More information about the coreboot-gerrit
mailing list