diff mbox series

[2/2] qla2xxx: Initialize vha->unknown_atio_[list, work] for NPIV hosts

Message ID 376c89a2-a9ac-bcf9-bf0f-dfe89a02fd4b@scst.dev
State New
Headers show
Series [1/2] qla2xxx: Remove duplicate of vha->iocb_work initialization | expand

Commit Message

Gleb Chesnokov Nov. 15, 2022, 9:38 a.m. UTC
Initialization of vha->unknown_atio_list and vha->unknown_atio_work
only happens for base_vha in qla_probe_one_stage1(). But there is no
initialization for NPIV hosts that are created in qla24xx_vport_create().

This causes a crash when trying to access these NPIV host fields.

Fix this by adding initialization to qla_vport_create().

Signed-off-by: Gleb Chesnokov <gleb.chesnokov@scst.dev>
---
  drivers/scsi/qla2xxx/qla_target.c | 3 +++
  1 file changed, 3 insertions(+)

  	/*

Comments

Martin K. Petersen Dec. 1, 2022, 3:45 a.m. UTC | #1
On Tue, 15 Nov 2022 12:38:08 +0300, Gleb Chesnokov wrote:

> Initialization of vha->unknown_atio_list and vha->unknown_atio_work
> only happens for base_vha in qla_probe_one_stage1(). But there is no
> initialization for NPIV hosts that are created in qla24xx_vport_create().
> 
> This causes a crash when trying to access these NPIV host fields.
> 
> Fix this by adding initialization to qla_vport_create().
> 
> [...]

Applied to 6.2/scsi-queue, thanks!

[2/2] qla2xxx: Initialize vha->unknown_atio_[list, work] for NPIV hosts
      https://git.kernel.org/mkp/scsi/c/95da5e58172c
diff mbox series

Patch

diff --git a/drivers/scsi/qla2xxx/qla_target.c 
b/drivers/scsi/qla2xxx/qla_target.c
index bb754a950802..548f22705ddc 100644
--- a/drivers/scsi/qla2xxx/qla_target.c
+++ b/drivers/scsi/qla2xxx/qla_target.c
@@ -6741,6 +6741,9 @@  qlt_vport_create(struct scsi_qla_host *vha, struct 
qla_hw_data *ha)
  	mutex_init(&vha->vha_tgt.tgt_mutex);
  	mutex_init(&vha->vha_tgt.tgt_host_action_mutex);

+	INIT_LIST_HEAD(&vha->unknown_atio_list);
+	INIT_DELAYED_WORK(&vha->unknown_atio_work, qlt_unknown_atio_work_fn);
+
  	qlt_clear_mode(vha);