Page MenuHomeFreeBSD
Differential D41998

tcp: Eliminate one usage of TUNABLE_INT_FETCH
ClosedPublic
Actions

Authored by zlei on Sep 27 2023, 10:26 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, May 15, 8:01 PM
Unknown Object (File)
Fri, May 15, 8:00 PM
Unknown Object (File)
Fri, May 15, 7:54 PM
Unknown Object (File)
Wed, May 6, 2:24 AM
Unknown Object (File)
Thu, Apr 30, 5:42 PM
Unknown Object (File)
Wed, Apr 29, 10:53 AM
Unknown Object (File)
Apr 28 2026, 12:38 AM
Unknown Object (File)
Apr 25 2026, 7:38 PM
View All Files
Subscribers
cc
imp
melifaro

Details

Reviewers
glebius
tuexen
cc
rscheff
Group Reviewers
transport
Commits
rG0a31dbecd43d: tcp: Simplify the initialization of loader tunable 'net.inet.tcp.tcbhashsize'
rG379f41a117eb: tcp: Simplify the initialization of loader tunable 'net.inet.tcp.tcbhashsize'
rG3a97686fc11a: tcp: Simplify the initialization of loader tunable 'net.inet.tcp.tcbhashsize'
rG38ecc80b2a4e: tcp: Simplify the initialization of loader tunable 'net.inet.tcp.tcbhashsize'
Summary

The code will be simpler.

No functional change intended.

MFC after: 1 week

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

zlei created this revision.Sep 27 2023, 10:26 AM
Herald added 1 blocking reviewer(s): transport. · View Herald TranscriptSep 27 2023, 10:26 AM
Herald added subscribers: melifaro, imp. · View Herald Transcript
zlei requested review of this revision.Sep 27 2023, 10:26 AM
zlei added inline comments.
sys/netinet/tcp_subr.c
318

@glebius Does it need CTLFLAG_NEEDGIANT ?

zlei added a reviewer: tuexen.Oct 2 2023, 12:01 AM
cc accepted this revision as: cc.Oct 2 2023, 9:25 PM
cc added a subscriber: cc.

My understanding too here is that CTLFLAG_NOFETCH is not used, and so is CTLFLAG_NEEDGIANT not needed.

rscheff accepted this revision.Oct 3 2023, 7:14 AM
This revision is now accepted and ready to land.Oct 3 2023, 7:14 AM
Closed by commit rG38ecc80b2a4e: tcp: Simplify the initialization of loader tunable 'net.inet.tcp.tcbhashsize' (authored by zlei). · Explain WhyOct 8 2023, 10:06 AM
This revision was automatically updated to reflect the committed changes.
zlei added a commit: rG38ecc80b2a4e: tcp: Simplify the initialization of loader tunable 'net.inet.tcp.tcbhashsize'.
zlei added a commit: rG3a97686fc11a: tcp: Simplify the initialization of loader tunable 'net.inet.tcp.tcbhashsize'.Oct 16 2023, 3:18 PM
zlei added a commit: rG379f41a117eb: tcp: Simplify the initialization of loader tunable 'net.inet.tcp.tcbhashsize'.Oct 16 2023, 3:44 PM
zlei added a commit: rG0a31dbecd43d: tcp: Simplify the initialization of loader tunable 'net.inet.tcp.tcbhashsize'.Oct 16 2023, 3:51 PM

Revision Contents
Changeset List

PathSize
sys/
netinet/
30 lines

Diff 128419

View Options

sys/netinet/tcp_subr.c

Loading...