diff options
author | 2019-02-15 19:13:07 +0800 | |
---|---|---|
committer | 2019-02-15 08:40:10 -0700 | |
commit | 8a2ee44a371c8cbef587ea609908c3cbf1645231 (patch) | |
tree | c9ecfab2ea5857d120f4768fe8b110203cab9ef9 /include/linux/bio.h | |
parent | block: avoid setting none scheduler if it's already none (diff) | |
download | linux-8a2ee44a371c8cbef587ea609908c3cbf1645231.tar.gz linux-8a2ee44a371c8cbef587ea609908c3cbf1645231.tar.bz2 linux-8a2ee44a371c8cbef587ea609908c3cbf1645231.zip |
btrfs: look at bi_size for repair decisions
bio_readpage_error currently uses bi_vcnt to decide if it is worth
retrying an I/O. But the vector count is mostly an implementation
artifact - it really should figure out if there is more than a
single sector worth retrying. Use bi_size for that and shift by
PAGE_SHIFT. This really should be blocks/sectors, but given that
btrfs doesn't support a sector size different from the PAGE_SIZE
using the page size keeps the changes to a minimum.
Reviewed-by: Omar Sandoval <osandov@fb.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'include/linux/bio.h')
-rw-r--r-- | include/linux/bio.h | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/include/linux/bio.h b/include/linux/bio.h index 7380b094dcca..72b4f7be2106 100644 --- a/include/linux/bio.h +++ b/include/linux/bio.h @@ -263,12 +263,6 @@ static inline void bio_get_last_bvec(struct bio *bio, struct bio_vec *bv) bv->bv_len = iter.bi_bvec_done; } -static inline unsigned bio_pages_all(struct bio *bio) -{ - WARN_ON_ONCE(bio_flagged(bio, BIO_CLONED)); - return bio->bi_vcnt; -} - static inline struct bio_vec *bio_first_bvec_all(struct bio *bio) { WARN_ON_ONCE(bio_flagged(bio, BIO_CLONED)); |