btrfs: rename clean_io_failure and remove extraneous args

This is exported, so rename it to btrfs_clean_io_failure.  Additionally
we are passing in the io tree's and such from the inode, so instead of
doing all that simply pass in the inode itself and get all the
components we need directly inside of btrfs_clean_io_failure.

Signed-off-by: Josef Bacik <josef@toxicpanda.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>

diff --git a/fs/btrfs/compression.c b/fs/btrfs/compression.c
index 1c77de3239bc466752c5d84748d92b765ac574ca..cac0eeceb8154cae47bab5955b831c08a2a9db0b 100644 (file)
--- a/fs/btrfs/compression.c
+++ b/fs/btrfs/compression.c
@@ -183,9 +183,8 @@ static void end_compressed_bio_read(struct btrfs_bio *bbio)
                if (!status &&
                    (!csum || !btrfs_check_data_csum(inode, bbio, offset,
                                                     bv.bv_page, bv.bv_offset))) {
-                       clean_io_failure(fs_info, &bi->io_failure_tree,
-                                        &bi->io_tree, start, bv.bv_page,
-                                        btrfs_ino(bi), bv.bv_offset);
+                       btrfs_clean_io_failure(bi, start, bv.bv_page,
+                                              bv.bv_offset);
                } else {
                        int ret;
 

diff --git a/fs/btrfs/extent-io-tree.h b/fs/btrfs/extent-io-tree.h
index ec2f8b8e6faa769cebf677f5f47c0fb4d1e6f243..bb71b4a690224ecca049c301ef18622c0a8a6aa6 100644 (file)
--- a/fs/btrfs/extent-io-tree.h
+++ b/fs/btrfs/extent-io-tree.h
@@ -256,9 +256,7 @@ void btrfs_free_io_failure_record(struct btrfs_inode *inode, u64 start,
 int free_io_failure(struct extent_io_tree *failure_tree,
                    struct extent_io_tree *io_tree,
                    struct io_failure_record *rec);
-int clean_io_failure(struct btrfs_fs_info *fs_info,
-                    struct extent_io_tree *failure_tree,
-                    struct extent_io_tree *io_tree, u64 start,
-                    struct page *page, u64 ino, unsigned int pg_offset);
+int btrfs_clean_io_failure(struct btrfs_inode *inode, u64 start,
+                          struct page *page, unsigned int pg_offset);
 
 #endif /* BTRFS_EXTENT_IO_TREE_H */

diff --git a/fs/btrfs/extent_io.c b/fs/btrfs/extent_io.c
index 47367ca6d0d79543ca14a3d9a1c9602b9a52fa90..927f499c210f14043fe0d579a670f27509fcf8a0 100644 (file)
--- a/fs/btrfs/extent_io.c
+++ b/fs/btrfs/extent_io.c
@@ -2431,11 +2431,13 @@ static int prev_mirror(const struct io_failure_record *failrec, int cur_mirror)
  * each time an IO finishes, we do a fast check in the IO failure tree
  * to see if we need to process or clean up an io_failure_record
  */
-int clean_io_failure(struct btrfs_fs_info *fs_info,
-                    struct extent_io_tree *failure_tree,
-                    struct extent_io_tree *io_tree, u64 start,
-                    struct page *page, u64 ino, unsigned int pg_offset)
+int btrfs_clean_io_failure(struct btrfs_inode *inode, u64 start,
+                          struct page *page, unsigned int pg_offset)
 {
+       struct btrfs_fs_info *fs_info = inode->root->fs_info;
+       struct extent_io_tree *failure_tree = &inode->io_failure_tree;
+       struct extent_io_tree *io_tree = &inode->io_tree;
+       u64 ino = btrfs_ino(inode);
        u64 private;
        struct io_failure_record *failrec;
        struct extent_state *state;
@@ -2963,7 +2965,6 @@ static void end_bio_extent_readpage(struct btrfs_bio *bbio)
 {
        struct bio *bio = &bbio->bio;
        struct bio_vec *bvec;
-       struct extent_io_tree *tree, *failure_tree;
        struct processed_extent processed = { 0 };
        /*
         * The offset to the beginning of a bio, since one bio can never be
@@ -2990,8 +2991,6 @@ static void end_bio_extent_readpage(struct btrfs_bio *bbio)
                        "end_bio_extent_readpage: bi_sector=%llu, err=%d, mirror=%u",
                        bio->bi_iter.bi_sector, bio->bi_status,
                        bbio->mirror_num);
-               tree = &BTRFS_I(inode)->io_tree;
-               failure_tree = &BTRFS_I(inode)->io_failure_tree;
 
                /*
                 * We always issue full-sector reads, but if some block in a
@@ -3032,9 +3031,7 @@ static void end_bio_extent_readpage(struct btrfs_bio *bbio)
                        loff_t i_size = i_size_read(inode);
                        pgoff_t end_index = i_size >> PAGE_SHIFT;
 
-                       clean_io_failure(BTRFS_I(inode)->root->fs_info,
-                                        failure_tree, tree, start, page,
-                                        btrfs_ino(BTRFS_I(inode)), 0);
+                       btrfs_clean_io_failure(BTRFS_I(inode), start, page, 0);
 
                        /*
                         * Zero out the remaining part if this range straddles

diff --git a/fs/btrfs/inode.c b/fs/btrfs/inode.c
index 10849db7f3a58a12c8d884d50063a8d6fdc5b014..548665299e57dc26ddcde75a125940f841dc5a75 100644 (file)
--- a/fs/btrfs/inode.c
+++ b/fs/btrfs/inode.c
@@ -7869,8 +7869,6 @@ static blk_status_t btrfs_check_read_dio_bio(struct btrfs_dio_private *dip,
 {
        struct inode *inode = dip->inode;
        struct btrfs_fs_info *fs_info = BTRFS_I(inode)->root->fs_info;
-       struct extent_io_tree *failure_tree = &BTRFS_I(inode)->io_failure_tree;
-       struct extent_io_tree *io_tree = &BTRFS_I(inode)->io_tree;
        const bool csum = !(BTRFS_I(inode)->flags & BTRFS_INODE_NODATASUM);
        blk_status_t err = BLK_STS_OK;
        struct bvec_iter iter;
@@ -7883,9 +7881,8 @@ static blk_status_t btrfs_check_read_dio_bio(struct btrfs_dio_private *dip,
                if (uptodate &&
                    (!csum || !btrfs_check_data_csum(inode, bbio, offset, bv.bv_page,
                                               bv.bv_offset))) {
-                       clean_io_failure(fs_info, failure_tree, io_tree, start,
-                                        bv.bv_page, btrfs_ino(BTRFS_I(inode)),
-                                        bv.bv_offset);
+                       btrfs_clean_io_failure(BTRFS_I(inode), start,
+                                              bv.bv_page, bv.bv_offset);
                } else {
                        int ret;