Re: [Intel-wired-lan] [PATCH iwl-next v5 2/2] ice: Implement 'flow-type ether' rules

[Alexander Lobakin <aleksander.lobakin@intel.com>]

From: Lukasz Plachno <lukasz.plachno@intel.com>
Date: Tue,  6 Feb 2024 17:33:37 +0100

> From: Jakub Buchocki <jakubx.buchocki@intel.com>
> 
> Add support for 'flow-type ether' Flow Director rules via ethtool.
> 
> Create packet segment info for filter configuration based on ethtool
> command parameters. Reuse infrastructure already created for
> ipv4 and ipv6 flows to convert packet segment into
> extraction sequence, which is later used to program the filter
> inside Flow Director block of the Rx pipeline.

[...]

> diff --git a/drivers/net/ethernet/intel/ice/ice_fdir.c b/drivers/net/ethernet/intel/ice/ice_fdir.c
> index 1f7b26f38818..5fe0bad00fd7 100644
> --- a/drivers/net/ethernet/intel/ice/ice_fdir.c
> +++ b/drivers/net/ethernet/intel/ice/ice_fdir.c
> @@ -4,6 +4,8 @@
>  #include "ice_common.h"
>  
>  /* These are training packet headers used to program flow director filters. */
> +static const u8 ice_fdir_eth_pkt[22] = {0};

I believe this zeroing is not needed, just declare it and the compiler
will zero it automatically.

[...]

> @@ -97,6 +100,12 @@ struct ice_rx_flow_userdef {
>  	u16 flex_fltr;
>  };
>  
> +struct ice_fdir_eth {
> +	u8 dst[ETH_ALEN];
> +	u8 src[ETH_ALEN];
> +	__be16 type;
> +};

This is clearly `struct ethhdr`, please remove this duplicating
definition and just use the generic structure.

Thanks,
Olek