Re: [PATCH 21.11] net/bnxt: fix tunnel stateless offloads

[Kevin Traynor <ktraynor@redhat.com>]

On 22/06/2022 09:37, Kalesh A P wrote:
> From: Kalesh AP<kalesh-anakkur.purayil@broadcom.com>
> 
> [ upstream commit c0278f6e52399f7612f9f1a9d52434071ac49921 ]
> 
> The HW only supports tunnel header parsing globally for supported tunnel
> types. When a function uses one default VNIC to receive both the tunnel
> and non-tunnel packets, applying the same stateless offload operation to
> both tunnel and non-tunnel packets can cause problems in certain scenarios.
> To workaround these problems, the firmware advertises no tunnel header
> parsing capabilities to the driver using the HWRM_FUNC_QCAPS.
> The driver must check this flag setting and accordingly not advertise
> tunnel packet stateless offload capabilities to the stack.
> 
> If the device supports VXLAN, GRE, IPIP and GENEVE tunnel parsing,
> then reports RX_OFFLOAD_OUTER_IPV4_CKSUM, RX_OFFLOAD_OUTER_UDP_CKSUM
> and TX_OFFLOAD_OUTER_IPV4_CKSUM in the Rx/Tx offload capabilities of
> the device.
> Also, advertise tunnel TSO capabilities based on FW support.
> 
> Backported the needed changes from "2646926002d8" to fix build failures.
> 
> Fixes: 0a6d2a720078 ("net/bnxt: get device infos")
> 
> Signed-off-by: Kalesh AP<kalesh-anakkur.purayil@broadcom.com>
> Reviewed-by: Ajit Khaparde<ajit.khaparde@broadcom.com>
> Reviewed-by: Somnath Kotur<somnath.kotur@broadcom.com>

Thanks, applied.