Page MenuHomeFreeBSD

blackhole(4): disable for locally originated TCP/UDP packets
ClosedPublic

Authored by glebius on Oct 28 2021, 10:37 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Nov 12, 1:12 AM
Unknown Object (File)
Tue, Nov 12, 12:40 AM
Unknown Object (File)
Mon, Oct 28, 12:27 AM
Unknown Object (File)
Tue, Oct 22, 11:15 AM
Unknown Object (File)
Tue, Oct 22, 7:01 AM
Unknown Object (File)
Sun, Oct 20, 2:51 AM
Unknown Object (File)
Mon, Oct 14, 9:52 PM
Unknown Object (File)
Oct 7 2024, 4:39 AM

Details

Summary

In most cases blackholing for locally originated packets is undesired,
leads to different kind of lags and delays. Provide sysctls to enforce
it, e.g. for debugging purposes.

Diff Detail

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

Event Timeline

rrs added inline comments.
sys/netinet/tcp_input.c
147

A boolean should have two values "true" and "false". Yet you use this
V_blackhole =1 or V_blackhole = 2

I think you need to change it back to an int since its used with multiple values.

sys/netinet/tcp_input.c
147

My boolean is V_blackhole_local. The V_blackhole remains int as it was.

Does it make sense to add some text to the blackhole man-page? Once this is in the tree, I can add the corresponding change for SCTP...

sys/netinet/tcp_input.c
147

He uses V_blackhole_local only as a boolean variable. This variable is different from V_blackhole.

rrs added a reviewer: rrs.
rrs added inline comments.
sys/netinet/tcp_input.c
147

ahh I see.. got confused by the variables :)

This revision is now accepted and ready to land.Nov 3 2021, 7:53 PM