diff options
| author | Andrew Price <anprice@redhat.com> | 2015-11-12 21:07:52 +0100 |
|---|---|---|
| committer | Bob Peterson <rpeterso@redhat.com> | 2015-11-16 18:57:59 +0100 |
| commit | 3dd1dd8c696bdb7c8dcc9456cb23558ad1b336b8 (patch) | |
| tree | 3ec8d2b7d442737c15f6b83999713c1c38cbdfc7 /samples/Makefile | |
| parent | GFS2: Delete an unnecessary check before the function call "iput" (diff) | |
| download | linux-3dd1dd8c696bdb7c8dcc9456cb23558ad1b336b8.tar.xz linux-3dd1dd8c696bdb7c8dcc9456cb23558ad1b336b8.zip | |
GFS2: Use rht_for_each_entry_rcu in glock_hash_walk
This lockdep splat was being triggered on umount:
[55715.973122] ===============================
[55715.980169] [ INFO: suspicious RCU usage. ]
[55715.981021] 4.3.0-11553-g8d3de01-dirty #15 Tainted: G W
[55715.982353] -------------------------------
[55715.983301] fs/gfs2/glock.c:1427 suspicious rcu_dereference_protected() usage!
The code it refers to is the rht_for_each_entry_safe usage in
glock_hash_walk. The condition that triggers the warning is
lockdep_rht_bucket_is_held(tbl, hash) which is checked in the
__rcu_dereference_protected macro.
The rhashtable buckets are not changed in glock_hash_walk so it's safe
to rely on the rcu protection. Replace the rht_for_each_entry_safe()
usage with rht_for_each_entry_rcu(), which doesn't care whether the
bucket lock is held if the rcu read lock is held.
Signed-off-by: Andrew Price <anprice@redhat.com>
Signed-off-by: Bob Peterson <rpeterso@redhat.com>
Acked-by: Steven Whitehouse <swhiteho@redhat.com>
Diffstat (limited to 'samples/Makefile')
0 files changed, 0 insertions, 0 deletions