diff options
author | Bart Van Assche <bart.vanassche@sandisk.com> | 2017-03-17 20:02:03 -0400 |
---|---|---|
committer | Martin K. Petersen <martin.petersen@oracle.com> | 2017-03-19 13:16:37 -0400 |
commit | 0aeccdfe220c360ab888816da06b8eb67d910ff6 (patch) | |
tree | ce2f86fff0cefca61b328c9d2e9b038dc3def3b8 | |
parent | 7cb689fe42927281b8d98606ae5450173fcc66a9 (diff) |
scsi: scsi_dh_alua: Warn if the first argument of alua_rtpg_queue() is NULL
Callers must provide a valid port group to alua_rtpg_queue(). Issue a
kernel warning if that is not the case.
Signed-off-by: Bart Van Assche <bart.vanassche@sandisk.com>
Cc: Hannes Reinecke <hare@suse.de>
Cc: Tang Junhui <tang.junhui@zte.com.cn>
Reviewed-by: Hannes Reinecke <hare@suse.de>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
-rw-r--r-- | drivers/scsi/device_handler/scsi_dh_alua.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/scsi/device_handler/scsi_dh_alua.c b/drivers/scsi/device_handler/scsi_dh_alua.c index b6849d3ecefe..c01b47e5b55a 100644 --- a/drivers/scsi/device_handler/scsi_dh_alua.c +++ b/drivers/scsi/device_handler/scsi_dh_alua.c | |||
@@ -876,7 +876,7 @@ static bool alua_rtpg_queue(struct alua_port_group *pg, | |||
876 | unsigned long flags; | 876 | unsigned long flags; |
877 | struct workqueue_struct *alua_wq = kaluad_wq; | 877 | struct workqueue_struct *alua_wq = kaluad_wq; |
878 | 878 | ||
879 | if (!pg || scsi_device_get(sdev)) | 879 | if (WARN_ON_ONCE(!pg) || scsi_device_get(sdev)) |
880 | return false; | 880 | return false; |
881 | 881 | ||
882 | spin_lock_irqsave(&pg->lock, flags); | 882 | spin_lock_irqsave(&pg->lock, flags); |