diff options
author | Chen, Gong <gong.chen@linux.intel.com> | 2014-06-11 04:34:51 -0400 |
---|---|---|
committer | Tony Luck <tony.luck@intel.com> | 2014-06-25 13:27:26 -0700 |
commit | 7c76bb5f7a3d052339b873374333dd0dcc35ce28 (patch) | |
tree | d37e9133b755a9765436c9a45f4d7dbd2f16826b /drivers | |
parent | d6cae935ec5b7873a8ccd8f0331bef2df729e86a (diff) | |
download | blackbird-op-linux-7c76bb5f7a3d052339b873374333dd0dcc35ce28.tar.gz blackbird-op-linux-7c76bb5f7a3d052339b873374333dd0dcc35ce28.zip |
RAS, extlog: Adjust init flow
Unless the platform has eMCA related capability, don't
need to check if there is conflict with EDAC driver.
Signed-off-by: Chen, Gong <gong.chen@linux.intel.com>
Acked-by: Borislav Petkov <bp@suse.de>
Signed-off-by: Tony Luck <tony.luck@intel.com>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/acpi/acpi_extlog.c | 13 |
1 files changed, 5 insertions, 8 deletions
diff --git a/drivers/acpi/acpi_extlog.c b/drivers/acpi/acpi_extlog.c index a99d4a6156dc..0ad6f389d922 100644 --- a/drivers/acpi/acpi_extlog.c +++ b/drivers/acpi/acpi_extlog.c @@ -223,19 +223,16 @@ static int __init extlog_init(void) u64 cap; int rc; + rdmsrl(MSR_IA32_MCG_CAP, cap); + + if (!(cap & MCG_ELOG_P) || !extlog_get_l1addr()) + return -ENODEV; + if (get_edac_report_status() == EDAC_REPORTING_FORCE) { pr_warn("Not loading eMCA, error reporting force-enabled through EDAC.\n"); return -EPERM; } - rc = -ENODEV; - rdmsrl(MSR_IA32_MCG_CAP, cap); - if (!(cap & MCG_ELOG_P)) - return rc; - - if (!extlog_get_l1addr()) - return rc; - rc = -EINVAL; /* get L1 header to fetch necessary information */ l1_hdr_size = sizeof(struct extlog_l1_head); |