|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