Page MenuHomeFreeBSD

Use m_collapse(9) instead of m_defrag(9) in e1000 drivers
ClosedPublic

Authored by marius on Dec 28 2015, 12:26 AM.
Tags
None
Referenced Files
F83069091: D4717.diff
Sun, May 5, 8:58 PM
Unknown Object (File)
Mon, Apr 29, 10:33 AM
Unknown Object (File)
Mon, Apr 29, 10:30 AM
Unknown Object (File)
Mon, Apr 29, 10:28 AM
Unknown Object (File)
Sun, Apr 28, 4:30 PM
Unknown Object (File)
Jan 9 2024, 11:42 AM
Unknown Object (File)
Dec 22 2023, 9:52 PM
Unknown Object (File)
Nov 4 2023, 3:57 AM
Subscribers

Details

Summary

Given that em(4), lem(4) and igb(4) hardware doesn't require the
alignment guarantees provided by m_defrag(9), use m_collapse(9)
instead for performance reasons.
While at it, sanitize the statistics softc members, i. e. retire
unused ones and add SYSCTL nodes missing for actually used ones.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

marius retitled this revision from to Use m_collapse(9) instead of m_defrag(9) in e1000 drivers.
marius updated this object.
marius edited the test plan for this revision. (Show Details)
marius added a reviewer: erj.
marius set the repository for this revision to rS FreeBSD src repository - subversion.
sbruno added a reviewer: sbruno.
sbruno added a subscriber: sbruno.

This looks fine and an improvement.

This revision is now accepted and ready to land.Jan 5 2016, 5:41 PM
This revision was automatically updated to reflect the committed changes.