From nobody Fri Sep 20 11:05:38 2024 X-Original-To: dev-commits-src-main@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 4X98gk6snTz5Vnb3; Fri, 20 Sep 2024 11:05:38 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from mxrelay.nyi.freebsd.org (mxrelay.nyi.freebsd.org [IPv6:2610:1c1:1:606c::19:3]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "mxrelay.nyi.freebsd.org", Issuer "R11" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4X98gk6Fzwz4g4Y; Fri, 20 Sep 2024 11:05:38 +0000 (UTC) (envelope-from git@FreeBSD.org) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1726830338; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=1LjI01F/WM9Fgv7MYxOmg0U6gGeFY/GGXXKhk37TVxk=; b=E7SaoYLtlkVkihNjq7HCpKUYT52M+o7zj7I24rmMVadkCB4K8Vu20YH83tB5WVYTHTw4iU UcZ65nIAoC0FIxknla1ZQUbDe2xGjRs0y93nl+BOovCpQCorh8AmZLuU8DaJiY88lun/3p 8qv/Aj55n/KuA3AkNWu4VIZSFOPPrucAejiJEhrnXc4OYU3RDbogFBkbMHUlFGwFQBzv0F m9aeQNDyK8ZLAG4XvDQjmzQOFey1B1ozJmsCrE+EeMUivsfCIHSaZi4qDhcF6nVwSX9nvy VV/Kqek4rRSX/eSOhqustvwfrrTX9e2ISIXRzYY0zSd6cuaUnFzgXcFmNrVUeg== ARC-Seal: i=1; s=dkim; d=freebsd.org; t=1726830338; a=rsa-sha256; cv=none; b=EkRwIN3tqFEt6VA3qfhDGx5VZBvSWakDTHpkD1PcLQDOfVUiUXdU9ZSfbIGnRc9xEvg/0s KCpvO0x6AtvLdakY6E/wWsn+adgM77DjiNKnXCIFsPmjrFvhG8BbrmejO1rJfHlXhNKnE4 yIqSDLjlQS5scM6s1hgYGwn2qWVw6w/InYXRxTF5x0UejAeWfIL5ScLSlKfOXj5s0OcZ3K VzlT93wnBTuQjKv7IaSaGtgSWdOnb3YMkFki65rDlSd3OTKGrYI0yNquNta8p8XsWss4qc ibtyLRPlyNNidgU9GF9QWuyP8o5UY7lc8PVaDusfnDeYOaGorgEhvYdcegXxVA== ARC-Authentication-Results: i=1; mx1.freebsd.org; none ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1726830338; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=1LjI01F/WM9Fgv7MYxOmg0U6gGeFY/GGXXKhk37TVxk=; b=WPLXyecIAXwMKZmoV9mU1130emBCnScV8aUG4rJ0tAFu1dczFeeS1NAz+ua93aWEenusRe Br5Jqlu3bQuB5lY7uSR8zU+u1T5sx0pTuwc1EY7MvOIxTxS3Sbq0YhLgRB6CtiH2O0HVG4 l3WO83U0d6G18uFsMH18yfWP82rO1xE/mEbVnncAWHZLL+H16ZpbqSMV1KUvTDtxUHURo8 HxSTyo1gLfjjE/BSop9pBHjd341Ts4rM+/oml3M0e6ob8kshbA46KTEg2ZpIgdUXVk1uOW TZZuE4HNj0ogol9aCH55NQWF1xFsoW7tZTm/dZbvXTS9KL72PL+zCTCjjYWXiw== Received: from gitrepo.freebsd.org (gitrepo.freebsd.org [IPv6:2610:1c1:1:6068::e6a:5]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (Client did not present a certificate) by mxrelay.nyi.freebsd.org (Postfix) with ESMTPS id 4X98gk5s3nznHf; Fri, 20 Sep 2024 11:05:38 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from gitrepo.freebsd.org ([127.0.1.44]) by gitrepo.freebsd.org (8.18.1/8.18.1) with ESMTP id 48KB5c3u053282; Fri, 20 Sep 2024 11:05:38 GMT (envelope-from git@gitrepo.freebsd.org) Received: (from git@localhost) by gitrepo.freebsd.org (8.18.1/8.18.1/Submit) id 48KB5cS2053279; Fri, 20 Sep 2024 11:05:38 GMT (envelope-from git) Date: Fri, 20 Sep 2024 11:05:38 GMT Message-Id: <202409201105.48KB5cS2053279@gitrepo.freebsd.org> To: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org From: Michael Tuexen Subject: git: 87fbd9fc7fc5 - main - tcp: remove unused socket option names List-Id: Commit messages for the main branch of the src repository List-Archive: https://lists.freebsd.org/archives/dev-commits-src-main List-Help: List-Post: List-Subscribe: List-Unsubscribe: X-BeenThere: dev-commits-src-main@freebsd.org Sender: owner-dev-commits-src-main@FreeBSD.org MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-Git-Committer: tuexen X-Git-Repository: src X-Git-Refname: refs/heads/main X-Git-Reftype: branch X-Git-Commit: 87fbd9fc7fc5f8d79fe5e3dcd13ad02b11a67ef0 Auto-Submitted: auto-generated The branch main has been updated by tuexen: URL: https://cgit.FreeBSD.org/src/commit/?id=87fbd9fc7fc5f8d79fe5e3dcd13ad02b11a67ef0 commit 87fbd9fc7fc5f8d79fe5e3dcd13ad02b11a67ef0 Author: Michael Tuexen AuthorDate: 2024-09-20 11:03:53 +0000 Commit: Michael Tuexen CommitDate: 2024-09-20 11:03:53 +0000 tcp: remove unused socket option names These IPPROTO_TCP-level socket option names correspond to socket options, which are not implemented. So remove them. Thanks to Peter Lei for suggesting this change. Reviewed by: rscheff, thj Sponsored by: Netflix, Inc. Differential Revision: https://reviews.freebsd.org/D46623 --- sys/netinet/tcp.h | 47 ++++++++++++++++++++------------------------ sys/netinet/tcp_stacks/bbr.c | 2 -- 2 files changed, 21 insertions(+), 28 deletions(-) diff --git a/sys/netinet/tcp.h b/sys/netinet/tcp.h index 6d92f078029b..f789898e5785 100644 --- a/sys/netinet/tcp.h +++ b/sys/netinet/tcp.h @@ -211,7 +211,7 @@ __tcp_set_flags(struct tcphdr *th, uint16_t flags) #define TCP_CONGESTION 64 /* get/set congestion control algorithm */ #define TCP_CCALGOOPT 65 /* get/set cc algorithm specific options */ #define TCP_MAXUNACKTIME 68 /* maximum time without making progress (sec) */ -#define TCP_MAXPEAKRATE 69 /* maximum peak rate allowed (kbps) */ + /* unused; was TCP_MAXPEAKRATE */ #define TCP_IDLE_REDUCE 70 /* Reduce cwnd on idle input */ #define TCP_REMOTE_UDP_ENCAPS_PORT 71 /* Enable TCP over UDP tunneling via the specified port */ #define TCP_DELACK 72 /* socket option for delayed ack */ @@ -233,57 +233,53 @@ __tcp_set_flags(struct tcphdr *th, uint16_t flags) /* Options for Rack and BBR */ #define TCP_REUSPORT_LB_NUMA 1026 /* set listen socket numa domain */ #define TCP_RACK_MBUF_QUEUE 1050 /* Do we allow mbuf queuing if supported */ -#define TCP_RACK_PROP 1051 /* Not used */ + /* unused; was TCP_RACK_PROP */ #define TCP_RACK_TLP_REDUCE 1052 /* RACK TLP cwnd reduction (bool) */ -#define TCP_RACK_PACE_REDUCE 1053 /* RACK Pacingv reduction factor (divisor) */ + /* unused; was TCP_RACK_PACE_REDUCE */ #define TCP_RACK_PACE_MAX_SEG 1054 /* Max TSO size we will send */ #define TCP_RACK_PACE_ALWAYS 1055 /* Use the always pace method */ -#define TCP_RACK_PROP_RATE 1056 /* Not used */ + /* unused; was TCP_RACK_PROP_RATE */ #define TCP_RACK_PRR_SENDALOT 1057 /* Allow PRR to send more than one seg */ #define TCP_RACK_MIN_TO 1058 /* Minimum time between rack t-o's in ms */ -#define TCP_RACK_EARLY_RECOV 1059 /* Not used */ + /* unused; was TCP_RACK_EARLY_RECOV */ #define TCP_RACK_EARLY_SEG 1060 /* If early recovery max segments */ #define TCP_RACK_REORD_THRESH 1061 /* RACK reorder threshold (shift amount) */ #define TCP_RACK_REORD_FADE 1062 /* Does reordering fade after ms time */ #define TCP_RACK_TLP_THRESH 1063 /* RACK TLP theshold i.e. srtt+(srtt/N) */ #define TCP_RACK_PKT_DELAY 1064 /* RACK added ms i.e. rack-rtt + reord + N */ -#define TCP_RACK_TLP_INC_VAR 1065 /* Does TLP include rtt variance in t-o */ + /* unused; was TCP_RACK_TLP_INC_VAR */ + /* unused */ #define TCP_BBR_IWINTSO 1067 /* Initial TSO window for BBRs first sends */ -#define TCP_BBR_RECFORCE 1068 /* Enter recovery force out a segment disregard pacer no longer valid */ + /* unused; was TCP_BBR_RECFORCE */ #define TCP_BBR_STARTUP_PG 1069 /* Startup pacing gain */ #define TCP_BBR_DRAIN_PG 1070 /* Drain pacing gain */ -#define TCP_BBR_RWND_IS_APP 1071 /* Rwnd limited is considered app limited */ + /* unused; was TCP_BBR_RWND_IS_APP */ #define TCP_BBR_PROBE_RTT_INT 1072 /* How long in useconds between probe-rtt */ -#define TCP_BBR_ONE_RETRAN 1073 /* Is only one segment allowed out during retran */ + /* unused; was TCP_BBR_ONE_RETRAN */ #define TCP_BBR_STARTUP_LOSS_EXIT 1074 /* Do we exit a loss during startup if not 20% incr */ -#define TCP_BBR_USE_LOWGAIN 1075 /* lower the gain in PROBE_BW enable */ -#define TCP_BBR_LOWGAIN_THRESH 1076 /* Unused after 2.3 morphs to TSLIMITS >= 2.3 */ + /* unused; was TCP_BBR_USE_LOWGAIN */ #define TCP_BBR_TSLIMITS 1076 /* Do we use experimental Timestamp limiting for our algo */ -#define TCP_BBR_LOWGAIN_HALF 1077 /* Unused after 2.3 */ #define TCP_BBR_PACE_OH 1077 /* Reused in 4.2 for pacing overhead setting */ -#define TCP_BBR_LOWGAIN_FD 1078 /* Unused after 2.3 */ -#define TCP_BBR_HOLD_TARGET 1078 /* For 4.3 on */ + /* unused; was TCP_BBR_HOLD_TARGET */ #define TCP_BBR_USEDEL_RATE 1079 /* Enable use of delivery rate for loss recovery */ #define TCP_BBR_MIN_RTO 1080 /* Min RTO in milliseconds */ #define TCP_BBR_MAX_RTO 1081 /* Max RTO in milliseconds */ -#define TCP_BBR_REC_OVER_HPTS 1082 /* Recovery override htps settings 0/1/3 */ -#define TCP_BBR_UNLIMITED 1083 /* Not used before 2.3 and morphs to algorithm >= 2.3 */ + /* unused; was TCP_BBR_REC_OVER_HPTS */ #define TCP_BBR_ALGORITHM 1083 /* What measurement algo does BBR use netflix=0, google=1 */ -#define TCP_BBR_DRAIN_INC_EXTRA 1084 /* Does the 3/4 drain target include the extra gain */ -#define TCP_BBR_STARTUP_EXIT_EPOCH 1085 /* what epoch gets us out of startup */ + /* unused; was TCP_BBR_DRAIN_INC_EXTRA */ + /* unused; was TCP_BBR_STARTUP_EXIT_EPOCH */ #define TCP_BBR_PACE_PER_SEC 1086 #define TCP_BBR_PACE_DEL_TAR 1087 #define TCP_BBR_PACE_SEG_MAX 1088 #define TCP_BBR_PACE_SEG_MIN 1089 #define TCP_BBR_PACE_CROSS 1090 -#define TCP_RACK_IDLE_REDUCE_HIGH 1092 /* Reduce the highest cwnd seen to IW on idle */ -#define TCP_RACK_MIN_PACE 1093 /* Do we enforce rack min pace time */ -#define TCP_RACK_MIN_PACE_SEG 1094 /* If so what is the seg threshould */ -#define TCP_RACK_GP_INCREASE 1094 /* After 4.1 its the GP increase in older rack */ + /* unused */ + /* unused; was TCP_RACK_IDLE_REDUCE_HIGH */ + /* unused; was TCP_RACK_MIN_PACE */ + /* unused; was TCP_RACK_GP_INCREASE */ #define TCP_RACK_TLP_USE 1095 -#define TCP_BBR_ACK_COMP_ALG 1096 /* Not used */ #define TCP_BBR_TMR_PACE_OH 1096 /* Recycled in 4.2 */ -#define TCP_BBR_EXTRA_GAIN 1097 + /* unused; was TCP_BBR_EXTRA_GAIN */ #define TCP_RACK_DO_DETECTION 1097 /* Recycle of extra gain for rack, attack detection */ #define TCP_BBR_RACK_RTT_USE 1098 /* what RTT should we use 0, 1, or 2? */ #define TCP_BBR_RETRAN_WTSO 1099 @@ -302,7 +298,6 @@ __tcp_set_flags(struct tcphdr *th, uint16_t flags) #define TCP_BBR_POLICER_DETECT 1111 /* Turn on/off google mode policer detection */ #define TCP_BBR_RACK_INIT_RATE 1112 /* Set an initial pacing rate for when we have no b/w in kbits per sec */ #define TCP_RACK_RR_CONF 1113 /* Rack rapid recovery configuration control*/ -#define TCP_RACK_CHEAT_NOT_CONF_RATE TCP_RACK_RR_CONF #define TCP_RACK_GP_INCREASE_CA 1114 /* GP increase for Congestion Avoidance */ #define TCP_RACK_GP_INCREASE_SS 1115 /* GP increase for Slow Start */ #define TCP_RACK_GP_INCREASE_REC 1116 /* GP increase for Recovery */ @@ -325,7 +320,7 @@ __tcp_set_flags(struct tcphdr *th, uint16_t flags) #define TCP_REC_ABC_VAL 1134 /* Do we use the ABC value for recovery or the override one from sysctl */ #define TCP_RACK_MEASURE_CNT 1135 /* How many measurements are required in GP pacing */ #define TCP_DEFER_OPTIONS 1136 /* Defer options until the proper number of measurements occur, does not defer TCP_RACK_MEASURE_CNT */ -#define TCP_FAST_RSM_HACK 1137 /* Not used in modern stacks */ + /* unused; was TCP_FAST_RSM_HACK */ #define TCP_RACK_PACING_BETA 1138 /* Changing the beta for pacing */ #define TCP_RACK_PACING_BETA_ECN 1139 /* Changing the beta for ecn with pacing */ #define TCP_RACK_TIMER_SLOP 1140 /* Set or get the timer slop used */ diff --git a/sys/netinet/tcp_stacks/bbr.c b/sys/netinet/tcp_stacks/bbr.c index 3f8c749583b4..866ae29eef45 100644 --- a/sys/netinet/tcp_stacks/bbr.c +++ b/sys/netinet/tcp_stacks/bbr.c @@ -14215,10 +14215,8 @@ bbr_set_sockopt(struct tcpcb *tp, struct sockopt *sopt) case TCP_BBR_ALGORITHM: case TCP_BBR_TSLIMITS: case TCP_BBR_IWINTSO: - case TCP_BBR_RECFORCE: case TCP_BBR_STARTUP_PG: case TCP_BBR_DRAIN_PG: - case TCP_BBR_RWND_IS_APP: case TCP_BBR_PROBE_RTT_INT: case TCP_BBR_PROBE_RTT_GAIN: case TCP_BBR_PROBE_RTT_LEN: