On Sun, Feb 14, 2016 at 11:12:13PM +0530, Varun Prakash wrote:
> clear tx_thread_active for ISCSI_TCP_CXGB4
> transport in logout_post_handler functions.
> 
> Signed-off-by: Varun Prakash <va...@chelsio.com>
> ---
>  drivers/target/iscsi/iscsi_target.c | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/target/iscsi/iscsi_target.c 
> b/drivers/target/iscsi/iscsi_target.c
> index 858f6e4..3dd7ba2 100644
> --- a/drivers/target/iscsi/iscsi_target.c
> +++ b/drivers/target/iscsi/iscsi_target.c
> @@ -4579,7 +4579,8 @@ static void iscsit_logout_post_handler_closesession(
>        * always sleep waiting for RX/TX thread shutdown to complete
>        * within iscsit_close_connection().
>        */
> -     if (conn->conn_transport->transport_type == ISCSI_TCP)
> +     if ((conn->conn_transport->transport_type == ISCSI_TCP) ||
> +         (conn->conn_transport->transport_type == ISCSI_TCP_CXGB4))
>               sleep = cmpxchg(&conn->tx_thread_active, true, false);

Just move the call to iscsit_response_queue, and this is handled much
saner.

>  
>       atomic_set(&conn->conn_logout_remove, 0);
> @@ -4596,7 +4597,8 @@ static void iscsit_logout_post_handler_samecid(
>  {
>       int sleep = 1;
>  
> -     if (conn->conn_transport->transport_type == ISCSI_TCP)
> +     if ((conn->conn_transport->transport_type == ISCSI_TCP) ||
> +         (conn->conn_transport->transport_type == ISCSI_TCP_CXGB4))
>               sleep = cmpxchg(&conn->tx_thread_active, true, false);

Same here.
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to