Commit 6d5db466 authored by Borislav Petkov's avatar Borislav Petkov
Browse files

EDAC, MCE: Fix NB error formatting 



Minor formatting fixup since the information which core was associated
with the MCE is not always valid.

Signed-off-by: default avatarBorislav Petkov <borislav.petkov@amd.com>
parent 50adbbd8

drivers/edac/mce_amd.c

+10 −7
Original line number Diff line number Diff line
@@ -597,24 +597,27 @@ void amd_decode_nb_mce(int node_id, struct mce *m, u32 nbcfg) 
	u16 ec   = EC(m->status);

	u8 xec   = XEC(m->status, 0x1f);

	u32 nbsh = (u32)(m->status >> 32);

	int core = -1;



	pr_emerg(HW_ERR "Northbridge Error, node %d: ", node_id);

	pr_emerg(HW_ERR "Northbridge Error (node %d", node_id);



	/*

	 * F10h, revD can disable ErrCpu[3:0] so check that first and also the

	 * value encoding has changed so interpret those differently

	 */

	/* F10h, revD can disable ErrCpu[3:0] through ErrCpuVal */

	if ((boot_cpu_data.x86 == 0x10) &&

	    (boot_cpu_data.x86_model > 7)) {

		if (nbsh & K8_NBSH_ERR_CPU_VAL)

			pr_cont(", core: %u", (u8)(nbsh & nb_err_cpumask));

			core = nbsh & nb_err_cpumask;

	} else {

		u8 assoc_cpus = nbsh & nb_err_cpumask;



		if (assoc_cpus > 0)

			pr_cont(", core: %d", fls(assoc_cpus) - 1);

			core = fls(assoc_cpus) - 1;

	}



	if (core >= 0)

		pr_cont(", core %d): ", core);

	else

		pr_cont("): ");



	switch (xec) {

	case 0x2:

		pr_cont("Sync error (sync packets on HT link detected).\n");