[prev in list] [next in list] [prev in thread] [next in thread]
List: linux-block
Subject: Re: [PATCH 6/7] blk-mq-sched: remove hack that bypasses scheduler for reserved requests
From: Ming Lei <ming.lei () redhat ! com>
Date: 2017-04-28 15:22:59
Message-ID: 20170428152258.GB25592 () ming ! t460p
[Download RAW message or body]
On Fri, Apr 28, 2017 at 08:55:39AM -0600, Jens Axboe wrote:
> We have update the troublesome driver (mtip32xx) to deal with this
> appropriately. So kill the hack that bypassed scheduler allocation
> and insertion for reserved requests.
>
> Signed-off-by: Jens Axboe <axboe@fb.com>
> ---
> block/blk-mq-sched.c | 6 +-----
> 1 file changed, 1 insertion(+), 5 deletions(-)
>
> diff --git a/block/blk-mq-sched.c b/block/blk-mq-sched.c
> index 8b361e192e8a..e79e9f18d7c2 100644
> --- a/block/blk-mq-sched.c
> +++ b/block/blk-mq-sched.c
> @@ -82,11 +82,7 @@ struct request *blk_mq_sched_get_request(struct request_queue *q,
> if (likely(!data->hctx))
> data->hctx = blk_mq_map_queue(q, data->ctx->cpu);
>
> - /*
> - * For a reserved tag, allocate a normal request since we might
> - * have driver dependencies on the value of the internal tag.
> - */
> - if (e && !(data->flags & BLK_MQ_REQ_RESERVED)) {
> + if (e) {
> data->flags |= BLK_MQ_REQ_INTERNAL;
>
> /*
Reviewed-by: Ming Lei <ming.lei@redhat.com>
Thanks,
Ming
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic