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);
| ^~~~~~~~~~~
