f2fs: fix to update cached_en of extent tree properly
authorFan Li <fanofcode.li@samsung.com>
Tue, 18 Aug 2015 09:13:13 +0000 (17:13 +0800)
committerJaegeuk Kim <jaegeuk@kernel.org>
Sat, 22 Aug 2015 05:45:06 +0000 (22:45 -0700)
commitf8b703da2c23f9bfda7299bd14e4f7201c2be3c8
tree82fe2fc35b8597ed8659b67623b35045630e98c0
parent217940d4f0c4ec4f0852f7046fa419d0edf65c17
f2fs: fix to update cached_en of extent tree properly

In f2fs_lookup_extent_tree, et->cached_en was read and updated with only
read lock held,
it could cause __lookup_extent_tree within return entirely wrong
extent_node, if other
thread update et->cached_en just before __lookup_extent_tree return.

However, there are two things about this patch that need to be noticed:
1. It does no good to arrange the order of concurrent read/write, the result
would still
be random in such case.
2. It's built on this assumption: the mix up of reads and writes on a single
pointer would
not make the pointer partially wrong at any time. Please let me know if I'm
wrong, thx.

Signed-off-by: Fan li <fanofcode.li@samsung.com>
Reviewed-by: Chao Yu <chao2.yu@samsung.com>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
fs/f2fs/extent_cache.c