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
F81915402: D33024.id98613.diff
Tue, Apr 23, 2:58 AM
Unknown Object (File)
Sat, Apr 20, 2:15 AM
Unknown Object (File)
Feb 22 2024, 6:30 PM
Unknown Object (File)
Feb 22 2024, 6:30 PM
Unknown Object (File)
Feb 22 2024, 6:30 PM
Unknown Object (File)
Feb 22 2024, 6:30 PM
Unknown Object (File)
Feb 22 2024, 12:50 AM
Unknown Object (File)
Jan 11 2024, 11:30 PM
View All 20 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 42882
Build 39770: 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 98729

View Options

sys/netinet/tcp_hpts.h

Loading...
View Options

sys/netinet/tcp_hpts.c

Loading...