On Tue, Jul 2, 2019 at 7:28 PM Maurizio Lombardi <mlombard@xxxxxxxxxx> wrote:
>
> With a possibly faulty disk the following messages may appear in the logs:
>
> kernel: sd 0:0:9:0: timing out command, waited 180s
> kernel: sd 0:0:9:0: timing out command, waited 20s
> kernel: sd 0:0:9:0: timing out command, waited 20s
> kernel: sd 0:0:9:0: timing out command, waited 60s
> kernel: sd 0:0:9:0: timing out command, waited 20s
>
> This is not very informative because it's not possible to identify
> the command that timed out.
>
> This patch replaces sdev_printk() with scmd_printk().
>
> Signed-off-by: Maurizio Lombardi <mlombard@xxxxxxxxxx>
> ---
> drivers/scsi/scsi_lib.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/scsi/scsi_lib.c b/drivers/scsi/scsi_lib.c
> index f6437b98296b..97ed233fa469 100644
> --- a/drivers/scsi/scsi_lib.c
> +++ b/drivers/scsi/scsi_lib.c
> @@ -1501,7 +1501,7 @@ static void scsi_softirq_done(struct request *rq)
> disposition = scsi_decide_disposition(cmd);
> if (disposition != SUCCESS &&
> time_before(cmd->jiffies_at_alloc + wait_for, jiffies)) {
> - sdev_printk(KERN_ERR, cmd->device,
> + scmd_printk(KERN_ERR, cmd,
> "timing out command, waited %lus\n",
> wait_for/HZ);
> disposition = SUCCESS;
> --
> Maurizio Lombardi
>
Reviewed-by: Ming Lei <ming.lei@xxxxxxxxxx>
Thanks,
Ming Lei
