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
F109376446: D41998.diff
Tue, Feb 4, 6:31 AM
F109355250: D41998.id127855.diff
Mon, Feb 3, 11:40 PM
Unknown Object (File)
Mon, Feb 3, 6:10 AM
Unknown Object (File)
Sun, Jan 26, 7:21 PM
Unknown Object (File)
Sat, Jan 25, 2:03 PM
Unknown Object (File)
Nov 30 2024, 6:35 PM
Unknown Object (File)
Nov 25 2024, 5:39 AM
Unknown Object (File)
Nov 23 2024, 1:37 AM
View All 38 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...