Page MenuHomeFreeBSD
Differential D33024

tcp_hpts: make struct tcp_hpts_entry private to the module.
ClosedPublic
Actions

Authored by glebius on Nov 16 2021, 9:39 PM.
Tags
None
Referenced Files
F133013874: D33024.id98729.diff
Wed, Oct 22, 1:58 AM
F132991743: D33024.diff
Tue, Oct 21, 9:27 PM
Unknown Object (File)
Mon, Oct 20, 3:09 AM
Unknown Object (File)
Fri, Oct 17, 10:11 AM
Unknown Object (File)
Thu, Oct 16, 2:07 PM
Unknown Object (File)
Sep 19 2025, 5:36 AM
Unknown Object (File)
Sep 9 2025, 7:25 AM
Unknown Object (File)
Sep 7 2025, 6:18 AM
View All Files
Subscribers
imp
melifaro
rrs

Details

Reviewers
rrs
Group Reviewers
transport
Commits
rGb0a7c008cbc6: tcp_hpts: make struct tcp_hpts_entry private to the module.
Summary

Also, make some of the functions also private to the module. Remove
unused functions discovered after that.

Diff Detail

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

Event Timeline

glebius created this revision.Nov 16 2021, 9:39 PM
Herald added a reviewer: transport. · View Herald TranscriptNov 16 2021, 9:39 PM
Herald added subscribers: melifaro, imp. · View Herald Transcript
glebius requested review of this revision.Nov 16 2021, 9:39 PM
glebius added a parent revision: D33023: tcp_hpts: provide tcp_in_hpts()..
Harbormaster completed remote builds in B42824: Diff 98613.Nov 16 2021, 9:39 PM
glebius added a child revision: D33025: tcp_hpts: rename input queue to drop queue and trim dead code.Nov 16 2021, 9:39 PM
rrs accepted this revision.Nov 17 2021, 7:25 PM
rrs added a subscriber: rrs.

I am usually not a big fan of this type of actions. I like
structures to be in a header file instead of buried in the
.c file.. but if you really think its a good idea.. i guess its ok.. my
feelings are not that strong about it.

This revision is now accepted and ready to land.Nov 17 2021, 7:25 PM
glebius updated this revision to Diff 98729.Nov 19 2021, 4:39 AM

Rebased on newer main.

This revision now requires review to proceed.Nov 19 2021, 4:39 AM
Harbormaster completed remote builds in B42882: Diff 98729.Nov 19 2021, 4:39 AM
This revision was not accepted when it landed; it landed in state Needs Review.Dec 2 2021, 6:52 PM
Closed by commit rGb0a7c008cbc6: tcp_hpts: make struct tcp_hpts_entry private to the module. (authored by glebius). · Explain Why
This revision was automatically updated to reflect the committed changes.
glebius added a commit: rGb0a7c008cbc6: tcp_hpts: make struct tcp_hpts_entry private to the module..

Revision Contents
Changeset List

PathSize
sys/
netinet/
105 lines
151 lines

Diff 98613

View Options

sys/netinet/tcp_hpts.h

Loading...
View Options

sys/netinet/tcp_hpts.c

Loading...