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 <varun@xxxxxxxxxxx>
> ---
> 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 target-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
