diff mbox series

[07/12] scsi: iscsi_tcp: Drop target_alloc use

Message ID 20220308002747.122682-8-michael.christie@oracle.com
State Superseded
Headers show
Series misc iscsi patches | expand

Commit Message

Mike Christie March 8, 2022, 12:27 a.m. UTC
For software iscsi, we do a session per host so there is no need to set
the target's can_queue since its the same as the host one. It just results
in extra atomic checks in the main IO path.

Signed-off-by: Mike Christie <michael.christie@oracle.com>
---
 drivers/scsi/iscsi_tcp.c | 1 -
 1 file changed, 1 deletion(-)

Comments

Wu Bo March 9, 2022, 1:31 a.m. UTC | #1
On 2022/3/8 8:27, Mike Christie wrote:
> For software iscsi, we do a session per host so there is no need to set
> the target's can_queue since its the same as the host one. It just results
> in extra atomic checks in the main IO path.
> 
> Signed-off-by: Mike Christie <michael.christie@oracle.com>
> ---
>   drivers/scsi/iscsi_tcp.c | 1 -
>   1 file changed, 1 deletion(-)
> 
> diff --git a/drivers/scsi/iscsi_tcp.c b/drivers/scsi/iscsi_tcp.c
> index 3bdefc4b6b17..974245eab605 100644
> --- a/drivers/scsi/iscsi_tcp.c
> +++ b/drivers/scsi/iscsi_tcp.c
> @@ -1053,7 +1053,6 @@ static struct scsi_host_template iscsi_sw_tcp_sht = {
>   	.eh_target_reset_handler = iscsi_eh_recover_target,
>   	.dma_boundary		= PAGE_SIZE - 1,
>   	.slave_configure        = iscsi_sw_tcp_slave_configure,
> -	.target_alloc		= iscsi_target_alloc,
>   	.proc_name		= "iscsi_tcp",
>   	.this_id		= -1,
>   	.track_queue_depth	= 1,
> 
Reviewed-by: Wu Bo <wubo40@huawei.com>

.
Lee Duncan March 12, 2022, 7:43 p.m. UTC | #2
On 3/7/22 16:27, Mike Christie wrote:
> For software iscsi, we do a session per host so there is no need to set
> the target's can_queue since its the same as the host one. It just results
> in extra atomic checks in the main IO path.
> 
> Signed-off-by: Mike Christie <michael.christie@oracle.com>
> ---
>   drivers/scsi/iscsi_tcp.c | 1 -
>   1 file changed, 1 deletion(-)
> 
> diff --git a/drivers/scsi/iscsi_tcp.c b/drivers/scsi/iscsi_tcp.c
> index 3bdefc4b6b17..974245eab605 100644
> --- a/drivers/scsi/iscsi_tcp.c
> +++ b/drivers/scsi/iscsi_tcp.c
> @@ -1053,7 +1053,6 @@ static struct scsi_host_template iscsi_sw_tcp_sht = {
>   	.eh_target_reset_handler = iscsi_eh_recover_target,
>   	.dma_boundary		= PAGE_SIZE - 1,
>   	.slave_configure        = iscsi_sw_tcp_slave_configure,
> -	.target_alloc		= iscsi_target_alloc,
>   	.proc_name		= "iscsi_tcp",
>   	.this_id		= -1,
>   	.track_queue_depth	= 1,

Reviewed-by: Lee Duncan <lduncan@suse.com>
diff mbox series

Patch

diff --git a/drivers/scsi/iscsi_tcp.c b/drivers/scsi/iscsi_tcp.c
index 3bdefc4b6b17..974245eab605 100644
--- a/drivers/scsi/iscsi_tcp.c
+++ b/drivers/scsi/iscsi_tcp.c
@@ -1053,7 +1053,6 @@  static struct scsi_host_template iscsi_sw_tcp_sht = {
 	.eh_target_reset_handler = iscsi_eh_recover_target,
 	.dma_boundary		= PAGE_SIZE - 1,
 	.slave_configure        = iscsi_sw_tcp_slave_configure,
-	.target_alloc		= iscsi_target_alloc,
 	.proc_name		= "iscsi_tcp",
 	.this_id		= -1,
 	.track_queue_depth	= 1,