ACK: [c/azure][PATCH] blk-mq: remove the request_list usage

Stefan Bader stefan.bader at canonical.com
Tue Apr 23 13:17:32 UTC 2019


On 22.04.19 22:32, Marcelo Henrique Cerri wrote:
> From: Jens Axboe <axboe at kernel.dk>
> 
> BugLink: http://bugs.launchpad.net/bugs/1819689
> 
> We don't do anything with it, that's just the legacy path.
> 
> Reviewed-by: Hannes Reinecke <hare at suse.com>
> Tested-by: Ming Lei <ming.lei at redhat.com>
> Reviewed-by: Omar Sandoval <osandov at fb.com>
> Signed-off-by: Jens Axboe <axboe at kernel.dk>
> (cherry picked from commit 7ac257b862f2cfba3a909d1051499d390cffad6c)
> Signed-off-by: Marcelo Henrique Cerri <marcelo.cerri at canonical.com>
Acked-by: Stefan Bader <stefan.bader at canonical.com>
> ---
>  block/blk-mq.c | 5 -----
>  1 file changed, 5 deletions(-)
> 
> diff --git a/block/blk-mq.c b/block/blk-mq.c
> index 90e8058ae557..76702ea84dd5 100644
> --- a/block/blk-mq.c
> +++ b/block/blk-mq.c
> @@ -508,9 +508,6 @@ void blk_mq_free_request(struct request *rq)
>  
>  	wbt_done(q->rq_wb, rq);
>  
> -	if (blk_rq_rl(rq))
> -		blk_put_rl(blk_rq_rl(rq));
> -
>  	WRITE_ONCE(rq->state, MQ_RQ_IDLE);
>  	if (refcount_dec_and_test(&rq->ref))
>  		__blk_mq_free_request(rq);
> @@ -1626,8 +1623,6 @@ static void blk_mq_bio_to_request(struct request *rq, struct bio *bio)
>  {
>  	blk_init_request_from_bio(rq, bio);
>  
> -	blk_rq_set_rl(rq, blk_get_rl(rq->q, bio));
> -
>  	blk_account_io_start(rq, true);
>  }
>  
> 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20190423/7f45b4bb/attachment.sig>


More information about the kernel-team mailing list