diff options
| author | Veaceslav Falico <vfalico@redhat.com> | 2014-01-10 11:59:44 +0100 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2014-01-14 07:22:27 +0100 |
| commit | 768b954922e69a92a363bd4041cb93040ae4e9cf (patch) | |
| tree | a6de632160b5525aaebc9e462a7790c1b41d932f /net/xfrm/xfrm_policy.c | |
| parent | bonding: fix bond_3ad_set_carrier() RCU usage (diff) | |
| download | linux-768b954922e69a92a363bd4041cb93040ae4e9cf.tar.xz linux-768b954922e69a92a363bd4041cb93040ae4e9cf.zip | |
bonding: fix __get_first_agg RCU usage
Currently, the RCU read lock usage is just wrong - it gets the slave struct
under RCU and continues to use it when RCU lock is released.
However, it's still safe to do this cause we didn't need the
rcu_read_lock() initially - all of the __get_first_agg() callers are either
holding RCU read lock or the RTNL lock, so that we can't sync while in it.
Fixes: be79bd048 ("bonding: add RCU for bond_3ad_state_machine_handler()")
CC: dingtianhong@huawei.com
CC: Jay Vosburgh <fubar@us.ibm.com>
CC: Andy Gospodarek <andy@greyhouse.net>
Signed-off-by: Veaceslav Falico <vfalico@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/xfrm/xfrm_policy.c')
0 files changed, 0 insertions, 0 deletions