diff options
author | jiangyiwen <jiangyiwen@huawei.com> | 2014-04-03 14:47:13 -0700 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2014-04-03 16:20:57 -0700 |
commit | 43b10a20372d9a1c08391f33f1c8bd86179ddc5f (patch) | |
tree | ed5fd7ca5fd34c4d84393e7fb3d65b8512594382 /fs/ocfs2/xattr.c | |
parent | 7dc3e83901b342ea7fe36262329c3784f2937361 (diff) | |
download | op-kernel-dev-43b10a20372d9a1c08391f33f1c8bd86179ddc5f.zip op-kernel-dev-43b10a20372d9a1c08391f33f1c8bd86179ddc5f.tar.gz |
ocfs2: avoid system inode ref confusion by adding mutex lock
The following case may lead to the same system inode ref in confusion.
A thread B thread
ocfs2_get_system_file_inode
->get_local_system_inode
->_ocfs2_get_system_file_inode
because of *arr == NULL,
ocfs2_get_system_file_inode
->get_local_system_inode
->_ocfs2_get_system_file_inode
gets first ref thru
_ocfs2_get_system_file_inode,
gets second ref thru igrab and
set *arr = inode
at the moment, B thread also gets
two refs, so lead to one more
inode ref.
So add mutex lock to avoid multi thread set two inode ref once at the
same time.
Signed-off-by: jiangyiwen <jiangyiwen@huawei.com>
Reviewed-by: Joseph Qi <joseph.qi@huawei.com>
Cc: Joel Becker <jlbec@evilplan.org>
Cc: Mark Fasheh <mfasheh@suse.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs/ocfs2/xattr.c')
0 files changed, 0 insertions, 0 deletions