diff options
author | Ming Lei <ming.lei@redhat.com> | 2020-04-23 04:07:13 +0200 |
---|---|---|
committer | Martin K. Petersen <martin.petersen@oracle.com> | 2020-04-25 00:21:15 +0200 |
commit | f983622ae60516d634008c7b1ff9ffff4f7bb8ae (patch) | |
tree | 11f5a5aa347e1ebec3429d67f33665c3b4283ced /include/target | |
parent | scsi: BusLogic: Remove conversion to bool in blogic_inquiry() (diff) | |
download | linux-f983622ae60516d634008c7b1ff9ffff4f7bb8ae.tar.xz linux-f983622ae60516d634008c7b1ff9ffff4f7bb8ae.zip |
scsi: core: Avoid calling synchronize_rcu() for each device in scsi_host_block()
scsi_host_block() calls scsi_internal_device_block() for each scsi_device and
scsi_internal_device_block() calls blk_mq_quiesce_queue() for each LUN.
Since synchronize_rcu() is called from blk_mq_quiesce_queue(), this can cause
substantial slowdowns on systems with many LUNs.
Use scsi_internal_device_block_nowait() to implement scsi_host_block() so it
is sufficient to run synchronize_rcu() once. This is safe since SCSI does not
set the BLK_MQ_F_BLOCKING flag.
[mkp: commit desc and comment tweaks]
Link: https://lore.kernel.org/r/20200423020713.332743-1-ming.lei@redhat.com
Cc: Steffen Maier <maier@linux.ibm.com>
Cc: Bart Van Assche <bvanassche@acm.org>
Cc: Christoph Hellwig <hch@lst.de>
Cc: Dexuan Cui <decui@microsoft.com>
Cc: Hannes Reinecke <hare@suse.de>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Hannes Reinecke <hare@suse.de>
Signed-off-by: Ming Lei <ming.lei@redhat.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
Diffstat (limited to 'include/target')
0 files changed, 0 insertions, 0 deletions