Proxmox-Port/linux-loongson
1
0
Fork 0
mirror of https://git.kernel.org/pub/scm/linux/kernel/git/chenhuacai/linux-loongson synced 2025-08-30 21:52:21 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

btrfs: Remove extent_io_ops::readpage_io_failed_hook

Browse Source 
For data inodes this hook does nothing but to return -EAGAIN which is
used to signal to the endio routines that this bio belongs to a data
inode. If this is the case the actual retrying is handled by
bio_readpage_error. Alternatively, if this bio belongs to the btree
inode then btree_io_failed_hook just does some cleanup and doesn't retry
anything.

This patch simplifies the code flow by eliminating
readpage_io_failed_hook and instead open-coding btree_io_failed_hook in
end_bio_extent_readpage. Also eliminate some needless checks since IO is
always performed on either data inode or btree inode, both of which are
guaranteed to have their extent_io_tree::ops set.

Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
This commit is contained in:
Nikolay Borisov 2018-11-22 10:17:49 +02:00 committed by David Sterba
parent 7b41ba71c1
commit 78e62c02ab
4 changed files with 35 additions and 60 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
fs/btrfs/disk-io.c
View File

@ -673,19 +673,6 @@ static int btree_readpage_end_io_hook(struct btrfs_io_bio *io_bio,
return ret; return ret;
} }
static int btree_io_failed_hook(struct page *page, int failed_mirror)
{
struct extent_buffer *eb;
eb = (struct extent_buffer *)page->private;
set_bit(EXTENT_BUFFER_READ_ERR, &eb->bflags);
eb->read_mirror = failed_mirror;
atomic_dec(&eb->io_pages);
if (test_and_clear_bit(EXTENT_BUFFER_READAHEAD, &eb->bflags))
btree_readahead_hook(eb, -EIO);
return -EIO; /* we fixed nothing */
}
static void end_workqueue_bio(struct bio *bio) static void end_workqueue_bio(struct bio *bio)
{ {
struct btrfs_end_io_wq *end_io_wq = bio->bi_private; struct btrfs_end_io_wq *end_io_wq = bio->bi_private;
@ -4560,7 +4547,4 @@ static const struct extent_io_ops btree_extent_io_ops = {
/* mandatory callbacks */ /* mandatory callbacks */
.submit_bio_hook = btree_submit_bio_hook, .submit_bio_hook = btree_submit_bio_hook,
.readpage_end_io_hook = btree_readpage_end_io_hook, .readpage_end_io_hook = btree_readpage_end_io_hook,
.readpage_io_failed_hook = btree_io_failed_hook,
/* optional callbacks */
}; };

61
fs/btrfs/extent_io.c
View File

@ -2333,13 +2333,11 @@ struct bio *btrfs_create_repair_bio(struct inode *inode, struct bio *failed_bio,
} }
/* /*
* this is a generic handler for readpage errors (default * This is a generic handler for readpage errors. If other copies exist, read
* readpage_io_failed_hook). if other copies exist, read those and write back * those and write back good data to the failed position. Does not investigate
* good data to the failed position. does not investigate in remapping the * in remapping the failed extent elsewhere, hoping the device will be smart
* failed extent elsewhere, hoping the device will be smart enough to do this as * enough to do this as needed
* needed
*/ */
static int bio_readpage_error(struct bio *failed_bio, u64 phy_offset, static int bio_readpage_error(struct bio *failed_bio, u64 phy_offset,
struct page *page, u64 start, u64 end, struct page *page, u64 start, u64 end,
int failed_mirror) int failed_mirror)
@ -2501,6 +2499,8 @@ static void end_bio_extent_readpage(struct bio *bio)
struct page *page = bvec->bv_page; struct page *page = bvec->bv_page;
struct inode *inode = page->mapping->host; struct inode *inode = page->mapping->host;
struct btrfs_fs_info *fs_info = btrfs_sb(inode->i_sb); struct btrfs_fs_info *fs_info = btrfs_sb(inode->i_sb);
bool data_inode = btrfs_ino(BTRFS_I(inode))
!= BTRFS_BTREE_INODE_OBJECTID;
btrfs_debug(fs_info, btrfs_debug(fs_info,
"end_bio_extent_readpage: bi_sector=%llu, err=%d, mirror=%u", "end_bio_extent_readpage: bi_sector=%llu, err=%d, mirror=%u",
@ -2530,7 +2530,7 @@ static void end_bio_extent_readpage(struct bio *bio)
len = bvec->bv_len; len = bvec->bv_len;
mirror = io_bio->mirror_num; mirror = io_bio->mirror_num;
if (likely(uptodate && tree->ops)) { if (likely(uptodate)) {
ret = tree->ops->readpage_end_io_hook(io_bio, offset, ret = tree->ops->readpage_end_io_hook(io_bio, offset,
page, start, end, page, start, end,
mirror); mirror);
@ -2546,38 +2546,37 @@ static void end_bio_extent_readpage(struct bio *bio)
if (likely(uptodate)) if (likely(uptodate))
goto readpage_ok; goto readpage_ok;
if (tree->ops) { if (data_inode) {
ret = tree->ops->readpage_io_failed_hook(page, mirror);
if (ret == -EAGAIN) {
/* /*
* Data inode's readpage_io_failed_hook() always * The generic bio_readpage_error handles errors the
* returns -EAGAIN. * following way: If possible, new read requests are
* * created and submitted and will end up in
* The generic bio_readpage_error handles errors * end_bio_extent_readpage as well (if we're lucky,
* the following way: If possible, new read * not in the !uptodate case). In that case it returns
* requests are created and submitted and will * 0 and we just go on with the next page in our bio.
* end up in end_bio_extent_readpage as well (if * If it can't handle the error it will return -EIO and
* we're lucky, not in the !uptodate case). In * we remain responsible for that page.
* that case it returns 0 and we just go on with
* the next page in our bio. If it can't handle
* the error it will return -EIO and we remain
* responsible for that page.
*/ */
ret = bio_readpage_error(bio, offset, page, ret = bio_readpage_error(bio, offset, page, start, end,
start, end, mirror); mirror);
if (ret == 0) { if (ret == 0) {
uptodate = !bio->bi_status; uptodate = !bio->bi_status;
offset += len; offset += len;
continue; continue;
} }
} } else {
struct extent_buffer *eb;
/* eb = (struct extent_buffer *)page->private;
* metadata's readpage_io_failed_hook() always returns set_bit(EXTENT_BUFFER_READ_ERR, &eb->bflags);
* -EIO and fixes nothing. -EIO is also returned if eb->read_mirror = mirror;
* data inode error could not be fixed. atomic_dec(&eb->io_pages);
*/ if (test_and_clear_bit(EXTENT_BUFFER_READAHEAD,
ASSERT(ret == -EIO); &eb->bflags))
btree_readahead_hook(eb, -EIO);
ret = -EIO;
} }
readpage_ok: readpage_ok:
if (likely(uptodate)) { if (likely(uptodate)) {

1
fs/btrfs/extent_io.h
View File

@ -101,7 +101,6 @@ struct extent_io_ops {
int (*readpage_end_io_hook)(struct btrfs_io_bio *io_bio, u64 phy_offset, int (*readpage_end_io_hook)(struct btrfs_io_bio *io_bio, u64 phy_offset,
struct page *page, u64 start, u64 end, struct page *page, u64 start, u64 end,
int mirror); int mirror);
int (*readpage_io_failed_hook)(struct page *page, int failed_mirror);
}; };
struct extent_io_tree { struct extent_io_tree {

7
fs/btrfs/inode.c
View File

@ -10409,12 +10409,6 @@ static int btrfs_tmpfile(struct inode *dir, struct dentry *dentry, umode_t mode)
return ret; return ret;
} }
__attribute__((const))
static int btrfs_readpage_io_failed_hook(struct page *page, int failed_mirror)
{
return -EAGAIN;
}
void btrfs_set_range_writeback(struct extent_io_tree *tree, u64 start, u64 end) void btrfs_set_range_writeback(struct extent_io_tree *tree, u64 start, u64 end)
{ {
struct inode *inode = tree->private_data; struct inode *inode = tree->private_data;
@ -10810,7 +10804,6 @@ static const struct extent_io_ops btrfs_extent_io_ops = {
/* mandatory callbacks */ /* mandatory callbacks */
.submit_bio_hook = btrfs_submit_bio_hook, .submit_bio_hook = btrfs_submit_bio_hook,
.readpage_end_io_hook = btrfs_readpage_end_io_hook, .readpage_end_io_hook = btrfs_readpage_end_io_hook,
.readpage_io_failed_hook = btrfs_readpage_io_failed_hook,
}; };
/* /*