[Xen-changelog] [xen-unstable] linux: Remove unnecessary locking from pcifront free_root_bus_devs().

[Xen patchbot-unstable <patchbot-unstable@xxxxxxxxxxxxxxxxxxx>]

# HG changeset patch
# User kfraser@xxxxxxxxxxxxxxxxxxxxx
# Date 1173103759 0
# Node ID 4159a8a43a97441404edeeccd3534c9282795b4a
# Parent  d99e6b6c294de17a9d94dbce15f4241879928bed
linux: Remove unnecessary locking from pcifront free_root_bus_devs().
Signed-off-by: Keir Fraser <keir@xxxxxxxxxxxxx>
---
 linux-2.6-xen-sparse/drivers/xen/pcifront/pci_op.c |   11 +++--------
 1 files changed, 3 insertions(+), 8 deletions(-)

diff -r d99e6b6c294d -r 4159a8a43a97 
linux-2.6-xen-sparse/drivers/xen/pcifront/pci_op.c
--- a/linux-2.6-xen-sparse/drivers/xen/pcifront/pci_op.c        Mon Mar 05 
14:04:45 2007 +0000
+++ b/linux-2.6-xen-sparse/drivers/xen/pcifront/pci_op.c        Mon Mar 05 
14:09:19 2007 +0000
@@ -239,17 +239,12 @@ static void free_root_bus_devs(struct pc
 {
        struct pci_dev *dev;
 
-       down_write(&pci_bus_sem);
        while (!list_empty(&bus->devices)) {
-               dev = container_of(bus->devices.next, struct pci_dev, bus_list);
-               up_write(&pci_bus_sem);
-
+               dev = container_of(bus->devices.next, struct pci_dev,
+                                  bus_list);
                dev_dbg(&dev->dev, "removing device\n");
                pci_remove_bus_device(dev);
-
-               down_write(&pci_bus_sem);
-       }
-       up_write(&pci_bus_sem);
+       }
 }
 
 void pcifront_free_roots(struct pcifront_device *pdev)

_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-changelog