Re: [PATCH 2/6] scsi: ensure that a request is valid when returning it from scsi_find_tag()

From: Christoph Hellwig
Date: Mon Apr 13 2015 - 05:47:25 EST


On Tue, Apr 07, 2015 at 11:18:36AM -0600, Jens Axboe wrote:
> For the scsi-mq case, ensure that the request has been started before
> returning it from scsi_find_tag(). This is akin to the ->special
> check for the non-mq case, it ensures that the request has been setup
> and issued.
>
> Signed-off-by: Jens Axboe <axboe@xxxxxx>

Shouldn't this be done in blk_mq_tag_to_rq? In general when a driver
looks for an active tag it wants an active tag, so working around
this in the driver seems like a bad idea.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/