diff options
| author | Veaceslav Falico <vfalico@redhat.com> | 2014-01-10 11:59:44 +0100 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2014-01-13 22:22:27 -0800 |
| commit | 768b954922e69a92a363bd4041cb93040ae4e9cf (patch) | |
| tree | a6de632160b5525aaebc9e462a7790c1b41d932f /scripts/tracing | |
| parent | c1bc9644ecd1f663f918d42b33b41a508683b0ee (diff) | |
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 'scripts/tracing')
0 files changed, 0 insertions, 0 deletions