Page MenuHomeFreeBSD

if_dwc: Multicast and FCS fixes
ClosedPublic

Authored by jmcneill on May 31 2016, 12:27 AM.
Tags
Referenced Files
F131805477: D6653.diff
Sat, Oct 11, 7:45 AM
Unknown Object (File)
Sun, Sep 21, 3:02 AM
Unknown Object (File)
Sep 4 2025, 4:16 AM
Unknown Object (File)
Aug 29 2025, 4:35 AM
Unknown Object (File)
Aug 24 2025, 12:24 AM
Unknown Object (File)
Aug 15 2025, 10:25 AM
Unknown Object (File)
Jul 26 2025, 11:50 PM
Unknown Object (File)
Jul 9 2025, 11:02 PM
Subscribers

Details

Summary

Multicast filters are different between the two versions of dwc supported by the driver. This patch adds support for the second (DWC_GMAC_ALT_DESC mode) type of filters.

In addition, remove trailing FCS in receive path. dhclient complains:

ip length 334 disagrees with bytes received 338.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

jmcneill retitled this revision from to if_dwc: Fix multicast in DWC_GMAC_ALT_DESC mode.
jmcneill updated this object.
jmcneill edited the test plan for this revision. (Show Details)
jmcneill added reviewers: ARM, loos.
jmcneill set the repository for this revision to rS FreeBSD src repository - subversion.
jmcneill added a project: ARM.
jmcneill retitled this revision from if_dwc: Fix multicast in DWC_GMAC_ALT_DESC mode to if_dwc: Multicast and FCS fixes.
jmcneill updated this object.

Remove trailing FCS in receive path. Fixes "ip length XXX disagrees with bytes received YYY" message from dhclient.

sys/dev/dwc/if_dwc.c
772 ↗(On Diff #17143)

I wanted to use M_HASFCS here but if_ethersubr.c passes the mbuf to BPF before stripping FCS (so dhclient still complained).

loos edited edge metadata.
This revision is now accepted and ready to land.Jun 4 2016, 3:09 PM
This revision was automatically updated to reflect the committed changes.