MCE, AMD: Drop too granulary family model checks (ec3e82d6) · Commits · 戴 / test

MCA details seldom change inbetween the models of a family so don't be too conservative and enable decoding on everything starting from K8 onwards. Minor adjustments can come in later but most importantly, we have some decoding infrastructure in place for upcoming models by default. Signed-off-by:

Borislav Petkov <borislav.petkov@amd.com>

drivers/edac/mce_amd.c

+2 −4

Original line number	Diff line number	Diff line
		@@ -754,9 +754,7 @@ static int __init mce_amd_init(void)
		if (c->x86_vendor != X86_VENDOR_AMD)
		return 0;

		if ((c->x86 < 0xf \|\| c->x86 > 0x12) &&
		(c->x86 != 0x14 \|\| c->x86_model > 0xf) &&
		(c->x86 != 0x15 \|\| c->x86_model > 0xf))
		if (c->x86 < 0xf \|\| c->x86 > 0x15)
		return 0;

		fam_ops = kzalloc(sizeof(struct amd_decoder_ops), GFP_KERNEL);
		@@ -797,7 +795,7 @@ static int __init mce_amd_init(void)
		break;

		default:
		printk(KERN_WARNING "Huh? What family is that: %d?!\n", c->x86);
		printk(KERN_WARNING "Huh? What family is it: 0x%x?!\n", c->x86);
		kfree(fam_ops);
		return -EINVAL;
		}

Admin message