diff options
| author | Huang Ying <ying.huang@intel.com> | 2014-09-07 07:05:20 +0400 |
|---|---|---|
| committer | Jaegeuk Kim <jaegeuk@kernel.org> | 2014-09-10 00:15:07 +0400 |
| commit | 9a01b56b1a79e210e9d2d67c9b18906d31d536c4 (patch) | |
| tree | e87eb9e201717999a21e78dabb04cf2748f0dbf3 /virt/kvm | |
| parent | 721bd4d5c3f957f98157b6dcac9c4a4dd828e3ff (diff) | |
| download | linux-9a01b56b1a79e210e9d2d67c9b18906d31d536c4.tar.xz | |
f2fs: avoid node page to be written twice in gc_node_segment
In gc_node_segment, if node page gc is run concurrently with node page
writeback, and check_valid_map and get_node_page run after page locked
and before cur_valid_map is updated as below, it is possible for the
page to be written twice unnecessarily.
sync_node_pages
try_lock_page
...
check_valid_map f2fs_write_node_page
...
write_node_page
do_write_page
allocate_data_block
...
refresh_sit_entry /* update cur_valid_map */
...
...
unlock_page
get_node_page
...
set_page_dirty
...
f2fs_put_page
unlock_page
This can be solved via calling check_valid_map after get_node_page again.
Signed-off-by: Huang, Ying <ying.huang@intel.com>
Signed-off-by: Jaegeuk Kim <jaegeuk@kernel.org>
Diffstat (limited to 'virt/kvm')
0 files changed, 0 insertions, 0 deletions