Page MenuHomeFreeBSD

em: skip rxcsum offload processing when disabled
ClosedPublic

Authored by vmaffione on Nov 28 2021, 10:14 PM.
Tags
None
Referenced Files
F107148527: D33155.diff
Fri, Jan 10, 9:04 PM
Unknown Object (File)
Sat, Dec 28, 4:49 AM
Unknown Object (File)
Thu, Dec 12, 2:05 PM
Unknown Object (File)
Dec 5 2024, 11:14 AM
Unknown Object (File)
Nov 15 2024, 1:11 AM
Unknown Object (File)
Nov 15 2024, 12:54 AM
Unknown Object (File)
Nov 13 2024, 3:30 PM
Unknown Object (File)
Nov 9 2024, 5:54 PM
Subscribers

Details

Summary

Similarly to the other Intel drivers, don't try to process
RX checksum offloads when this feature (IFCAP_RXCSUM) is
disabled.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

I don't like that the drivers are dealing in ifps and calling things like if_getcapenable(). Establishing an interface that allowed drivers to have no knowledge of these details was one of the motivations for iflib.

The alternative would be to have iflib itself tell the driver if rxcsum offload was desired by setting a flag in the if_rxd_info_t

Also, if we stick with this approach, the ifp is passed into the driver via if_rxd_info_t's iri_ifp, so the iflib_get_ifp() call is not needed.

Thanks. I think we may get there (getting rid of ifp) at a later step, since some reworking would actually needed within iflib.
This change is meant to simply align em(4) to the other drivers.

This revision is now accepted and ready to land.Nov 30 2021, 10:22 PM