Page MenuHomeFreeBSD
Differential D38822

netinet: Remove the IP(V6)_RSS_LISTEN_BUCKET socket option
ClosedPublic
Actions

Authored by markj on Feb 28 2023, 2:03 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Jul 8, 7:58 PM
Unknown Object (File)
Thu, Jul 2, 12:40 AM
Unknown Object (File)
Tue, Jun 30, 11:05 PM
Unknown Object (File)
Fri, Jun 26, 10:57 PM
Unknown Object (File)
May 10 2026, 1:08 AM
Unknown Object (File)
May 10 2026, 1:04 AM
Unknown Object (File)
May 6 2026, 8:14 AM
Unknown Object (File)
May 5 2026, 4:50 PM
View All Files
Subscribers
ae
imp
melifaro
mjg

Details

Reviewers
glebius
mjg
Group Reviewers
network
Commits
rG317fa5169d35: netinet: Remove the IP(V6)_RSS_LISTEN_BUCKET socket option
Summary

It has no effect, and an exp-run revealed that it is not in use.

No functional change intended.

PR: 261398 (exp-run)

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

markj created this revision.Feb 28 2023, 2:03 PM
Herald added subscribers: melifaro, ae, imp. · View Herald TranscriptFeb 28 2023, 2:03 PM
markj requested review of this revision.Feb 28 2023, 2:03 PM
Harbormaster completed remote builds in B50062: Diff 118076.Feb 28 2023, 2:03 PM
markj added a parent revision: D38821: librss: Remove rss_socket_set_rss_bucket().Feb 28 2023, 2:03 PM
mjg added a subscriber: mjg.Feb 28 2023, 2:08 PM
mjg added inline comments.
sys/netinet/in_pcb.h
672

unused opts are denoted with an explicit comment, so this should be

/*                              0x00000080 */

something to that extent should probably be also slapped in the IP_RSS_LISTEN_BUCKET case

mjg added inline comments.Feb 28 2023, 2:10 PM
sys/netinet/in_pcb.h
672

same with IPV6_RSS_LISTEN_BUCKET even though that one already has unannotated gaps

markj updated this revision to Diff 118078.Feb 28 2023, 2:15 PM
markj marked an inline comment as done.

Add comments for removed socket options and inpcb flags.

Harbormaster completed remote builds in B50063: Diff 118078.Feb 28 2023, 2:15 PM
markj marked an inline comment as done.Feb 28 2023, 2:15 PM
mjg accepted this revision.Feb 28 2023, 2:21 PM
This revision is now accepted and ready to land.Feb 28 2023, 2:21 PM
markj updated this revision to Diff 118079.Feb 28 2023, 2:28 PM

Remove a case that I missed.

This revision now requires review to proceed.Feb 28 2023, 2:28 PM
Harbormaster completed remote builds in B50064: Diff 118079.Feb 28 2023, 2:28 PM
glebius accepted this revision as: glebius.Feb 28 2023, 4:14 PM
This revision is now accepted and ready to land.Feb 28 2023, 4:14 PM
Closed by commit rG317fa5169d35: netinet: Remove the IP(V6)_RSS_LISTEN_BUCKET socket option (authored by markj). · Explain WhyFeb 28 2023, 9:06 PM
This revision was automatically updated to reflect the committed changes.
markj added a commit: rG317fa5169d35: netinet: Remove the IP(V6)_RSS_LISTEN_BUCKET socket option.

Revision Contents
Changeset List

PathSize
sys/
netinet/
3 lines
6 lines
1 line
14 lines
netinet6/
4 lines
16 lines

Diff 118099

View Options

sys/netinet/in.h

Loading...
View Options

sys/netinet/in_pcb.h

Loading...
View Options

sys/netinet/in_pcb.c

Loading...
View Options

sys/netinet/ip_output.c

Loading...
View Options

sys/netinet6/in6.h

Loading...
View Options

sys/netinet6/ip6_output.c

Loading...