Page MenuHomeFreeBSD
Differential D3699

Eliminate nd6_nud_hint() and its TCP bindings.
ClosedPublic
Actions

Authored by melifaro on Sep 20 2015, 3:14 PM.
Tags
None
Referenced Files
F112757634: D3699.id8951.diff
Sat, Mar 22, 9:49 AM
Unknown Object (File)
Mon, Mar 17, 9:38 AM
Unknown Object (File)
Thu, Feb 27, 8:26 PM
Unknown Object (File)
Thu, Feb 20, 5:59 PM
Unknown Object (File)
Thu, Feb 20, 5:33 PM
Unknown Object (File)
Feb 4 2025, 4:19 PM
Unknown Object (File)
Nov 29 2024, 9:10 PM
Unknown Object (File)
Nov 21 2024, 5:25 PM
View All Files
Subscribers
imp

Details

Reviewers
hrs
gnn
Group Reviewers
network
Commits
rS288301: Eliminate nd6_nud_hint() and its TCP bindings.
Summary

Initially function was introduced in r53541 (KAME initial commit) to "provide hints from upper layer protocols that indicate a connection is making "forward progress"" (quote from RFC 2461 7.3.1 Reachability Confirmation).
However, it was converted to do nothing (e.g. just return) in r122922 (tcp_hostcache implementation) back in 2003. Some defines were moved to tcp_var.h in r169541. Then, it was broken (for non-corner cases) by r186119 (L2<>L3 split) in 2008 (NULL ifp in nd6_lookup). So, right now this code is broken and has no "real" base users.

Even if ability to provide reachability hint from upper-layer protocol like TCP might sound as a good idea, it is not trivial to do efficient implementation. Personally I don't think it is needed ATM - if one really needs to reduce number of ND confirmations, "basereachable" variable can be increased to sufficient value. Given that, I'd like to remove the part of code in question.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 617
Build 617: arc lint + arc unit

Event Timeline

melifaro updated this revision to Diff 8849.Sep 20 2015, 3:14 PM
melifaro retitled this revision from to Eliminate nd6_nud_hint() and its TCP bindings..
melifaro updated this object.
melifaro edited the test plan for this revision. (Show Details)
melifaro added a reviewer: network.
Herald added a subscriber: imp. · View Herald TranscriptSep 20 2015, 3:14 PM
hrs accepted this revision.Sep 20 2015, 5:45 PM
hrs added a reviewer: hrs.
This revision is now accepted and ready to land.Sep 20 2015, 5:45 PM
gnn accepted this revision.Sep 22 2015, 8:38 PM
gnn added a reviewer: gnn.
melifaro updated this revision to Diff 8950.Sep 27 2015, 5:12 AM
melifaro edited edge metadata.

Do pre-commit sync.

This revision now requires review to proceed.Sep 27 2015, 5:12 AM
melifaro updated this revision to Diff 8951.Sep 27 2015, 5:18 AM
melifaro edited edge metadata.

Remove non-relevant __noinline definitions.

Closed by commit rS288301: Eliminate nd6_nud_hint() and its TCP bindings. (authored by melifaro). · Explain WhySep 27 2015, 5:29 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
sys/
net/
1 line
1 line
netinet/
15 lines
1 line
12 lines
netinet6/
1 line
43 lines
12 lines

Diff 8950

View Options

sys/net/if_llatbl.h

  • This file uses an unknown character encoding.
# Created by: Anton Törnqvist <antont@inbox.lv> # Created by: Anton Tornqvist <antont@inbox.lv>
# $FreeBSD$ # $FreeBSD$
PORTNAME= sxhkd PORTNAME= sxhkd
PORTVERSION= 0.5.7 PORTVERSION= 0.5.8
CATEGORIES= x11 CATEGORIES= x11
MAINTAINER= gor@clogic.com.ua MAINTAINER= gor@clogic.com.ua
COMMENT= Simple X hotkey daemon COMMENT= Simple X hotkey daemon
LICENSE= BSD2CLAUSE LICENSE= BSD2CLAUSE
LICENSE_FILE= ${WRKSRC}/LICENSE LICENSE_FILE= ${WRKSRC}/LICENSE
LIB_DEPENDS= libxcb-keysyms.so:x11/xcb-util-keysyms LIB_DEPENDS= libxcb-keysyms.so:x11/xcb-util-keysyms
USES= gmake pkgconfig
USE_XORG= xcb
matUnsubmitted
Not Done
Inline Actions

Why the += ?

mat: Why the += ?
USE_GITHUB= yes USE_GITHUB= yes
GH_ACCOUNT= baskerville GH_ACCOUNT= baskerville
USES= gmake pkgconfig
PORTEXAMPLES= * PORTEXAMPLES= *
PLIST_FILES= bin/sxhkd \ PLIST_FILES= bin/sxhkd \
man/man1/sxhkd.1.gz man/man1/sxhkd.1.gz
OPTIONS_DEFINE= EXAMPLES OPTIONS_DEFINE= EXAMPLES
.include <bsd.port.mk> .include <bsd.port.mk>
View Options

sys/net/if_llatbl.c

  • This file uses an unknown character encoding.
