Page MenuHomeFreeBSD
Differential D34824

if_bnxt: Allow bnxt interfaces to use vlans with promisc mode
ClosedPublic
Actions

Authored by thj on Apr 7 2022, 4:35 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Aug 23, 2:08 PM
Unknown Object (File)
Fri, Aug 21, 8:25 PM
Unknown Object (File)
Thu, Aug 13, 11:48 AM
Unknown Object (File)
Wed, Aug 12, 10:42 PM
Unknown Object (File)
Wed, Aug 12, 10:38 PM
Unknown Object (File)
Wed, Aug 12, 9:36 PM
Unknown Object (File)
Wed, Aug 12, 6:37 PM
Unknown Object (File)
Tue, Aug 11, 4:29 PM
View All Files
Subscribers
imp
pi

Details

Reviewers
shurd
allanjude
rpokala
mav
Group Reviewers
network
Klara
Commits
rG5acfa48bb258: if_bnxt: Allow bnxt interfaces to use vlans
rGe54505277c39: if_bnxt: Allow bnxt interfaces to use vlans
rG0c6a2fa33e36: if_bnxt: Allow bnxt interfaces to use vlans
Summary
  • Add patch 203393 from Stephen Hurd
  • Add patch 229498 from misho

These patches are from this bz thread:
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=236983

Diff Detail

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

Event Timeline

thj created this revision.Apr 7 2022, 4:35 PM
Herald added a subscriber: imp. · View Herald TranscriptApr 7 2022, 4:35 PM
thj requested review of this revision.Apr 7 2022, 4:35 PM
Harbormaster completed remote builds in B45056: Diff 104729.Apr 7 2022, 4:35 PM
thj added a reviewer: mav.Apr 7 2022, 4:50 PM
thj added a reviewer: rpokala.Apr 7 2022, 4:54 PM
mav resigned from this revision.Apr 7 2022, 5:15 PM

I am not really familiar with this hardware to review this. Only fixed couple mechanical issues.

pi added a subscriber: pi.Apr 7 2022, 6:16 PM

Tested, packets only flow if interface is in promisc mode. The patch from

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=236983#c23

works without the promisc.

thj updated this revision to Diff 104912.Apr 12 2022, 4:46 PM

With just misho's patch from bz

Harbormaster completed remote builds in B45127: Diff 104912.Apr 12 2022, 4:46 PM
This revision was not accepted when it landed; it landed in state Needs Review.Apr 21 2022, 8:32 AM
Closed by commit rG0c6a2fa33e36: if_bnxt: Allow bnxt interfaces to use vlans (authored by misho_elwix.org, committed by thj). · Explain Why
This revision was automatically updated to reflect the committed changes.
thj added a commit: rG0c6a2fa33e36: if_bnxt: Allow bnxt interfaces to use vlans.
thj added a commit: rGe54505277c39: if_bnxt: Allow bnxt interfaces to use vlans.Apr 27 2022, 1:16 PM
thj added a commit: rG5acfa48bb258: if_bnxt: Allow bnxt interfaces to use vlans.Apr 27 2022, 1:56 PM

Revision Contents
Changeset List

PathSize
sys/
dev/
bnxt/
6 lines

Diff 105261

View Options

sys/dev/bnxt/if_bnxt.c

Loading...