Jon,
I must NACK this patch to the lpfc driver and recommend that all other patches
which replace pci_find_capability(pdef, PCI_CAP_ID_EXP) with
"pci_is_pcie(pdev)" are NACK'd as well.
The reason is due to an issue on PPC platforms whereby use of "pdev->is_pcie"
and pci_is_pcie() will erroneously fail under some conditions, but explicit
search for the capability struct via pci_find_capability() is always
successful. I expect this to be due a shadowing of pci config space in the
hal/platform that isn't sufficiently built up. We detected this issue while
testing AER/EEH, and are functional only if the pci_find_capability() option
is used.
-- james s
On 6/27/2011 1:39 PM, Jon Mason wrote:
The PCIE capability offset is saved during PCI bus walking. It will
remove an unnecessary search in the PCI configuration space if this
value is referenced instead of reacquiring it. Also, pci_is_pcie is a
better way of determining if the device is PCIE or not (as it uses the
same saved PCIE capability offset).
Signed-off-by: Jon Mason<jdmason@xxxxxxxx>
---
drivers/scsi/lpfc/lpfc_init.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/scsi/lpfc/lpfc_init.c b/drivers/scsi/lpfc/lpfc_init.c
index 148b98d..9000ad0 100644
--- a/drivers/scsi/lpfc/lpfc_init.c
+++ b/drivers/scsi/lpfc/lpfc_init.c
@@ -3970,7 +3970,7 @@ lpfc_enable_pci_dev(struct lpfc_hba *phba)
pci_save_state(pdev);
/* PCIe EEH recovery on powerpc platforms needs fundamental reset */
- if (pci_find_capability(pdev, PCI_CAP_ID_EXP))
+ if (pci_is_pcie(pdev))
pdev->needs_freset = 1;
return 0;
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
