On 4/21/2025 7:04 AM, Vishal Badole wrote:
According to the XGMAC specification, enabling features such as Layer 3 and Layer 4 Packet Filtering, Split Header, Receive Side Scaling (RSS), and Virtualized Network support automatically selects the IPC Full Checksum Offload Engine on the receive side.
When RX checksum offload is disabled, these dependent features must also be disabled to prevent abnormal behavior caused by mismatched feature dependencies.
Ensure that toggling RX checksum offload (disabling or enabling) properly disables or enables all dependent features, maintaining consistent and expected behavior in the network device.
My understanding based on previous changes I've made to Intel drivers, the netdev community opinion here is that the driver shouldn't automatically change user configuration like this. Instead, it should reject requests to disable a feature if that isn't possible due to the other requirements.
In this case, that means checking and rejecting disable of Rx checksum offload whenever the features which depend on it are enabled, and reject requests to enable the features when Rx checksum is disabled.