From: Sergiy Kibrik <Sergiy_Kibrik@epam.com>
Date: Wed, 29 May 2024 07:56:15 +0000 (+0200)
Subject: x86/MCE: add default switch case in init_nonfatal_mce_checker()
X-Git-Tag: 4.19.0-rc1~118
X-Git-Url: http://xenbits.xensource.com/gitweb?a=commitdiff_plain;h=5effd810c629d9a836e46ee321ca7409dad27212;p=xen.git

x86/MCE: add default switch case in init_nonfatal_mce_checker()

The default switch case block is wanted here, to handle situation
e.g. of unexpected c->x86_vendor value -- then no mcheck init is done, but
misleading message still gets logged anyway.

Signed-off-by: Sergiy Kibrik <Sergiy_Kibrik@epam.com>
Acked-by: Jan Beulich <jbeulich@suse.com>
---

diff --git a/xen/arch/x86/cpu/mcheck/non-fatal.c b/xen/arch/x86/cpu/mcheck/non-fatal.c
index 33cacd15c2..5a53bcd0b7 100644
--- a/xen/arch/x86/cpu/mcheck/non-fatal.c
+++ b/xen/arch/x86/cpu/mcheck/non-fatal.c
@@ -29,9 +29,14 @@ static int __init cf_check init_nonfatal_mce_checker(void)
 		/* Assume we are on K8 or newer AMD or Hygon CPU here */
 		amd_nonfatal_mcheck_init(c);
 		break;
+
 	case X86_VENDOR_INTEL:
 		intel_nonfatal_mcheck_init(c);
 		break;
+
+	default:
+		/* unhandled vendor isn't really an error */
+		return 0;
 	}
 	printk(KERN_INFO "mcheck_poll: Machine check polling timer started.\n");
 	return 0;