Page MenuHomeFreeBSD

em: skip rxcsum offload processing when disabled
ClosedPublic

Authored by vmaffione on Nov 28 2021, 10:14 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Mar 29, 3:10 PM
Unknown Object (File)
Mar 16 2024, 4:26 PM
Unknown Object (File)
Feb 14 2024, 1:55 PM
Unknown Object (File)
Feb 13 2024, 7:11 PM
Unknown Object (File)
Jan 15 2024, 8:58 AM
Unknown Object (File)
Dec 20 2023, 5:51 AM
Unknown Object (File)
Oct 18 2023, 4:36 AM
Unknown Object (File)
Sep 16 2023, 12:24 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