Paths

Table of Contentst

Differential D52296

tcp: micro-optimize SYN-cookie expansion
ClosedPublic
Actions

Authored by tuexen on Sat, Aug 30, 10:32 PM.

Tags

None

Referenced Files

	F128643536: D52296.id161342.diff
	Fri, Sep 12, 10:28 PM

	Unknown Object (File)
	Fri, Sep 12, 2:16 AM

	Unknown Object (File)
	Thu, Sep 4, 8:20 AM

	Unknown Object (File)
	Tue, Sep 2, 4:25 PM

	Unknown Object (File)
	Tue, Sep 2, 3:36 PM

	Unknown Object (File)
	Tue, Sep 2, 1:13 PM

	Unknown Object (File)
	Mon, Sep 1, 10:24 PM

	Unknown Object (File)
	Mon, Sep 1, 10:28 AM

View All 11 Files

Subscribers

Details

Reviewers

rscheff
cc
peter.lei_ieee.org
nickbanks_netflix.com
lstewart
glebius

Group Reviewers

Commits

rG5d65c9eeb805: tcp: micro-optimize SYN-cookie expansion
rG341d1aabc13e: tcp: micro-optimize SYN-cookie expansion

Summary

Only compute wscale when it is actually used.
No functional change intended.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

tuexen created this revision.Sat, Aug 30, 10:32 PM

Herald added a reviewer: transport. · View Herald TranscriptSat, Aug 30, 10:32 PM

Herald added subscribers: glebius, melifaro, imp. · View Herald Transcript

tuexen requested review of this revision.Sat, Aug 30, 10:32 PM

rscheff accepted this revision.Mon, Sep 1, 6:42 AM

This revision is now accepted and ready to land.Mon, Sep 1, 6:42 AM

glebius added inline comments.Mon, Sep 1, 5:02 PM

sys/netinet/tcp_syncache.c
2327	I'd suggest to make it u_int. Negative right operand of << is undefined behavior. I know we don't got into negative here, but using unsigned operand would make it tidier.

Use u_int instead of int as the type for wscale as suggested by glebius@.

This revision now requires review to proceed.Mon, Sep 1, 8:45 PM

glebius accepted this revision.Mon, Sep 1, 10:13 PM

This revision is now accepted and ready to land.Mon, Sep 1, 10:13 PM

Closed by commit rG341d1aabc13e: tcp: micro-optimize SYN-cookie expansion (authored by tuexen). · Explain WhyMon, Sep 1, 10:24 PM

This revision was automatically updated to reflect the committed changes.

tuexen added a commit: rG341d1aabc13e: tcp: micro-optimize SYN-cookie expansion.

tuexen added a commit: rG5d65c9eeb805: tcp: micro-optimize SYN-cookie expansion.Mon, Sep 8, 10:02 PM

Revision Contents
Changeset List

Path

Size

sys/

netinet/

14 lines

Diff 161342

sys/netinet/tcp_syncache.c

Loading...