diff options
author | David Sterba <dsterba@suse.com> | 2015-12-03 13:08:59 +0100 |
---|---|---|
committer | David Sterba <dsterba@suse.com> | 2015-12-07 15:06:45 +0100 |
commit | b5227c075b5c11d5cf653bac5c01b9f7f03f2a8f (patch) | |
tree | c75377d2b70d9e6b5163eb8ffc7d0d0df4376a73 /fs/btrfs/extent_io.c | |
parent | a9d93e1778f3a92852d3816cd203cbec179b8813 (diff) | |
download | op-kernel-dev-b5227c075b5c11d5cf653bac5c01b9f7f03f2a8f.zip op-kernel-dev-b5227c075b5c11d5cf653bac5c01b9f7f03f2a8f.tar.gz |
btrfs: make end_extent_writepage return void
Does not return any errors, nor anything from the callgraph. The branch
in end_bio_extent_writepage has been skipped since
5fd02043553b ("Btrfs: finish ordered extents in their own thread").
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/extent_io.c')
-rw-r--r-- | fs/btrfs/extent_io.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/fs/btrfs/extent_io.c b/fs/btrfs/extent_io.c index b7261ea..0ea3e99 100644 --- a/fs/btrfs/extent_io.c +++ b/fs/btrfs/extent_io.c @@ -2515,7 +2515,7 @@ static int bio_readpage_error(struct bio *failed_bio, u64 phy_offset, /* lots and lots of room for performance fixes in the end_bio funcs */ -int end_extent_writepage(struct page *page, int err, u64 start, u64 end) +void end_extent_writepage(struct page *page, int err, u64 start, u64 end) { int uptodate = (err == 0); struct extent_io_tree *tree; @@ -2536,7 +2536,6 @@ int end_extent_writepage(struct page *page, int err, u64 start, u64 end) ret = ret < 0 ? ret : -EIO; mapping_set_error(page->mapping, ret); } - return 0; } /* @@ -2578,9 +2577,7 @@ static void end_bio_extent_writepage(struct bio *bio) start = page_offset(page); end = start + bvec->bv_offset + bvec->bv_len - 1; - if (end_extent_writepage(page, bio->bi_error, start, end)) - continue; - + end_extent_writepage(page, bio->bi_error, start, end); end_page_writeback(page); } |