netlink: fix lockdep splats
authorEric Dumazet <eric.dumazet@gmail.com>
Tue, 5 Aug 2014 05:50:07 +0000 (07:50 +0200)
committerDavid S. Miller <davem@davemloft.net>
Tue, 5 Aug 2014 05:58:06 +0000 (22:58 -0700)
With netlink_lookup() conversion to RCU, we need to use appropriate
rcu dereference in netlink_seq_socket_idx() & netlink_seq_next()

Reported-by: Sasha Levin <sasha.levin@oracle.com>
Signed-off-by: Eric Dumazet <edumazet@google.com>
Fixes: e341694e3eb57fc ("netlink: Convert netlink_lookup() to use RCU protected hash table")
Signed-off-by: David S. Miller <davem@davemloft.net>
net/netlink/af_netlink.c

index 0b89ca51a3afb3c70b7805d0c0390f6392d20e6e..479a344563d836a9ea13250665cb9c04da7fc587 100644 (file)
@@ -2899,7 +2899,7 @@ static struct sock *netlink_seq_socket_idx(struct seq_file *seq, loff_t pos)
 
        for (i = 0; i < MAX_LINKS; i++) {
                struct rhashtable *ht = &nl_table[i].hash;
-               const struct bucket_table *tbl = rht_dereference(ht->tbl, ht);
+               const struct bucket_table *tbl = rht_dereference_rcu(ht->tbl, ht);
 
                for (j = 0; j < tbl->size; j++) {
                        rht_for_each_entry_rcu(nlk, tbl->buckets[j], node) {
@@ -2950,7 +2950,7 @@ static void *netlink_seq_next(struct seq_file *seq, void *v, loff_t *pos)
 
        do {
                struct rhashtable *ht = &nl_table[i].hash;
-               const struct bucket_table *tbl = rht_dereference(ht->tbl, ht);
+               const struct bucket_table *tbl = rht_dereference_rcu(ht->tbl, ht);
 
                for (; j < tbl->size; j++) {
                        rht_for_each_entry_rcu(nlk, tbl->buckets[j], node) {