[PATCH] cpu: msr: do not check SYSCALL MSRs

Alex Hung alex.hung at canonical.com
Fri Apr 6 05:38:40 UTC 2018


This includes IA32_STAR, IA32_LSTAR and IA32_FMASK registers

Signed-off-by: Alex Hung <alex.hung at canonical.com>
---
 src/cpu/msr/msr.c | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/src/cpu/msr/msr.c b/src/cpu/msr/msr.c
index 362bb33..70d8965 100644
--- a/src/cpu/msr/msr.c
+++ b/src/cpu/msr/msr.c
@@ -294,9 +294,12 @@ static const msr_info AMD_MSRs[] = {
 	{ "PAT",			0x00000277,	0x0707070707070703ULL, NULL },
 	{ "MTRR_DEF_TYPE",		0x000002ff,	0x0000000000000c0fULL, NULL },
 	{ "EFER",			0xc0000080,	0x0000000000000d01ULL, NULL },
+	/* Do not check syscall MSRs, they will be different on
+	 * each CPU, so checking them across CPUs is incorrect
 	{ "STAR",			0xc0000081,	0xffffffffffffffffULL, NULL },
 	{ "LSTAR",			0xc0000082,	0xffffffffffffffffULL, NULL },
 	{ "FMASK",			0xc0000084,	0xffffffffffffffffULL, NULL },
+	 */
 	//{ "FS_BASE",			0xc0000100,	0xffffffffffffffffULL, NULL },
 	//{ "GS_BASE",			0xc0000101, 	0xffffffffffffffffULL, NULL },
 	{ "KERNEL_GS_BASE",		0xc0000102, 	0xffffffffffffffffULL, NULL },
@@ -437,9 +440,12 @@ static const msr_info IA32_MSRs[] = {
 	{ "A_PMC6",			0x000004c7,	0xffffffffffffffffULL, NULL },
 	{ "A_PMC7",			0x000004c8,	0xffffffffffffffffULL, NULL },
 	{ "EFER",			0xc0000080,	0x0000000000000d01ULL, NULL },
+	/* Do not check syscall MSRs, they will be different on
+	 * each CPU, so checking them across CPUs is incorrect
 	{ "STAR",			0xc0000081,	0xffffffffffffffffULL, NULL },
 	{ "LSTAR",			0xc0000082,	0xffffffffffffffffULL, NULL },
 	{ "FMASK",			0xc0000084,	0xffffffffffffffffULL, NULL },
+	 */
 	//{ "FS_BASE",			0xc0000100,	0xffffffffffffffffULL, NULL },
 	//{ "GS_BASE",			0xc0000101, 	0xffffffffffffffffULL, NULL },
 	{ "KERNEL_GS_BASE",		0xc0000102, 	0xffffffffffffffffULL, NULL },
-- 
2.7.4




More information about the fwts-devel mailing list