diff options
| author | Kyösti Mälkki <kyosti.malkki@gmail.com> | 2017-03-02 13:53:21 +0200 |
|---|---|---|
| committer | Kyösti Mälkki <kyosti.malkki@gmail.com> | 2017-03-09 12:44:28 +0100 |
| commit | 4f74c895929594598c323e026b27772f22d68eaa (patch) | |
| tree | be1cd6fb07658d768d5807fc33139471bca53d9e | |
| parent | 50e6daff95b7bd5c67a12a6b27cfdf7eedf304bd (diff) | |
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@gmail.com>
Reviewed-on: https://review.coreboot.org/18542
Tested-by: build bot (Jenkins)
Reviewed-by: Marc Jones <marc@marcjonesconsulting.com>
Reviewed-by: Marshall Dawson <marshalldawson3rd@gmail.com>
6 files changed, 18 insertions, 0 deletions
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); |