Btrfs: cleanup error handling in extent_write_cached_pages
authorLiu Bo <bo.li.liu@oracle.com>
Tue, 8 Mar 2016 00:56:22 +0000 (16:56 -0800)
committerDavid Sterba <dsterba@suse.com>
Thu, 28 Apr 2016 08:41:47 +0000 (10:41 +0200)
Now that we bail out immediately if ->writepage() returns an error,
we don't need an extra error to retain the error code.

Signed-off-by: Liu Bo <bo.li.liu@oracle.com>
Signed-off-by: David Sterba <dsterba@suse.com>
fs/btrfs/extent_io.c

index fd41a730d239ecfc0fb0b13ae98db74bc4be964a..b67d6d24440b8ae5723f9861d188273f5e817f3f 100644 (file)
@@ -3916,7 +3916,6 @@ static int extent_write_cache_pages(struct extent_io_tree *tree,
        struct inode *inode = mapping->host;
        int ret = 0;
        int done = 0;
-       int err = 0;
        int nr_to_write_done = 0;
        struct pagevec pvec;
        int nr_pages;
@@ -4009,8 +4008,6 @@ retry:
                                unlock_page(page);
                                ret = 0;
                        }
-                       if (!err && ret < 0)
-                               err = ret;
                        if (ret < 0) {
                                /*
                                 * done_index is set past this page,
@@ -4036,7 +4033,7 @@ retry:
                pagevec_release(&pvec);
                cond_resched();
        }
-       if (!scanned && !done && !err) {
+       if (!scanned && !done) {
                /*
                 * We hit the last page and there is more work to be done: wrap
                 * back to the start of the file
@@ -4050,7 +4047,7 @@ retry:
                mapping->writeback_index = done_index;
 
        btrfs_add_delayed_iput(inode);
-       return err;
+       return ret;
 }
 
 static void flush_epd_write_bio(struct extent_page_data *epd)