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)
	Sun, Oct 19, 3:36 PM

	Unknown Object (File)
	Wed, Oct 15, 6:48 PM

	Unknown Object (File)
	Wed, Oct 15, 6:48 PM

	Unknown Object (File)
	Wed, Oct 15, 6:48 PM

	Unknown Object (File)
	Wed, Oct 15, 8:22 AM

	Unknown Object (File)
	Mon, Oct 6, 7:25 AM

	Unknown Object (File)
	Tue, Sep 30, 5:03 AM

	Unknown Object (File)
	Wed, Sep 24, 7:28 AM

View All 57 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 Skipped

Unit

Tests Skipped

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 144056

sys/netinet/tcp_syncache.c

Loading...