diff options
| author | Shlomo Pongratz <shlomop@mellanox.com> | 2014-03-30 15:26:29 +0300 |
|---|---|---|
| committer | Christoph Hellwig <hch@lst.de> | 2014-05-19 13:33:00 +0200 |
| commit | 72b9740201d5f0e24b0b8326a4949786a30ff628 (patch) | |
| tree | e74603446a6e204dc3b9906cd41bbd3275102fe7 /drivers/scsi/libiscsi.c | |
| parent | abb14148c0f850e7201efc3e7aea1762f993606b (diff) | |
| download | linux-72b9740201d5f0e24b0b8326a4949786a30ff628.tar.gz linux-72b9740201d5f0e24b0b8326a4949786a30ff628.tar.bz2 linux-72b9740201d5f0e24b0b8326a4949786a30ff628.zip | |
scsi/libiscsi: Fix static checker warning on bh locking
Commit 659743b "[SCSI] libiscsi: Reduce locking contention in fast path" introduced a
new smatch warning on libiscsi.c "iscsi_xmit_task() warn: inconsistent returns
bottom_half:: locked (1410 [(-61)]) unlocked (1425 [0], 1425 [s32min-(-1),1-s32max])",
which we can eliminate by using non bh locking on the nested spin_lock call.
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Shlomo Pongratz <shlomop@mellanox.com>
Signed-off-by: Or Gerlitz <ogerlitz@mellanox.com>
Reviewed-by: Mike Christie <michaelc@cs.wisc.edu>
Signed-off-by: Christoph Hellwig <hch@lst.de>
Diffstat (limited to 'drivers/scsi/libiscsi.c')
| -rw-r--r-- | drivers/scsi/libiscsi.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/scsi/libiscsi.c b/drivers/scsi/libiscsi.c index 26dc005bb0f0..ecd7bd304efe 100644 --- a/drivers/scsi/libiscsi.c +++ b/drivers/scsi/libiscsi.c @@ -1442,9 +1442,9 @@ static int iscsi_xmit_task(struct iscsi_conn *conn) conn->task = NULL; } /* regular RX path uses back_lock */ - spin_lock_bh(&conn->session->back_lock); + spin_lock(&conn->session->back_lock); __iscsi_put_task(task); - spin_unlock_bh(&conn->session->back_lock); + spin_unlock(&conn->session->back_lock); return rc; } |