> @@ -646,14 +703,20 @@ static int scsi_try_target_reset(struct scsi_cmnd *scmd)
>  static int scsi_try_bus_device_reset(struct scsi_cmnd *scmd)
>  {
>         int rtn;
> -       struct scsi_host_template *hostt = scmd->device->host->hostt;
> +       struct Scsi_Host *host = scmd->device->host;
> +       struct scsi_host_template *hostt = host->hostt;
> 
>         if (!hostt->eh_device_reset_handler)
>                 return FAILED;
> 
> +       if (scsi_begin_eh(host))
> +               return FAST_IO_FAIL;
> +
>         rtn = hostt->eh_device_reset_handler(scmd);
>         if (rtn == SUCCESS)
>                 __scsi_report_device_reset(scmd->device, NULL);
> +       scsi_end_eh(host);
> +
>         return rtn;
>  }

As the new eh from Hannes haven't make it into mainline, maybe we still
need also check scsi_try_to_abort_cmd?

Jack
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to