diff mbox series

[03/11] qla2xxx: Remove a comment that refers to the SCSI host lock

Message ID 20190404194448.121489-4-bvanassche@acm.org (mailing list archive)
State Accepted
Headers show
Series qla2xxx patches for kernel v5.2 | expand

Commit Message

Bart Van Assche April 4, 2019, 7:44 p.m. UTC
Since qla2xxx_queuecommand() no longer takes the SCSI host lock, remove
a comment that refers to the host lock.

Cc: Himanshu Madhani <hmadhani@marvell.com>
Cc: Giridhar Malavali <gmalavali@marvell.com>
Signed-off-by: Bart Van Assche <bvanassche@acm.org>
---
 drivers/scsi/qla2xxx/qla_os.c | 3 ---
 1 file changed, 3 deletions(-)

Comments

Himanshu Madhani April 8, 2019, 4:42 p.m. UTC | #1
On 4/4/19, 12:45 PM, "Bart Van Assche" <bvanassche@acm.org> wrote:

    External Email
    
    ----------------------------------------------------------------------
    Since qla2xxx_queuecommand() no longer takes the SCSI host lock, remove
    a comment that refers to the host lock.
    
    Cc: Himanshu Madhani <hmadhani@marvell.com>
    Cc: Giridhar Malavali <gmalavali@marvell.com>
    Signed-off-by: Bart Van Assche <bvanassche@acm.org>
    ---
     drivers/scsi/qla2xxx/qla_os.c | 3 ---
     1 file changed, 3 deletions(-)
    
    diff --git a/drivers/scsi/qla2xxx/qla_os.c b/drivers/scsi/qla2xxx/qla_os.c
    index 57e1041e4ca2..dddfadc27b75 100644
    --- a/drivers/scsi/qla2xxx/qla_os.c
    +++ b/drivers/scsi/qla2xxx/qla_os.c
    @@ -889,9 +889,6 @@ qla2xxx_qpair_sp_compl(void *ptr, int res)
     	qla2xxx_rel_qpair_sp(sp->qpair, sp);
     }
     
    -/* If we are SP1 here, we need to still take and release the host_lock as SP1
    - * does not have the changes necessary to avoid taking host->host_lock.
    - */
     static int
     qla2xxx_queuecommand(struct Scsi_Host *host, struct scsi_cmnd *cmd)
     {
    -- 
    2.21.0.196.g041f5ea1cf98
    
 Thanks for the patch.

Acked-by: Himanshu Madhani <hmadhani@marvell.com>
diff mbox series

Patch

diff --git a/drivers/scsi/qla2xxx/qla_os.c b/drivers/scsi/qla2xxx/qla_os.c
index 57e1041e4ca2..dddfadc27b75 100644
--- a/drivers/scsi/qla2xxx/qla_os.c
+++ b/drivers/scsi/qla2xxx/qla_os.c
@@ -889,9 +889,6 @@  qla2xxx_qpair_sp_compl(void *ptr, int res)
 	qla2xxx_rel_qpair_sp(sp->qpair, sp);
 }
 
-/* If we are SP1 here, we need to still take and release the host_lock as SP1
- * does not have the changes necessary to avoid taking host->host_lock.
- */
 static int
 qla2xxx_queuecommand(struct Scsi_Host *host, struct scsi_cmnd *cmd)
 {