Re: [PATCH] VSOCK: Don't set sk_state to TCP_CLOSE before testing it

2017-11-28 Thread David Miller
From: Jorgen Hansen Date: Mon, 27 Nov 2017 05:29:32 -0800 > A recent commit (3b4477d2dcf2) converted the sk_state to use > TCP constants. In that change, vmci_transport_handle_detach > was changed such that sk->sk_state was set to TCP_CLOSE before > we test whether it is

Re: [PATCH] VSOCK: Don't set sk_state to TCP_CLOSE before testing it

2017-11-28 Thread Stefan Hajnoczi
On Mon, Nov 27, 2017 at 05:29:32AM -0800, Jorgen Hansen wrote: > A recent commit (3b4477d2dcf2) converted the sk_state to use > TCP constants. In that change, vmci_transport_handle_detach > was changed such that sk->sk_state was set to TCP_CLOSE before > we test whether it is TCP_SYN_SENT. This

[PATCH] VSOCK: Don't set sk_state to TCP_CLOSE before testing it

2017-11-27 Thread Jorgen Hansen
A recent commit (3b4477d2dcf2) converted the sk_state to use TCP constants. In that change, vmci_transport_handle_detach was changed such that sk->sk_state was set to TCP_CLOSE before we test whether it is TCP_SYN_SENT. This change moves the sk_state change back to the original locations in that