diff options
| author | Bart Van Assche <bart.vanassche@wdc.com> | 2018-06-22 14:53:07 -0700 |
|---|---|---|
| committer | Martin K. Petersen <martin.petersen@oracle.com> | 2018-07-02 16:44:32 -0400 |
| commit | e936a38ac92dd40867ac3b52cfd8f3f70fe717a5 (patch) | |
| tree | ce514ffd9409a92eceafb7c53b063f622c6a802f /drivers/target/target_core_device.c | |
| parent | 325c1e8b248ed12a685d0a904b774abef3821e8e (diff) | |
| download | linux-e936a38ac92dd40867ac3b52cfd8f3f70fe717a5.tar.gz linux-e936a38ac92dd40867ac3b52cfd8f3f70fe717a5.tar.bz2 linux-e936a38ac92dd40867ac3b52cfd8f3f70fe717a5.zip | |
scsi: target: Remove se_dev_entry.ua_count
se_dev_entry.ua_count is only used to check whether or not
se_dev_entry.ua_list is empty. Use list_empty_careful() instead. Checking
whether or not ua_list is empty without holding the lock that protects that
list is fine because the code that dequeues from that list will check again
whether or not that list is empty.
Signed-off-by: Bart Van Assche <bart.vanassche@wdc.com>
Reviewed-by: Mike Christie <mchristi@redhat.com>
Cc: Mike Christie <mchristi@redhat.com>
Cc: Christoph Hellwig <hch@lst.de>
Cc: Hannes Reinecke <hare@suse.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
Diffstat (limited to 'drivers/target/target_core_device.c')
| -rw-r--r-- | drivers/target/target_core_device.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/target/target_core_device.c b/drivers/target/target_core_device.c index e5c90afb40e6..73675eec740d 100644 --- a/drivers/target/target_core_device.c +++ b/drivers/target/target_core_device.c @@ -336,7 +336,6 @@ int core_enable_device_list_for_node( return -ENOMEM; } - atomic_set(&new->ua_count, 0); spin_lock_init(&new->ua_lock); INIT_LIST_HEAD(&new->ua_list); INIT_LIST_HEAD(&new->lun_link); |