The output routines prepend an IP header with some extra wiggle room on
platforms which we declare to require strict alignment (i.e., everything
except x86). When doing so we may end up cutting off that wiggle room,
which involves updating the mbuf length. In this case we must update
the mbuf header as well.
Details
Details
- Reviewers
- None
- Group Reviewers
network
Diff Detail
Diff Detail
- Repository
- rS FreeBSD src repository - subversion
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 46746 Build 43635: arc lint + arc unit
Event Timeline
Comment Actions
but I got distracted by if_gif being broken on aarch64 (and one of the bridge tests uses gif)
@kp You might be interested in this ;)
Comment Actions
Ah, yes. I came up with this earlier, because it breaks the if_bridge_test:gif test, but even with that fix it's still broken (it no longer panics with this patch, but the test case still fails. if_gif generates incorrect packets for the '-s 2000' ping test).
My current version just deletes the code entirely, because it's both wrong and seemingly not actually useful.