Because the bus notifier calls eeh_rmv_from_parent_pe() (via
eeh_remove_device()) when a VF is removed, the call in
remove_sriov_vf_pdns() is redundant.

So remove the call.

Signed-off-by: Sam Bobroff <sbobr...@linux.ibm.com>
---
 arch/powerpc/kernel/pci_dn.c | 9 +--------
 1 file changed, 1 insertion(+), 8 deletions(-)

diff --git a/arch/powerpc/kernel/pci_dn.c b/arch/powerpc/kernel/pci_dn.c
index 4e654df55969..f6ac25f7af63 100644
--- a/arch/powerpc/kernel/pci_dn.c
+++ b/arch/powerpc/kernel/pci_dn.c
@@ -236,14 +236,7 @@ void remove_sriov_vf_pdns(struct pci_dev *pdev)
                         */
                        edev = pdn_to_eeh_dev(pdn);
                        if (edev) {
-                               /*
-                                * We allocate pci_dn's for the totalvfs count,
-                                * but only only the vfs that were activated
-                                * have a configured PE.
-                                */
-                               if (edev->pe)
-                                       eeh_rmv_from_parent_pe(edev);
-
+                               WARN_ON_ONCE(edev->pe);
                                pdn->edev = NULL;
                                kfree(edev);
                        }
-- 
2.22.0.216.g00a2a96fc9

Reply via email to