[PATCH] cpu: msr: do not check SYSENTER_* MSRs (LP: #1582005)

Colin King colin.king at canonical.com
Mon May 16 08:44:31 UTC 2016


From: Colin Ian King <colin.king at canonical.com>

We should not checking consistency across CPUs for the SYSENTER_*
MSRs, so disable these.

Kudos to Rudolf Marek for spotting this

Signed-off-by: Colin Ian King <colin.king at canonical.com>
---
 src/cpu/msr/msr.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/src/cpu/msr/msr.c b/src/cpu/msr/msr.c
index 6e9842b..913f5ab 100644
--- a/src/cpu/msr/msr.c
+++ b/src/cpu/msr/msr.c
@@ -253,9 +253,14 @@ typedef struct {
 /* From AMD Architecture Programmer's Manual, Volume 2: System Programming, Appending A */
 static const msr_info AMD_MSRs[] = {
 	{ "MTRRCAP",			0x000000fe,	0x0000000000000fffULL, NULL },
+	/*
+	 * LP#1582005 - Do not check sysenter MSRs, they will be different on
+	 * each CPU, so checking them across CPUs is incorrect
+ 	 *
 	{ "SYSENTER_CS",		0x00000174,	0x000000000000ffffULL, NULL },
 	{ "SYSENTER_ESP",		0x00000175,	0xffffffffffffffffULL, NULL },
 	{ "SYSENTER_EIP",		0x00000176,	0xffffffffffffffffULL, NULL },
+	 */
 	{ "MCG_CAP",			0x00000179,	0x0000000001ff0fffULL, NULL },
 	{ "MCG_STATUS",			0x0000017a,	0xffffffffffffffffULL, NULL },
 	{ "MCG_CTL",			0x0000017b,	0xffffffffffffffffULL, NULL },
-- 
2.8.1




More information about the fwts-devel mailing list