summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMing Lei <ming.lei@canonical.com>2015-10-20 23:13:55 +0800
committerJens Axboe <axboe@fb.com>2015-10-21 15:00:54 -0600
commit7460d389c01741f0dfff733af93d3b3abd9b974e (patch)
treeec3904b3fab8e38bb5b2c583cdde993ef2d7f451
parente18378a60e27ad7b3e11ecc4e2c92159585dee68 (diff)
downloadop-kernel-dev-7460d389c01741f0dfff733af93d3b3abd9b974e.zip
op-kernel-dev-7460d389c01741f0dfff733af93d3b3abd9b974e.tar.gz
block: check bio_mergeable() early before merging
After bio splitting is introduced, one bio can be splitted and it is marked as NOMERGE because it is too fat to be merged, so check bio_mergeable() earlier to avoid to try to merge it unnecessarily. Signed-off-by: Ming Lei <ming.lei@canonical.com> Reviewed-by: Jeff Moyer <jmoyer@redhat.com> Signed-off-by: Jens Axboe <axboe@fb.com>
-rw-r--r--block/elevator.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/block/elevator.c b/block/elevator.c
index 84d6394..c3555c9 100644
--- a/block/elevator.c
+++ b/block/elevator.c
@@ -420,7 +420,7 @@ int elv_merge(struct request_queue *q, struct request **req, struct bio *bio)
* noxmerges: Only simple one-hit cache try
* merges: All merge tries attempted
*/
- if (blk_queue_nomerges(q))
+ if (blk_queue_nomerges(q) || !bio_mergeable(bio))
return ELEVATOR_NO_MERGE;
/*
OpenPOWER on IntegriCloud