Re: [RESEND PATCH] scsi: core: use eh_timeout to timeout start_unit command

From: Christoph Hellwig
Date: Tue Nov 09 2021 - 05:03:35 EST


On Mon, Nov 08, 2021 at 10:19:16AM +0800, brookxu wrote:
> From: Chunguang Xu <brookxu@xxxxxxxxxxx>
>
> In some abnormal scenarios, STU may timeout. The recovery time
> of 30 seconds is relatively long. Now we need to adjusting
> rq_timeout to adjust STU timeout value, but it will affect the
> actual IO.
>
> ptach 9728c081(make scsi_eh_try_stu use block timeout) uses
> rq_timeout to timeout the STU command, but after pathc 0816c92(

patch is mispelled in two different ways here. But you probably
want to use commit anyway and use 12 charater commit hashes.

> - rtn = scsi_send_eh_cmnd(scmd, stu_command, 6, scmd->device->request_queue->rq_timeout, 0);
> + rtn = scsi_send_eh_cmnd(scmd, stu_command, 6, scmd->device->eh_timeout, 0);

Both the old and new coe is completely unreadable. Please wrap lines
after 80 characters

Otherwise this looks good.