diff options
author | Aristeu Rozanski <aris@redhat.com> | 2014-06-02 14:15:29 -0400 |
---|---|---|
committer | Mauro Carvalho Chehab <m.chehab@samsung.com> | 2014-06-26 14:46:51 -0400 |
commit | adc61bcd91aa8364efaa496e7a4699950c362c21 (patch) | |
tree | 6732353201fac40ccfc0b019ecbc61844774af1a | |
parent | d7c660b7dcb2f3c3835432cf733fba03507f4387 (diff) |
sb_edac: remove bogus assumption on mc ordering
When a MC is handled, the correct sbridge_dev is searched based on the node,
checking again later with the assumption the first memory controller found is
the first socket's memory controller is a bogus assumption. Get rid of it.
Cc: Tony Luck <tony.luck@intel.com>
Signed-off-by: Aristeu Rozanski <aris@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
-rw-r--r-- | drivers/edac/sb_edac.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/drivers/edac/sb_edac.c b/drivers/edac/sb_edac.c index 4342926f1d32..24d8c7fea0bc 100644 --- a/drivers/edac/sb_edac.c +++ b/drivers/edac/sb_edac.c | |||
@@ -1810,10 +1810,6 @@ static int sbridge_mce_check_error(struct notifier_block *nb, unsigned long val, | |||
1810 | "%u APIC %x\n", mce->cpuvendor, mce->cpuid, | 1810 | "%u APIC %x\n", mce->cpuvendor, mce->cpuid, |
1811 | mce->time, mce->socketid, mce->apicid); | 1811 | mce->time, mce->socketid, mce->apicid); |
1812 | 1812 | ||
1813 | /* Only handle if it is the right mc controller */ | ||
1814 | if (cpu_data(mce->cpu).phys_proc_id != pvt->sbridge_dev->mc) | ||
1815 | return NOTIFY_DONE; | ||
1816 | |||
1817 | smp_rmb(); | 1813 | smp_rmb(); |
1818 | if ((pvt->mce_out + 1) % MCE_LOG_LEN == pvt->mce_in) { | 1814 | if ((pvt->mce_out + 1) % MCE_LOG_LEN == pvt->mce_in) { |
1819 | smp_wmb(); | 1815 | smp_wmb(); |