OSDN Git Service

Revert "PCI: Use hotplug-safe pci_get_domain_bus_and_slot()"
authorBjorn Helgaas <bhelgaas@google.com>
Thu, 20 Sep 2012 23:09:48 +0000 (17:09 -0600)
committerBjorn Helgaas <bhelgaas@google.com>
Thu, 20 Sep 2012 23:10:54 +0000 (17:10 -0600)
commit94bb346480f8646871e5547491b5746ae0a643c3
tree56bec45b0079145ea4a0e8aec552ab44dd1f51f9
parent9b9a6d261616bed589302bc6244c5bd7c99a733f
Revert "PCI: Use hotplug-safe pci_get_domain_bus_and_slot()"

This reverts commit 433efd2247b0cbf5e7e86275e1f21281d3b99047.

When we remove an SR-IOV device, we have this call chain:

    driver .remove() method
        pci_disable_sriov()
            sriov_disable()
                virtfn_remove()
                    pci_get_domain_bus_and_slot()

sriov_disable() is only called for PFs, not for VFs.  When it's called
for a PF, it loops through all the VFs and calls virtfn_remove() for
each.  But we stop and remove VFs before PFs, so by the time we get
to virtfn_remove(), the VFs have already been stopped and deleted
from the device list.  Now pci_get_domain_bus_and_slot(), which uses
bus_find_device() and relies on that device list, doesn't find the
VFs, so the VF references aren't released correctly.

Reported-by: Yinghai Lu <yinghai@kernel.org>
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
drivers/pci/iov.c