Thread (22 messages) 22 messages, 2 authors, 2021-07-09

RE: [smartpqi updates PATCH 4/9] smartpqi: add SCSI cmd info for resets

From: <Don.Brace@microchip.com>
Date: 2021-07-08 20:20:25
Also in: lkml

From: Paul Menzel [mailto:pmenzel@molgen.mpg.de] 
Subject: Re: [smartpqi updates PATCH 4/9] smartpqi: add SCSI cmd info for resets

Dear Murthy, dear Don,


Am 06.07.21 um 20:16 schrieb Don Brace:
From: Murthy Bhat <redacted>

Report on SCSI command that has triggered the reset.
  - Also add check for 0 length SCSI commands.
Can you please add an example log message line to the git commit message summary?

Don: Thanks for your review. I added an example.

Kind regards,

Paul

quoted hunk ↗ jump to hunk
Reviewed-by: Kevin Barnett <redacted>
Reviewed-by: Mike McGowen <redacted>
Reviewed-by: Scott Benesh <redacted>
Reviewed-by: Scott Teel <redacted>
Signed-off-by: Murthy Bhat <redacted>
Signed-off-by: Don Brace <don.brace@microchip.com>
---
  drivers/scsi/smartpqi/smartpqi_init.c | 6 ++++--
  1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/drivers/scsi/smartpqi/smartpqi_init.c 
b/drivers/scsi/smartpqi/smartpqi_init.c
index 5ce1c41a758d..c2ddb66b5c2d 100644
--- a/drivers/scsi/smartpqi/smartpqi_init.c
+++ b/drivers/scsi/smartpqi/smartpqi_init.c
@@ -6033,8 +6033,10 @@ static int pqi_eh_device_reset_handler(struct scsi_cmnd *scmd)
      mutex_lock(&ctrl_info->lun_reset_mutex);

      dev_err(&ctrl_info->pci_dev->dev,
-             "resetting scsi %d:%d:%d:%d\n",
-             shost->host_no, device->bus, device->target, device->lun);
+             "resetting scsi %d:%d:%d:%d due to cmd 0x%02x\n",
+             shost->host_no,
+             device->bus, device->target, device->lun,
+             scmd->cmd_len > 0 ? scmd->cmnd[0] : 0xff);

      pqi_check_ctrl_health(ctrl_info);
      if (pqi_ctrl_offline(ctrl_info))
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help