Skip to content

Commit

Permalink
MCE, AMD: Drop too granulary family model checks
Browse files Browse the repository at this point in the history
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>
  • Loading branch information
Borislav Petkov committed Apr 4, 2012
1 parent dd775ae commit ec3e82d
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions drivers/edac/mce_amd.c
Original file line number Diff line number Diff line change
Expand Up @@ -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);
Expand Down Expand Up @@ -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;
}
Expand Down

0 comments on commit ec3e82d

Please sign in to comment.