Skip to content

Commit

Permalink
EDAC, sb_edac: Modify H/W event reporting policy
Browse files Browse the repository at this point in the history
Newer Intel platforms support more than one method to report H/W event.
On this kind of platform, H/W event report can adopt new method and
traditional EDAC method should be disabled. Moreover, if EDAC event
report method is set to *force*, it means event must be reported via
EDAC interface. IOW, it overrides the default event report policy.

Signed-off-by: Chen, Gong <gong.chen@linux.intel.com>
Acked-by: Tony Luck <tony.luck@intel.com>
Link: http://lkml.kernel.org/r/1386310630-12529-3-git-send-email-gong.chen@linux.intel.com
[ Boris: massage commit and error messages ]
Signed-off-by: Borislav Petkov <bp@suse.de>
  • Loading branch information
Chen, Gong authored and Borislav Petkov committed Dec 11, 2013
1 parent c700f01 commit fd52103
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion drivers/edac/sb_edac.c
Original file line number Diff line number Diff line change
Expand Up @@ -1829,6 +1829,9 @@ static int sbridge_mce_check_error(struct notifier_block *nb, unsigned long val,
struct mem_ctl_info *mci;
struct sbridge_pvt *pvt;

if (get_edac_report_status() == EDAC_REPORTING_DISABLED)
return NOTIFY_DONE;

mci = get_mci_for_node_id(mce->socketid);
if (!mci)
return NOTIFY_BAD;
Expand Down Expand Up @@ -2142,9 +2145,10 @@ static int __init sbridge_init(void)
opstate_init();

pci_rc = pci_register_driver(&sbridge_driver);

if (pci_rc >= 0) {
mce_register_decode_chain(&sbridge_mce_dec);
if (get_edac_report_status() == EDAC_REPORTING_DISABLED)
sbridge_printk(KERN_WARNING, "Loading driver, error reporting disabled.\n");
return 0;
}

Expand Down

0 comments on commit fd52103

Please sign in to comment.