Skip to content

Commit fcd343a

Browse files
skoralahbp3tk0v
authored andcommitted
x86/mce: Add support for Extended Physical Address MCA changes
Newer AMD CPUs support more physical address bits. That is, the MCA_ADDR registers on Scalable MCA systems contain the ErrorAddr in bits [56:0] instead of [55:0]. Hence, the existing LSB field from bits [61:56] in MCA_ADDR must be moved around to accommodate the larger ErrorAddr size. MCA_CONFIG[McaLsbInStatusSupported] indicates this change. If set, the LSB field will be found in MCA_STATUS rather than MCA_ADDR. Each logical CPU has unique MCA bank in hardware and is not shared with other logical CPUs. Additionally, on SMCA systems, each feature bit may be different for each bank within same logical CPU. Check for MCA_CONFIG[McaLsbInStatusSupported] for each MCA bank and for each CPU. Additionally, all MCA banks do not support maximum ErrorAddr bits in MCA_ADDR. Some banks might support fewer bits but the remaining bits are marked as reserved. [ Yazen: Rebased and fixed up formatting. bp: Massage comments. ] Signed-off-by: Smita Koralahalli <[email protected]> Signed-off-by: Yazen Ghannam <[email protected]> Signed-off-by: Borislav Petkov (AMD) <[email protected]> Link: https://lore.kernel.org/r/[email protected]
1 parent 2117654 commit fcd343a

File tree

3 files changed

+33
-8
lines changed

3 files changed

+33
-8
lines changed

arch/x86/kernel/cpu/mce/amd.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -306,6 +306,8 @@ static void smca_configure(unsigned int bank, unsigned int cpu)
306306
if ((low & BIT(5)) && !((high >> 5) & 0x3))
307307
high |= BIT(5);
308308

309+
this_cpu_ptr(mce_banks_array)[bank].lsb_in_status = !!(low & BIT(8));
310+
309311
wrmsr(smca_config, low, high);
310312
}
311313

arch/x86/kernel/cpu/mce/core.c

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -67,13 +67,7 @@ DEFINE_PER_CPU(unsigned, mce_exception_count);
6767

6868
DEFINE_PER_CPU_READ_MOSTLY(unsigned int, mce_num_banks);
6969

70-
struct mce_bank {
71-
u64 ctl; /* subevents to enable */
72-
73-
__u64 init : 1, /* initialise bank? */
74-
__reserved_1 : 63;
75-
};
76-
static DEFINE_PER_CPU_READ_MOSTLY(struct mce_bank[MAX_NR_BANKS], mce_banks_array);
70+
DEFINE_PER_CPU_READ_MOSTLY(struct mce_bank[MAX_NR_BANKS], mce_banks_array);
7771

7872
#define ATTR_LEN 16
7973
/* One object for each MCE bank, shared by all CPUs */

arch/x86/kernel/cpu/mce/internal.h

Lines changed: 30 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -177,6 +177,24 @@ struct mce_vendor_flags {
177177

178178
extern struct mce_vendor_flags mce_flags;
179179

180+
struct mce_bank {
181+
/* subevents to enable */
182+
u64 ctl;
183+
184+
/* initialise bank? */
185+
__u64 init : 1,
186+
187+
/*
188+
* (AMD) MCA_CONFIG[McaLsbInStatusSupported]: When set, this bit indicates
189+
* the LSB field is found in MCA_STATUS and not in MCA_ADDR.
190+
*/
191+
lsb_in_status : 1,
192+
193+
__reserved_1 : 62;
194+
};
195+
196+
DECLARE_PER_CPU_READ_MOSTLY(struct mce_bank[MAX_NR_BANKS], mce_banks_array);
197+
180198
enum mca_msr {
181199
MCA_CTL,
182200
MCA_STATUS,
@@ -190,14 +208,25 @@ extern bool filter_mce(struct mce *m);
190208
#ifdef CONFIG_X86_MCE_AMD
191209
extern bool amd_filter_mce(struct mce *m);
192210

193-
/* Extract [55:<lsb>] where lsb is the LS-*valid* bit of the address bits. */
211+
/*
212+
* If MCA_CONFIG[McaLsbInStatusSupported] is set, extract ErrAddr in bits
213+
* [56:0] of MCA_STATUS, else in bits [55:0] of MCA_ADDR.
214+
*/
194215
static __always_inline void smca_extract_err_addr(struct mce *m)
195216
{
196217
u8 lsb;
197218

198219
if (!mce_flags.smca)
199220
return;
200221

222+
if (this_cpu_ptr(mce_banks_array)[m->bank].lsb_in_status) {
223+
lsb = (m->status >> 24) & 0x3f;
224+
225+
m->addr &= GENMASK_ULL(56, lsb);
226+
227+
return;
228+
}
229+
201230
lsb = (m->addr >> 56) & 0x3f;
202231

203232
m->addr &= GENMASK_ULL(55, lsb);

0 commit comments

Comments
 (0)