[dpdk-test-report] |WARNING| pw35333 [PATCH v2 1/2] Add RIB library

checkpatch at dpdk.org checkpatch at dpdk.org
Wed Feb 21 22:09:57 CET 2018


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

_coding style issues_


WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#26: 
 - For dir24_8 implementation it is possible to remove rte_lpm_tbl_entry.depth

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#165: FILE: lib/librte_rib/rte_dir24_8.c:37:
+	uint64_t *next_hops, const unsigned n)				\

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#170: FILE: lib/librte_rib/rte_dir24_8.c:42:
+	uint32_t prefetch_offset = RTE_MIN((unsigned)bulk_prefetch, n);	\

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#699: FILE: lib/librte_rib/rte_dir24_8.h:83:
+	uint64_t *next_hops, const unsigned n);

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#701: FILE: lib/librte_rib/rte_dir24_8.h:85:
+	uint64_t *next_hops, const unsigned n);

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#703: FILE: lib/librte_rib/rte_dir24_8.h:87:
+	uint64_t *next_hops, const unsigned n);

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#705: FILE: lib/librte_rib/rte_dir24_8.h:89:
+	uint64_t *next_hops, const unsigned n);

WARNING:MACRO_WITH_FLOW_CONTROL: Macros with flow control statements should be avoided
#1285: FILE: lib/librte_rib/rte_rib.h:15:
+#define RTE_RIB_RETURN_IF_TRUE(cond, retval) do {	\
+	if (cond)					\
+		return retval;				\
+} while (0)

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#1358: FILE: lib/librte_rib/rte_rib.h:88:
+	uint64_t *next_hops, const unsigned n);

total: 0 errors, 9 warnings, 1520 lines checked


More information about the test-report mailing list