diff options
author | Veaceslav Falico <vfalico@redhat.com> | 2013-03-26 00:10:02 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2013-03-26 12:49:40 -0400 |
commit | ad999eee669d6a0439f5b9734e87eed50e776e32 (patch) | |
tree | 71d37f47c97109f0767b5d80c33d1a8dcd8ff3a5 | |
parent | e7333e3c7c7ce2d7019fac2f514ceeea2b57f438 (diff) |
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>
-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) | |||
796 | { | 796 | { |
797 | struct bonding *bond = container_of(work, struct bonding, | 797 | struct bonding *bond = container_of(work, struct bonding, |
798 | mcast_work.work); | 798 | mcast_work.work); |
799 | rcu_read_lock(); | 799 | |
800 | bond_resend_igmp_join_requests(bond); | 800 | bond_resend_igmp_join_requests(bond); |
801 | rcu_read_unlock(); | ||
802 | } | 801 | } |
803 | 802 | ||
804 | /* | 803 | /* |