Hi Trond,
On 02/08/2015 10:07 PM, Trond Myklebust wrote:
> Now that we no longer use the partial shutdown code when closing the
> socket, we no longer need to worry about the TCP linger2 state.
Can you also remove the "#define XS_TCP_LINGER_TO" and xs_tcp_fin_timeout from the top of xprtsock.c? It looks like they don't have any users after this patch.
Thanks,
Anna
>
> Signed-off-by: Trond Myklebust <trond.myklebust@xxxxxxxxxxxxxxx>
> ---
> net/sunrpc/xprtsock.c | 35 -----------------------------------
> 1 file changed, 35 deletions(-)
>
> diff --git a/net/sunrpc/xprtsock.c b/net/sunrpc/xprtsock.c
> index b5dfb4f14ef9..ca45f1c1c36d 100644
> --- a/net/sunrpc/xprtsock.c
> +++ b/net/sunrpc/xprtsock.c
> @@ -1399,37 +1399,6 @@ out:
> read_unlock_bh(&sk->sk_callback_lock);
> }
>
> -/*
> - * Do the equivalent of linger/linger2 handling for dealing with
> - * broken servers that don't close the socket in a timely
> - * fashion
> - */
> -static void xs_tcp_schedule_linger_timeout(struct rpc_xprt *xprt,
> - unsigned long timeout)
> -{
> - struct sock_xprt *transport;
> -
> - if (xprt_test_and_set_connecting(xprt))
> - return;
> - set_bit(XPRT_CONNECTION_ABORT, &xprt->state);
> - transport = container_of(xprt, struct sock_xprt, xprt);
> - queue_delayed_work(rpciod_workqueue, &transport->connect_worker,
> - timeout);
> -}
> -
> -static void xs_tcp_cancel_linger_timeout(struct rpc_xprt *xprt)
> -{
> - struct sock_xprt *transport;
> -
> - transport = container_of(xprt, struct sock_xprt, xprt);
> -
> - if (!test_bit(XPRT_CONNECTION_ABORT, &xprt->state) ||
> - !cancel_delayed_work(&transport->connect_worker))
> - return;
> - clear_bit(XPRT_CONNECTION_ABORT, &xprt->state);
> - xprt_clear_connecting(xprt);
> -}
> -
> static void xs_sock_mark_closed(struct rpc_xprt *xprt)
> {
> xs_sock_reset_connection_flags(xprt);
> @@ -1485,7 +1454,6 @@ static void xs_tcp_state_change(struct sock *sk)
> clear_bit(XPRT_CONNECTED, &xprt->state);
> clear_bit(XPRT_CLOSE_WAIT, &xprt->state);
> smp_mb__after_atomic();
> - xs_tcp_schedule_linger_timeout(xprt, xs_tcp_fin_timeout);
> break;
> case TCP_CLOSE_WAIT:
> /* The server initiated a shutdown of the socket */
> @@ -1502,13 +1470,11 @@ static void xs_tcp_state_change(struct sock *sk)
> break;
> case TCP_LAST_ACK:
> set_bit(XPRT_CLOSING, &xprt->state);
> - xs_tcp_schedule_linger_timeout(xprt, xs_tcp_fin_timeout);
> smp_mb__before_atomic();
> clear_bit(XPRT_CONNECTED, &xprt->state);
> smp_mb__after_atomic();
> break;
> case TCP_CLOSE:
> - xs_tcp_cancel_linger_timeout(xprt);
> xs_sock_mark_closed(xprt);
> }
> out:
> @@ -2106,7 +2072,6 @@ static int xs_tcp_finish_connecting(struct rpc_xprt *xprt, struct socket *sock)
>
> /* socket options */
> sock_reset_flag(sk, SOCK_LINGER);
> - tcp_sk(sk)->linger2 = 0;
> tcp_sk(sk)->nonagle |= TCP_NAGLE_OFF;
>
> xprt_clear_connected(xprt);
>
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
