On 10/17/2010 7:35 PM, Mike Christie wrote: > On 10/12/2010 05:50 PM, John Fastabend wrote: >> case FC_CMD_ABORTED: >> - FC_FCP_DBG(fsp, "Returning DID_ERROR to scsi-ml " >> - "due to FC_CMD_ABORTED\n"); >> - sc_cmd->result = (DID_ERROR<< 16) | fsp->io_status; >> - break; >> - case FC_CMD_RECOVERY: >> - sc_cmd->result = (DID_BUS_BUSY<< 16) | fsp->io_status; >> + FC_FCP_DBG(fsp, "Returning DID_ABORT to scsi-ml " >> + "due to FC_CMD_ABORTED\n"); >> + sc_cmd->result = (DID_ABORT<< 16) | fsp->cdb_status; > > > For where there command is aborted due to internal recovery (recovery > that was not started by the scsi eh) we do not want to use DID_ABORT > because we want the IO to be retried.
OK, I'll set this to DID_ERROR. Thanks. _______________________________________________ devel mailing list [email protected] http://www.open-fcoe.org/mailman/listinfo/devel
