Hi,
Thanks for your patch.
This patch is from mainline and stable tree, please leave the original author and commit message.
And, if this patch could fix your bug, please file a issue at, https://gitee.com/openeuler/kernel/issues
and attach the url of issue at commit message.
On 2021/9/23 20:53, yinxiujiang wrote:
From: yin-xiujiang yinxiujiang@kylinos.cn
Use the original author above.
Add stable inclusion or mainline inclusion here.
Fix incorrect call to free srb in qla2xxx_mqueuecommand(), as srb is now allocated by upper layers. This fixes smatch warning of srb unintended free.
Leave the original Signed-off-by.
Signed-off-by: yin-xiujiang yinxiujiang@kylinos.cn
drivers/scsi/qla2xxx/qla_os.c | 7 ------- 1 file changed, 7 deletions(-)
diff --git a/drivers/scsi/qla2xxx/qla_os.c b/drivers/scsi/qla2xxx/qla_os.c index bfbf213b15c0..8e9d386146ac 100644 --- a/drivers/scsi/qla2xxx/qla_os.c +++ b/drivers/scsi/qla2xxx/qla_os.c @@ -1028,8 +1028,6 @@ qla2xxx_mqueuecommand(struct Scsi_Host *host, struct scsi_cmnd *cmd, if (rval != QLA_SUCCESS) { ql_dbg(ql_dbg_io + ql_dbg_verbose, vha, 0x3078, "Start scsi failed rval=%d for cmd=%p.\n", rval, cmd);
if (rval == QLA_INTERFACE_ERROR)goto qc24_free_sp_fail_command;@@ -1044,11 +1042,6 @@ qla2xxx_mqueuecommand(struct Scsi_Host *host, struct scsi_cmnd *cmd, qc24_target_busy: return SCSI_MLQUEUE_TARGET_BUSY;
-qc24_free_sp_fail_command:
- sp->free(sp);
- CMD_SP(cmd) = NULL;
- qla2xxx_rel_qpair_sp(sp->qpair, sp);
qc24_fail_command: cmd->scsi_done(cmd);