Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 4 Aug 2023 17:31:12 GMT
From:      Ed Maste <emaste@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Subject:   git: 348bea10b6f2 - main - openssh: retire HPN option handling
Message-ID:  <202308041731.374HVCxU039889@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch main has been updated by emaste:

URL: https://cgit.FreeBSD.org/src/commit/?id=348bea10b6f2b86f757011652ea798ccb3f696c1

commit 348bea10b6f2b86f757011652ea798ccb3f696c1
Author:     Ed Maste <emaste@FreeBSD.org>
AuthorDate: 2023-08-02 14:37:12 +0000
Commit:     Ed Maste <emaste@FreeBSD.org>
CommitDate: 2023-08-04 17:29:04 +0000

    openssh: retire HPN option handling
    
    The HPN patch set was removed from base system SSH in January 2016, in
    commit 60c59fad8806.  We retained the option parsing (using OpenSSH's
    support for deprecated options) to avoid breaking existing installations
    upon upgrade, but sufficient time has now passed that we can remove this
    special case.
    
    Approved by:    des
    Relnotes:       Yes
    Sponsored by:   The FreeBSD Foundation
    Differential Revision: https://reviews.freebsd.org/D41291
---
 crypto/openssh/FREEBSD-upgrade | 6 ++----
 crypto/openssh/readconf.c      | 7 -------
 crypto/openssh/servconf.c      | 6 ------
 3 files changed, 2 insertions(+), 17 deletions(-)

diff --git a/crypto/openssh/FREEBSD-upgrade b/crypto/openssh/FREEBSD-upgrade
index 905b65ec4e3d..8d3f3a14f823 100644
--- a/crypto/openssh/FREEBSD-upgrade
+++ b/crypto/openssh/FREEBSD-upgrade
@@ -182,10 +182,8 @@
 
 11) Retired patches
 
-   We no longer have HPN patches (adaptive buffer size for increased
-   throughput on high-BxD links) or client-side VersionAddendum, but we
-   recognize and ignore the configuration options to avoid breaking
-   existing configurations.
+   We no longer have client-side VersionAddendum, but we recognize and ignore
+   the configuration options to avoid breaking existing configurations.
 
 12) PrintLastLog bugfix
 
diff --git a/crypto/openssh/readconf.c b/crypto/openssh/readconf.c
index 616a6e8b9bd4..3798c2507717 100644
--- a/crypto/openssh/readconf.c
+++ b/crypto/openssh/readconf.c
@@ -323,13 +323,6 @@ static struct {
 	{ "requiredrsasize", oRequiredRSASize },
 	{ "enableescapecommandline", oEnableEscapeCommandline },
 
-	/* HPN patch - retired in 60c59fad8806 */
-	{ "hpndisabled", oDeprecated },
-	{ "hpnbuffersize", oDeprecated },
-	{ "tcprcvbufpoll", oDeprecated },
-	{ "tcprcvbuf", oDeprecated },
-	{ "noneenabled", oUnsupported },
-	{ "noneswitch", oUnsupported },
 	/* Client VersionAddendum - retired in bffe60ead024 */
 	{ "versionaddendum", oDeprecated },
 
diff --git a/crypto/openssh/servconf.c b/crypto/openssh/servconf.c
index de45475dbc33..2f9632992751 100644
--- a/crypto/openssh/servconf.c
+++ b/crypto/openssh/servconf.c
@@ -708,12 +708,6 @@ static struct {
 	{ "useblacklist", sUseBlacklist, SSHCFG_GLOBAL },
 	{ "useblocklist", sUseBlacklist, SSHCFG_GLOBAL }, /* alias */
 
-	/* HPN patch - retired in 60c59fad8806 */
-	{ "noneenabled", sUnsupported, SSHCFG_ALL },
-	{ "hpndisabled", sDeprecated, SSHCFG_ALL },
-	{ "hpnbuffersize", sDeprecated, SSHCFG_ALL },
-	{ "tcprcvbufpoll", sDeprecated, SSHCFG_ALL },
-
 	{ NULL, sBadOption, 0 }
 };
 



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202308041731.374HVCxU039889>