diff options
| author | Shaohua Li <shli@fb.com> | 2015-05-08 19:51:33 +0200 |
|---|---|---|
| committer | Jens Axboe <axboe@fb.com> | 2015-05-08 22:17:23 +0200 |
| commit | 5b3f341f098d60da2970758db6a05bd851eb6b39 (patch) | |
| tree | fd25ff2f67ac583a02f2cd288b7419609d373d00 /block/blk.h | |
| parent | blk-mq: do limited block plug for multiple queue case (diff) | |
| download | linux-5b3f341f098d60da2970758db6a05bd851eb6b39.tar.xz linux-5b3f341f098d60da2970758db6a05bd851eb6b39.zip | |
blk-mq: make plug work for mutiple disks and queues
Last patch makes plug work for multiple queue case. However it only
works for single disk case, because it assumes only one request in the
plug list. If a task is accessing multiple disks, eg MD/DM, the
assumption is wrong. Let blk_attempt_plug_merge() record request from
the same queue.
V2: use NULL parameter in !mq case. Fix a bug. Add comments in
blk_attempt_plug_merge to make it less (hopefully) confusion.
Cc: Jens Axboe <axboe@fb.com>
Cc: Christoph Hellwig <hch@lst.de>
Signed-off-by: Shaohua Li <shli@fb.com>
Signed-off-by: Jens Axboe <axboe@fb.com>
Diffstat (limited to '')
| -rw-r--r-- | block/blk.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/block/blk.h b/block/blk.h index 4b48d55e588e..026d9594142b 100644 --- a/block/blk.h +++ b/block/blk.h @@ -78,7 +78,8 @@ bool bio_attempt_front_merge(struct request_queue *q, struct request *req, bool bio_attempt_back_merge(struct request_queue *q, struct request *req, struct bio *bio); bool blk_attempt_plug_merge(struct request_queue *q, struct bio *bio, - unsigned int *request_count); + unsigned int *request_count, + struct request **same_queue_rq); void blk_account_io_start(struct request *req, bool new_io); void blk_account_io_completion(struct request *req, unsigned int bytes); |