[coreboot-gerrit] New patch to review for coreboot: AGESA: Apply a threshold on event logging

[Kyösti Mälkki (kyosti.malkki@gmail.com)]

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);