diff options
| author | Veaceslav Falico <vfalico@redhat.com> | 2013-03-26 08:10:02 +0400 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2013-03-26 20:49:40 +0400 |
| commit | ad999eee669d6a0439f5b9734e87eed50e776e32 (patch) | |
| tree | 71d37f47c97109f0767b5d80c33d1a8dcd8ff3a5 /drivers/net/bonding | |
| parent | e7333e3c7c7ce2d7019fac2f514ceeea2b57f438 (diff) | |
| download | linux-ad999eee669d6a0439f5b9734e87eed50e776e32.tar.xz | |
bonding: cleanup unneeded rcu_read_lock()
bond_resend_igmp_join_requests_delayed() calls _resend_igmp_join_requests()
under rcu_read_lock(), while it gets its own rcu_read_lock() for the whole
function. Remove the lock from the _delayed function.
Signed-off-by: Veaceslav Falico <vfalico@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/bonding')
| -rw-r--r-- | drivers/net/bonding/bond_main.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/net/bonding/bond_main.c b/drivers/net/bonding/bond_main.c index 6bbd90e1123c..11a8cb381b85 100644 --- a/drivers/net/bonding/bond_main.c +++ b/drivers/net/bonding/bond_main.c @@ -796,9 +796,8 @@ static void bond_resend_igmp_join_requests_delayed(struct work_struct *work) { struct bonding *bond = container_of(work, struct bonding, mcast_work.work); - rcu_read_lock(); + bond_resend_igmp_join_requests(bond); - rcu_read_unlock(); } /* |