Bart, thanks for reviewing. Will clean it up.
Regards,
Quinn Tran
-----Original Message-----
From: <linux-scsi-owner@xxxxxxxxxxxxxxx> on behalf of Bart Van Assche <Bart.VanAssche@xxxxxxxxxxx>
Date: Wednesday, February 8, 2017 at 11:22 AM
To: "hch@xxxxxxxxxxxxx" <hch@xxxxxxxxxxxxx>, "Madhani, Himanshu" <Himanshu.Madhani@xxxxxxxxxx>, target-devel <target-devel@xxxxxxxxxxxxxxx>, Nicholas Bellinger <nab@xxxxxxxxxxxxxxx>
Cc: "linux-scsi@xxxxxxxxxxxxxxx" <linux-scsi@xxxxxxxxxxxxxxx>, "Malavali, Giridhar" <Giridhar.Malavali@xxxxxxxxxx>
Subject: Re: [PATCH v2 10/14] qla2xxx: Fix request queue corruption.
Resent-From: <quinn.tran@xxxxxxxxxx>
On Fri, 2017-02-03 at 14:40 -0800, Himanshu Madhani wrote:
> From: Quinn Tran <quinn.tran@xxxxxxxxxx>
>
> When FW notify driver or driver detects low FW resource,
> driver tries to send out Busy SCSI Status to tell Initiator
> side to back off. During the send process, the lock was not held.
>
> Signed-off-by: Quinn Tran <quinn.tran@xxxxxxxxxx>
> Signed-off-by: Himanshu Madhani <himanshu.madhani@xxxxxxxxxx>
> ---
> drivers/scsi/qla2xxx/qla_target.c | 12 +++++++++---
> 1 file changed, 9 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/scsi/qla2xxx/qla_target.c b/drivers/scsi/qla2xxx/qla_target.c
> index b61cbb8..b5fb9c55 100644
> --- a/drivers/scsi/qla2xxx/qla_target.c
> +++ b/drivers/scsi/qla2xxx/qla_target.c
> @@ -5170,16 +5170,22 @@ static int __qlt_send_busy(struct scsi_qla_host *vha,
>
> static int
> qlt_chk_qfull_thresh_hold(struct scsi_qla_host *vha,
> - struct atio_from_isp *atio)
> + struct atio_from_isp *atio, uint8_t ha_locked)
> {
> struct qla_hw_data *ha = vha->hw;
> uint16_t status;
> + unsigned long flags;
>
> if (ha->tgt.num_pend_cmds < Q_FULL_THRESH_HOLD(ha))
> return 0;
>
> + if (!ha_locked)
> + spin_lock_irqsave(&ha->hardware_lock, flags);
> status = temp_sam_status;
> qlt_send_busy(vha, atio, status);
> + if (!ha_locked)
> + spin_unlock_irqrestore(&ha->hardware_lock, flags);
> +
> return 1;
> }
>
> @@ -5224,7 +5230,7 @@ static void qlt_24xx_atio_pkt(struct scsi_qla_host *vha,
>
>
> if (likely(atio->u.isp24.fcp_cmnd.task_mgmt_flags == 0)) {
> - rc = qlt_chk_qfull_thresh_hold(vha, atio);
> + rc = qlt_chk_qfull_thresh_hold(vha, atio, ha_locked);
> if (rc != 0) {
> tgt->atio_irq_cmd_count--;
> return;
> @@ -5347,7 +5353,7 @@ static void qlt_response_pkt(struct scsi_qla_host *vha, response_t *pkt)
> break;
> }
>
> - rc = qlt_chk_qfull_thresh_hold(vha, atio);
> + rc = qlt_chk_qfull_thresh_hold(vha, atio, 1);
> if (rc != 0) {
> tgt->irq_cmd_count--;
> return;
Hello Quinn,
Please consider to use bool instead of uint8_t for ha_locked and true /
false instead of 1 / 0.
Bart.
