diff options
| author | Fan Li <fanofcode.li@samsung.com> | 2015-08-18 17:13:13 +0800 |
|---|---|---|
| committer | Jaegeuk Kim <jaegeuk@kernel.org> | 2015-08-21 22:45:06 -0700 |
| commit | f8b703da2c23f9bfda7299bd14e4f7201c2be3c8 (patch) | |
| tree | 82fe2fc35b8597ed8659b67623b35045630e98c0 /include/linux/f2fs_fs.h | |
| parent | 217940d4f0c4ec4f0852f7046fa419d0edf65c17 (diff) | |
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>
Diffstat (limited to 'include/linux/f2fs_fs.h')
0 files changed, 0 insertions, 0 deletions