From 2de3e790f537301e0a1cf202d022afca122acdbc Mon Sep 17 00:00:00 2001 From: Gleb Smirnoff Date: Thu, 21 Jan 2016 22:34:51 +0000 Subject: [PATCH] - Rename cc.h to more meaningful tcp_cc.h. - Declare it a kernel only include, which it already is. - Don't include tcp.h implicitly from tcp_cc.h --- sys/netinet/cc/cc.c | 3 ++- sys/netinet/cc/cc_cdg.c | 4 ++-- sys/netinet/cc/cc_chd.c | 4 ++-- sys/netinet/cc/cc_cubic.c | 4 ++-- sys/netinet/cc/cc_dctcp.c | 4 ++-- sys/netinet/cc/cc_hd.c | 4 ++-- sys/netinet/cc/cc_htcp.c | 4 ++-- sys/netinet/cc/cc_newreno.c | 4 ++-- sys/netinet/cc/cc_vegas.c | 5 ++--- sys/netinet/{cc.h => tcp_cc.h} | 11 ++++++----- sys/netinet/tcp_input.c | 3 ++- sys/netinet/tcp_output.c | 3 ++- sys/netinet/tcp_stacks/fastpath.c | 3 ++- sys/netinet/tcp_subr.c | 3 ++- sys/netinet/tcp_timer.c | 3 ++- sys/netinet/tcp_usrreq.c | 3 ++- 16 files changed, 36 insertions(+), 29 deletions(-) rename sys/netinet/{cc.h => tcp_cc.h} (97%) diff --git a/sys/netinet/cc/cc.c b/sys/netinet/cc/cc.c index f1cf328757b7..718bfb4009e7 100644 --- a/sys/netinet/cc/cc.c +++ b/sys/netinet/cc/cc.c @@ -65,10 +65,11 @@ __FBSDID("$FreeBSD$"); #include -#include #include #include +#include #include +#include #include diff --git a/sys/netinet/cc/cc_cdg.c b/sys/netinet/cc/cc_cdg.c index b0bf19103f5e..ec4e73c8c304 100644 --- a/sys/netinet/cc/cc_cdg.c +++ b/sys/netinet/cc/cc_cdg.c @@ -65,11 +65,11 @@ __FBSDID("$FreeBSD$"); #include -#include +#include #include #include #include - +#include #include #include diff --git a/sys/netinet/cc/cc_chd.c b/sys/netinet/cc/cc_chd.c index 42062376c33c..67cf5f369352 100644 --- a/sys/netinet/cc/cc_chd.c +++ b/sys/netinet/cc/cc_chd.c @@ -67,11 +67,11 @@ __FBSDID("$FreeBSD$"); #include -#include +#include #include #include #include - +#include #include #include diff --git a/sys/netinet/cc/cc_cubic.c b/sys/netinet/cc/cc_cubic.c index 339ee6fc0e94..5e9404a7d7cd 100644 --- a/sys/netinet/cc/cc_cubic.c +++ b/sys/netinet/cc/cc_cubic.c @@ -59,11 +59,11 @@ __FBSDID("$FreeBSD$"); #include -#include +#include #include #include #include - +#include #include #include diff --git a/sys/netinet/cc/cc_dctcp.c b/sys/netinet/cc/cc_dctcp.c index 069c2b0ac99c..63e4250f4748 100644 --- a/sys/netinet/cc/cc_dctcp.c +++ b/sys/netinet/cc/cc_dctcp.c @@ -50,10 +50,10 @@ __FBSDID("$FreeBSD$"); #include -#include +#include #include #include - +#include #include #define CAST_PTR_INT(X) (*((int*)(X))) diff --git a/sys/netinet/cc/cc_hd.c b/sys/netinet/cc/cc_hd.c index d6ba1fd69978..a9d6f872d34c 100644 --- a/sys/netinet/cc/cc_hd.c +++ b/sys/netinet/cc/cc_hd.c @@ -68,11 +68,11 @@ __FBSDID("$FreeBSD$"); #include -#include +#include #include #include #include - +#include #include #include diff --git a/sys/netinet/cc/cc_htcp.c b/sys/netinet/cc/cc_htcp.c index 3f1681c60e6c..e5d1c9bb2fbc 100644 --- a/sys/netinet/cc/cc_htcp.c +++ b/sys/netinet/cc/cc_htcp.c @@ -62,11 +62,11 @@ __FBSDID("$FreeBSD$"); #include -#include +#include #include #include #include - +#include #include /* Fixed point math shifts. */ diff --git a/sys/netinet/cc/cc_newreno.c b/sys/netinet/cc/cc_newreno.c index 31b70cc13192..e887d0db9a1b 100644 --- a/sys/netinet/cc/cc_newreno.c +++ b/sys/netinet/cc/cc_newreno.c @@ -62,10 +62,10 @@ __FBSDID("$FreeBSD$"); #include -#include +#include #include #include - +#include #include static void newreno_ack_received(struct cc_var *ccv, uint16_t type); diff --git a/sys/netinet/cc/cc_vegas.c b/sys/netinet/cc/cc_vegas.c index 8e193ae4ef75..db894ab047b0 100644 --- a/sys/netinet/cc/cc_vegas.c +++ b/sys/netinet/cc/cc_vegas.c @@ -69,11 +69,10 @@ __FBSDID("$FreeBSD$"); #include -#include -#include +#include #include #include - +#include #include #include diff --git a/sys/netinet/cc.h b/sys/netinet/tcp_cc.h similarity index 97% rename from sys/netinet/cc.h rename to sys/netinet/tcp_cc.h index 481b401a1ebd..4a2b0c80ac21 100644 --- a/sys/netinet/cc.h +++ b/sys/netinet/tcp_cc.h @@ -48,11 +48,12 @@ * http://caia.swin.edu.au/urp/newtcp/ */ -#ifndef _NETINET_CC_H_ -#define _NETINET_CC_H_ +#ifndef _NETINET_TCP_CC_H_ +#define _NETINET_TCP_CC_H_ -/* XXX: TCP_CA_NAME_MAX define lives in tcp.h for compat reasons. */ -#include +#if !defined(_KERNEL) +#error "no user-servicable parts inside" +#endif /* Global CC vars. */ extern STAILQ_HEAD(cc_head, cc_algo) cc_list; @@ -171,4 +172,4 @@ extern struct rwlock cc_list_lock; #define CC_LIST_WUNLOCK() rw_wunlock(&cc_list_lock) #define CC_LIST_LOCK_ASSERT() rw_assert(&cc_list_lock, RA_LOCKED) -#endif /* _NETINET_CC_H_ */ +#endif /* _NETINET_TCP_CC_H_ */ diff --git a/sys/netinet/tcp_input.c b/sys/netinet/tcp_input.c index bb50d3b15de5..7b2193e5aad3 100644 --- a/sys/netinet/tcp_input.c +++ b/sys/netinet/tcp_input.c @@ -82,7 +82,6 @@ __FBSDID("$FreeBSD$"); #define TCPSTATES /* for logging */ -#include #include #include #include @@ -101,12 +100,14 @@ __FBSDID("$FreeBSD$"); #ifdef TCP_RFC7413 #include #endif +#include #include #include #include #include #include #include +#include #ifdef TCPPCAP #include #endif diff --git a/sys/netinet/tcp_output.c b/sys/netinet/tcp_output.c index 4a4e7eb5061a..55cf209cba44 100644 --- a/sys/netinet/tcp_output.c +++ b/sys/netinet/tcp_output.c @@ -55,7 +55,6 @@ __FBSDID("$FreeBSD$"); #include #include -#include #include #include #include @@ -71,12 +70,14 @@ __FBSDID("$FreeBSD$"); #ifdef TCP_RFC7413 #include #endif +#include #define TCPOUTFLAGS #include #include #include #include #include +#include #ifdef TCPPCAP #include #endif diff --git a/sys/netinet/tcp_stacks/fastpath.c b/sys/netinet/tcp_stacks/fastpath.c index 626a7dcbddd7..c3664df25a94 100644 --- a/sys/netinet/tcp_stacks/fastpath.c +++ b/sys/netinet/tcp_stacks/fastpath.c @@ -86,7 +86,6 @@ __FBSDID("$FreeBSD$"); #define TCPSTATES /* for logging */ -#include #include #include #include @@ -100,6 +99,7 @@ __FBSDID("$FreeBSD$"); #include #include #include +#include #include #include #include @@ -107,6 +107,7 @@ __FBSDID("$FreeBSD$"); #include #include #include +#include #ifdef TCPDEBUG #include #endif /* TCPDEBUG */ diff --git a/sys/netinet/tcp_subr.c b/sys/netinet/tcp_subr.c index d68a8a687d1f..7de745e27cce 100644 --- a/sys/netinet/tcp_subr.c +++ b/sys/netinet/tcp_subr.c @@ -68,7 +68,6 @@ __FBSDID("$FreeBSD$"); #include #include -#include #include #include #include @@ -88,11 +87,13 @@ __FBSDID("$FreeBSD$"); #ifdef TCP_RFC7413 #include #endif +#include #include #include #include #include #include +#include #ifdef INET6 #include #endif diff --git a/sys/netinet/tcp_timer.c b/sys/netinet/tcp_timer.c index fb4ff11a4a72..0dfeb13564f3 100644 --- a/sys/netinet/tcp_timer.c +++ b/sys/netinet/tcp_timer.c @@ -55,7 +55,6 @@ __FBSDID("$FreeBSD$"); #include #include -#include #include #include #include @@ -65,9 +64,11 @@ __FBSDID("$FreeBSD$"); #include #endif #include +#include #include #include #include +#include #ifdef INET6 #include #endif diff --git a/sys/netinet/tcp_usrreq.c b/sys/netinet/tcp_usrreq.c index 76bc8aac0d99..c084f92e7b3f 100644 --- a/sys/netinet/tcp_usrreq.c +++ b/sys/netinet/tcp_usrreq.c @@ -69,7 +69,6 @@ __FBSDID("$FreeBSD$"); #include #include -#include #include #include #include @@ -85,11 +84,13 @@ __FBSDID("$FreeBSD$"); #ifdef TCP_RFC7413 #include #endif +#include #include #include #include #include #include +#include #ifdef TCPPCAP #include #endif