>>>>> "Li" == Li Zhong <zh...@linux.vnet.ibm.com> writes:

> @@ -845,8 +844,11 @@ static int scsi_send_eh_cmnd(struct scsi_cmnd *scmd, 
> unsigned char *cmnd,
> 
>       scsi_eh_restore_cmnd(scmd, &ses);
> 
>-      if (sdrv && sdrv->eh_action)
>-              rtn = sdrv->eh_action(scmd, cmnd, cmnd_size, rtn);
>+      if (scmd->request->cmd_type == REQ_TYPE_FS) {
>+              struct scsi_driver *sdrv = scsi_cmd_to_driver(scmd);
>+              if (sdrv->eh_action)
>+                      rtn = sdrv->eh_action(scmd, cmnd, cmnd_size, rtn);
>+      }
> 
>       return rtn;
> }

My only concern is whether our device lifetime rules guarantee that the
ULD is always attached when we service an error handling command?

-- 
Martin K. Petersen      Oracle Linux Engineering
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to