From 3c3346b7cb7ed4fb41486d72586bf1c0cf1229f1 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: [PATCH] UPSTREAM: 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. BUG=none BRANCH=none TEST=none Change-Id: I41a257d8482bdeb568689045511547484c33e3c0 Signed-off-by: Patrick Georgi Original-Commit-Id: 4f74c895929594598c323e026b27772f22d68eaa Original-Change-Id: Ic9e45b1473b4fee46a1ad52d439e8682d961dc03 Original-Signed-off-by: Kysti Mlkki Original-Reviewed-on: https://review.coreboot.org/18542 Original-Tested-by: build bot (Jenkins) Original-Reviewed-by: Marc Jones Original-Reviewed-by: Marshall Dawson Reviewed-on: https://chromium-review.googlesource.com/452466 --- 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);