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)
Wed, Oct 9, 1:37 AM
Unknown Object (File)
Tue, Oct 8, 12:24 AM
Unknown Object (File)
Mon, Oct 7, 11:38 PM
Unknown Object (File)
Mon, Oct 7, 11:31 PM
Unknown Object (File)
Mon, Sep 30, 3:02 AM
Unknown Object (File)
Thu, Sep 12, 4:49 PM
Unknown Object (File)
Thu, Sep 12, 9:39 AM
Unknown Object (File)
Wed, Sep 11, 4:08 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 Not Applicable
Unit
Tests Not Applicable

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