diff options
| author | Darrick J. Wong <djwong@us.ibm.com> | 2011-05-18 13:53:20 -0400 |
|---|---|---|
| committer | Theodore Ts'o <tytso@mit.edu> | 2011-05-18 13:53:20 -0400 |
| commit | 7cb1a5351da8ac499d965a78e94c79ad27891f43 (patch) | |
| tree | b78e2ff5d432f14e2840e8911654789b656b26db /fs | |
| parent | ed3ce80a52c4658f48ef8fc825bdedf8b3fcaa89 (diff) | |
| download | op-kernel-dev-7cb1a5351da8ac499d965a78e94c79ad27891f43.zip op-kernel-dev-7cb1a5351da8ac499d965a78e94c79ad27891f43.tar.gz | |
ext4: clean up some wait_on_page_writeback calls
wait_on_page_writeback already checks the writeback bit, so callers of it
needn't do that test.
Signed-off-by: Darrick J. Wong <djwong@us.ibm.com>
Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
Diffstat (limited to 'fs')
| -rw-r--r-- | fs/ext4/inode.c | 4 | ||||
| -rw-r--r-- | fs/ext4/move_extent.c | 3 |
2 files changed, 2 insertions, 5 deletions
diff --git a/fs/ext4/inode.c b/fs/ext4/inode.c index 3424e82..f6caaac 100644 --- a/fs/ext4/inode.c +++ b/fs/ext4/inode.c @@ -2796,9 +2796,7 @@ static int write_cache_pages_da(struct address_space *mapping, continue; } - if (PageWriteback(page)) - wait_on_page_writeback(page); - + wait_on_page_writeback(page); BUG_ON(PageWriteback(page)); if (mpd->next_page != page->index) diff --git a/fs/ext4/move_extent.c b/fs/ext4/move_extent.c index b9f3e78..2b8304b 100644 --- a/fs/ext4/move_extent.c +++ b/fs/ext4/move_extent.c @@ -876,8 +876,7 @@ move_extent_per_page(struct file *o_filp, struct inode *donor_inode, * It needs to call wait_on_page_writeback() to wait for the * writeback of the page. */ - if (PageWriteback(page)) - wait_on_page_writeback(page); + wait_on_page_writeback(page); /* Release old bh and drop refs */ try_to_release_page(page, 0); |
