aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAristeu Rozanski <aris@redhat.com>2014-06-02 14:15:29 -0400
committerMauro Carvalho Chehab <m.chehab@samsung.com>2014-06-26 14:46:51 -0400
commitadc61bcd91aa8364efaa496e7a4699950c362c21 (patch)
tree6732353201fac40ccfc0b019ecbc61844774af1a
parentd7c660b7dcb2f3c3835432cf733fba03507f4387 (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.c4
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();