Paths

Table of Contentst

Differential D46850

tcp: small cleanup
ClosedPublic
Actions

Authored by tuexen on Sep 30 2024, 6:56 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, Apr 22, 11:10 AM

	Unknown Object (File)
	Tue, Apr 21, 10:07 PM

	Unknown Object (File)
	Sun, Apr 19, 3:53 PM

	Unknown Object (File)
	Sat, Apr 11, 1:03 PM

	Unknown Object (File)
	Fri, Apr 10, 12:13 PM

	Unknown Object (File)
	Fri, Apr 10, 1:55 AM

	Unknown Object (File)
	Wed, Apr 8, 5:36 PM

	Unknown Object (File)
	Wed, Apr 8, 10:04 AM

View All 96 Files

Subscribers

Details

Reviewers

markj
rscheff
cc
peter.lei_ieee.org
lstewart
glebius

Group Reviewers

Commits

rG5818c74e34a8: tcp: small cleanup
rGb785f83e987d: tcp: small cleanup
rG2eacb0841c7d: tcp: small cleanup

Summary

No functional change intended.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

tuexen created this revision.Sep 30 2024, 6:56 PM

Herald added a reviewer: transport. · View Herald TranscriptSep 30 2024, 6:56 PM

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

tuexen requested review of this revision.Sep 30 2024, 6:56 PM

glebius accepted this revision.Sep 30 2024, 7:00 PM

glebius added inline comments.

sys/netinet/tcp_syncache.c
1588	No objection on this change, but I personally preferred the original style.

tuexen marked an inline comment as done.Sep 30 2024, 7:14 PM

tuexen added inline comments.

sys/netinet/tcp_syncache.c
1588	Me too. But then the line is too long. The alternative would be: if ((sc = uma_zalloc(V_tcp_syncache.zone, M_NOWAIT \| M_ZERO)) == NULL) { I think the way proposed is simpler to read.

tuexen marked an inline comment as done.Sep 30 2024, 7:38 PM

markj accepted this revision.Oct 1 2024, 7:42 AM

rscheff accepted this revision.Oct 1 2024, 7:51 AM

This revision is now accepted and ready to land.Oct 1 2024, 7:51 AM

Looks good to me. Thanks for removing the goto label skip_alloc that improves reading.

Closed by commit rG2eacb0841c7d: tcp: small cleanup (authored by tuexen). · Explain WhyOct 1 2024, 3:34 PM

This revision was automatically updated to reflect the committed changes.

tuexen added a commit: rG2eacb0841c7d: tcp: small cleanup.

tuexen added a commit: rGb785f83e987d: tcp: small cleanup.Oct 31 2024, 1:59 PM

tuexen added a commit: rG5818c74e34a8: tcp: small cleanup.Oct 31 2024, 4:59 PM

Revision Contents
Changeset List

Path

Size

sys/

netinet/

59 lines

Diff 144095

sys/netinet/tcp_syncache.c

Loading...