Skip to content

Commit

Permalink
ext4: clean up some wait_on_page_writeback calls
Browse files Browse the repository at this point in the history
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>
  • Loading branch information
Darrick J. Wong authored and Theodore Ts'o committed May 18, 2011
1 parent ed3ce80 commit 7cb1a53
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 5 deletions.
4 changes: 1 addition & 3 deletions fs/ext4/inode.c
Original file line number Diff line number Diff line change
Expand Up @@ -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)
Expand Down
3 changes: 1 addition & 2 deletions fs/ext4/move_extent.c
Original file line number Diff line number Diff line change
Expand Up @@ -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);
Expand Down

0 comments on commit 7cb1a53

Please sign in to comment.