diff options
author | Ming Lei <ming.lei@canonical.com> | 2014-09-25 11:23:47 -0400 |
---|---|---|
committer | Jens Axboe <axboe@fb.com> | 2014-09-25 17:22:45 -0400 |
commit | f70ced09170761acb69840cafaace4abc72cba4b (patch) | |
tree | bc62f5926a5e8b74be30316196a41b25ece12368 /block/blk-sysfs.c | |
parent | e97c293cdf77263abdc021de280516e0017afc84 (diff) |
blk-mq: support per-distpatch_queue flush machinery
This patch supports to run one single flush machinery for
each blk-mq dispatch queue, so that:
- current init_request and exit_request callbacks can
cover flush request too, then the buggy copying way of
initializing flush request's pdu can be fixed
- flushing performance gets improved in case of multi hw-queue
In fio sync write test over virtio-blk(4 hw queues, ioengine=sync,
iodepth=64, numjobs=4, bs=4K), it is observed that througput gets
increased a lot over my test environment:
- throughput: +70% in case of virtio-blk over null_blk
- throughput: +30% in case of virtio-blk over SSD image
The multi virtqueue feature isn't merged to QEMU yet, and patches for
the feature can be found in below tree:
git://kernel.ubuntu.com/ming/qemu.git v2.1.0-mq.4
And simply passing 'num_queues=4 vectors=5' should be enough to
enable multi queue(quad queue) feature for QEMU virtio-blk.
Suggested-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Ming Lei <ming.lei@canonical.com>
Signed-off-by: Jens Axboe <axboe@fb.com>
Diffstat (limited to 'block/blk-sysfs.c')
-rw-r--r-- | block/blk-sysfs.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/block/blk-sysfs.c b/block/blk-sysfs.c index 718cffc4c678..e8f38a36c625 100644 --- a/block/blk-sysfs.c +++ b/block/blk-sysfs.c | |||
@@ -517,10 +517,10 @@ static void blk_release_queue(struct kobject *kobj) | |||
517 | if (q->queue_tags) | 517 | if (q->queue_tags) |
518 | __blk_queue_free_tags(q); | 518 | __blk_queue_free_tags(q); |
519 | 519 | ||
520 | blk_free_flush_queue(q->fq); | ||
521 | |||
522 | if (q->mq_ops) | 520 | if (q->mq_ops) |
523 | blk_mq_free_queue(q); | 521 | blk_mq_free_queue(q); |
522 | else | ||
523 | blk_free_flush_queue(q->fq); | ||
524 | 524 | ||
525 | blk_trace_shutdown(q); | 525 | blk_trace_shutdown(q); |
526 | 526 | ||