kernel_connect() will modify the rpc_xprt socket address in contexts where eBPF programs perform NAT instead of iptables. In these contexts, it is common for an NFS mount to be mounted to be a static virtual IP while the server has an ephemeral IP leading to a problem where the virtual IP gets overwritten and forgotten. When the endpoint IP changes, reconnect attempts fail and the mount never recovers. This patch protects addr from being modified in these scenarios, allowing NFS reconnects to work as intended. Link: https://github.com/cilium/cilium/issues/21541#issuecomment-1386857338 Signed-off-by: Jordan Rife <jrife@xxxxxxxxxx> --- include/linux/sunrpc/xprt.h | 1 + net/sunrpc/xprtsock.c | 17 +++++++++++++++-- 2 files changed, 16 insertions(+), 2 deletions(-) diff --git a/include/linux/sunrpc/xprt.h b/include/linux/sunrpc/xprt.h index b52411bcfe4e7..ddde79b025c53 100644 --- a/include/linux/sunrpc/xprt.h +++ b/include/linux/sunrpc/xprt.h @@ -211,6 +211,7 @@ struct rpc_xprt { const struct rpc_timeout *timeout; /* timeout parms */ struct sockaddr_storage addr; /* server address */ + struct sockaddr_storage m_addr; /* mutable server address */ size_t addrlen; /* size of server address */ int prot; /* IP protocol */ diff --git a/net/sunrpc/xprtsock.c b/net/sunrpc/xprtsock.c index 9f010369100a2..4100e0bf5ebba 100644 --- a/net/sunrpc/xprtsock.c +++ b/net/sunrpc/xprtsock.c @@ -236,6 +236,18 @@ static inline struct sockaddr *xs_addr(struct rpc_xprt *xprt) return (struct sockaddr *) &xprt->addr; } +static inline struct sockaddr *xs_m_addr(struct rpc_xprt *xprt) +{ + /* kernel_connect() may modify the address in contexts where NAT is + * performed by eBPF programs instead of iptables. Make a copy to ensure + * that our original address, xprt->addr, is not modified. Without this, + * NFS reconnects may fail if the endpoint address changes. + */ + memcpy(&xprt->m_addr, &xprt->addr, xprt->addrlen); + + return (struct sockaddr *) &xprt->m_addr; +} + static inline struct sockaddr_un *xs_addr_un(struct rpc_xprt *xprt) { return (struct sockaddr_un *) &xprt->addr; @@ -1954,7 +1966,7 @@ static int xs_local_finish_connecting(struct rpc_xprt *xprt, xs_stream_start_connect(transport); - return kernel_connect(sock, xs_addr(xprt), xprt->addrlen, 0); + return kernel_connect(sock, xs_m_addr(xprt), xprt->addrlen, 0); } /** @@ -2334,7 +2346,8 @@ static int xs_tcp_finish_connecting(struct rpc_xprt *xprt, struct socket *sock) /* Tell the socket layer to start connecting... */ set_bit(XPRT_SOCK_CONNECTING, &transport->sock_state); - return kernel_connect(sock, xs_addr(xprt), xprt->addrlen, O_NONBLOCK); + + return kernel_connect(sock, xs_m_addr(xprt), xprt->addrlen, O_NONBLOCK); } /** -- 2.42.0.rc1.204.g551eb34607-goog
