Skip to content
Commit 1da26860 authored by Íñigo Huguet's avatar Íñigo Huguet Committed by Greg Kroah-Hartman
Browse files

sfc: ef10: don't overwrite offload features at NIC reset



[ Upstream commit ca4a80e4 ]

At NIC reset, some offload features related to encapsulated traffic
might have changed (this mainly happens if the firmware-variant is
changed with the sfboot userspace tool). Because of this, features are
checked and set again at reset time.

However, this was not done right, and some features were improperly
overwritten at NIC reset:
- Tunneled IPv6 segmentation was always disabled
- Features disabled with ethtool were reenabled
- Features that becomes unsupported after the reset were not disabled

Also, checking if the device supports IPV6_CSUM to enable TSO6 is no
longer necessary because all currently supported devices support it.
Additionally, move the assignment of some other features to the
EF10_OFFLOAD_FEATURES macro, like it is done in ef100, leaving the
selection of features in efx_pci_probe_post_io a bit cleaner.

Fixes: ffffd245 ("sfc: correctly advertise tunneled IPv6 segmentation")
Fixes: 24b2c375 ("sfc: advertise encapsulated offloads on EF10")
Reported-by: default avatarTianhao Zhao <tizhao@redhat.com>
Suggested-by: default avatarJonathan Cooper <jonathan.s.cooper@amd.com>
Tested-by: default avatarJonathan Cooper <jonathan.s.cooper@amd.com>
Signed-off-by: default avatarÍñigo Huguet <ihuguet@redhat.com>
Acked-by: default avatarEdward Cree <ecree.xilinx@gmail.com>
Link: https://lore.kernel.org/r/20230323083417.7345-1-ihuguet@redhat.com


Signed-off-by: default avatarJakub Kicinski <kuba@kernel.org>
Signed-off-by: default avatarSasha Levin <sashal@kernel.org>
parent c5a159d5
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment