From d6552ebf26bc5bb605c65fd05deb4301574f396b Mon Sep 17 00:00:00 2001
From: Enji Cooper <ngie@FreeBSD.org>
Date: Wed, 29 Jan 2025 13:24:53 -0800
Subject: [PATCH] mca_log: add missing breaks when handling thresholds
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

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 b293fcedbd8..4ba49469d3a 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;
 			}
 		}
 	}