From 4f74c895929594598c323e026b27772f22d68eaa Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ky=C3=B6sti=20M=C3=A4lkki?= Date: Thu, 2 Mar 2017 13:53:21 +0200 Subject: AGESA: Apply a threshold on event logging MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 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 Reviewed-on: https://review.coreboot.org/18542 Tested-by: build bot (Jenkins) Reviewed-by: Marc Jones Reviewed-by: Marshall Dawson --- 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 3c41f5dcda..4c70418eb6 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 f8025427a7..7df57641e1 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 e0ad3c6330..74a72a914e 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 98d818c3de..1810764431 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 f73f50b777..169de7d8f8 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 4b6a9f9fb9..ad3fa397be 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); -- cgit v1.2.3