Commit 8cc07c80 authored Feb 19, 2018 by Will Deacon Committed by Al Viro Feb 25, 2018

fs: dcache: Use READ_ONCE when accessing i_dir_seq



i_dir_seq is subject to concurrent modification by a cmpxchg or
store-release operation, so ensure that the relaxed access in
d_alloc_parallel uses READ_ONCE.

Reported-by: Peter Zijlstra <peterz@infradead.org>
Signed-off-by: Will Deacon <will.deacon@arm.com>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>

parent 015555fd

fs/dcache.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -2507,7 +2507,7 @@ struct dentry d_alloc_parallel(struct dentry parent,
		}

		hlist_bl_lock(b);
		if (unlikely(parent->d_inode->i_dir_seq != seq)) {
		if (unlikely(READ_ONCE(parent->d_inode->i_dir_seq) != seq)) {
		hlist_bl_unlock(b);
		rcu_read_unlock();
		goto retry;