net/tstun: remove buffer on outbound channel

The buffer is not necessary, I was unable to observe any optimization,
and having an unbuffered channel offers potential scheduler
optimization.

Signed-off-by: James Tucker <james@tailscale.com>
This commit is contained in:
James Tucker 2022-08-25 10:34:25 -07:00
parent 58f35261d0
commit 0c34fc7b5b
No known key found for this signature in database

View File

@ -202,8 +202,7 @@ func wrap(logf logger.Logf, tdev tun.Device, isTAP bool) *Wrapper {
// a goroutine should not block when setting it, even with no listeners.
bufferConsumed: make(chan struct{}, 1),
closed: make(chan struct{}),
// outbound can be unbuffered; the buffer is an optimization.
outbound: make(chan tunReadResult, 1),
outbound: make(chan tunReadResult),
eventsUpDown: make(chan tun.Event),
eventsOther: make(chan tun.Event),
// TODO(dmytro): (highly rate-limited) hexdumps should happen on unknown packets.