Page MenuHomeFreeBSD
Differential D51260

bridge: refactor local packet handling
Needs ReviewPublic
Actions

Authored by ivy on Fri, Jul 11, 12:15 PM.

Details

Reviewers
des
kevans
Group Reviewers
network
Summary

Remove the GRAB_OUR_PACKETS() macro and sprinkle its remains over
bridge_input():

  • check much earlier if we should punt the packet to vlan(4), or if it's filtered by pfil, or if we need to learn the address.
  • actual local packet handling is moved to a new function, bridge_try_local().

While here, change handling of frames received from discarding (STP)
ports to drop the frames instead of passing them to the host IP stack.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 65357
Build 62240: arc lint + arc unit

Event Timeline

ivy created this revision.Fri, Jul 11, 12:15 PM
Herald added subscribers: glebius, melifaro, ae, imp. · View Herald TranscriptFri, Jul 11, 12:15 PM
ivy requested review of this revision.Fri, Jul 11, 12:15 PM
Harbormaster completed remote builds in B65355: Diff 158328.Fri, Jul 11, 12:15 PM
ivy added a parent revision: D51252: ifconfig: don't print bridge member ifmaxaddr if it's 0.Fri, Jul 11, 12:16 PM
ivy updated this revision to Diff 158331.Fri, Jul 11, 12:46 PM

no need to call bridge_rtupdate() in bridge_forward() now

Harbormaster completed remote builds in B65357: Diff 158331.Fri, Jul 11, 12:46 PM
ivy updated this revision to Diff 158347.Sat, Jul 12, 2:37 AM

improve code a little; add more comments; ensure the bif is set correctly for netmap-injected frames

Harbormaster completed remote builds in B65368: Diff 158347.Sat, Jul 12, 2:37 AM
ivy updated this revision to Diff 158348.Sat, Jul 12, 3:18 AM

change the order of input processing: do filtering, then monitoring, then span,
so monitor and span see the same frames and don't see filtered frames.

improve some comments.

fix the member_ifaddrs_enabled test (the bridge needs to be up to pass frames now).

Harbormaster completed remote builds in B65369: Diff 158348.Sat, Jul 12, 3:18 AM
ivy added a child revision: D51310: bridge: do not allow a bridge SVI in a bridge.Mon, Jul 14, 4:42 PM

Revision Contents
Changeset List

PathSize
sys/
net/
320 lines

Diff 158331

View Options

sys/net/if_bridge.c

Loading...