Re: [PATCH v5 07/27] net/i40e: use proper flex len define

Medvedkin, Vladimir
2026-02-17T17:10:25Z

Acked-by: Vladimir Medvedkin <vladimir.medvedkin@intel.com>

On 2/17/2026 12:13 PM, Anatoly Burakov wrote:
> For FDIR, we have byte arrays that are supposed to be limited to whatever
> the HW supports in terms of flex descriptor matching. However, in the
> structure definition, spec and mask bytes are using different array length
> defines, and the only reason why this works is because they evaluate to the
> same value.
>
> Use the i40e-specific definition instead.
>
> Signed-off-by: Anatoly Burakov <anatoly.burakov@intel.com>
> ---
> drivers/net/intel/i40e/i40e_ethdev.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/net/intel/i40e/i40e_ethdev.h b/drivers/net/intel/i40e/i40e_ethdev.h
> index ca6638b32c..d57c53f661 100644
> --- a/drivers/net/intel/i40e/i40e_ethdev.h
> +++ b/drivers/net/intel/i40e/i40e_ethdev.h
> @@ -631,7 +631,7 @@ struct i40e_fdir_flex_pit {
> /* A structure used to contain extend input of flow */
> struct i40e_fdir_flow_ext {
> uint16_t vlan_tci;
> - uint8_t flexbytes[RTE_ETH_FDIR_MAX_FLEXLEN];
> + uint8_t flexbytes[I40E_FDIR_MAX_FLEX_LEN];
> /* It is filled by the flexible payload to match. */
> uint8_t flex_mask[I40E_FDIR_MAX_FLEX_LEN];
> uint8_t raw_id;

--
Regards,
Vladimir