|WARNING| pw156924 [PATCH v1] support multiple PF process flow table

checkpatch at dpdk.org checkpatch at dpdk.org
Thu Sep 25 04:57:33 CEST 2025


Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/156924

_coding style issues_


CHECK:MACRO_ARG_REUSE: Macro argument reuse 'DEV_ID' - possible side-effects?
#521: FILE: drivers/net/zxdh/zxdh_np.c:402:
+#define ZXDH_DTB_TAB_UP_VIR_ADDR_GET(DEV_ID, QUEUE_ID, INDEX)   \
+		(p_dpp_dtb_mgr[(ZXDH_DEV_SLOT_ID(DEV_ID))]->            \
+		queue_info[(QUEUE_ID)].tab_up.start_vir_addr +          \
+		(INDEX) * p_dpp_dtb_mgr[(ZXDH_DEV_SLOT_ID(DEV_ID))]->   \
+		queue_info[(QUEUE_ID)].tab_up.item_size)

CHECK:MACRO_ARG_REUSE: Macro argument reuse 'QUEUE_ID' - possible side-effects?
#521: FILE: drivers/net/zxdh/zxdh_np.c:402:
+#define ZXDH_DTB_TAB_UP_VIR_ADDR_GET(DEV_ID, QUEUE_ID, INDEX)   \
+		(p_dpp_dtb_mgr[(ZXDH_DEV_SLOT_ID(DEV_ID))]->            \
+		queue_info[(QUEUE_ID)].tab_up.start_vir_addr +          \
+		(INDEX) * p_dpp_dtb_mgr[(ZXDH_DEV_SLOT_ID(DEV_ID))]->   \
+		queue_info[(QUEUE_ID)].tab_up.item_size)

CHECK:MACRO_ARG_REUSE: Macro argument reuse 'DEV_ID' - possible side-effects?
#530: FILE: drivers/net/zxdh/zxdh_np.c:408:
+#define ZXDH_DTB_TAB_DOWN_VIR_ADDR_GET(DEV_ID, QUEUE_ID, INDEX) \
+		(p_dpp_dtb_mgr[ZXDH_DEV_SLOT_ID(DEV_ID)]->              \
+		queue_info[QUEUE_ID].tab_down.start_vir_addr +          \
+		(INDEX) * p_dpp_dtb_mgr[ZXDH_DEV_SLOT_ID(DEV_ID)]->     \
+		queue_info[QUEUE_ID].tab_down.item_size)

CHECK:MACRO_ARG_REUSE: Macro argument reuse 'QUEUE_ID' - possible side-effects?
#530: FILE: drivers/net/zxdh/zxdh_np.c:408:
+#define ZXDH_DTB_TAB_DOWN_VIR_ADDR_GET(DEV_ID, QUEUE_ID, INDEX) \
+		(p_dpp_dtb_mgr[ZXDH_DEV_SLOT_ID(DEV_ID)]->              \
+		queue_info[QUEUE_ID].tab_down.start_vir_addr +          \
+		(INDEX) * p_dpp_dtb_mgr[ZXDH_DEV_SLOT_ID(DEV_ID)]->     \
+		queue_info[QUEUE_ID].tab_down.item_size)

WARNING:MULTILINE_DEREFERENCE: Avoid multiple line dereference - prefer 'p_dpp_dtb_mgr[ZXDH_DEV_SLOT_ID(dev_id)]->queue_info[queue_id].tab_down.start_phy_addr'
#1013: FILE: drivers/net/zxdh/zxdh_np.c:5248:
+	phy_addr = p_dpp_dtb_mgr[ZXDH_DEV_SLOT_ID(dev_id)]->queue_info[queue_id].
+		tab_down.start_phy_addr +

WARNING:MULTILINE_DEREFERENCE: Avoid multiple line dereference - prefer 'p_dpp_dtb_mgr[ZXDH_DEV_SLOT_ID(dev_id)]->queue_info[queue_id].tab_down.item_size'
#1015: FILE: drivers/net/zxdh/zxdh_np.c:5250:
+		item_index * p_dpp_dtb_mgr[ZXDH_DEV_SLOT_ID(dev_id)]->queue_info[queue_id].
+		tab_down.item_size;

total: 0 errors, 2 warnings, 4 checks, 1357 lines checked


More information about the test-report mailing list