Re: [PATCH net v2 3/3] idpf: fix offloads support for encapsulated packets

[Willem de Bruijn <willemdebruijn.kernel@gmail.com>]

Tony Nguyen wrote:
> From: Madhu Chittim <madhu.chittim@intel.com>
> 
> Split offloads into csum, tso and other offloads so that tunneled
> packets do not by default have all the offloads enabled.
> 
> Stateless offloads for encapsulated packets are not yet supported in
> firmware/software but in the driver we were setting the features same as
> non encapsulated features.
> 
> Fixed naming to clarify CSUM bits are being checked for Tx.
> 
> Inherit netdev features to VLAN interfaces as well.
> 
> Fixes: 0fe45467a104 ("idpf: add create vport and netdev configuration")
> Reviewed-by: Sridhar Samudrala <sridhar.samudrala@intel.com>
> Signed-off-by: Madhu Chittim <madhu.chittim@intel.com>
> Tested-by: Zachary Goldstein <zachmgoldstein@google.com>
> Tested-by: Samuel Salin <Samuel.salin@intel.com>
> Signed-off-by: Tony Nguyen <anthony.l.nguyen@intel.com>

Reviewed-by: Willem de Bruijn <willemb@google.com>

The issue resolved was that checksum offload support was advertised
for tunneled packets, but not implemented in (some) firmware.

Support for tunnel encap without checksum offload (e.g., plain GRE)
and even GSO_PARTIAL may still work. Not sure. If so, that can be
reenabled at a later date.