From nobody Thu Feb 2 00:03:30 2023 X-Original-To: dev-commits-src-all@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 4P6fBq5vkyz3cQ17; Thu, 2 Feb 2023 00:03:31 +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 "R3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4P6fBp4tDKz3k0j; Thu, 2 Feb 2023 00:03:30 +0000 (UTC) (envelope-from git@FreeBSD.org) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1675296210; 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=Yw5xBr3/sD8U9yuN1kHQ+LFat0Y4KrMR/t/osEAMGCE=; b=weYOiZ9rRh4vr7lJSxB63EoVNlZU+yTiqd+v/AatPGroQJfs12jJLRcIq3gXz3cDfYX7oM KZxuok2G9itomk2qQUClCqyxFIQigMfl7flKyND8a2em8ziwKbEzLWZur6CFvVxo4OVWHf P7chRXHRnayWOlENSdVv/ms49MIys2Q5neRmscTJwU5uRnPLm35Pkbdo8sEv2g2G8o9xGi fwiqOtCrsX6P9K7rFp0Zl0IePUHNUbEu+4Q8LEj0N9+WV4sI0us4GbfP3TxXr2PsJAYSDS RBOu1syK32B3YpD4HTICBYT9J66+rTa5Br6cNHKZ+iNvQboANxvuMSG2jfG+4g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1675296210; 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=Yw5xBr3/sD8U9yuN1kHQ+LFat0Y4KrMR/t/osEAMGCE=; b=IPXmnLvs9qZ+vJsBNW5OgFHzurhRikVj0E5QYrJh65tcDjCyzAAkUEcZEeayHGf7YVfQff T+K9GWtjnGeEDCq46ObYF/dk81eM5yCH0QZQwXx2JjCuF7yBEZiPf9zXotXgzBu+cDh3ox SYkA3Nc670BSA/0tpxjh75T7R8H4sUIrzo8fbOnV9kwNgRz0W2qAQfSLV465xipi6/z8wz nVCaEA8qUKuSEQnkbH/7GTsvhI+jcxS44zcguvt4Ha6P9YJpjTK2NrprDc2WWakbICb30k 3laUpVoucxIz3olIpxBEyI0GsFiDI1B3CSLgqQSP7bYtEdbqGSJ4Cw1qcRDOAw== ARC-Authentication-Results: i=1; mx1.freebsd.org; none ARC-Seal: i=1; s=dkim; d=freebsd.org; t=1675296210; a=rsa-sha256; cv=none; b=lxkwYP14hLd9byyoYZ91+IJjVaih1aHeSbZDeeZmQVDSDN0RZ4au54PP/sMtCeS6sIcAlZ nbXTxYegImXxLfezbu24F2Zbk0ORl2nMiTuZ5L/uOVeUYyjOOsW+ddJt+7yf7DEGuvKDJ5 cDp6mYcLl6Tde+oFBkOusBL7AuLAsp7s7oWU/EjiuUheqcERYdOThxvaF5jFYSUorWPlSM LP4P0eYgm/EF1Hz/Ou0PktVuHTi4CRuIghmyRxXZGeg+5JGFPJjAZgSvzRkePMgapfwWVK wV8zJuo4Sbz2agzdMtxAN69bMe5HMNbv/mHWRhHf/NvXH9artTIb8O89R4/voA== 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 4P6fBp3rXVzYmx; Thu, 2 Feb 2023 00:03:30 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from gitrepo.freebsd.org ([127.0.1.44]) by gitrepo.freebsd.org (8.16.1/8.16.1) with ESMTP id 31203UiV020345; Thu, 2 Feb 2023 00:03:30 GMT (envelope-from git@gitrepo.freebsd.org) Received: (from git@localhost) by gitrepo.freebsd.org (8.16.1/8.16.1/Submit) id 31203U5L020344; Thu, 2 Feb 2023 00:03:30 GMT (envelope-from git) Date: Thu, 2 Feb 2023 00:03:30 GMT Message-Id: <202302020003.31203U5L020344@gitrepo.freebsd.org> To: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org From: Michael Tuexen Subject: git: e97522a1b7ea - stable/13 - tcp: remove unused t_rttbest List-Id: Commit messages for all branches of the src repository List-Archive: https://lists.freebsd.org/archives/dev-commits-src-all List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-dev-commits-src-all@freebsd.org X-BeenThere: dev-commits-src-all@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/stable/13 X-Git-Reftype: branch X-Git-Commit: e97522a1b7ea023961d604221eb38960fd170fa9 Auto-Submitted: auto-generated X-ThisMailContainsUnwantedMimeParts: N The branch stable/13 has been updated by tuexen: URL: https://cgit.FreeBSD.org/src/commit/?id=e97522a1b7ea023961d604221eb38960fd170fa9 commit e97522a1b7ea023961d604221eb38960fd170fa9 Author: Michael Tuexen AuthorDate: 2022-11-16 10:22:13 +0000 Commit: Michael Tuexen CommitDate: 2023-02-02 00:02:30 +0000 tcp: remove unused t_rttbest No functional change intended. Reviewed by: rscheff@ Sponsored by: Netflix, Inc. Differential Revision: https://reviews.freebsd.org/D37401 (cherry picked from commit bd4f986644b3804e19b92d39f1f90ffabff83e14) --- sys/netinet/tcp_input.c | 4 ---- sys/netinet/tcp_stacks/bbr.c | 3 --- sys/netinet/tcp_stacks/rack.c | 3 --- sys/netinet/tcp_usrreq.c | 5 ++--- sys/netinet/tcp_var.h | 2 -- 5 files changed, 2 insertions(+), 15 deletions(-) diff --git a/sys/netinet/tcp_input.c b/sys/netinet/tcp_input.c index 94e9d0b71cb8..1c2fe014c7f3 100644 --- a/sys/netinet/tcp_input.c +++ b/sys/netinet/tcp_input.c @@ -410,7 +410,6 @@ cc_conn_init(struct tcpcb *tp) if (tp->t_srtt == 0 && (rtt = metrics.rmx_rtt)) { tp->t_srtt = rtt; - tp->t_rttbest = tp->t_srtt + TCP_RTT_SCALE; TCPSTAT_INC(tcps_usedrtt); if (metrics.rmx_rttvar) { tp->t_rttvar = metrics.rmx_rttvar; @@ -3642,8 +3641,6 @@ tcp_xmit_timer(struct tcpcb *tp, int rtt) delta -= tp->t_rttvar >> (TCP_RTTVAR_SHIFT - TCP_DELTA_SHIFT); if ((tp->t_rttvar += delta) <= 0) tp->t_rttvar = 1; - if (tp->t_rttbest > tp->t_srtt + tp->t_rttvar) - tp->t_rttbest = tp->t_srtt + tp->t_rttvar; } else { /* * No rtt measurement yet - use the unsmoothed rtt. @@ -3652,7 +3649,6 @@ tcp_xmit_timer(struct tcpcb *tp, int rtt) */ tp->t_srtt = rtt << TCP_RTT_SHIFT; tp->t_rttvar = rtt << (TCP_RTTVAR_SHIFT - 1); - tp->t_rttbest = tp->t_srtt + tp->t_rttvar; } tp->t_rtttime = 0; tp->t_rxtshift = 0; diff --git a/sys/netinet/tcp_stacks/bbr.c b/sys/netinet/tcp_stacks/bbr.c index 42c66ac57891..1b4abfb6f5c4 100644 --- a/sys/netinet/tcp_stacks/bbr.c +++ b/sys/netinet/tcp_stacks/bbr.c @@ -6395,8 +6395,6 @@ tcp_bbr_xmit_timer_commit(struct tcp_bbr *bbr, struct tcpcb *tp, uint32_t cts) tp->t_rttvar += delta; if (tp->t_rttvar <= 0) tp->t_rttvar = 1; - if (tp->t_rttbest > tp->t_srtt + tp->t_rttvar) - tp->t_rttbest = tp->t_srtt + tp->t_rttvar; } else { /* * No rtt measurement yet - use the unsmoothed rtt. Set the @@ -6405,7 +6403,6 @@ tcp_bbr_xmit_timer_commit(struct tcp_bbr *bbr, struct tcpcb *tp, uint32_t cts) */ tp->t_srtt = rtt_ticks << TCP_RTT_SHIFT; tp->t_rttvar = rtt_ticks << (TCP_RTTVAR_SHIFT - 1); - tp->t_rttbest = tp->t_srtt + tp->t_rttvar; } KMOD_TCPSTAT_INC(tcps_rttupdated); tp->t_rttupdated++; diff --git a/sys/netinet/tcp_stacks/rack.c b/sys/netinet/tcp_stacks/rack.c index 162b1c928ef7..ead81528ace5 100644 --- a/sys/netinet/tcp_stacks/rack.c +++ b/sys/netinet/tcp_stacks/rack.c @@ -7737,8 +7737,6 @@ tcp_rack_xmit_timer_commit(struct tcp_rack *rack, struct tcpcb *tp) tp->t_rttvar += (delta >> 3); if (tp->t_rttvar <= 0) tp->t_rttvar = 1; - if (tp->t_rttbest > tp->t_srtt + tp->t_rttvar) - tp->t_rttbest = tp->t_srtt + tp->t_rttvar; } else { /* * No rtt measurement yet - use the unsmoothed rtt. Set the @@ -7747,7 +7745,6 @@ tcp_rack_xmit_timer_commit(struct tcp_rack *rack, struct tcpcb *tp) */ tp->t_srtt = rtt; tp->t_rttvar = rtt >> 1; - tp->t_rttbest = tp->t_srtt + tp->t_rttvar; } rack->rc_srtt_measure_made = 1; KMOD_TCPSTAT_INC(tcps_rttupdated); diff --git a/sys/netinet/tcp_usrreq.c b/sys/netinet/tcp_usrreq.c index 74bf2925870f..0706e55f439b 100644 --- a/sys/netinet/tcp_usrreq.c +++ b/sys/netinet/tcp_usrreq.c @@ -2919,9 +2919,8 @@ db_print_tcpcb(struct tcpcb *tp, const char *name, int indent) tp->t_rxtcur, tp->t_maxseg, tp->t_srtt); db_print_indent(indent); - db_printf("t_rttvar: %d t_rxtshift: %d t_rttmin: %u " - "t_rttbest: %u\n", tp->t_rttvar, tp->t_rxtshift, tp->t_rttmin, - tp->t_rttbest); + db_printf("t_rttvar: %d t_rxtshift: %d t_rttmin: %u\n", + tp->t_rttvar, tp->t_rxtshift, tp->t_rttmin); db_print_indent(indent); db_printf("t_rttupdated: %lu max_sndwnd: %u t_softerror: %d\n", diff --git a/sys/netinet/tcp_var.h b/sys/netinet/tcp_var.h index 4a05f0f5ab2e..2474f3787903 100644 --- a/sys/netinet/tcp_var.h +++ b/sys/netinet/tcp_var.h @@ -212,8 +212,6 @@ struct tcpcb { int t_blackhole_exit; /* when to exit blackhole detection */ u_int t_rttmin; /* minimum rtt allowed */ - u_int t_rttbest; /* best rtt we've seen */ - int t_softerror; /* possible error not yet reported */ uint32_t max_sndwnd; /* largest window peer has offered */ /* Cache line 5 */