diff options
| author | Keith Busch <kbusch@kernel.org> | 2024-10-22 15:48:48 -0700 |
|---|---|---|
| committer | Bjorn Helgaas <bhelgaas@google.com> | 2024-11-11 13:06:02 -0600 |
| commit | e3f30d563a388220a7c4e3b9a7b52ac0b0324b26 (patch) | |
| tree | 8a0fcef5bcf8b3f4618c0e333477e49505d566e7 /drivers/pci/remove.c | |
| parent | 93093ea1f05928b123dae38b710631362bef1601 (diff) | |
| download | linux-e3f30d563a388220a7c4e3b9a7b52ac0b0324b26.tar.gz linux-e3f30d563a388220a7c4e3b9a7b52ac0b0324b26.tar.bz2 linux-e3f30d563a388220a7c4e3b9a7b52ac0b0324b26.zip | |
PCI: Make pci_destroy_dev() concurrent safe
Use an atomic flag instead of the racy check against the device's kobj
parent. We shouldn't be poking into device implementation details at this
level anyway.
Link: https://lore.kernel.org/r/20241022224851.340648-3-kbusch@meta.com
Signed-off-by: Keith Busch <kbusch@kernel.org>
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
Reviewed-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Diffstat (limited to 'drivers/pci/remove.c')
| -rw-r--r-- | drivers/pci/remove.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/pci/remove.c b/drivers/pci/remove.c index 78863f241a88..1f35945459fd 100644 --- a/drivers/pci/remove.c +++ b/drivers/pci/remove.c @@ -46,7 +46,7 @@ static void pci_stop_dev(struct pci_dev *dev) static void pci_destroy_dev(struct pci_dev *dev) { - if (!dev->dev.kobj.parent) + if (pci_dev_test_and_set_removed(dev)) return; pci_npem_remove(dev); |