From bb1d472d79f718296a2e9487dd0c219a9b67b2ff Mon Sep 17 00:00:00 2001 From: Richard Scheffenegger Date: Tue, 13 Sep 2022 12:01:53 +0200 Subject: [PATCH] tcp: make CUBIC the default congestion control mechanism. This changes the default TCP Congestion Control (CC) to CUBIC. For small, transactional exchanges (e.g. web objects <15kB), this will not have a material effect. However, for long duration data transfers, CUBIC allocates a slightly higher fraction of the available bandwidth, when competing against NewReno CC. Reviewed By: tuexen, mav, #transport, guest-ccui, emaste Relnotes: Yes Sponsored by: NetApp, Inc. Differential Revision: https://reviews.freebsd.org/D36537 --- share/man/man4/cc_cubic.4 | 3 ++- share/man/man4/cc_newreno.4 | 3 +-- share/man/man4/mod_cc.4 | 14 +++++++------- sys/amd64/conf/DEFAULTS | 2 +- sys/arm/conf/DEFAULTS | 2 +- sys/arm/conf/std.armv6 | 3 +-- sys/arm/conf/std.armv7 | 3 +-- sys/arm64/conf/DEFAULTS | 2 +- sys/arm64/conf/std.arm64 | 3 +-- sys/conf/NOTES | 16 ++++++++-------- sys/i386/conf/DEFAULTS | 2 +- sys/netinet/cc/cc.c | 2 +- sys/powerpc/conf/DEFAULTS | 2 +- sys/riscv/conf/DEFAULTS | 2 +- 14 files changed, 28 insertions(+), 31 deletions(-) diff --git a/share/man/man4/cc_cubic.4 b/share/man/man4/cc_cubic.4 index 806d6aec2441..9ceaaa871777 100644 --- a/share/man/man4/cc_cubic.4 +++ b/share/man/man4/cc_cubic.4 @@ -30,7 +30,7 @@ .\" .\" $FreeBSD$ .\" -.Dd May 13, 2021 +.Dd September 13, 2022 .Dt CC_CUBIC 4 .Os .Sh NAME @@ -39,6 +39,7 @@ .Sh DESCRIPTION The CUBIC congestion control algorithm was designed to provide increased throughput in fast and long-distance networks. +The CUBIC congestion control algorithm is the default for TCP. It attempts to maintain fairness when competing with legacy NewReno TCP in lower speed scenarios where NewReno is able to operate adequately. .Pp diff --git a/share/man/man4/cc_newreno.4 b/share/man/man4/cc_newreno.4 index 76dd3b2559bb..0d0f0d000d38 100644 --- a/share/man/man4/cc_newreno.4 +++ b/share/man/man4/cc_newreno.4 @@ -30,7 +30,7 @@ .\" .\" $FreeBSD$ .\" -.Dd May 13, 2021 +.Dd September 13, 2022 .Dt CC_NEWRENO 4 .Os .Sh NAME @@ -39,7 +39,6 @@ .Sh SYNOPSIS .In netinet/cc/cc_newreno.h .Sh DESCRIPTION -The NewReno congestion control algorithm is the default for TCP. Details about the algorithm can be found in RFC5681. .Sh Socket Options The diff --git a/share/man/man4/mod_cc.4 b/share/man/man4/mod_cc.4 index a00f17ede5e4..a03ff0630c15 100644 --- a/share/man/man4/mod_cc.4 +++ b/share/man/man4/mod_cc.4 @@ -29,7 +29,7 @@ .\" .\" $FreeBSD$ .\" -.Dd August 6, 2019 +.Dd September 13, 2022 .Dt MOD_CC 4 .Os .Sh NAME @@ -47,7 +47,7 @@ using the .Xr kld 4 facility. .Pp -The default algorithm is NewReno, and all connections use the default unless +The default algorithm is CUBIC, and all connections use the default unless explicitly overridden using the .Dv TCP_CONGESTION socket option (see @@ -127,7 +127,7 @@ For further explanation please see the internet-draft. .Pp Each congestion control module may also expose other MIB variables to control their behaviour. -Note that both newreno and cubic now support hystart++ based on the version 3 of the internet-draft. +Note that both NewReno and CUBIC now support Hystart++ based on the version 3 of the internet-draft. .Sh Kernel Configuration All of the available congestion control modules may also be loaded via kernel configutation options. @@ -138,10 +138,10 @@ Compilation of the kernel will fail if these two conditions are not met. The framework exposes the following kernel configuration options. .Bl -tag -width ".Va CC_NEWRENO" .It Va CC_NEWRENO -This directive loads the newreno congestion control algorithm and is included -in GENERIC by default. +This directive loads the NewReno congestion control algorithm. .It Va CC_CUBIC -This directive loads the cubic congestion control algorithm. +This directive loads the CUBIC congestion control algorithm and is included +in GENERIC by default. .It Va CC_VEGAS This directive loads the vegas congestion control algorithm, note that this algorithm also requires the TCP_HHOOK option as well. @@ -160,7 +160,7 @@ this algorithm also requires the TCP_HHOOK option as well. This directive loads the htcp congestion control algorithm. .It Va CC_DEFAULT This directive specifies the string that represents the name of the system default algorithm, the GENERIC kernel -defaults this to newreno. +defaults this to CUBIC. .El .Sh SEE ALSO .Xr cc_cdg 4 , diff --git a/sys/amd64/conf/DEFAULTS b/sys/amd64/conf/DEFAULTS index 5ea20c19f791..733823e931d6 100644 --- a/sys/amd64/conf/DEFAULTS +++ b/sys/amd64/conf/DEFAULTS @@ -22,6 +22,6 @@ options GEOM_PART_MBR options GEOM_PART_GPT # Default congestion control algorithm -options CC_NEWRENO # include newreno congestion control +options CC_CUBIC # include CUBIC congestion control options NEW_PCIB diff --git a/sys/arm/conf/DEFAULTS b/sys/arm/conf/DEFAULTS index 2e145585c7e8..e5533850aa36 100644 --- a/sys/arm/conf/DEFAULTS +++ b/sys/arm/conf/DEFAULTS @@ -6,5 +6,5 @@ device mem # Default congestion control algorithm -options CC_NEWRENO # include newreno congestion control +options CC_CUBIC # include CUBIC congestion control diff --git a/sys/arm/conf/std.armv6 b/sys/arm/conf/std.armv6 index 47724f176f4a..4826980da480 100644 --- a/sys/arm/conf/std.armv6 +++ b/sys/arm/conf/std.armv6 @@ -8,8 +8,7 @@ options PREEMPTION # Enable kernel thread preemption options VIMAGE # Subsystem virtualization, e.g. VNET options INET # InterNETworking options INET6 # IPv6 communications protocols -options CC_NEWRENO # include newreno congestion control -options CC_DEFAULT=\"newreno\" # define our default CC module it should be compiled in. +options CC_CUBIC # include CUBIC congestion control options TCP_HHOOK # hhook(9) framework for TCP device crypto # core crypto support options IPSEC_SUPPORT # Allow kldload of ipsec and tcpmd5 diff --git a/sys/arm/conf/std.armv7 b/sys/arm/conf/std.armv7 index 88c963643a25..9412a8e92855 100644 --- a/sys/arm/conf/std.armv7 +++ b/sys/arm/conf/std.armv7 @@ -8,8 +8,7 @@ options PREEMPTION # Enable kernel thread preemption options VIMAGE # Subsystem virtualization, e.g. VNET options INET # InterNETworking options INET6 # IPv6 communications protocols -options CC_NEWRENO # include newreno congestion control -options CC_DEFAULT=\"newreno\" # define our default CC module it should be compiled in. +options CC_CUBIC # include CUBIC congestion control options TCP_HHOOK # hhook(9) framework for TCP device crypto # core crypto support options IPSEC_SUPPORT # Allow kldload of ipsec and tcpmd5 diff --git a/sys/arm64/conf/DEFAULTS b/sys/arm64/conf/DEFAULTS index e2b034e94967..b0d2aa6c473b 100644 --- a/sys/arm64/conf/DEFAULTS +++ b/sys/arm64/conf/DEFAULTS @@ -14,6 +14,6 @@ options GEOM_PART_MBR options GEOM_PART_GPT # Default congestion control algorithm -options CC_NEWRENO # include newreno congestion control +options CC_CUBIC # include CUBIC congestion control options NEW_PCIB diff --git a/sys/arm64/conf/std.arm64 b/sys/arm64/conf/std.arm64 index 553e961dc0a2..c2a7e2b09afc 100644 --- a/sys/arm64/conf/std.arm64 +++ b/sys/arm64/conf/std.arm64 @@ -11,8 +11,7 @@ options PREEMPTION # Enable kernel thread preemption options VIMAGE # Subsystem virtualization, e.g. VNET options INET # InterNETworking options INET6 # IPv6 communications protocols -options CC_NEWRENO # include newreno congestion control -options CC_DEFAULT=\"newreno\" # define our default CC module it should be compiled in. +options CC_CUBIC # include CUBIC congestion control options IPSEC_SUPPORT # Allow kldload of ipsec and tcpmd5 options ROUTE_MPATH # Multipath routing support options FIB_ALGO # Modular fib lookups diff --git a/sys/conf/NOTES b/sys/conf/NOTES index 455ae61ce135..434c739c8b21 100644 --- a/sys/conf/NOTES +++ b/sys/conf/NOTES @@ -651,13 +651,13 @@ options INET6 #IPv6 communications protocols # # Note if you include INET/INET6 or both options # You *must* define at least one of the congestion control -# options or the compile will fail. Generic defines -# options CC_NEWRENO. You also will need to specify -# a default or the compile of your kernel will fail -# as well. The string in default is the name of the +# options or the compile will fail. GENERIC defines +# options CC_CUBIC. You may want to specify a default +# if multiple congestion controls are compiled in. +# The string in default is the name of the # cc module as it would appear in the sysctl for -# setting the default. Generic defines newreno -# as shown below. +# setting the default. The code defines CUBIC +# as default, or the sole cc_module compiled in. # options CC_CDG options CC_CHD @@ -667,8 +667,8 @@ options CC_HD options CC_HTCP options CC_NEWRENO options CC_VEGAS -options CC_DEFAULT=\"newreno\" -options RATELIMIT # TX rate limiting support +options CC_DEFAULT=\"cubic\" +options RATELIMIT # TX rate limiting support options ROUTETABLES=2 # allocated fibs up to 65536. default is 1. # but that would be a bad idea as they are large. diff --git a/sys/i386/conf/DEFAULTS b/sys/i386/conf/DEFAULTS index 67328043a537..820ff0ff30f0 100644 --- a/sys/i386/conf/DEFAULTS +++ b/sys/i386/conf/DEFAULTS @@ -23,7 +23,7 @@ options GEOM_PART_MBR options GEOM_PART_GPT # Default congestion control algorithm -options CC_NEWRENO # include newreno congestion control +options CC_CUBIC # include CUBIC congestion control # enable support for native hardware device atpic diff --git a/sys/netinet/cc/cc.c b/sys/netinet/cc/cc.c index a009998ca920..0c2d6cb0e86d 100644 --- a/sys/netinet/cc/cc.c +++ b/sys/netinet/cc/cc.c @@ -81,7 +81,7 @@ __FBSDID("$FreeBSD$"); * Have a sane default if no CC_DEFAULT is specified in the kernel config file. */ #ifndef CC_DEFAULT -#define CC_DEFAULT "newreno" +#define CC_DEFAULT "cubic" #endif uint32_t hystart_minrtt_thresh = 4000; diff --git a/sys/powerpc/conf/DEFAULTS b/sys/powerpc/conf/DEFAULTS index 9bf5e706b04a..984283c5bf5f 100644 --- a/sys/powerpc/conf/DEFAULTS +++ b/sys/powerpc/conf/DEFAULTS @@ -13,6 +13,6 @@ options GEOM_PART_BSD options GEOM_PART_MBR # Default congestion control algorithm -options CC_NEWRENO # include newreno congestion control +options CC_CUBIC # include CUBIC congestion control options NEW_PCIB diff --git a/sys/riscv/conf/DEFAULTS b/sys/riscv/conf/DEFAULTS index f18fdca0c001..d3d7229e0acc 100644 --- a/sys/riscv/conf/DEFAULTS +++ b/sys/riscv/conf/DEFAULTS @@ -13,6 +13,6 @@ options GEOM_PART_BSD options GEOM_PART_MBR # Default congestion control algorithm -options CC_NEWRENO # include newreno congestion control +options CC_CUBIC # include CUBIC congestion control options NEW_PCIB