diff options
author | Chao Yu <chao2.yu@samsung.com> | 2013-11-03 21:28:33 -0500 |
---|---|---|
committer | Jaegeuk Kim <jaegeuk.kim@samsung.com> | 2013-11-03 22:24:01 -0500 |
commit | 4bf08ff6f9f381fa625d65e36c3e8e57313943cf (patch) | |
tree | c9d93308315b39fb7b9a56ec9d847f6a338fc1a7 /fs/f2fs/node.c | |
parent | 66e960c692e9def8451d51e9cdb1ffc294dc27b2 (diff) |
f2fs: remove unnecessary TestClearPageError when wait pages writeback
In wait_on_node_pages_writeback we will test and clear error flag for all
pages in radix tree, but not necessary.
So we only do this for pages belong to the specified inode.
Signed-off-by: Chao Yu <chao2.yu@samsung.com>
Signed-off-by: Jaegeuk Kim <jaegeuk.kim@samsung.com>
Diffstat (limited to 'fs/f2fs/node.c')
-rw-r--r-- | fs/f2fs/node.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/fs/f2fs/node.c b/fs/f2fs/node.c index b527ed4c0225..4ac4150d421d 100644 --- a/fs/f2fs/node.c +++ b/fs/f2fs/node.c | |||
@@ -1170,10 +1170,11 @@ int wait_on_node_pages_writeback(struct f2fs_sb_info *sbi, nid_t ino) | |||
1170 | if (page->index > end) | 1170 | if (page->index > end) |
1171 | continue; | 1171 | continue; |
1172 | 1172 | ||
1173 | if (ino && ino_of_node(page) == ino) | 1173 | if (ino && ino_of_node(page) == ino) { |
1174 | wait_on_page_writeback(page); | 1174 | wait_on_page_writeback(page); |
1175 | if (TestClearPageError(page)) | 1175 | if (TestClearPageError(page)) |
1176 | ret = -EIO; | 1176 | ret = -EIO; |
1177 | } | ||
1177 | } | 1178 | } |
1178 | pagevec_release(&pvec); | 1179 | pagevec_release(&pvec); |
1179 | cond_resched(); | 1180 | cond_resched(); |