[dpdk-dev] [PATCH v2 0/9] net/mlx5: add flex item support

[Ferruh Yigit]

On 11/2/2021 8:53 AM, Gregory Etelson wrote:
> Gregory Etelson (4):
>    common/mlx5: extend flex parser capabilities
>    common/mlx5: fix flex parser DevX creation routine
>    net/mlx5: add flex parser DevX object management
>    net/mlx5: handle flex item in flows
> 
> Viacheslav Ovsiienko (5):
>    common/mlx5: refactor HCA attributes query
>    net/mlx5: update eCPRI flex parser structures
>    net/mlx5: add flex item API
>    net/mlx5: translate flex item configuration
>    net/mlx5: translate flex item pattern into matcher
> 

Hi Gregory,

Can you please check following doc error [1] and build error [2]:

[1]
$ ./devtools/check-doc-vs-code.sh
rte_flow doc out of sync for mlx5
         item flex



[2]
                  from ../lib/eal/x86/include/rte_rwlock.h:13,
                  from ../lib/eal/include/rte_fbarray.h:40,
                  from ../lib/eal/include/rte_memory.h:25,
                  from ../lib/eal/include/rte_malloc.h:17,
                  from ../drivers/net/mlx5/mlx5_flow_flex.c:4:
../drivers/net/mlx5/mlx5_flow_flex.c: In function ‘mlx5_flex_flow_translate_item’:
../drivers/net/mlx5/mlx5_flow_flex.c:240:37: error: ‘priv’ undeclared (first use in this function)
   240 |         MLX5_ASSERT(mlx5_flex_index(priv, tp) >= 0);
       |                                     ^~~~
../lib/eal/include/rte_branch_prediction.h:38:45: note: in definition of macro ‘unlikely’
    38 | #define unlikely(x)     __builtin_expect(!!(x), 0)
       |                                             ^
../lib/eal/include/rte_debug.h:47:25: note: in expansion of macro ‘RTE_VERIFY’
    47 | #define RTE_ASSERT(exp) RTE_VERIFY(exp)
       |                         ^~~~~~~~~~
../drivers/common/mlx5/mlx5_common.h:104:26: note: in expansion of macro ‘RTE_ASSERT’
   104 | #define MLX5_ASSERT(exp) RTE_ASSERT(exp)
       |                          ^~~~~~~~~~
../drivers/net/mlx5/mlx5_flow_flex.c:240:9: note: in expansion of macro ‘MLX5_ASSERT’
   240 |         MLX5_ASSERT(mlx5_flex_index(priv, tp) >= 0);
       |         ^~~~~~~~~~~
../drivers/net/mlx5/mlx5_flow_flex.c:240:37: note: each undeclared identifier is reported only once for each function it appears in
   240 |         MLX5_ASSERT(mlx5_flex_index(priv, tp) >= 0);
       |                                     ^~~~
../lib/eal/include/rte_branch_prediction.h:38:45: note: in definition of macro ‘unlikely’
    38 | #define unlikely(x)     __builtin_expect(!!(x), 0)
       |                                             ^
../lib/eal/include/rte_debug.h:47:25: note: in expansion of macro ‘RTE_VERIFY’
    47 | #define RTE_ASSERT(exp) RTE_VERIFY(exp)
       |                         ^~~~~~~~~~
../drivers/common/mlx5/mlx5_common.h:104:26: note: in expansion of macro ‘RTE_ASSERT’
   104 | #define MLX5_ASSERT(exp) RTE_ASSERT(exp)
       |                          ^~~~~~~~~~
../drivers/net/mlx5/mlx5_flow_flex.c:240:9: note: in expansion of macro ‘MLX5_ASSERT’
   240 |         MLX5_ASSERT(mlx5_flex_index(priv, tp) >= 0);
       |         ^~~~~~~~~~~