[coreboot-gerrit] Patch set updated for coreboot: AGESA: Apply a threshold on event logging

Kyösti Mälkki (kyosti.malkki@gmail.com) gerrit at coreboot.org
Wed Mar 8 06:42:50 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 d9a93dc68232f3939c4b0ab608d815249d53380f
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 threshold as described in AMD.h, and do not add
    entries below STATUS_LOG_LEVEL 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..4c70418 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..7df5764 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..74a72a9 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..1810764 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..169de7d 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..ad3fa39 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