Re: [PATCH 4/16][BUG] PCI slot: Add missing semaphore for slotrelease (Not for mainline!)

From: Alex Chiang
Date: Fri Mar 21 2008 - 12:58:10 EST


* Kenji Kaneshige <kaneshige.kenji@xxxxxxxxxxxxxx>:
> Add missing pci_bus sem into pci_destroy_slot(). There is possible
> race condition because current code doesn't have any serialization
> in the destroy path.

Merged, thanks.

/ac

>
> Signed-off-by: Kenji Kaneshige <kaneshige.kenji@xxxxxxxxxxxxxx>
>
> ---
> drivers/pci/slot.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> Index: linux-2.6.25-rc6/drivers/pci/slot.c
> ===================================================================
> --- linux-2.6.25-rc6.orig/drivers/pci/slot.c
> +++ linux-2.6.25-rc6/drivers/pci/slot.c
> @@ -196,7 +196,9 @@ EXPORT_SYMBOL_GPL(pci_create_slot);
>
> int pci_destroy_slot(struct pci_slot *slot)
> {
> + down_write(&pci_bus_sem);
> kobject_put(&slot->kobj);
> + up_write(&pci_bus_sem);
>
> dbg("%s: decreased refcount to %d on %x:%d\n", __func__,
> slot->kobj.kref.refcount, slot->bus->number, slot->number);
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/