aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJeff Layton <jlayton@redhat.com>2017-02-24 13:25:23 -0500
committerJ. Bruce Fields <bfields@redhat.com>2017-02-24 16:55:46 -0500
commit362142b25843fb059941aaa01b91501d5d8652cc (patch)
treed8069aae1c1aa73a17ee47ec0df3391cb01fdee2
parent05a45a2db42543c5f1a32e08f545aebbd7cb4790 (diff)
sunrpc: flag transports as having congestion control
NFSv4 requires a transport protocol with congestion control in most cases. On an IP network, that means that NFSv4 over UDP should be forbidden. The situation with RDMA is a bit more nuanced, but most RDMA transports are suitable for this. For now, we assume that all RDMA transports are suitable, but we may need to revise that at some point. Signed-off-by: Jeff Layton <jlayton@redhat.com> Signed-off-by: J. Bruce Fields <bfields@redhat.com>
-rw-r--r--include/linux/sunrpc/svc_xprt.h1
-rw-r--r--net/sunrpc/svcsock.c1
-rw-r--r--net/sunrpc/xprtrdma/svc_rdma_transport.c8
3 files changed, 10 insertions, 0 deletions
diff --git a/include/linux/sunrpc/svc_xprt.h b/include/linux/sunrpc/svc_xprt.h
index 7440290f64ac..ddb7f94a9d06 100644
--- a/include/linux/sunrpc/svc_xprt.h
+++ b/include/linux/sunrpc/svc_xprt.h
@@ -67,6 +67,7 @@ struct svc_xprt {
67#define XPT_CACHE_AUTH 11 /* cache auth info */ 67#define XPT_CACHE_AUTH 11 /* cache auth info */
68#define XPT_LOCAL 12 /* connection from loopback interface */ 68#define XPT_LOCAL 12 /* connection from loopback interface */
69#define XPT_KILL_TEMP 13 /* call xpo_kill_temp_xprt before closing */ 69#define XPT_KILL_TEMP 13 /* call xpo_kill_temp_xprt before closing */
70#define XPT_CONG_CTRL 14 /* has congestion control */
70 71
71 struct svc_serv *xpt_server; /* service for transport */ 72 struct svc_serv *xpt_server; /* service for transport */
72 atomic_t xpt_reserved; /* space on outq that is rsvd */ 73 atomic_t xpt_reserved; /* space on outq that is rsvd */
diff --git a/net/sunrpc/svcsock.c b/net/sunrpc/svcsock.c
index de066acdb34e..1956b8b96b2d 100644
--- a/net/sunrpc/svcsock.c
+++ b/net/sunrpc/svcsock.c
@@ -1306,6 +1306,7 @@ static void svc_tcp_init(struct svc_sock *svsk, struct svc_serv *serv)
1306 svc_xprt_init(sock_net(svsk->sk_sock->sk), &svc_tcp_class, 1306 svc_xprt_init(sock_net(svsk->sk_sock->sk), &svc_tcp_class,
1307 &svsk->sk_xprt, serv); 1307 &svsk->sk_xprt, serv);
1308 set_bit(XPT_CACHE_AUTH, &svsk->sk_xprt.xpt_flags); 1308 set_bit(XPT_CACHE_AUTH, &svsk->sk_xprt.xpt_flags);
1309 set_bit(XPT_CONG_CTRL, &svsk->sk_xprt.xpt_flags);
1309 if (sk->sk_state == TCP_LISTEN) { 1310 if (sk->sk_state == TCP_LISTEN) {
1310 dprintk("setting up TCP socket for listening\n"); 1311 dprintk("setting up TCP socket for listening\n");
1311 set_bit(XPT_LISTENER, &svsk->sk_xprt.xpt_flags); 1312 set_bit(XPT_LISTENER, &svsk->sk_xprt.xpt_flags);
diff --git a/net/sunrpc/xprtrdma/svc_rdma_transport.c b/net/sunrpc/xprtrdma/svc_rdma_transport.c
index ab2fd5377f94..9d9e4e16080f 100644
--- a/net/sunrpc/xprtrdma/svc_rdma_transport.c
+++ b/net/sunrpc/xprtrdma/svc_rdma_transport.c
@@ -569,6 +569,14 @@ static struct svcxprt_rdma *rdma_create_xprt(struct svc_serv *serv,
569 spin_lock_init(&cma_xprt->sc_ctxt_lock); 569 spin_lock_init(&cma_xprt->sc_ctxt_lock);
570 spin_lock_init(&cma_xprt->sc_map_lock); 570 spin_lock_init(&cma_xprt->sc_map_lock);
571 571
572 /*
573 * Note that this implies that the underlying transport support
574 * has some form of congestion control (see RFC 7530 section 3.1
575 * paragraph 2). For now, we assume that all supported RDMA
576 * transports are suitable here.
577 */
578 set_bit(XPT_CONG_CTRL, &cma_xprt->sc_xprt.xpt_flags);
579
572 if (listener) 580 if (listener)
573 set_bit(XPT_LISTENER, &cma_xprt->sc_xprt.xpt_flags); 581 set_bit(XPT_LISTENER, &cma_xprt->sc_xprt.xpt_flags);
574 582