|WARNING| pw105365 [PATCH] net/mlx5: fix RSS expansion with explicit next protocol
checkpatch at dpdk.org
checkpatch at dpdk.org
Thu Dec 23 14:10:10 CET 2021
Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/105365
_coding style issues_
CHECK:MACRO_ARG_REUSE: Macro argument reuse 'fld' - possible side-effects?
#235: FILE: drivers/net/mlx5/mlx5_flow.c:246:
+#define MLX5_XSET_ITEM_MASK_SPEC(type, fld) \
+ do { \
+ const void *m = item->mask; \
+ const void *s = item->spec; \
+ mask = m ? \
+ ((const struct rte_flow_item_##type *)m)->fld : \
+ rte_flow_item_##type##_mask.fld; \
+ spec = ((const struct rte_flow_item_##type *)s)->fld; \
+ } while (0)
CHECK:MACRO_ARG_PRECEDENCE: Macro argument 'fld' may be better as '(fld)' to avoid precedence issues
#235: FILE: drivers/net/mlx5/mlx5_flow.c:246:
+#define MLX5_XSET_ITEM_MASK_SPEC(type, fld) \
+ do { \
+ const void *m = item->mask; \
+ const void *s = item->spec; \
+ mask = m ? \
+ ((const struct rte_flow_item_##type *)m)->fld : \
+ rte_flow_item_##type##_mask.fld; \
+ spec = ((const struct rte_flow_item_##type *)s)->fld; \
+ } while (0)
total: 0 errors, 0 warnings, 2 checks, 236 lines checked
More information about the test-report
mailing list