Commit 57bf595a authored by Bart Van Assche's avatar Bart Van Assche Committed by Martin K. Petersen

scsi: qla2xxx: Verify locking assumptions at runtime

Make sure that locking assumptions are verified at runtime if kernel
debugging is enabled.

Cc: Himanshu Madhani <hmadhani@marvell.com>
Signed-off-by: default avatarBart Van Assche <bvanassche@acm.org>
Tested-by: default avatarHimanshu Madhani <hmadhani@marvell.com>
Reviewed-by: default avatarHimanshu Madhani <hmadhani@marvell.com>
Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
parent 9e75b5e2
...@@ -789,6 +789,8 @@ qlt_plogi_ack_find_add(struct scsi_qla_host *vha, port_id_t *id, ...@@ -789,6 +789,8 @@ qlt_plogi_ack_find_add(struct scsi_qla_host *vha, port_id_t *id,
{ {
struct qlt_plogi_ack_t *pla; struct qlt_plogi_ack_t *pla;
lockdep_assert_held(&vha->hw->hardware_lock);
list_for_each_entry(pla, &vha->plogi_ack_list, list) { list_for_each_entry(pla, &vha->plogi_ack_list, list) {
if (pla->id.b24 == id->b24) { if (pla->id.b24 == id->b24) {
ql_dbg(ql_dbg_disc + ql_dbg_verbose, vha, 0x210d, ql_dbg(ql_dbg_disc + ql_dbg_verbose, vha, 0x210d,
...@@ -4713,6 +4715,8 @@ static int qlt_handle_login(struct scsi_qla_host *vha, ...@@ -4713,6 +4715,8 @@ static int qlt_handle_login(struct scsi_qla_host *vha,
struct qlt_plogi_ack_t *pla; struct qlt_plogi_ack_t *pla;
unsigned long flags; unsigned long flags;
lockdep_assert_held(&vha->hw->hardware_lock);
wwn = wwn_to_u64(iocb->u.isp24.port_name); wwn = wwn_to_u64(iocb->u.isp24.port_name);
port_id.b.domain = iocb->u.isp24.port_id[2]; port_id.b.domain = iocb->u.isp24.port_id[2];
...@@ -4886,6 +4890,8 @@ static int qlt_24xx_handle_els(struct scsi_qla_host *vha, ...@@ -4886,6 +4890,8 @@ static int qlt_24xx_handle_els(struct scsi_qla_host *vha,
int res = 0; int res = 0;
unsigned long flags; unsigned long flags;
lockdep_assert_held(&ha->hardware_lock);
wwn = wwn_to_u64(iocb->u.isp24.port_name); wwn = wwn_to_u64(iocb->u.isp24.port_name);
port_id.b.domain = iocb->u.isp24.port_id[2]; port_id.b.domain = iocb->u.isp24.port_id[2];
...@@ -5162,6 +5168,8 @@ static void qlt_handle_imm_notify(struct scsi_qla_host *vha, ...@@ -5162,6 +5168,8 @@ static void qlt_handle_imm_notify(struct scsi_qla_host *vha,
int send_notify_ack = 1; int send_notify_ack = 1;
uint16_t status; uint16_t status;
lockdep_assert_held(&ha->hardware_lock);
status = le16_to_cpu(iocb->u.isp2x.status); status = le16_to_cpu(iocb->u.isp2x.status);
switch (status) { switch (status) {
case IMM_NTFY_LIP_RESET: case IMM_NTFY_LIP_RESET:
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment