diff options
| author | Amol Grover <frextrite@gmail.com> | 2020-02-19 15:30:11 +0530 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2020-02-19 10:55:42 -0800 |
| commit | a7a9456e8d28e81030f7cf6f1f59f907089916a9 (patch) | |
| tree | 82a639d30e61c709964114dab8272ef01604296c /net/hsr | |
| parent | 7822dee5d17a02c2b77da60c1cb50089ea64fd9b (diff) | |
| download | linux-a7a9456e8d28e81030f7cf6f1f59f907089916a9.tar.bz2 | |
net: hsr: Pass lockdep expression to RCU lists
node_db is traversed using list_for_each_entry_rcu
outside an RCU read-side critical section but under the protection
of hsr->list_lock.
Hence, add corresponding lockdep expression to silence false-positive
warnings, and harden RCU lists.
Signed-off-by: Amol Grover <frextrite@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/hsr')
| -rw-r--r-- | net/hsr/hsr_framereg.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/net/hsr/hsr_framereg.c b/net/hsr/hsr_framereg.c index 364ea2cc028e..3ba7f61be107 100644 --- a/net/hsr/hsr_framereg.c +++ b/net/hsr/hsr_framereg.c @@ -155,7 +155,8 @@ static struct hsr_node *hsr_add_node(struct hsr_priv *hsr, new_node->seq_out[i] = seq_out; spin_lock_bh(&hsr->list_lock); - list_for_each_entry_rcu(node, node_db, mac_list) { + list_for_each_entry_rcu(node, node_db, mac_list, + lockdep_is_held(&hsr->list_lock)) { if (ether_addr_equal(node->macaddress_A, addr)) goto out; if (ether_addr_equal(node->macaddress_B, addr)) |