x86/spec-ctrl: Enable Zen2 chickenbit

... as instructed in the Branch Type Confusion whitepaper.

This is part of XSA-407.

Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
(cherry picked from commit 9deaf2d932)
This commit is contained in:
Andrew Cooper committed 2022-07-12 16:33:19 +01:00
1 parent 3c71016e68
commit 3feba68e15
4 files changed
+36

No files matched your search

+28
View File
@@ -590,6 +590,31 @@ void amd_init_ssbd(const struct cpuinfo_x86 *c)
printk_once(XENLOG_ERR "No SSBD controls available\n");
}
/*
* On Zen2 we offer this chicken (bit) on the altar of Speculation.
*
* Refer to the AMD Branch Type Confusion whitepaper:
* https://XXX
*
* Setting this unnamed bit supposedly causes prediction information on
* non-branch instructions to be ignored. It is to be set unilaterally in
* newer microcode.
*
* This chickenbit is something unrelated on Zen1, and Zen1 vs Zen2 isn't a
* simple model number comparison, so use STIBP as a heuristic to separate the
* two uarches in Fam17h(AMD)/18h(Hygon).
*/
void amd_init_spectral_chicken(void)
{
uint64_t val, chickenbit = 1 << 1;
if (cpu_has_hypervisor || !boot_cpu_has(X86_FEATURE_AMD_STIBP))
return;
if (rdmsr_safe(MSR_AMD64_DE_CFG2, val) == 0 && !(val & chickenbit))
wrmsr_safe(MSR_AMD64_DE_CFG2, val | chickenbit);
}
static void init_amd(struct cpuinfo_x86 *c)
{
u32 l, h;
@@ -668,6 +693,9 @@ static void init_amd(struct cpuinfo_x86 *c)
amd_init_ssbd(c);
if (c->x86 == 0x17)
amd_init_spectral_chicken();
/* MFENCE stops RDTSC speculation */
if (!cpu_has_lfence_dispatch)
__set_bit(X86_FEATURE_MFENCE_RDTSC, c->x86_capability);
+1
View File
@@ -20,3 +20,4 @@ extern bool detect_extended_topology(struct cpuinfo_x86 *c);
void early_init_amd(struct cpuinfo_x86 *c);
void amd_init_ssbd(const struct cpuinfo_x86 *c);
void amd_init_spectral_chicken(void);
+6
View File
@@ -61,6 +61,12 @@ static void init_hygon(struct cpuinfo_x86 *c)
amd_init_ssbd(c);
/*
* TODO: Check heuristic safety with Hygon first
if (c->x86 == 0x18)
amd_init_spectral_chicken();
*/
/* MFENCE stops RDTSC speculation */
if (!cpu_has_lfence_dispatch)
__set_bit(X86_FEATURE_MFENCE_RDTSC, c->x86_capability);
+1
View File
@@ -311,6 +311,7 @@
#define MSR_AMD64_DC_CFG 0xc0011022
#define MSR_AMD64_DE_CFG 0xc0011029
#define AMD64_DE_CFG_LFENCE_SERIALISE (_AC(1, ULL) << 1)
#define MSR_AMD64_DE_CFG2 0xc00110e3
#define MSR_AMD64_DR0_ADDRESS_MASK 0xc0011027
#define MSR_AMD64_DR1_ADDRESS_MASK 0xc0011019
Reference in new issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.