2017-01-17 16:03:22 +03:00
/*
* blk - mq scheduling framework
*
* Copyright ( C ) 2016 Jens Axboe
*/
# include <linux/kernel.h>
# include <linux/module.h>
# include <linux/blk-mq.h>
# include <trace/events/block.h>
# include "blk.h"
# include "blk-mq.h"
2017-05-04 17:24:40 +03:00
# include "blk-mq-debugfs.h"
2017-01-17 16:03:22 +03:00
# include "blk-mq-sched.h"
# include "blk-mq-tag.h"
# include "blk-wbt.h"
void blk_mq_sched_free_hctx_data ( struct request_queue * q ,
void ( * exit ) ( struct blk_mq_hw_ctx * ) )
{
struct blk_mq_hw_ctx * hctx ;
int i ;
queue_for_each_hw_ctx ( q , hctx , i ) {
if ( exit & & hctx - > sched_data )
exit ( hctx ) ;
kfree ( hctx - > sched_data ) ;
hctx - > sched_data = NULL ;
}
}
EXPORT_SYMBOL_GPL ( blk_mq_sched_free_hctx_data ) ;
2017-06-16 19:15:25 +03:00
void blk_mq_sched_assign_ioc ( struct request * rq , struct bio * bio )
2017-01-17 16:03:22 +03:00
{
2017-06-16 19:15:25 +03:00
struct request_queue * q = rq - > q ;
struct io_context * ioc = rq_ioc ( bio ) ;
2017-01-17 16:03:22 +03:00
struct io_cq * icq ;
spin_lock_irq ( q - > queue_lock ) ;
icq = ioc_lookup_icq ( ioc , q ) ;
spin_unlock_irq ( q - > queue_lock ) ;
if ( ! icq ) {
icq = ioc_create_icq ( ioc , q , GFP_ATOMIC ) ;
if ( ! icq )
return ;
}
2017-06-16 19:15:20 +03:00
get_io_context ( icq - > ioc ) ;
2017-06-16 19:15:25 +03:00
rq - > elv . icq = icq ;
2017-01-17 16:03:22 +03:00
}
2017-06-21 02:56:13 +03:00
/*
* Mark a hardware queue as needing a restart . For shared queues , maintain
* a count of how many hardware queues are marked for restart .
*/
static void blk_mq_sched_mark_restart_hctx ( struct blk_mq_hw_ctx * hctx )
{
if ( test_bit ( BLK_MQ_S_SCHED_RESTART , & hctx - > state ) )
return ;
if ( hctx - > flags & BLK_MQ_F_TAG_SHARED ) {
struct request_queue * q = hctx - > queue ;
if ( ! test_and_set_bit ( BLK_MQ_S_SCHED_RESTART , & hctx - > state ) )
atomic_inc ( & q - > shared_hctx_restart ) ;
} else
set_bit ( BLK_MQ_S_SCHED_RESTART , & hctx - > state ) ;
}
blk-mq: don't handle TAG_SHARED in restart
Now restart is used in the following cases, and TAG_SHARED is for
SCSI only.
1) .get_budget() returns BLK_STS_RESOURCE
- if resource in target/host level isn't satisfied, this SCSI device
will be added in shost->starved_list, and the whole queue will be rerun
(via SCSI's built-in RESTART) in scsi_end_request() after any request
initiated from this host/targe is completed. Forget to mention, host level
resource can't be an issue for blk-mq at all.
- the same is true if resource in the queue level isn't satisfied.
- if there isn't outstanding request on this queue, then SCSI's RESTART
can't work(blk-mq's can't work too), and the queue will be run after
SCSI_QUEUE_DELAY, and finally all starved sdevs will be handled by SCSI's
RESTART when this request is finished
2) scsi_dispatch_cmd() returns BLK_STS_RESOURCE
- if there isn't onprogressing request on this queue, the queue
will be run after SCSI_QUEUE_DELAY
- otherwise, SCSI's RESTART covers the rerun.
3) blk_mq_get_driver_tag() failed
- BLK_MQ_S_TAG_WAITING covers the cross-queue RESTART for driver
allocation.
In one word, SCSI's built-in RESTART is enough to cover the queue
rerun, and we don't need to pay special attention to TAG_SHARED wrt. restart.
In my test on scsi_debug(8 luns), this patch improves IOPS by 20% ~ 30% when
running I/O on these 8 luns concurrently.
Aslo Roman Pen reported the current RESTART is very expensive especialy
when there are lots of LUNs attached in one host, such as in his
test, RESTART causes half of IOPS be cut.
Fixes: https://marc.info/?l=linux-kernel&m=150832216727524&w=2
Fixes: 6d8c6c0f97ad ("blk-mq: Restart a single queue if tag sets are shared")
Signed-off-by: Ming Lei <ming.lei@redhat.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2017-10-27 07:43:29 +03:00
void blk_mq_sched_restart ( struct blk_mq_hw_ctx * hctx )
2017-06-21 02:56:13 +03:00
{
if ( ! test_bit ( BLK_MQ_S_SCHED_RESTART , & hctx - > state ) )
blk-mq: don't handle TAG_SHARED in restart
Now restart is used in the following cases, and TAG_SHARED is for
SCSI only.
1) .get_budget() returns BLK_STS_RESOURCE
- if resource in target/host level isn't satisfied, this SCSI device
will be added in shost->starved_list, and the whole queue will be rerun
(via SCSI's built-in RESTART) in scsi_end_request() after any request
initiated from this host/targe is completed. Forget to mention, host level
resource can't be an issue for blk-mq at all.
- the same is true if resource in the queue level isn't satisfied.
- if there isn't outstanding request on this queue, then SCSI's RESTART
can't work(blk-mq's can't work too), and the queue will be run after
SCSI_QUEUE_DELAY, and finally all starved sdevs will be handled by SCSI's
RESTART when this request is finished
2) scsi_dispatch_cmd() returns BLK_STS_RESOURCE
- if there isn't onprogressing request on this queue, the queue
will be run after SCSI_QUEUE_DELAY
- otherwise, SCSI's RESTART covers the rerun.
3) blk_mq_get_driver_tag() failed
- BLK_MQ_S_TAG_WAITING covers the cross-queue RESTART for driver
allocation.
In one word, SCSI's built-in RESTART is enough to cover the queue
rerun, and we don't need to pay special attention to TAG_SHARED wrt. restart.
In my test on scsi_debug(8 luns), this patch improves IOPS by 20% ~ 30% when
running I/O on these 8 luns concurrently.
Aslo Roman Pen reported the current RESTART is very expensive especialy
when there are lots of LUNs attached in one host, such as in his
test, RESTART causes half of IOPS be cut.
Fixes: https://marc.info/?l=linux-kernel&m=150832216727524&w=2
Fixes: 6d8c6c0f97ad ("blk-mq: Restart a single queue if tag sets are shared")
Signed-off-by: Ming Lei <ming.lei@redhat.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2017-10-27 07:43:29 +03:00
return ;
2017-06-21 02:56:13 +03:00
blk-mq: don't handle TAG_SHARED in restart
Now restart is used in the following cases, and TAG_SHARED is for
SCSI only.
1) .get_budget() returns BLK_STS_RESOURCE
- if resource in target/host level isn't satisfied, this SCSI device
will be added in shost->starved_list, and the whole queue will be rerun
(via SCSI's built-in RESTART) in scsi_end_request() after any request
initiated from this host/targe is completed. Forget to mention, host level
resource can't be an issue for blk-mq at all.
- the same is true if resource in the queue level isn't satisfied.
- if there isn't outstanding request on this queue, then SCSI's RESTART
can't work(blk-mq's can't work too), and the queue will be run after
SCSI_QUEUE_DELAY, and finally all starved sdevs will be handled by SCSI's
RESTART when this request is finished
2) scsi_dispatch_cmd() returns BLK_STS_RESOURCE
- if there isn't onprogressing request on this queue, the queue
will be run after SCSI_QUEUE_DELAY
- otherwise, SCSI's RESTART covers the rerun.
3) blk_mq_get_driver_tag() failed
- BLK_MQ_S_TAG_WAITING covers the cross-queue RESTART for driver
allocation.
In one word, SCSI's built-in RESTART is enough to cover the queue
rerun, and we don't need to pay special attention to TAG_SHARED wrt. restart.
In my test on scsi_debug(8 luns), this patch improves IOPS by 20% ~ 30% when
running I/O on these 8 luns concurrently.
Aslo Roman Pen reported the current RESTART is very expensive especialy
when there are lots of LUNs attached in one host, such as in his
test, RESTART causes half of IOPS be cut.
Fixes: https://marc.info/?l=linux-kernel&m=150832216727524&w=2
Fixes: 6d8c6c0f97ad ("blk-mq: Restart a single queue if tag sets are shared")
Signed-off-by: Ming Lei <ming.lei@redhat.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2017-10-27 07:43:29 +03:00
clear_bit ( BLK_MQ_S_SCHED_RESTART , & hctx - > state ) ;
2017-06-21 02:56:13 +03:00
if ( blk_mq_hctx_has_pending ( hctx ) ) {
blk_mq_run_hw_queue ( hctx , true ) ;
blk-mq: don't handle TAG_SHARED in restart
Now restart is used in the following cases, and TAG_SHARED is for
SCSI only.
1) .get_budget() returns BLK_STS_RESOURCE
- if resource in target/host level isn't satisfied, this SCSI device
will be added in shost->starved_list, and the whole queue will be rerun
(via SCSI's built-in RESTART) in scsi_end_request() after any request
initiated from this host/targe is completed. Forget to mention, host level
resource can't be an issue for blk-mq at all.
- the same is true if resource in the queue level isn't satisfied.
- if there isn't outstanding request on this queue, then SCSI's RESTART
can't work(blk-mq's can't work too), and the queue will be run after
SCSI_QUEUE_DELAY, and finally all starved sdevs will be handled by SCSI's
RESTART when this request is finished
2) scsi_dispatch_cmd() returns BLK_STS_RESOURCE
- if there isn't onprogressing request on this queue, the queue
will be run after SCSI_QUEUE_DELAY
- otherwise, SCSI's RESTART covers the rerun.
3) blk_mq_get_driver_tag() failed
- BLK_MQ_S_TAG_WAITING covers the cross-queue RESTART for driver
allocation.
In one word, SCSI's built-in RESTART is enough to cover the queue
rerun, and we don't need to pay special attention to TAG_SHARED wrt. restart.
In my test on scsi_debug(8 luns), this patch improves IOPS by 20% ~ 30% when
running I/O on these 8 luns concurrently.
Aslo Roman Pen reported the current RESTART is very expensive especialy
when there are lots of LUNs attached in one host, such as in his
test, RESTART causes half of IOPS be cut.
Fixes: https://marc.info/?l=linux-kernel&m=150832216727524&w=2
Fixes: 6d8c6c0f97ad ("blk-mq: Restart a single queue if tag sets are shared")
Signed-off-by: Ming Lei <ming.lei@redhat.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2017-10-27 07:43:29 +03:00
return ;
2017-06-21 02:56:13 +03:00
}
}
2017-10-27 07:43:30 +03:00
/*
* Only SCSI implements . get_budget and . put_budget , and SCSI restarts
* its queue by itself in its completion handler , so we don ' t need to
* restart queue if . get_budget ( ) returns BLK_STS_NO_RESOURCE .
*/
static void blk_mq_do_dispatch_sched ( struct blk_mq_hw_ctx * hctx )
2017-10-14 12:22:26 +03:00
{
struct request_queue * q = hctx - > queue ;
struct elevator_queue * e = q - > elevator ;
LIST_HEAD ( rq_list ) ;
do {
2017-10-14 12:22:29 +03:00
struct request * rq ;
2017-10-14 12:22:26 +03:00
2017-10-14 12:22:29 +03:00
if ( e - > type - > ops . mq . has_work & &
! e - > type - > ops . mq . has_work ( hctx ) )
2017-10-14 12:22:26 +03:00
break ;
2017-10-14 12:22:29 +03:00
2017-11-04 21:21:12 +03:00
if ( ! blk_mq_get_dispatch_budget ( hctx ) )
2017-10-27 07:43:30 +03:00
break ;
2017-10-14 12:22:29 +03:00
rq = e - > type - > ops . mq . dispatch_request ( hctx ) ;
if ( ! rq ) {
blk_mq_put_dispatch_budget ( hctx ) ;
break ;
}
/*
* Now this rq owns the budget which has to be released
* if this rq won ' t be queued to driver via . queue_rq ( )
* in blk_mq_dispatch_rq_list ( ) .
*/
2017-10-14 12:22:26 +03:00
list_add ( & rq - > queuelist , & rq_list ) ;
2017-10-14 12:22:29 +03:00
} while ( blk_mq_dispatch_rq_list ( q , & rq_list , true ) ) ;
2017-10-14 12:22:26 +03:00
}
blk-mq-sched: improve dispatching from sw queue
SCSI devices use host-wide tagset, and the shared driver tag space is
often quite big. However, there is also a queue depth for each lun(
.cmd_per_lun), which is often small, for example, on both lpfc and
qla2xxx, .cmd_per_lun is just 3.
So lots of requests may stay in sw queue, and we always flush all
belonging to same hw queue and dispatch them all to driver.
Unfortunately it is easy to cause queue busy because of the small
.cmd_per_lun. Once these requests are flushed out, they have to stay in
hctx->dispatch, and no bio merge can happen on these requests, and
sequential IO performance is harmed.
This patch introduces blk_mq_dequeue_from_ctx for dequeuing a request
from a sw queue, so that we can dispatch them in scheduler's way. We can
then avoid dequeueing too many requests from sw queue, since we don't
flush ->dispatch completely.
This patch improves dispatching from sw queue by using the .get_budget
and .put_budget callbacks.
Reviewed-by: Omar Sandoval <osandov@fb.com>
Signed-off-by: Ming Lei <ming.lei@redhat.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2017-10-14 12:22:30 +03:00
static struct blk_mq_ctx * blk_mq_next_ctx ( struct blk_mq_hw_ctx * hctx ,
struct blk_mq_ctx * ctx )
{
unsigned idx = ctx - > index_hw ;
if ( + + idx = = hctx - > nr_ctx )
idx = 0 ;
return hctx - > ctxs [ idx ] ;
}
2017-10-27 07:43:30 +03:00
/*
* Only SCSI implements . get_budget and . put_budget , and SCSI restarts
* its queue by itself in its completion handler , so we don ' t need to
* restart queue if . get_budget ( ) returns BLK_STS_NO_RESOURCE .
*/
static void blk_mq_do_dispatch_ctx ( struct blk_mq_hw_ctx * hctx )
blk-mq-sched: improve dispatching from sw queue
SCSI devices use host-wide tagset, and the shared driver tag space is
often quite big. However, there is also a queue depth for each lun(
.cmd_per_lun), which is often small, for example, on both lpfc and
qla2xxx, .cmd_per_lun is just 3.
So lots of requests may stay in sw queue, and we always flush all
belonging to same hw queue and dispatch them all to driver.
Unfortunately it is easy to cause queue busy because of the small
.cmd_per_lun. Once these requests are flushed out, they have to stay in
hctx->dispatch, and no bio merge can happen on these requests, and
sequential IO performance is harmed.
This patch introduces blk_mq_dequeue_from_ctx for dequeuing a request
from a sw queue, so that we can dispatch them in scheduler's way. We can
then avoid dequeueing too many requests from sw queue, since we don't
flush ->dispatch completely.
This patch improves dispatching from sw queue by using the .get_budget
and .put_budget callbacks.
Reviewed-by: Omar Sandoval <osandov@fb.com>
Signed-off-by: Ming Lei <ming.lei@redhat.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2017-10-14 12:22:30 +03:00
{
struct request_queue * q = hctx - > queue ;
LIST_HEAD ( rq_list ) ;
struct blk_mq_ctx * ctx = READ_ONCE ( hctx - > dispatch_from ) ;
do {
struct request * rq ;
if ( ! sbitmap_any_bit_set ( & hctx - > ctx_map ) )
break ;
2017-11-04 21:21:12 +03:00
if ( ! blk_mq_get_dispatch_budget ( hctx ) )
2017-10-27 07:43:30 +03:00
break ;
blk-mq-sched: improve dispatching from sw queue
SCSI devices use host-wide tagset, and the shared driver tag space is
often quite big. However, there is also a queue depth for each lun(
.cmd_per_lun), which is often small, for example, on both lpfc and
qla2xxx, .cmd_per_lun is just 3.
So lots of requests may stay in sw queue, and we always flush all
belonging to same hw queue and dispatch them all to driver.
Unfortunately it is easy to cause queue busy because of the small
.cmd_per_lun. Once these requests are flushed out, they have to stay in
hctx->dispatch, and no bio merge can happen on these requests, and
sequential IO performance is harmed.
This patch introduces blk_mq_dequeue_from_ctx for dequeuing a request
from a sw queue, so that we can dispatch them in scheduler's way. We can
then avoid dequeueing too many requests from sw queue, since we don't
flush ->dispatch completely.
This patch improves dispatching from sw queue by using the .get_budget
and .put_budget callbacks.
Reviewed-by: Omar Sandoval <osandov@fb.com>
Signed-off-by: Ming Lei <ming.lei@redhat.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2017-10-14 12:22:30 +03:00
rq = blk_mq_dequeue_from_ctx ( hctx , ctx ) ;
if ( ! rq ) {
blk_mq_put_dispatch_budget ( hctx ) ;
break ;
}
/*
* Now this rq owns the budget which has to be released
* if this rq won ' t be queued to driver via . queue_rq ( )
* in blk_mq_dispatch_rq_list ( ) .
*/
list_add ( & rq - > queuelist , & rq_list ) ;
/* round robin for fair dispatch */
ctx = blk_mq_next_ctx ( hctx , rq - > mq_ctx ) ;
} while ( blk_mq_dispatch_rq_list ( q , & rq_list , true ) ) ;
WRITE_ONCE ( hctx - > dispatch_from , ctx ) ;
}
2017-10-14 12:22:29 +03:00
/* return true if hw queue need to be run again */
2017-10-27 07:43:30 +03:00
void blk_mq_sched_dispatch_requests ( struct blk_mq_hw_ctx * hctx )
2017-01-17 16:03:22 +03:00
{
blk-mq: use the right hctx when getting a driver tag fails
While dispatching requests, if we fail to get a driver tag, we mark the
hardware queue as waiting for a tag and put the requests on a
hctx->dispatch list to be run later when a driver tag is freed. However,
blk_mq_dispatch_rq_list() may dispatch requests from multiple hardware
queues if using a single-queue scheduler with a multiqueue device. If
blk_mq_get_driver_tag() fails, it doesn't update the hardware queue we
are processing. This means we end up using the hardware queue of the
previous request, which may or may not be the same as that of the
current request. If it isn't, the wrong hardware queue will end up
waiting for a tag, and the requests will be on the wrong dispatch list,
leading to a hang.
The fix is twofold:
1. Make sure we save which hardware queue we were trying to get a
request for in blk_mq_get_driver_tag() regardless of whether it
succeeds or not.
2. Make blk_mq_dispatch_rq_list() take a request_queue instead of a
blk_mq_hw_queue to make it clear that it must handle multiple
hardware queues, since I've already messed this up on a couple of
occasions.
This didn't appear in testing with nvme and mq-deadline because nvme has
more driver tags than the default number of scheduler tags. However,
with the blk_mq_update_nr_hw_queues() fix, it showed up with nbd.
Signed-off-by: Omar Sandoval <osandov@fb.com>
Signed-off-by: Jens Axboe <axboe@fb.com>
2017-04-07 17:56:26 +03:00
struct request_queue * q = hctx - > queue ;
struct elevator_queue * e = q - > elevator ;
2017-02-17 21:39:26 +03:00
const bool has_sched_dispatch = e & & e - > type - > ops . mq . dispatch_request ;
2017-01-17 16:03:22 +03:00
LIST_HEAD ( rq_list ) ;
2017-06-18 23:24:27 +03:00
/* RCU or SRCU read lock is needed before checking quiesced flag */
if ( unlikely ( blk_mq_hctx_stopped ( hctx ) | | blk_queue_quiesced ( q ) ) )
2017-10-27 07:43:30 +03:00
return ;
2017-01-17 16:03:22 +03:00
hctx - > run + + ;
/*
* If we have previous entries on our dispatch list , grab them first for
* more fair dispatch .
*/
if ( ! list_empty_careful ( & hctx - > dispatch ) ) {
spin_lock ( & hctx - > lock ) ;
if ( ! list_empty ( & hctx - > dispatch ) )
list_splice_init ( & hctx - > dispatch , & rq_list ) ;
spin_unlock ( & hctx - > lock ) ;
}
/*
* Only ask the scheduler for requests , if we didn ' t have residual
* requests from the dispatch list . This is to avoid the case where
* we only ever dispatch a fraction of the requests available because
* of low device queue depth . Once we pull requests out of the IO
* scheduler , we can no longer merge or sort them . So it ' s best to
* leave them there for as long as we can . Mark the hw queue as
* needing a restart in that case .
2017-10-14 12:22:26 +03:00
*
* We want to dispatch from the scheduler if there was nothing
* on the dispatch list or we were able to dispatch from the
* dispatch list .
2017-01-17 16:03:22 +03:00
*/
2017-01-26 22:40:07 +03:00
if ( ! list_empty ( & rq_list ) ) {
2017-02-22 21:58:30 +03:00
blk_mq_sched_mark_restart_hctx ( hctx ) ;
blk-mq-sched: improve dispatching from sw queue
SCSI devices use host-wide tagset, and the shared driver tag space is
often quite big. However, there is also a queue depth for each lun(
.cmd_per_lun), which is often small, for example, on both lpfc and
qla2xxx, .cmd_per_lun is just 3.
So lots of requests may stay in sw queue, and we always flush all
belonging to same hw queue and dispatch them all to driver.
Unfortunately it is easy to cause queue busy because of the small
.cmd_per_lun. Once these requests are flushed out, they have to stay in
hctx->dispatch, and no bio merge can happen on these requests, and
sequential IO performance is harmed.
This patch introduces blk_mq_dequeue_from_ctx for dequeuing a request
from a sw queue, so that we can dispatch them in scheduler's way. We can
then avoid dequeueing too many requests from sw queue, since we don't
flush ->dispatch completely.
This patch improves dispatching from sw queue by using the .get_budget
and .put_budget callbacks.
Reviewed-by: Omar Sandoval <osandov@fb.com>
Signed-off-by: Ming Lei <ming.lei@redhat.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2017-10-14 12:22:30 +03:00
if ( blk_mq_dispatch_rq_list ( q , & rq_list , false ) ) {
if ( has_sched_dispatch )
2017-10-27 07:43:30 +03:00
blk_mq_do_dispatch_sched ( hctx ) ;
blk-mq-sched: improve dispatching from sw queue
SCSI devices use host-wide tagset, and the shared driver tag space is
often quite big. However, there is also a queue depth for each lun(
.cmd_per_lun), which is often small, for example, on both lpfc and
qla2xxx, .cmd_per_lun is just 3.
So lots of requests may stay in sw queue, and we always flush all
belonging to same hw queue and dispatch them all to driver.
Unfortunately it is easy to cause queue busy because of the small
.cmd_per_lun. Once these requests are flushed out, they have to stay in
hctx->dispatch, and no bio merge can happen on these requests, and
sequential IO performance is harmed.
This patch introduces blk_mq_dequeue_from_ctx for dequeuing a request
from a sw queue, so that we can dispatch them in scheduler's way. We can
then avoid dequeueing too many requests from sw queue, since we don't
flush ->dispatch completely.
This patch improves dispatching from sw queue by using the .get_budget
and .put_budget callbacks.
Reviewed-by: Omar Sandoval <osandov@fb.com>
Signed-off-by: Ming Lei <ming.lei@redhat.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2017-10-14 12:22:30 +03:00
else
2017-10-27 07:43:30 +03:00
blk_mq_do_dispatch_ctx ( hctx ) ;
blk-mq-sched: improve dispatching from sw queue
SCSI devices use host-wide tagset, and the shared driver tag space is
often quite big. However, there is also a queue depth for each lun(
.cmd_per_lun), which is often small, for example, on both lpfc and
qla2xxx, .cmd_per_lun is just 3.
So lots of requests may stay in sw queue, and we always flush all
belonging to same hw queue and dispatch them all to driver.
Unfortunately it is easy to cause queue busy because of the small
.cmd_per_lun. Once these requests are flushed out, they have to stay in
hctx->dispatch, and no bio merge can happen on these requests, and
sequential IO performance is harmed.
This patch introduces blk_mq_dequeue_from_ctx for dequeuing a request
from a sw queue, so that we can dispatch them in scheduler's way. We can
then avoid dequeueing too many requests from sw queue, since we don't
flush ->dispatch completely.
This patch improves dispatching from sw queue by using the .get_budget
and .put_budget callbacks.
Reviewed-by: Omar Sandoval <osandov@fb.com>
Signed-off-by: Ming Lei <ming.lei@redhat.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2017-10-14 12:22:30 +03:00
}
2017-10-14 12:22:26 +03:00
} else if ( has_sched_dispatch ) {
2017-10-27 07:43:30 +03:00
blk_mq_do_dispatch_sched ( hctx ) ;
blk-mq-sched: improve dispatching from sw queue
SCSI devices use host-wide tagset, and the shared driver tag space is
often quite big. However, there is also a queue depth for each lun(
.cmd_per_lun), which is often small, for example, on both lpfc and
qla2xxx, .cmd_per_lun is just 3.
So lots of requests may stay in sw queue, and we always flush all
belonging to same hw queue and dispatch them all to driver.
Unfortunately it is easy to cause queue busy because of the small
.cmd_per_lun. Once these requests are flushed out, they have to stay in
hctx->dispatch, and no bio merge can happen on these requests, and
sequential IO performance is harmed.
This patch introduces blk_mq_dequeue_from_ctx for dequeuing a request
from a sw queue, so that we can dispatch them in scheduler's way. We can
then avoid dequeueing too many requests from sw queue, since we don't
flush ->dispatch completely.
This patch improves dispatching from sw queue by using the .get_budget
and .put_budget callbacks.
Reviewed-by: Omar Sandoval <osandov@fb.com>
Signed-off-by: Ming Lei <ming.lei@redhat.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
2017-10-14 12:22:30 +03:00
} else if ( q - > mq_ops - > get_budget ) {
/*
* If we need to get budget before queuing request , we
* dequeue request one by one from sw queue for avoiding
* to mess up I / O merge when dispatch runs out of resource .
*
* TODO : get more budgets , and dequeue more requests in
* one time .
*/
2017-10-27 07:43:30 +03:00
blk_mq_do_dispatch_ctx ( hctx ) ;
2017-10-14 12:22:26 +03:00
} else {
2017-01-26 22:40:07 +03:00
blk_mq_flush_busy_ctxs ( hctx , & rq_list ) ;
2017-10-14 12:22:29 +03:00
blk_mq_dispatch_rq_list ( q , & rq_list , false ) ;
2017-02-17 21:39:26 +03:00
}
2017-01-17 16:03:22 +03:00
}
2017-02-03 19:48:28 +03:00
bool blk_mq_sched_try_merge ( struct request_queue * q , struct bio * bio ,
struct request * * merged_request )
2017-01-17 16:03:22 +03:00
{
struct request * rq ;
2017-02-08 16:46:48 +03:00
switch ( elv_merge ( q , & rq , bio ) ) {
case ELEVATOR_BACK_MERGE :
2017-01-17 16:03:22 +03:00
if ( ! blk_mq_sched_allow_merge ( q , rq , bio ) )
return false ;
2017-02-08 16:46:48 +03:00
if ( ! bio_attempt_back_merge ( q , rq , bio ) )
return false ;
* merged_request = attempt_back_merge ( q , rq ) ;
if ( ! * merged_request )
elv_merged_request ( q , rq , ELEVATOR_BACK_MERGE ) ;
return true ;
case ELEVATOR_FRONT_MERGE :
2017-01-17 16:03:22 +03:00
if ( ! blk_mq_sched_allow_merge ( q , rq , bio ) )
return false ;
2017-02-08 16:46:48 +03:00
if ( ! bio_attempt_front_merge ( q , rq , bio ) )
return false ;
* merged_request = attempt_front_merge ( q , rq ) ;
if ( ! * merged_request )
elv_merged_request ( q , rq , ELEVATOR_FRONT_MERGE ) ;
return true ;
default :
return false ;
2017-01-17 16:03:22 +03:00
}
}
EXPORT_SYMBOL_GPL ( blk_mq_sched_try_merge ) ;
2017-05-26 14:53:20 +03:00
/*
* Reverse check our software queue for entries that we could potentially
* merge with . Currently includes a hand - wavy stop count of 8 , to not spend
* too much time checking for merges .
*/
static bool blk_mq_attempt_merge ( struct request_queue * q ,
struct blk_mq_ctx * ctx , struct bio * bio )
{
struct request * rq ;
int checked = 8 ;
2017-06-20 21:15:47 +03:00
lockdep_assert_held ( & ctx - > lock ) ;
2017-05-26 14:53:20 +03:00
list_for_each_entry_reverse ( rq , & ctx - > rq_list , queuelist ) {
bool merged = false ;
if ( ! checked - - )
break ;
if ( ! blk_rq_merge_ok ( rq , bio ) )
continue ;
switch ( blk_try_merge ( rq , bio ) ) {
case ELEVATOR_BACK_MERGE :
if ( blk_mq_sched_allow_merge ( q , rq , bio ) )
merged = bio_attempt_back_merge ( q , rq , bio ) ;
break ;
case ELEVATOR_FRONT_MERGE :
if ( blk_mq_sched_allow_merge ( q , rq , bio ) )
merged = bio_attempt_front_merge ( q , rq , bio ) ;
break ;
case ELEVATOR_DISCARD_MERGE :
merged = bio_attempt_discard_merge ( q , rq , bio ) ;
break ;
default :
continue ;
}
if ( merged )
ctx - > rq_merged + + ;
return merged ;
}
return false ;
}
2017-01-17 16:03:22 +03:00
bool __blk_mq_sched_bio_merge ( struct request_queue * q , struct bio * bio )
{
struct elevator_queue * e = q - > elevator ;
2017-05-26 14:53:20 +03:00
struct blk_mq_ctx * ctx = blk_mq_get_ctx ( q ) ;
struct blk_mq_hw_ctx * hctx = blk_mq_map_queue ( q , ctx - > cpu ) ;
bool ret = false ;
2017-01-17 16:03:22 +03:00
2017-05-26 14:53:20 +03:00
if ( e & & e - > type - > ops . mq . bio_merge ) {
2017-01-17 16:03:22 +03:00
blk_mq_put_ctx ( ctx ) ;
return e - > type - > ops . mq . bio_merge ( hctx , bio ) ;
}
2017-05-26 14:53:20 +03:00
if ( hctx - > flags & BLK_MQ_F_SHOULD_MERGE ) {
/* default per sw-queue merge */
spin_lock ( & ctx - > lock ) ;
ret = blk_mq_attempt_merge ( q , ctx , bio ) ;
spin_unlock ( & ctx - > lock ) ;
}
blk_mq_put_ctx ( ctx ) ;
return ret ;
2017-01-17 16:03:22 +03:00
}
bool blk_mq_sched_try_insert_merge ( struct request_queue * q , struct request * rq )
{
return rq_mergeable ( rq ) & & elv_attempt_insert_merge ( q , rq ) ;
}
EXPORT_SYMBOL_GPL ( blk_mq_sched_try_insert_merge ) ;
void blk_mq_sched_request_inserted ( struct request * rq )
{
trace_block_rq_insert ( rq - > q , rq ) ;
}
EXPORT_SYMBOL_GPL ( blk_mq_sched_request_inserted ) ;
2017-02-03 02:42:39 +03:00
static bool blk_mq_sched_bypass_insert ( struct blk_mq_hw_ctx * hctx ,
2017-11-02 18:24:36 +03:00
bool has_sched ,
2017-02-03 02:42:39 +03:00
struct request * rq )
2017-01-17 16:03:22 +03:00
{
2017-11-02 18:24:36 +03:00
/* dispatch flush rq directly */
if ( rq - > rq_flags & RQF_FLUSH_SEQ ) {
spin_lock ( & hctx - > lock ) ;
list_add ( & rq - > queuelist , & hctx - > dispatch ) ;
spin_unlock ( & hctx - > lock ) ;
return true ;
}
if ( has_sched ) {
2017-01-17 16:03:22 +03:00
rq - > rq_flags | = RQF_SORTED ;
2017-11-02 18:24:36 +03:00
WARN_ON ( rq - > tag ! = - 1 ) ;
2017-01-17 16:03:22 +03:00
}
2017-11-02 18:24:36 +03:00
return false ;
2017-01-17 16:03:22 +03:00
}
2017-01-27 11:00:47 +03:00
/*
* Add flush / fua to the queue . If we fail getting a driver tag , then
* punt to the requeue list . Requeue will re - invoke us from a context
* that ' s safe to block from .
*/
static void blk_mq_sched_insert_flush ( struct blk_mq_hw_ctx * hctx ,
struct request * rq , bool can_block )
{
if ( blk_mq_get_driver_tag ( rq , & hctx , can_block ) ) {
blk_insert_flush ( rq ) ;
blk_mq_run_hw_queue ( hctx , true ) ;
} else
2017-02-17 21:37:14 +03:00
blk_mq_add_to_requeue_list ( rq , false , true ) ;
2017-01-27 11:00:47 +03:00
}
void blk_mq_sched_insert_request ( struct request * rq , bool at_head ,
bool run_queue , bool async , bool can_block )
{
struct request_queue * q = rq - > q ;
struct elevator_queue * e = q - > elevator ;
struct blk_mq_ctx * ctx = rq - > mq_ctx ;
struct blk_mq_hw_ctx * hctx = blk_mq_map_queue ( q , ctx - > cpu ) ;
2017-11-02 18:24:36 +03:00
/* flush rq in flush machinery need to be dispatched directly */
if ( ! ( rq - > rq_flags & RQF_FLUSH_SEQ ) & & op_is_flush ( rq - > cmd_flags ) ) {
2017-01-27 11:00:47 +03:00
blk_mq_sched_insert_flush ( hctx , rq , can_block ) ;
return ;
}
2017-11-02 18:24:36 +03:00
if ( blk_mq_sched_bypass_insert ( hctx , ! ! e , rq ) )
2017-02-03 02:42:39 +03:00
goto run ;
2017-01-27 11:00:47 +03:00
if ( e & & e - > type - > ops . mq . insert_requests ) {
LIST_HEAD ( list ) ;
list_add ( & rq - > queuelist , & list ) ;
e - > type - > ops . mq . insert_requests ( hctx , & list , at_head ) ;
} else {
spin_lock ( & ctx - > lock ) ;
__blk_mq_insert_request ( hctx , rq , at_head ) ;
spin_unlock ( & ctx - > lock ) ;
}
2017-02-03 02:42:39 +03:00
run :
2017-01-27 11:00:47 +03:00
if ( run_queue )
blk_mq_run_hw_queue ( hctx , async ) ;
}
void blk_mq_sched_insert_requests ( struct request_queue * q ,
struct blk_mq_ctx * ctx ,
struct list_head * list , bool run_queue_async )
{
struct blk_mq_hw_ctx * hctx = blk_mq_map_queue ( q , ctx - > cpu ) ;
struct elevator_queue * e = hctx - > queue - > elevator ;
2017-02-03 02:42:39 +03:00
if ( e ) {
struct request * rq , * next ;
/*
* We bypass requests that already have a driver tag assigned ,
* which should only be flushes . Flushes are only ever inserted
* as single requests , so we shouldn ' t ever hit the
* WARN_ON_ONCE ( ) below ( but let ' s handle it just in case ) .
*/
list_for_each_entry_safe ( rq , next , list , queuelist ) {
if ( WARN_ON_ONCE ( rq - > tag ! = - 1 ) ) {
list_del_init ( & rq - > queuelist ) ;
2017-11-02 18:24:36 +03:00
blk_mq_sched_bypass_insert ( hctx , true , rq ) ;
2017-02-03 02:42:39 +03:00
}
}
}
2017-01-27 11:00:47 +03:00
if ( e & & e - > type - > ops . mq . insert_requests )
e - > type - > ops . mq . insert_requests ( hctx , list , false ) ;
else
blk_mq_insert_requests ( hctx , ctx , list ) ;
blk_mq_run_hw_queue ( hctx , run_queue_async ) ;
}
2017-01-17 16:03:22 +03:00
static void blk_mq_sched_free_tags ( struct blk_mq_tag_set * set ,
struct blk_mq_hw_ctx * hctx ,
unsigned int hctx_idx )
{
if ( hctx - > sched_tags ) {
blk_mq_free_rqs ( set , hctx - > sched_tags , hctx_idx ) ;
blk_mq_free_rq_map ( hctx - > sched_tags ) ;
hctx - > sched_tags = NULL ;
}
}
2017-04-05 22:01:30 +03:00
static int blk_mq_sched_alloc_tags ( struct request_queue * q ,
struct blk_mq_hw_ctx * hctx ,
unsigned int hctx_idx )
{
struct blk_mq_tag_set * set = q - > tag_set ;
int ret ;
hctx - > sched_tags = blk_mq_alloc_rq_map ( set , hctx_idx , q - > nr_requests ,
set - > reserved_tags ) ;
if ( ! hctx - > sched_tags )
return - ENOMEM ;
ret = blk_mq_alloc_rqs ( set , hctx - > sched_tags , hctx_idx , q - > nr_requests ) ;
if ( ret )
blk_mq_sched_free_tags ( set , hctx , hctx_idx ) ;
return ret ;
}
2017-04-07 17:52:27 +03:00
static void blk_mq_sched_tags_teardown ( struct request_queue * q )
2017-01-17 16:03:22 +03:00
{
struct blk_mq_tag_set * set = q - > tag_set ;
struct blk_mq_hw_ctx * hctx ;
2017-04-05 22:01:30 +03:00
int i ;
queue_for_each_hw_ctx ( q , hctx , i )
blk_mq_sched_free_tags ( set , hctx , i ) ;
}
2017-04-05 22:01:31 +03:00
int blk_mq_sched_init_hctx ( struct request_queue * q , struct blk_mq_hw_ctx * hctx ,
unsigned int hctx_idx )
{
struct elevator_queue * e = q - > elevator ;
2017-04-05 22:01:34 +03:00
int ret ;
2017-04-05 22:01:31 +03:00
if ( ! e )
return 0 ;
2017-04-05 22:01:34 +03:00
ret = blk_mq_sched_alloc_tags ( q , hctx , hctx_idx ) ;
if ( ret )
return ret ;
if ( e - > type - > ops . mq . init_hctx ) {
ret = e - > type - > ops . mq . init_hctx ( hctx , hctx_idx ) ;
if ( ret ) {
blk_mq_sched_free_tags ( q - > tag_set , hctx , hctx_idx ) ;
return ret ;
}
}
2017-05-04 17:24:40 +03:00
blk_mq_debugfs_register_sched_hctx ( q , hctx ) ;
2017-04-05 22:01:34 +03:00
return 0 ;
2017-04-05 22:01:31 +03:00
}
void blk_mq_sched_exit_hctx ( struct request_queue * q , struct blk_mq_hw_ctx * hctx ,
unsigned int hctx_idx )
{
struct elevator_queue * e = q - > elevator ;
if ( ! e )
return ;
2017-05-04 17:24:40 +03:00
blk_mq_debugfs_unregister_sched_hctx ( hctx ) ;
2017-04-05 22:01:34 +03:00
if ( e - > type - > ops . mq . exit_hctx & & hctx - > sched_data ) {
e - > type - > ops . mq . exit_hctx ( hctx , hctx_idx ) ;
hctx - > sched_data = NULL ;
}
2017-04-05 22:01:31 +03:00
blk_mq_sched_free_tags ( q - > tag_set , hctx , hctx_idx ) ;
}
2017-04-05 22:01:30 +03:00
int blk_mq_init_sched ( struct request_queue * q , struct elevator_type * e )
{
struct blk_mq_hw_ctx * hctx ;
2017-04-05 22:01:34 +03:00
struct elevator_queue * eq ;
2017-04-05 22:01:30 +03:00
unsigned int i ;
int ret ;
if ( ! e ) {
q - > elevator = NULL ;
return 0 ;
}
2017-01-17 16:03:22 +03:00
/*
2017-07-03 15:37:14 +03:00
* Default to double of smaller one between hw queue_depth and 128 ,
* since we don ' t split into sync / async like the old code did .
* Additionally , this is a per - hw queue depth .
2017-01-17 16:03:22 +03:00
*/
2017-07-03 15:37:14 +03:00
q - > nr_requests = 2 * min_t ( unsigned int , q - > tag_set - > queue_depth ,
BLKDEV_MAX_RQ ) ;
2017-01-17 16:03:22 +03:00
queue_for_each_hw_ctx ( q , hctx , i ) {
2017-04-05 22:01:30 +03:00
ret = blk_mq_sched_alloc_tags ( q , hctx , i ) ;
2017-01-17 16:03:22 +03:00
if ( ret )
2017-04-05 22:01:30 +03:00
goto err ;
2017-01-17 16:03:22 +03:00
}
2017-04-05 22:01:30 +03:00
ret = e - > ops . mq . init_sched ( q , e ) ;
if ( ret )
goto err ;
2017-01-17 16:03:22 +03:00
2017-05-04 17:24:40 +03:00
blk_mq_debugfs_register_sched ( q ) ;
queue_for_each_hw_ctx ( q , hctx , i ) {
if ( e - > ops . mq . init_hctx ) {
2017-04-05 22:01:34 +03:00
ret = e - > ops . mq . init_hctx ( hctx , i ) ;
if ( ret ) {
eq = q - > elevator ;
blk_mq_exit_sched ( q , eq ) ;
kobject_put ( & eq - > kobj ) ;
return ret ;
}
}
2017-05-04 17:24:40 +03:00
blk_mq_debugfs_register_sched_hctx ( q , hctx ) ;
2017-04-05 22:01:34 +03:00
}
2017-01-17 16:03:22 +03:00
return 0 ;
2017-04-05 22:01:30 +03:00
err :
2017-04-07 17:52:27 +03:00
blk_mq_sched_tags_teardown ( q ) ;
q - > elevator = NULL ;
2017-04-05 22:01:30 +03:00
return ret ;
2017-01-17 16:03:22 +03:00
}
2017-01-14 00:43:58 +03:00
2017-04-07 17:52:27 +03:00
void blk_mq_exit_sched ( struct request_queue * q , struct elevator_queue * e )
{
2017-04-05 22:01:34 +03:00
struct blk_mq_hw_ctx * hctx ;
unsigned int i ;
2017-05-04 17:24:40 +03:00
queue_for_each_hw_ctx ( q , hctx , i ) {
blk_mq_debugfs_unregister_sched_hctx ( hctx ) ;
if ( e - > type - > ops . mq . exit_hctx & & hctx - > sched_data ) {
e - > type - > ops . mq . exit_hctx ( hctx , i ) ;
hctx - > sched_data = NULL ;
2017-04-05 22:01:34 +03:00
}
}
2017-05-04 17:24:40 +03:00
blk_mq_debugfs_unregister_sched ( q ) ;
2017-04-07 17:52:27 +03:00
if ( e - > type - > ops . mq . exit_sched )
e - > type - > ops . mq . exit_sched ( e ) ;
blk_mq_sched_tags_teardown ( q ) ;
q - > elevator = NULL ;
}
2017-01-14 00:43:58 +03:00
int blk_mq_sched_init ( struct request_queue * q )
{
int ret ;
mutex_lock ( & q - > sysfs_lock ) ;
ret = elevator_init ( q , NULL ) ;
mutex_unlock ( & q - > sysfs_lock ) ;
return ret ;
}