diff options
| author |  David Sterba <dsterba@suse.com>
| 2017-02-17 15:18:32 +0100 |
|---|---|---|
| committer | David Sterba <dsterba@suse.com>
| 2017-02-28 14:29:24 +0100 |
| commit | c3988d630a4dfec5c09f2b6496734f320949ea9c (patch) | |
| tree | a135de57adae888a7ea51845c85373ed41733cce /fs/btrfs/inode.c | |
| parent | btrfs: do proper error handling in btrfs_insert_xattr_item (diff) | |
| download | linux-c3988d630a4dfec5c09f2b6496734f320949ea9c.tar.gz linux-c3988d630a4dfec5c09f2b6496734f320949ea9c.tar.bz2 linux-c3988d630a4dfec5c09f2b6496734f320949ea9c.zip | |
btrfs: let writepage_end_io_hook return void
There's no error path in any of the instances, always return 0.
Reviewed-by: Liu Bo <bo.li.liu@oracle.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/inode.c')
| -rw-r--r-- | fs/btrfs/inode.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/fs/btrfs/inode.c b/fs/btrfs/inode.c index 0d932b9594de..b904c74f31fd 100644 --- a/fs/btrfs/inode.c +++ b/fs/btrfs/inode.c @@ -2972,7 +2972,7 @@ static void finish_ordered_fn(struct btrfs_work *work) btrfs_finish_ordered_io(ordered_extent); } -static int btrfs_writepage_end_io_hook(struct page *page, u64 start, u64 end, +static void btrfs_writepage_end_io_hook(struct page *page, u64 start, u64 end, struct extent_state *state, int uptodate) { struct inode *inode = page->mapping->host; @@ -2986,7 +2986,7 @@ static int btrfs_writepage_end_io_hook(struct page *page, u64 start, u64 end, ClearPagePrivate2(page); if (!btrfs_dec_test_ordered_pending(inode, &ordered_extent, start, end - start + 1, uptodate)) - return 0; + return; if (btrfs_is_free_space_inode(BTRFS_I(inode))) { wq = fs_info->endio_freespace_worker; @@ -2999,8 +2999,6 @@ static int btrfs_writepage_end_io_hook(struct page *page, u64 start, u64 end, btrfs_init_work(&ordered_extent->work, func, finish_ordered_fn, NULL, NULL); btrfs_queue_work(wq, &ordered_extent->work); - - return 0; } static int __readpage_endio_check(struct inode *inode, |