Re: [PATCH net-next 1/4] net: add netif_inherit_tso_max()

[Simon Horman <simon.horman@corigine.com>]

On Thu, May 05, 2022 at 07:51:31PM -0700, Jakub Kicinski wrote:
> To make later patches smaller create a helper for inheriting
> the TSO limitations of a lower device. The TSO in the name
> is not an accident, subsequent patches will replace GSO
> with TSO in more names.
> 
> Signed-off-by: Jakub Kicinski <kuba@kernel.org>
> ---
> CC: simon.horman@corigine.com
> CC: gustavoars@kernel.org
> CC: roopa@nvidia.com
> CC: keescook@chromium.org
> CC: william.xuanziyang@huawei.com
> CC: lucien.xin@gmail.com
> CC: oss-drivers@corigine.com
> ---
>  drivers/net/ethernet/netronome/nfp/nfp_net_repr.c |  3 +--
>  drivers/net/ipvlan/ipvlan_main.c                  |  6 ++----
>  drivers/net/macvlan.c                             |  6 ++----
>  drivers/net/veth.c                                |  3 +--
>  drivers/net/vxlan/vxlan_core.c                    |  3 +--
>  include/linux/netdevice.h                         |  3 +++
>  net/8021q/vlan.c                                  |  3 +--
>  net/8021q/vlan_dev.c                              |  3 +--
>  net/core/dev.c                                    | 12 ++++++++++++
>  9 files changed, 24 insertions(+), 18 deletions(-)
> 
> diff --git a/drivers/net/ethernet/netronome/nfp/nfp_net_repr.c b/drivers/net/ethernet/netronome/nfp/nfp_net_repr.c
> index ba3fa7eac98d..790e1d5e4b4a 100644
> --- a/drivers/net/ethernet/netronome/nfp/nfp_net_repr.c
> +++ b/drivers/net/ethernet/netronome/nfp/nfp_net_repr.c
> @@ -286,8 +286,7 @@ nfp_repr_transfer_features(struct net_device *netdev, struct net_device *lower)
>  	if (repr->dst->u.port_info.lower_dev != lower)
>  		return;
>  
> -	netif_set_gso_max_size(netdev, lower->gso_max_size);
> -	netif_set_gso_max_segs(netdev, lower->gso_max_segs);
> +	netif_inherit_tso_max(netdev, lower);
>  
>  	netdev_update_features(netdev);
>  }

NFP portion:

Reviewed-by: Simon Horman <simon.horman@corigine.com>

...