summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorBorislav Petkov <bp@suse.de>2015-06-10 16:17:13 +0200
committerBorislav Petkov <bp@suse.de>2015-06-24 18:17:37 +0200
commit6d1e9bf5b04887a2220fbb44be074e9e8376e332 (patch)
treecbf5c74f528ed51f177f91d9f1a78f879d259a2e
parentf2f3dca1b7a66cbc91f45b59b8606bb325fa869f (diff)
downloadop-kernel-dev-6d1e9bf5b04887a2220fbb44be074e9e8376e332.zip
op-kernel-dev-6d1e9bf5b04887a2220fbb44be074e9e8376e332.tar.gz
EDAC, mce_amd_inj: Move bit preparations before the injection
We do get_online_cpus() and then start noodling with the bits. Do that *before* we grab the hotplug lock. Signed-off-by: Borislav Petkov <bp@suse.de>
-rw-r--r--drivers/edac/mce_amd_inj.c8
1 files changed, 4 insertions, 4 deletions
diff --git a/drivers/edac/mce_amd_inj.c b/drivers/edac/mce_amd_inj.c
index bfb0f35..b92986b 100644
--- a/drivers/edac/mce_amd_inj.c
+++ b/drivers/edac/mce_amd_inj.c
@@ -196,16 +196,16 @@ static void do_inject(void)
return;
}
- get_online_cpus();
- if (!cpu_online(cpu))
- goto err;
-
/* prep MCE global settings for the injection */
mcg_status = MCG_STATUS_MCIP | MCG_STATUS_EIPV;
if (!(i_mce.status & MCI_STATUS_PCC))
mcg_status |= MCG_STATUS_RIPV;
+ get_online_cpus();
+ if (!cpu_online(cpu))
+ goto err;
+
toggle_hw_mce_inject(cpu, true);
wrmsr_on_cpu(cpu, MSR_IA32_MCG_STATUS,
OpenPOWER on IntegriCloud