Re: [PATCH 10/24] scsi-multipath: add scsi_mpath_{start,end}_request()

[John Garry <john.g.garry@oracle.com>]

On 04/03/2026 06:13, Benjamin Marzinski wrote:
>>   
>> +	scsi_mpath_end_request(req);
>> +
>>   	/*
>>   	 * In the MQ case the command gets freed by __blk_mq_end_request,
>>   	 * so we have to do all cleanup that depends on it earlier.
> This looks wrong. We start accounting in scsi_queue_rq(), and we need to
> end it whenever we complete or requeue the request, otherwise the
> accounting will get off. But not all requests go through
> scsi_end_request(). scsi_mpath_failover_req(), for instance, calls
> blk_mq_end_request() directly, and other functions, like
> scsi_queue_insert() call blk_mq_requeue_request(). I'm pretty sure that
> this should go in scsi_complete(), as well in the error path of
> scsi_queue_rq().

ok, let me check that further.

Thanks for the notice.