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)
Sat, Jan 18, 11:01 PM
Unknown Object (File)
Sat, Jan 11, 12:10 AM
Unknown Object (File)
Dec 8 2024, 1:08 AM
Unknown Object (File)
Nov 23 2024, 10:22 PM
Unknown Object (File)
Nov 12 2024, 1:12 AM
Unknown Object (File)
Nov 12 2024, 12:40 AM
Unknown Object (File)
Oct 28 2024, 12:27 AM
Unknown Object (File)
Oct 22 2024, 11:15 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