summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorGavin Shan <shangw@linux.vnet.ibm.com>2012-11-22 21:58:26 +0000
committerBenjamin Herrenschmidt <benh@kernel.crashing.org>2012-11-26 09:14:16 +1100
commite716e014384688d1a50d1aa5213ee74748c6d4e0 (patch)
treef619f557bc95ce22c6b787a408b2acc77f68d01f
parentbb4618823a3389086f157fafb2a0d97cade8d89f (diff)
downloadop-kernel-dev-e716e014384688d1a50d1aa5213ee74748c6d4e0.zip
op-kernel-dev-e716e014384688d1a50d1aa5213ee74748c6d4e0.tar.gz
powerpc/eeh: Do not invalidate PE properly
While the EEH does recovery on the specific PE that has PCI errors, the PCI devices belonging to the PE will be removed and the PE will be marked as invalid since we still need the information stored in the PE. We only invalidate the PE when it doesn't have associated EEH devices and valid child PEs. However, the code used to check that is wrong. The patch fixes that. Signed-off-by: Gavin Shan <shangw@linux.vnet.ibm.com> Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
-rw-r--r--arch/powerpc/platforms/pseries/eeh_pe.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/powerpc/platforms/pseries/eeh_pe.c b/arch/powerpc/platforms/pseries/eeh_pe.c
index 797cd18..d16c8de 100644
--- a/arch/powerpc/platforms/pseries/eeh_pe.c
+++ b/arch/powerpc/platforms/pseries/eeh_pe.c
@@ -449,7 +449,7 @@ int eeh_rmv_from_parent_pe(struct eeh_dev *edev, int purge_pe)
if (list_empty(&pe->edevs)) {
cnt = 0;
list_for_each_entry(child, &pe->child_list, child) {
- if (!(pe->type & EEH_PE_INVALID)) {
+ if (!(child->type & EEH_PE_INVALID)) {
cnt++;
break;
}
OpenPOWER on IntegriCloud