Page MenuHomeFreeBSD
Differential D31584

gre: simplify RSS ifdefs
ClosedPublic
Actions

Authored by franco_opnsense.org on Aug 17 2021, 9:43 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Jan 24, 5:13 PM
Unknown Object (File)
Sat, Jan 18, 5:22 PM
Unknown Object (File)
Oct 23 2024, 9:38 PM
Unknown Object (File)
Oct 22 2024, 6:06 PM
Unknown Object (File)
Sep 28 2024, 3:33 PM
Unknown Object (File)
Sep 16 2024, 11:34 PM
Unknown Object (File)
Sep 15 2024, 12:25 PM
Unknown Object (File)
Sep 15 2024, 12:25 PM
View All 62 Files
Subscribers
ae
imp
melifaro
Contributor Reviews (src)

Details

Reviewers
kbowling
melifaro
Commits
rG5e2076065c8b: gre: simplify RSS ifdefs
rG412a609da482: gre: simplify RSS ifdefs
rGbb250fae9e9e: gre: simplify RSS ifdefs
Summary

Use the early break to avoid else definitions. When RSS gains a
runtime option previous constructs would duplicate and convolute
the existing code.

While here init flowid and skip magic numbers and late default
assignment.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 41096
Build 37985: arc lint + arc unit

Event Timeline

franco_opnsense.org created this revision.Aug 17 2021, 9:43 AM
Herald added subscribers: Contributor Reviews (src), melifaro, ae, imp. · View Herald TranscriptAug 17 2021, 9:43 AM
franco_opnsense.org requested review of this revision.Aug 17 2021, 9:43 AM
Harbormaster completed remote builds in B41096: Diff 93818.Aug 17 2021, 9:43 AM
melifaro added inline comments.Aug 17 2021, 11:20 AM
sys/net/if_gre.c
658

this one always wins

franco_opnsense.org added inline comments.Aug 17 2021, 11:48 AM
sys/net/if_gre.c
658

no, RSS inserts an early break statement above

kbowling accepted this revision.Aug 18 2021, 7:24 AM
This revision is now accepted and ready to land.Aug 18 2021, 7:24 AM
melifaro accepted this revision.Aug 18 2021, 7:56 AM
Closed by commit rGbb250fae9e9e: gre: simplify RSS ifdefs (authored by franco_opnsense.org, committed by Kevin Bowling <kbowling@FreeBSD.org>). · Explain WhyAug 18 2021, 5:09 PM
This revision was automatically updated to reflect the committed changes.
Kevin Bowling <kbowling@FreeBSD.org> added a commit: rGbb250fae9e9e: gre: simplify RSS ifdefs.
Kevin Bowling <kbowling@FreeBSD.org> added a commit: rG412a609da482: gre: simplify RSS ifdefs.Aug 25 2021, 11:53 PM
Kevin Bowling <kbowling@FreeBSD.org> added a commit: rG5e2076065c8b: gre: simplify RSS ifdefs.Aug 26 2021, 12:04 AM

Revision Contents
Changeset List

PathSize
sys/
net/
35 lines

Diff 93818

View Options

sys/net/if_gre.c

Loading...