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

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

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

Xen patchbot-unstable
# HG changeset patch
# User [hidden email]
# Date 1173103759 0
# Node ID 4159a8a43a97441404edeeccd3534c9282795b4a
# Parent  d99e6b6c294de17a9d94dbce15f4241879928bed
linux: Remove unnecessary locking from pcifront free_root_bus_devs().
Signed-off-by: Keir Fraser <[hidden email]>
---
 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
[hidden email]
http://lists.xensource.com/xen-changelog