mirror of
https://github.com/edk2-porting/linux-next.git
synced 2024-12-25 05:34:00 +08:00
powerpc/eeh: Selectively enable IO for error log
According to the experiment I did, PCI config access is blocked on P7IOC frozen PE by hardware, but PHB3 doesn't do that. That means we always get 0xFF's while dumping PCI config space of the frozen PE on P7IOC. We don't have the problem on PHB3. So we have to enable I/O prioir to collecting error log. Otherwise, meaningless 0xFF's are always returned. The patch fixes it by EEH flag (EEH_ENABLE_IO_FOR_LOG), which is selectively set to indicate the case for: P7IOC on PowerNV platform, pSeries platform. Signed-off-by: Gavin Shan <gwshan@linux.vnet.ibm.com> Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
This commit is contained in:
parent
05b1721d9f
commit
dc561fb9e7
@ -34,10 +34,11 @@ struct device_node;
|
||||
#ifdef CONFIG_EEH
|
||||
|
||||
/* EEH subsystem flags */
|
||||
#define EEH_ENABLED 0x1 /* EEH enabled */
|
||||
#define EEH_FORCE_DISABLED 0x2 /* EEH disabled */
|
||||
#define EEH_PROBE_MODE_DEV 0x4 /* From PCI device */
|
||||
#define EEH_PROBE_MODE_DEVTREE 0x8 /* From device tree */
|
||||
#define EEH_ENABLED 0x01 /* EEH enabled */
|
||||
#define EEH_FORCE_DISABLED 0x02 /* EEH disabled */
|
||||
#define EEH_PROBE_MODE_DEV 0x04 /* From PCI device */
|
||||
#define EEH_PROBE_MODE_DEVTREE 0x08 /* From device tree */
|
||||
#define EEH_ENABLE_IO_FOR_LOG 0x10 /* Enable IO for log */
|
||||
|
||||
/*
|
||||
* Delay for PE reset, all in ms
|
||||
|
@ -252,7 +252,7 @@ void eeh_slot_error_detail(struct eeh_pe *pe, int severity)
|
||||
* 0xFF's is always returned from PCI config space.
|
||||
*/
|
||||
if (!(pe->type & EEH_PE_PHB)) {
|
||||
if (eeh_has_flag(EEH_PROBE_MODE_DEVTREE))
|
||||
if (eeh_has_flag(EEH_ENABLE_IO_FOR_LOG))
|
||||
eeh_pci_enable(pe, EEH_OPT_THAW_MMIO);
|
||||
eeh_ops->configure_bridge(pe);
|
||||
eeh_pe_restore_bars(pe);
|
||||
|
@ -45,6 +45,9 @@
|
||||
*/
|
||||
static int powernv_eeh_init(void)
|
||||
{
|
||||
struct pci_controller *hose;
|
||||
struct pnv_phb *phb;
|
||||
|
||||
/* We require OPALv3 */
|
||||
if (!firmware_has_feature(FW_FEATURE_OPALv3)) {
|
||||
pr_warning("%s: OPALv3 is required !\n", __func__);
|
||||
@ -54,6 +57,19 @@ static int powernv_eeh_init(void)
|
||||
/* Set probe mode */
|
||||
eeh_add_flag(EEH_PROBE_MODE_DEV);
|
||||
|
||||
/*
|
||||
* P7IOC blocks PCI config access to frozen PE, but PHB3
|
||||
* doesn't do that. So we have to selectively enable I/O
|
||||
* prior to collecting error log.
|
||||
*/
|
||||
list_for_each_entry(hose, &hose_list, list_node) {
|
||||
phb = hose->private_data;
|
||||
|
||||
if (phb->model == PNV_PHB_MODEL_P7IOC)
|
||||
eeh_add_flag(EEH_ENABLE_IO_FOR_LOG);
|
||||
break;
|
||||
}
|
||||
|
||||
return 0;
|
||||
}
|
||||
|
||||
|
@ -128,7 +128,7 @@ static int pseries_eeh_init(void)
|
||||
}
|
||||
|
||||
/* Set EEH probe mode */
|
||||
eeh_add_flag(EEH_PROBE_MODE_DEVTREE);
|
||||
eeh_add_flag(EEH_PROBE_MODE_DEVTREE | EEH_ENABLE_IO_FOR_LOG);
|
||||
|
||||
return 0;
|
||||
}
|
||||
|
Loading…
Reference in New Issue
Block a user