Paths

Table of Contentst

Differential D46669

tcp: remove TCP_OFFLOAD_DISABLE
ClosedPublic
Actions

Authored by tuexen on Sep 13 2024, 8:47 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, Nov 9, 9:14 PM

	Unknown Object (File)
	Sun, Oct 27, 5:42 PM

	Unknown Object (File)
	Sat, Oct 26, 4:33 AM

	Unknown Object (File)
	Tue, Oct 15, 8:22 PM

	Unknown Object (File)
	Tue, Oct 15, 8:21 PM

	Unknown Object (File)
	Tue, Oct 15, 8:21 PM

	Unknown Object (File)
	Tue, Oct 15, 8:21 PM

	Unknown Object (File)
	Oct 8 2024, 5:07 AM

View All 29 Files

Subscribers

Details

Reviewers

np
rrs
rscheff
peter.lei_ieee.org
lstewart

Group Reviewers

Commits

rG1c6bb4c5789c: tcp: remove TCP_OFFLOAD_DISABLE

Summary

TCP_OFFLOAD_DISABLE is nowhere else used or defined. So remove it.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

tuexen created this revision.Sep 13 2024, 8:47 PM

Herald added a reviewer: transport. · View Herald TranscriptSep 13 2024, 8:47 PM

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

tuexen requested review of this revision.Sep 13 2024, 8:47 PM

I'd be okay with removing the condition entirely. It is useful to have core data structures be a fixed size independent of configuration.

In D46669#1063601, @np wrote:

I'd be okay with removing the condition entirely. It is useful to have core data structures be a fixed size independent of configuration.

I'm not sure about the usefulness of having a fixed size. I'll bring that up on the fortnightly transport VC (agenda). The corresponding change would be a separate commit.

tuexen accepted this revision as: transport.Sep 16 2024, 10:52 AM

This revision is now accepted and ready to land.Sep 16 2024, 10:52 AM

Closed by commit rG1c6bb4c5789c: tcp: remove TCP_OFFLOAD_DISABLE (authored by tuexen). · Explain WhySep 16 2024, 10:57 AM

This revision was automatically updated to reflect the committed changes.

tuexen added a commit: rG1c6bb4c5789c: tcp: remove TCP_OFFLOAD_DISABLE.

Revision Contents
Changeset List

Path

Size

sys/

netinet/

2 lines

Diff 143321

sys/netinet/tcp_syncache.h

Loading...