diff options
| author | Christoph Hellwig | 2016-03-11 17:34:53 +0100 |
|---|---|---|
| committer | Jens Axboe | 2016-03-14 15:55:25 +0100 |
| commit | 2b885517110cbe8724fef30363778b6284d0a428 (patch) | |
| tree | 609935ecfa54b2e31752f375ec1dfb46cf3fca71 /block | |
| parent | block: cleanup bio_endio (diff) | |
| download | kernel-qcow2-linux-2b885517110cbe8724fef30363778b6284d0a428.tar.gz kernel-qcow2-linux-2b885517110cbe8724fef30363778b6284d0a428.tar.xz kernel-qcow2-linux-2b885517110cbe8724fef30363778b6284d0a428.zip | |
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>
Diffstat (limited to 'block')
| -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) void bio_endio(struct bio *bio) { again: - if (unlikely(!bio_remaining_done(bio))) + if (!bio_remaining_done(bio)) return; /* |