Xianting Tian, Thanks for the patch. > -----Original Message----- > From: Xianting Tian <tian.xianting@xxxxxxx> > Sent: Saturday, August 29, 2020 1:28 PM > To: Nilesh Javali <njavali@xxxxxxxxxxx>; GR-QLogic-Storage-Upstream <GR- > QLogic-Storage-Upstream@xxxxxxxxxxx>; jejb@xxxxxxxxxxxxx; > martin.petersen@xxxxxxxxxx > Cc: linux-scsi@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; Xianting Tian > <tian.xianting@xxxxxxx> > Subject: [EXT] [PATCH] scsi: qla2xxx: Fix the return value > > External Email > > ---------------------------------------------------------------------- > A negative error code should be returned. > > Signed-off-by: Xianting Tian <tian.xianting@xxxxxxx> > --- > drivers/scsi/qla2xxx/qla_target.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/scsi/qla2xxx/qla_target.c > b/drivers/scsi/qla2xxx/qla_target.c > index fbb80a043..612e001cc 100644 > --- a/drivers/scsi/qla2xxx/qla_target.c > +++ b/drivers/scsi/qla2xxx/qla_target.c > @@ -3781,7 +3781,7 @@ int qlt_abort_cmd(struct qla_tgt_cmd *cmd) > "multiple abort. %p transport_state %x, t_state %x, " > "se_cmd_flags %x\n", cmd, cmd->se_cmd.transport_state, > cmd->se_cmd.t_state, cmd->se_cmd.se_cmd_flags); > - return EIO; > + return -EIO; > } > cmd->aborted = 1; > cmd->trc_flags |= TRC_ABORT; > -- > 2.17.1 Acked-by: Nilesh Javali <njavali@xxxxxxxxxxx>
