On Sun, Jan 25, 2009 at 05:25:02AM +0800, Greg KH wrote: > On Sun, Jan 25, 2009 at 12:41:34AM +0800, Yu Zhao wrote: > > Signed-off-by: Yu Zhao <yu.zhao@xxxxxxxxx> > > --- > > drivers/pci/remove.c | 3 +++ > > 1 files changed, 3 insertions(+), 0 deletions(-) > > > > diff --git a/drivers/pci/remove.c b/drivers/pci/remove.c > > index 042e089..caf8e1e 100644 > > --- a/drivers/pci/remove.c > > +++ b/drivers/pci/remove.c > > @@ -71,6 +71,9 @@ void pci_remove_bus(struct pci_bus *pci_bus) > > down_write(&pci_bus_sem); > > list_del(&pci_bus->node); > > up_write(&pci_bus_sem); > > + if (!pci_bus->is_added) > > + return; > > What race condition is this solving? pci_bus_add_child() sets the 'is_added' if a bus is successfully registered, so I guess the reverse operation should check it too. Otherwise, the 'is_added' is useless and should be removed :-) -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
