| Message ID | 1431668786-30371-6-git-send-email-weiyang@linux.vnet.ibm.com (mailing list archive) |
|---|---|
| State | New, archived |
| Delegated to: | Bjorn Helgaas |
| Headers | show |
On Fri, May 15, 2015 at 01:46:20PM +0800, Wei Yang wrote: >EEH on powerpc platform needs eeh_dev structure to track the PCI device >status. Since VFs are created/released dynamically, VF's eeh_dev is also >dynamically created/released in system. > >This patch creates/removes eeh_dev when pci_dn is created/removed for VFs, >and marks it with EEH_DEV_VF type. > >Signed-off-by: Wei Yang <weiyang@linux.vnet.ibm.com> Acked-by: Gavin Shan <gwshan@linux.vnet.ibm.com> After removing the unnecessary line of code as below. >--- > arch/powerpc/include/asm/eeh.h | 1 + > arch/powerpc/kernel/eeh.c | 4 ++++ > arch/powerpc/kernel/pci_dn.c | 11 +++++++++++ > 3 files changed, 16 insertions(+) > >diff --git a/arch/powerpc/include/asm/eeh.h b/arch/powerpc/include/asm/eeh.h >index a52db28..1b3614d 100644 >--- a/arch/powerpc/include/asm/eeh.h >+++ b/arch/powerpc/include/asm/eeh.h >@@ -138,6 +138,7 @@ struct eeh_dev { > struct pci_controller *phb; /* Associated PHB */ > struct pci_dn *pdn; /* Associated PCI device node */ > struct pci_dev *pdev; /* Associated PCI device */ >+ struct pci_dev *physfn; /* Associated PF PORT */ > struct pci_bus *bus; /* PCI bus for partial hotplug */ > }; > >diff --git a/arch/powerpc/kernel/eeh.c b/arch/powerpc/kernel/eeh.c >index 6c7ce1b..221e280 100644 >--- a/arch/powerpc/kernel/eeh.c >+++ b/arch/powerpc/kernel/eeh.c >@@ -1135,6 +1135,10 @@ void eeh_add_device_late(struct pci_dev *dev) > } > > edev->pdev = dev; >+#ifdef CONFIG_PCI_IOV >+ if (dev->is_virtfn) >+ edev->physfn = dev->physfn; >+#endif > dev->dev.archdata.edev = edev; > > if (eeh_has_flag(EEH_PROBE_MODE_DEV)) >diff --git a/arch/powerpc/kernel/pci_dn.c b/arch/powerpc/kernel/pci_dn.c >index f771130..94806a4 100644 >--- a/arch/powerpc/kernel/pci_dn.c >+++ b/arch/powerpc/kernel/pci_dn.c >@@ -180,7 +180,9 @@ static struct pci_dn *add_one_dev_pci_data(struct pci_dn *parent, > struct pci_dn *add_dev_pci_data(struct pci_dev *pdev) > { > #ifdef CONFIG_PCI_IOV >+ struct pci_controller *hose = pci_bus_to_host(pdev->bus); > struct pci_dn *parent, *pdn; >+ struct eeh_dev *edev; > int i; > > /* Only support IOV for now */ >@@ -206,6 +208,8 @@ struct pci_dn *add_dev_pci_data(struct pci_dev *pdev) > __func__, i); > return NULL; > } >+ eeh_dev_init(pdn, hose); >+ edev = pdn_to_eeh_dev(pdn); Nothing is done to edev after getting it. So I think the last line of changes here isn't needed. Could you check and remove it if I'm correct? Thanks, Gavin > } > #endif /* CONFIG_PCI_IOV */ > >@@ -254,10 +258,17 @@ void remove_dev_pci_data(struct pci_dev *pdev) > for (i = 0; i < pci_sriov_get_totalvfs(pdev); i++) { > list_for_each_entry_safe(pdn, tmp, > &parent->child_list, list) { >+ struct eeh_dev *edev; > if (pdn->busno != pci_iov_virtfn_bus(pdev, i) || > pdn->devfn != pci_iov_virtfn_devfn(pdev, i)) > continue; > >+ edev = pdn_to_eeh_dev(pdn); >+ if (edev) { >+ pdn->edev = NULL; >+ kfree(edev); >+ } >+ > if (!list_empty(&pdn->list)) > list_del(&pdn->list); > >-- >1.7.9.5 > -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Fri, May 15, 2015 at 04:19:16PM +1000, Gavin Shan wrote: >On Fri, May 15, 2015 at 01:46:20PM +0800, Wei Yang wrote: >>EEH on powerpc platform needs eeh_dev structure to track the PCI device >>status. Since VFs are created/released dynamically, VF's eeh_dev is also >>dynamically created/released in system. >> >>This patch creates/removes eeh_dev when pci_dn is created/removed for VFs, >>and marks it with EEH_DEV_VF type. >> >>Signed-off-by: Wei Yang <weiyang@linux.vnet.ibm.com> > >Acked-by: Gavin Shan <gwshan@linux.vnet.ibm.com> > >After removing the unnecessary line of code as below. > >>--- >> arch/powerpc/include/asm/eeh.h | 1 + >> arch/powerpc/kernel/eeh.c | 4 ++++ >> arch/powerpc/kernel/pci_dn.c | 11 +++++++++++ >> 3 files changed, 16 insertions(+) >> >>diff --git a/arch/powerpc/include/asm/eeh.h b/arch/powerpc/include/asm/eeh.h >>index a52db28..1b3614d 100644 >>--- a/arch/powerpc/include/asm/eeh.h >>+++ b/arch/powerpc/include/asm/eeh.h >>@@ -138,6 +138,7 @@ struct eeh_dev { >> struct pci_controller *phb; /* Associated PHB */ >> struct pci_dn *pdn; /* Associated PCI device node */ >> struct pci_dev *pdev; /* Associated PCI device */ >>+ struct pci_dev *physfn; /* Associated PF PORT */ >> struct pci_bus *bus; /* PCI bus for partial hotplug */ >> }; >> >>diff --git a/arch/powerpc/kernel/eeh.c b/arch/powerpc/kernel/eeh.c >>index 6c7ce1b..221e280 100644 >>--- a/arch/powerpc/kernel/eeh.c >>+++ b/arch/powerpc/kernel/eeh.c >>@@ -1135,6 +1135,10 @@ void eeh_add_device_late(struct pci_dev *dev) >> } >> >> edev->pdev = dev; >>+#ifdef CONFIG_PCI_IOV >>+ if (dev->is_virtfn) >>+ edev->physfn = dev->physfn; >>+#endif >> dev->dev.archdata.edev = edev; >> >> if (eeh_has_flag(EEH_PROBE_MODE_DEV)) >>diff --git a/arch/powerpc/kernel/pci_dn.c b/arch/powerpc/kernel/pci_dn.c >>index f771130..94806a4 100644 >>--- a/arch/powerpc/kernel/pci_dn.c >>+++ b/arch/powerpc/kernel/pci_dn.c >>@@ -180,7 +180,9 @@ static struct pci_dn *add_one_dev_pci_data(struct pci_dn *parent, >> struct pci_dn *add_dev_pci_data(struct pci_dev *pdev) >> { >> #ifdef CONFIG_PCI_IOV >>+ struct pci_controller *hose = pci_bus_to_host(pdev->bus); >> struct pci_dn *parent, *pdn; >>+ struct eeh_dev *edev; >> int i; >> >> /* Only support IOV for now */ >>@@ -206,6 +208,8 @@ struct pci_dn *add_dev_pci_data(struct pci_dev *pdev) >> __func__, i); >> return NULL; >> } >>+ eeh_dev_init(pdn, hose); >>+ edev = pdn_to_eeh_dev(pdn); > >Nothing is done to edev after getting it. So I think the last line of changes >here isn't needed. Could you check and remove it if I'm correct? You are right, removed. > >Thanks, >Gavin > >> } >> #endif /* CONFIG_PCI_IOV */ >> >>@@ -254,10 +258,17 @@ void remove_dev_pci_data(struct pci_dev *pdev) >> for (i = 0; i < pci_sriov_get_totalvfs(pdev); i++) { >> list_for_each_entry_safe(pdn, tmp, >> &parent->child_list, list) { >>+ struct eeh_dev *edev; >> if (pdn->busno != pci_iov_virtfn_bus(pdev, i) || >> pdn->devfn != pci_iov_virtfn_devfn(pdev, i)) >> continue; >> >>+ edev = pdn_to_eeh_dev(pdn); >>+ if (edev) { >>+ pdn->edev = NULL; >>+ kfree(edev); >>+ } >>+ >> if (!list_empty(&pdn->list)) >> list_del(&pdn->list); >> >>-- >>1.7.9.5 >>
diff --git a/arch/powerpc/include/asm/eeh.h b/arch/powerpc/include/asm/eeh.h index a52db28..1b3614d 100644 --- a/arch/powerpc/include/asm/eeh.h +++ b/arch/powerpc/include/asm/eeh.h @@ -138,6 +138,7 @@ struct eeh_dev { struct pci_controller *phb; /* Associated PHB */ struct pci_dn *pdn; /* Associated PCI device node */ struct pci_dev *pdev; /* Associated PCI device */ + struct pci_dev *physfn; /* Associated PF PORT */ struct pci_bus *bus; /* PCI bus for partial hotplug */ }; diff --git a/arch/powerpc/kernel/eeh.c b/arch/powerpc/kernel/eeh.c index 6c7ce1b..221e280 100644 --- a/arch/powerpc/kernel/eeh.c +++ b/arch/powerpc/kernel/eeh.c @@ -1135,6 +1135,10 @@ void eeh_add_device_late(struct pci_dev *dev) } edev->pdev = dev; +#ifdef CONFIG_PCI_IOV + if (dev->is_virtfn) + edev->physfn = dev->physfn; +#endif dev->dev.archdata.edev = edev; if (eeh_has_flag(EEH_PROBE_MODE_DEV)) diff --git a/arch/powerpc/kernel/pci_dn.c b/arch/powerpc/kernel/pci_dn.c index f771130..94806a4 100644 --- a/arch/powerpc/kernel/pci_dn.c +++ b/arch/powerpc/kernel/pci_dn.c @@ -180,7 +180,9 @@ static struct pci_dn *add_one_dev_pci_data(struct pci_dn *parent, struct pci_dn *add_dev_pci_data(struct pci_dev *pdev) { #ifdef CONFIG_PCI_IOV + struct pci_controller *hose = pci_bus_to_host(pdev->bus); struct pci_dn *parent, *pdn; + struct eeh_dev *edev; int i; /* Only support IOV for now */ @@ -206,6 +208,8 @@ struct pci_dn *add_dev_pci_data(struct pci_dev *pdev) __func__, i); return NULL; } + eeh_dev_init(pdn, hose); + edev = pdn_to_eeh_dev(pdn); } #endif /* CONFIG_PCI_IOV */ @@ -254,10 +258,17 @@ void remove_dev_pci_data(struct pci_dev *pdev) for (i = 0; i < pci_sriov_get_totalvfs(pdev); i++) { list_for_each_entry_safe(pdn, tmp, &parent->child_list, list) { + struct eeh_dev *edev; if (pdn->busno != pci_iov_virtfn_bus(pdev, i) || pdn->devfn != pci_iov_virtfn_devfn(pdev, i)) continue; + edev = pdn_to_eeh_dev(pdn); + if (edev) { + pdn->edev = NULL; + kfree(edev); + } + if (!list_empty(&pdn->list)) list_del(&pdn->list);
EEH on powerpc platform needs eeh_dev structure to track the PCI device status. Since VFs are created/released dynamically, VF's eeh_dev is also dynamically created/released in system. This patch creates/removes eeh_dev when pci_dn is created/removed for VFs, and marks it with EEH_DEV_VF type. Signed-off-by: Wei Yang <weiyang@linux.vnet.ibm.com> --- arch/powerpc/include/asm/eeh.h | 1 + arch/powerpc/kernel/eeh.c | 4 ++++ arch/powerpc/kernel/pci_dn.c | 11 +++++++++++ 3 files changed, 16 insertions(+)