Page MenuHomeFreeBSD
Differential D33368

rack: drop connection synchronously, when we can
ClosedPublic
Actions

Authored by glebius on Dec 10 2021, 2:47 AM.
Tags
None
Referenced Files
F176309973: D33368.id100589.diff
Sat, Oct 17, 8:03 PM
Unknown Object (File)
Fri, Oct 16, 1:47 PM
Unknown Object (File)
Wed, Oct 14, 4:59 PM
Unknown Object (File)
Wed, Oct 14, 2:27 PM
Unknown Object (File)
Fri, Oct 2, 1:46 PM
Unknown Object (File)
Thu, Oct 1, 7:23 AM
Unknown Object (File)
Tue, Sep 22, 10:30 AM
Unknown Object (File)
Sep 16 2026, 2:42 AM
View All Files
Subscribers
imp
melifaro
tuexen

Details

Reviewers
tuexen
Group Reviewers
transport
Commits
rG66aeb0b53b39: rack: drop connection synchronously, when we can
Summary

For all functions that are leaves of tcp_input() call
ctf_do_dropwithreset_conn() instead of ctf_do_dropwithreset(), cause
we always got tp and we want it to be dropped.

Diff Detail

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

Event Timeline

glebius created this revision.Dec 10 2021, 2:47 AM
Herald added 1 blocking reviewer(s): transport. · View Herald TranscriptDec 10 2021, 2:47 AM
Herald added subscribers: melifaro, imp. · View Herald Transcript
glebius requested review of this revision.Dec 10 2021, 2:47 AM
Harbormaster completed remote builds in B43274: Diff 99809.Dec 10 2021, 2:47 AM
glebius added a parent revision: D33367: bbr: drop packet synchronously in ctf_do_dropwithreset_conn().Dec 10 2021, 2:47 AM
glebius added a child revision: D33369: rack: rack_do_compressed_ack_processing() can call tcp_drop().
tuexen accepted this revision.Dec 10 2021, 12:04 PM
tuexen added a subscriber: tuexen.

You might want to check what rrs@ thinks...

This revision is now accepted and ready to land.Dec 10 2021, 12:04 PM
glebius updated this revision to Diff 99950.Dec 13 2021, 6:14 PM

Rebase, update descr.

This revision now requires review to proceed.Dec 13 2021, 6:14 PM
Harbormaster completed remote builds in B43317: Diff 99950.Dec 13 2021, 6:14 PM
This revision was not accepted when it landed; it landed in state Needs Review.Dec 26 2021, 4:50 PM
Closed by commit rG66aeb0b53b39: rack: drop connection synchronously, when we can (authored by glebius). · Explain Why
This revision was automatically updated to reflect the committed changes.
glebius added a commit: rG66aeb0b53b39: rack: drop connection synchronously, when we can.

Revision Contents
Changeset List

PathSize
sys/
netinet/
tcp_stacks/
18 lines

Diff 99809

View Options

sys/netinet/tcp_stacks/rack.c

Loading...