diff options
| author |   Jay Vosburgh <fubar@us.ibm.com> | 2008-01-17 16:25:03 -0800 |
|---|---|---|
| committer |   Jeff Garzik <jeff@garzik.org> | 2008-01-18 14:38:39 -0500 |
| commit | 5655662dab4ef044be7efd155f2f5fef2e486545 (patch) | |
| tree | d4003635df3c8cba3554e023d831ff8d4e5828e5 /drivers | |
| parent | bonding: fix lock ordering for rtnl and bonding_rwsem (diff) | |
| download | linux-dev-5655662dab4ef044be7efd155f2f5fef2e486545.tar.xz linux-dev-5655662dab4ef044be7efd155f2f5fef2e486545.zip | |
bonding: Don't hold lock when calling rtnl_unlock
Change bond_mii_monitor to not hold any locks when calling rtnl_unlock,
as rtnl_unlock can sleep (when acquring another mutex in netdev_run_todo).
Bug reported by Makito SHIOKAWA <mshiokawa@miraclelinux.com>, who
included a different patch.
Signed-off-by: Jay Vosburgh <fubar@us.ibm.com>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/net/bonding/bond_main.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/net/bonding/bond_main.c b/drivers/net/bonding/bond_main.c index 2c6da4969382..49a198206e3d 100644 --- a/drivers/net/bonding/bond_main.c +++ b/drivers/net/bonding/bond_main.c @@ -2386,7 +2386,9 @@ void bond_mii_monitor(struct work_struct *work) rtnl_lock(); read_lock(&bond->lock); __bond_mii_monitor(bond, 1); - rtnl_unlock(); + read_unlock(&bond->lock); + rtnl_unlock(); /* might sleep, hold no other locks */ + read_lock(&bond->lock); } delay = ((bond->params.miimon * HZ) / 1000) ? : 1; |