# Created by: Anton Törnqvist <antont@inbox.lv> # Created by: Anton Tornqvist <antont@inbox.lv>
# $FreeBSD$ # $FreeBSD$
PORTNAME= sxhkd PORTNAME= sxhkd
PORTVERSION= 0.5.7 PORTVERSION= 0.5.8
CATEGORIES= x11 CATEGORIES= x11
MAINTAINER= gor@clogic.com.ua MAINTAINER= gor@clogic.com.ua
COMMENT= Simple X hotkey daemon COMMENT= Simple X hotkey daemon
LICENSE= BSD2CLAUSE LICENSE= BSD2CLAUSE
LICENSE_FILE= ${WRKSRC}/LICENSE LICENSE_FILE= ${WRKSRC}/LICENSE
LIB_DEPENDS= libxcb-keysyms.so:x11/xcb-util-keysyms LIB_DEPENDS= libxcb-keysyms.so:x11/xcb-util-keysyms
USES= gmake pkgconfig
USE_XORG= xcb
matUnsubmitted
Not Done
Inline Actions

Why the += ?

mat: Why the += ?
USE_GITHUB= yes USE_GITHUB= yes
GH_ACCOUNT= baskerville GH_ACCOUNT= baskerville
USES= gmake pkgconfig
PORTEXAMPLES= * PORTEXAMPLES= *
PLIST_FILES= bin/sxhkd \ PLIST_FILES= bin/sxhkd \
man/man1/sxhkd.1.gz man/man1/sxhkd.1.gz
OPTIONS_DEFINE= EXAMPLES OPTIONS_DEFINE= EXAMPLES
.include <bsd.port.mk> .include <bsd.port.mk>
View Options

sys/netinet/tcp_input.c

  • This file uses an unknown character encoding.
# Created by: Anton Törnqvist <antont@inbox.lv> # Created by: Anton Tornqvist <antont@inbox.lv>
# $FreeBSD$ # $FreeBSD$
PORTNAME= sxhkd PORTNAME= sxhkd
PORTVERSION= 0.5.7 PORTVERSION= 0.5.8
CATEGORIES= x11 CATEGORIES= x11
MAINTAINER= gor@clogic.com.ua MAINTAINER= gor@clogic.com.ua
COMMENT= Simple X hotkey daemon COMMENT= Simple X hotkey daemon
LICENSE= BSD2CLAUSE LICENSE= BSD2CLAUSE
LICENSE_FILE= ${WRKSRC}/LICENSE LICENSE_FILE= ${WRKSRC}/LICENSE
LIB_DEPENDS= libxcb-keysyms.so:x11/xcb-util-keysyms LIB_DEPENDS= libxcb-keysyms.so:x11/xcb-util-keysyms
USES= gmake pkgconfig
USE_XORG= xcb
matUnsubmitted
Not Done
Inline Actions

Why the += ?

mat: Why the += ?
USE_GITHUB= yes USE_GITHUB= yes
GH_ACCOUNT= baskerville GH_ACCOUNT= baskerville
USES= gmake pkgconfig
PORTEXAMPLES= * PORTEXAMPLES= *
PLIST_FILES= bin/sxhkd \ PLIST_FILES= bin/sxhkd \
man/man1/sxhkd.1.gz man/man1/sxhkd.1.gz
OPTIONS_DEFINE= EXAMPLES OPTIONS_DEFINE= EXAMPLES
.include <bsd.port.mk> .include <bsd.port.mk>
View Options

sys/netinet/tcp_reass.c

  • This file uses an unknown character encoding.
# Created by: Anton Törnqvist <antont@inbox.lv> # Created by: Anton Tornqvist <antont@inbox.lv>
# $FreeBSD$ # $FreeBSD$
PORTNAME= sxhkd PORTNAME= sxhkd
PORTVERSION= 0.5.7 PORTVERSION= 0.5.8
CATEGORIES= x11 CATEGORIES= x11
MAINTAINER= gor@clogic.com.ua MAINTAINER= gor@clogic.com.ua
COMMENT= Simple X hotkey daemon COMMENT= Simple X hotkey daemon
LICENSE= BSD2CLAUSE LICENSE= BSD2CLAUSE
LICENSE_FILE= ${WRKSRC}/LICENSE LICENSE_FILE= ${WRKSRC}/LICENSE
LIB_DEPENDS= libxcb-keysyms.so:x11/xcb-util-keysyms LIB_DEPENDS= libxcb-keysyms.so:x11/xcb-util-keysyms
USES= gmake pkgconfig
USE_XORG= xcb
matUnsubmitted
Not Done
Inline Actions

Why the += ?

mat: Why the += ?
USE_GITHUB= yes USE_GITHUB= yes
GH_ACCOUNT= baskerville GH_ACCOUNT= baskerville
USES= gmake pkgconfig
PORTEXAMPLES= * PORTEXAMPLES= *
PLIST_FILES= bin/sxhkd \ PLIST_FILES= bin/sxhkd \
man/man1/sxhkd.1.gz man/man1/sxhkd.1.gz
OPTIONS_DEFINE= EXAMPLES OPTIONS_DEFINE= EXAMPLES
.include <bsd.port.mk> .include <bsd.port.mk>
View Options

sys/netinet/tcp_var.h

Loading...
View Options

sys/netinet6/nd6.h

Loading...
View Options

sys/netinet6/nd6.c

Loading...
View Options

sys/netinet6/nd6_nbr.c

Loading...