git: d6552ebf26bc - stable/14 - mca_log: add missing breaks when handling thresholds
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Wed, 26 Mar 2025 21:35:25 UTC
The branch stable/14 has been updated by ngie:
URL: https://cgit.FreeBSD.org/src/commit/?id=d6552ebf26bc5bb605c65fd05deb4301574f396b
commit d6552ebf26bc5bb605c65fd05deb4301574f396b
Author: Enji Cooper <ngie@FreeBSD.org>
AuthorDate: 2025-01-29 21:24:53 +0000
Commit: Enji Cooper <ngie@FreeBSD.org>
CommitDate: 2025-03-26 21:27:24 +0000
mca_log: add missing breaks when handling thresholds
Per the "Intel® 64 and IA-32 Architectures Software Developer’s Manual
Combined Volumes: Vol. 3B 17-7", the Green and Yellow threshold
indicators are mutually exclusive. Add the missing `break` statements so
they're treated that way.
MFC after: 1 week
Reported by: Coverity
Differential Revision: https://reviews.freebsd.org/D48722
(cherry picked from commit 36d7d2044b934103ad9254cd03ea61245c16ccda)
---
sys/x86/x86/mca.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/sys/x86/x86/mca.c b/sys/x86/x86/mca.c
index b293fcedbd84..4ba49469d3a2 100644
--- a/sys/x86/x86/mca.c
+++ b/sys/x86/x86/mca.c
@@ -453,8 +453,10 @@ mca_log(const struct mca_record *rec)
switch ((rec->mr_status & MC_STATUS_TES_STATUS) >> 53) {
case 0x1:
printf("(Green) ");
+ break;
case 0x2:
printf("(Yellow) ");
+ break;
}
}
}