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;
  

  goto qc24_host_busy_free_sp; }

@@ -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);