Page MenuHomeFreeBSD

vtnet: Avoid ifdefs based on __NO_STRICT_ALIGNMENT
ClosedPublic

Authored by imp on Jan 29 2024, 8:53 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, May 28, 8:55 AM
Unknown Object (File)
Sun, May 12, 1:58 PM
Unknown Object (File)
Apr 17 2024, 7:29 PM
Unknown Object (File)
Apr 17 2024, 7:23 PM
Unknown Object (File)
Apr 17 2024, 7:04 PM
Unknown Object (File)
Apr 17 2024, 7:04 PM
Unknown Object (File)
Apr 16 2024, 4:31 PM
Unknown Object (File)
Feb 1 2024, 6:34 AM
Subscribers
None

Details

Summary

Some platforms require an adjustment of the ethernet hearders. Rather
than make this be on __NO_STRICT_ALIGNMENT being defined, define
VTNET_ETHER_ALIGN to be either 0 or ETHER_ALIGN (aka 2). Add a test to
the if statements to only do them when != 0. This eliminates the #ifdef
sprinkled in the code, still communicates the intent and gives the same
compiled results.

Sponsored by: Netflix

Diff Detail

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

Event Timeline

imp requested review of this revision.Jan 29 2024, 8:53 PM
imp created this revision.
sys/dev/virtio/network/if_vtnet.c
95

s/2/ETHER_ALIGN/ to understand the magic number

sys/dev/virtio/network/if_vtnet.c
95

Right. I'd meant to do that and has a braino

update to use ETHER_ALIGN

This revision is now accepted and ready to land.Jan 31 2024, 3:39 AM