diff options
author | Christoph Hellwig <hch@lst.de> | 2016-03-11 11:34:53 -0500 |
---|---|---|
committer | Jens Axboe <axboe@fb.com> | 2016-03-14 10:55:25 -0400 |
commit | 2b885517110cbe8724fef30363778b6284d0a428 (patch) | |
tree | 609935ecfa54b2e31752f375ec1dfb46cf3fca71 | |
parent | ba8c6967b7391aab8fa562611fe637a57850b4aa (diff) |
block: bio_remaining_done() isn't unlikely
We use bio chaining during most I/Os these days due to the delayed
bio splitting. Additionally XFS will start using it, and there is
a pending direct I/O rewrite also making heavy use for it. Don't
pretend it's always unlikely, and let the branch predictor do it's
job instead.
Signed-off-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Jens Axboe <axboe@fb.com>
-rw-r--r-- | block/bio.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/block/bio.c b/block/bio.c index e4682ec11fcd..0fde6e0e81f2 100644 --- a/block/bio.c +++ b/block/bio.c | |||
@@ -1746,7 +1746,7 @@ static inline bool bio_remaining_done(struct bio *bio) | |||
1746 | void bio_endio(struct bio *bio) | 1746 | void bio_endio(struct bio *bio) |
1747 | { | 1747 | { |
1748 | again: | 1748 | again: |
1749 | if (unlikely(!bio_remaining_done(bio))) | 1749 | if (!bio_remaining_done(bio)) |
1750 | return; | 1750 | return; |
1751 | 1751 | ||
1752 | /* | 1752 | /* |