Source-Changes-HG archive
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
[src/trunk]: src/sys/arch/x86 Implement a mitigation for SpectreV4 on AMD fam...
details: https://anonhg.NetBSD.org/src/rev/1dd79e687bc7
branches: trunk
changeset: 362000:1dd79e687bc7
user: maxv <maxv%NetBSD.org@localhost>
date: Tue May 22 10:20:04 2018 +0000
description:
Implement a mitigation for SpectreV4 on AMD families 15h and 16h. We use
a non-architectural MSR. This MSR is also available on 17h, but there SMT
is involved, and it needs more investigation.
Not tested (I have only 10h).
diffstat:
sys/arch/x86/include/specialreg.h | 4 +-
sys/arch/x86/x86/spectre.c | 67 +++++++++++++++++++++++++++++++-------
2 files changed, 57 insertions(+), 14 deletions(-)
diffs (141 lines):
diff -r 6883461b0694 -r 1dd79e687bc7 sys/arch/x86/include/specialreg.h
--- a/sys/arch/x86/include/specialreg.h Tue May 22 09:25:58 2018 +0000
+++ b/sys/arch/x86/include/specialreg.h Tue May 22 10:20:04 2018 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: specialreg.h,v 1.122 2018/05/22 07:24:08 maxv Exp $ */
+/* $NetBSD: specialreg.h,v 1.123 2018/05/22 10:20:04 maxv Exp $ */
/*-
* Copyright (c) 1991 The Regents of the University of California.
@@ -858,6 +858,8 @@
#define MSR_LS_CFG 0xc0011020
#define LS_CFG_DIS_LS2_SQUISH 0x02000000
+#define LS_CFG_DIS_SSB_F15H 0x0040000000000000ULL
+#define LS_CFG_DIS_SSB_F16H 0x0000000200000000ULL
#define MSR_IC_CFG 0xc0011021
#define IC_CFG_DIS_SEQ_PREFETCH 0x00000800
diff -r 6883461b0694 -r 1dd79e687bc7 sys/arch/x86/x86/spectre.c
--- a/sys/arch/x86/x86/spectre.c Tue May 22 09:25:58 2018 +0000
+++ b/sys/arch/x86/x86/spectre.c Tue May 22 10:20:04 2018 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: spectre.c,v 1.14 2018/05/22 09:25:58 maxv Exp $ */
+/* $NetBSD: spectre.c,v 1.15 2018/05/22 10:20:04 maxv Exp $ */
/*
* Copyright (c) 2018 NetBSD Foundation, Inc.
@@ -34,7 +34,7 @@
*/
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: spectre.c,v 1.14 2018/05/22 09:25:58 maxv Exp $");
+__KERNEL_RCSID(0, "$NetBSD: spectre.c,v 1.15 2018/05/22 10:20:04 maxv Exp $");
#include "opt_spectre.h"
@@ -60,7 +60,9 @@
enum v4_mitigation {
V4_MITIGATION_NONE,
V4_MITIGATION_INTEL_SSBD,
- V4_MITIGATION_INTEL_SSB_NO
+ V4_MITIGATION_INTEL_SSB_NO,
+ V4_MITIGATION_AMD_NONARCH_F15H,
+ V4_MITIGATION_AMD_NONARCH_F16H
};
static enum v2_mitigation v2_mitigation_method = V2_MITIGATION_NONE;
@@ -379,14 +381,18 @@
strlcat(name, "(none)", sizeof(name));
} else {
switch (v4_mitigation_method) {
+ case V4_MITIGATION_NONE:
+ panic("%s: impossible", __func__);
case V4_MITIGATION_INTEL_SSBD:
strlcat(name, "[Intel SSBD]", sizeof(name));
break;
case V4_MITIGATION_INTEL_SSB_NO:
strlcat(name, "[Intel SSB_NO]", sizeof(name));
break;
- default:
- panic("%s: impossible", __func__);
+ case V4_MITIGATION_AMD_NONARCH_F15H:
+ case V4_MITIGATION_AMD_NONARCH_F16H:
+ strlcat(name, "[AMD NONARCH]", sizeof(name));
+ break;
}
}
@@ -397,6 +403,7 @@
static void
v4_detect_method(void)
{
+ struct cpu_info *ci = curcpu();
u_int descs[4];
uint64_t msr;
@@ -421,6 +428,17 @@
return;
}
}
+ } else if (cpu_vendor == CPUVENDOR_AMD) {
+ switch (CPUID_TO_FAMILY(ci->ci_signature)) {
+ case 0x15:
+ v4_mitigation_method = V4_MITIGATION_AMD_NONARCH_F15H;
+ return;
+ case 0x16:
+ v4_mitigation_method = V4_MITIGATION_AMD_NONARCH_F16H;
+ return;
+ default:
+ break;
+ }
}
v4_mitigation_method = V4_MITIGATION_NONE;
@@ -431,15 +449,38 @@
{
uint64_t msr;
- msr = rdmsr(MSR_IA32_SPEC_CTRL);
-
- if (enabled) {
- msr |= IA32_SPEC_CTRL_SSBD;
- } else {
- msr &= ~IA32_SPEC_CTRL_SSBD;
+ switch (v4_mitigation_method) {
+ case V4_MITIGATION_NONE:
+ case V4_MITIGATION_INTEL_SSB_NO:
+ panic("impossible");
+ case V4_MITIGATION_INTEL_SSBD:
+ msr = rdmsr(MSR_IA32_SPEC_CTRL);
+ if (enabled) {
+ msr |= IA32_SPEC_CTRL_SSBD;
+ } else {
+ msr &= ~IA32_SPEC_CTRL_SSBD;
+ }
+ wrmsr(MSR_IA32_SPEC_CTRL, msr);
+ break;
+ case V4_MITIGATION_AMD_NONARCH_F15H:
+ msr = rdmsr(MSR_LS_CFG);
+ if (enabled) {
+ msr |= LS_CFG_DIS_SSB_F15H;
+ } else {
+ msr &= ~LS_CFG_DIS_SSB_F15H;
+ }
+ wrmsr(MSR_LS_CFG, msr);
+ break;
+ case V4_MITIGATION_AMD_NONARCH_F16H:
+ msr = rdmsr(MSR_LS_CFG);
+ if (enabled) {
+ msr |= LS_CFG_DIS_SSB_F16H;
+ } else {
+ msr &= ~LS_CFG_DIS_SSB_F16H;
+ }
+ wrmsr(MSR_LS_CFG, msr);
+ break;
}
-
- wrmsr(MSR_IA32_SPEC_CTRL, msr);
}
static void
Home |
Main Index |
Thread Index |
Old Index