|WARNING| pw135261 [PATCH v3 13/14] lib: replace logging helpers

checkpatch at dpdk.org checkpatch at dpdk.org
Mon Dec 18 10:32:19 CET 2023


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

_coding style issues_


ERROR:SPACING: space required after that ',' (ctx:VxB)
#3445: FILE: lib/ethdev/rte_ethdev.h:182:
+	RTE_LOG(level, ETHDEV, RTE_FMT(RTE_FMT_HEAD(__VA_ARGS__,) "
", \
 	                                                       ^

ERROR:SPACING: space required after that ',' (ctx:VxB)
#3446: FILE: lib/ethdev/rte_ethdev.h:183:
+		RTE_FMT_TAIL(__VA_ARGS__,)))
 		                        ^

ERROR:SPACING: space required after that ',' (ctx:VxB)
#4118: FILE: lib/member/member.h:12:
+		RTE_FMT("%s(): " RTE_FMT_HEAD(__VA_ARGS__,) "
", \
 		                                         ^

ERROR:SPACING: space required after that ',' (ctx:VxB)
#4119: FILE: lib/member/member.h:13:
+			__func__, RTE_FMT_TAIL(__VA_ARGS__,)))
 			                                  ^

ERROR:SPACING: space required after that ',' (ctx:VxB)
#5428: FILE: lib/regexdev/rte_regexdev.h:212:
+	RTE_LOG(level, REGEXDEV, RTE_FMT(RTE_FMT_HEAD(__VA_ARGS__,) "
", \
 	                                                         ^

ERROR:SPACING: space required after that ',' (ctx:VxB)
#5429: FILE: lib/regexdev/rte_regexdev.h:213:
+		RTE_FMT_TAIL(__VA_ARGS__,)))
 		                        ^

ERROR:SPACING: space required after that ',' (ctx:VxB)
#5476: FILE: lib/telemetry/telemetry.c:61:
+	RTE_LOG(l, TMTY,  RTE_FMT("TELEMETRY: " RTE_FMT_HEAD(__VA_ARGS__,) "
", \
 	                                                                ^

ERROR:SPACING: space required after that ',' (ctx:VxB)
#5477: FILE: lib/telemetry/telemetry.c:62:
+		RTE_FMT_TAIL(__VA_ARGS__,)))
 		                        ^

WARNING:TYPO_SPELLING: 'couldn' may be misspelled - perhaps 'could'?
#7377: FILE: lib/vhost/vhost_user.c:1246:
+		VHOST_CONFIG_LOG(dev->ifname, ERR, "couldn't get hugepage size through fstat");

total: 8 errors, 1 warnings, 7964 lines checked


More information about the test-report mailing list