[dpdk-test-report] |WARNING| pw19935 [PATCH RFCv2 2/4] ring: separate common and rte_ring specific functions

checkpatch at dpdk.org checkpatch at dpdk.org
Tue Jan 24 11:40:05 CET 2017


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

_coding style issues_


WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#95: FILE: lib/librte_ring/rte_common_ring.c:104:
+rte_common_ring_get_memsize(unsigned count)

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#104: FILE: lib/librte_ring/rte_common_ring.c:122:
+rte_common_ring_init(struct rte_ring *r, const char *name, unsigned count,

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#122: FILE: lib/librte_ring/rte_common_ring.c:162:
+rte_common_ring_create(const char *name, unsigned count, int socket_id,

ERROR:POINTER_LOCATION: "foo* bar" should be "foo *bar"
#131: FILE: lib/librte_ring/rte_common_ring.c:171:
+	struct rte_common_ring_list* ring_list = NULL;

WARNING:LONG_LINE_STRING: line over 90 characters
#172: FILE: lib/librte_ring/rte_common_ring.c:237:
+		RTE_LOG(ERR, RING, "Cannot free ring (not created with rte_common_ring_create()");

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#190: FILE: lib/librte_ring/rte_common_ring.c:272:
+rte_common_ring_set_water_mark(struct rte_ring *r, unsigned count)

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#278: FILE: lib/librte_ring/rte_common_ring.h:235:
+ssize_t rte_common_ring_get_memsize(unsigned count);

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#305: FILE: lib/librte_ring/rte_common_ring.h:271:
+int rte_common_ring_init(struct rte_ring *r, const char *name, unsigned count,

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#335: FILE: lib/librte_ring/rte_common_ring.h:313:
+struct rte_ring *rte_common_ring_create(const char *name, unsigned count,

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#353: FILE: lib/librte_ring/rte_common_ring.h:341:
+int rte_common_ring_set_water_mark(struct rte_ring *r, unsigned count);

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#922: FILE: lib/librte_ring/rte_ring.c:37:
+rte_ring_get_memsize(unsigned count)

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#928: FILE: lib/librte_ring/rte_ring.c:43:
+rte_ring_init(struct rte_ring *r, const char *name, unsigned count,

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#929: FILE: lib/librte_ring/rte_ring.c:44:
+	unsigned flags)

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#936: FILE: lib/librte_ring/rte_ring.c:51:
+rte_ring_create(const char *name, unsigned count, int socket_id,

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#937: FILE: lib/librte_ring/rte_ring.c:52:
+		unsigned flags)

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#949: FILE: lib/librte_ring/rte_ring.c:64:
+rte_ring_set_water_mark(struct rte_ring *r, unsigned count)

WARNING:TYPO_SPELLING: 'preemptable' may be misspelled - perhaps 'preemptible'?
#1038: FILE: lib/librte_ring/rte_ring.h:53:
+ * Note: the ring implementation is not preemptable. A lcore must not

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#1063: FILE: lib/librte_ring/rte_ring.h:78:
+ssize_t rte_ring_get_memsize(unsigned count);

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#1099: FILE: lib/librte_ring/rte_ring.h:114:
+int rte_ring_init(struct rte_ring *r, const char *name, unsigned count,

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#1100: FILE: lib/librte_ring/rte_ring.h:115:
+	unsigned flags);

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#1141: FILE: lib/librte_ring/rte_ring.h:156:
+struct rte_ring *rte_ring_create(const char *name, unsigned count,

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#1142: FILE: lib/librte_ring/rte_ring.h:157:
+				 int socket_id, unsigned flags);

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#1169: FILE: lib/librte_ring/rte_ring.h:184:
+int rte_ring_set_water_mark(struct rte_ring *r, unsigned count);

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#1201: FILE: lib/librte_ring/rte_ring.h:216:
+			 unsigned n)

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#1223: FILE: lib/librte_ring/rte_ring.h:238:
+			 unsigned n)

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#1249: FILE: lib/librte_ring/rte_ring.h:264:
+		      unsigned n)

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#1342: FILE: lib/librte_ring/rte_ring.h:357:
+rte_ring_mc_dequeue_bulk(struct rte_ring *r, void **obj_table, unsigned n)

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#1363: FILE: lib/librte_ring/rte_ring.h:378:
+rte_ring_sc_dequeue_bulk(struct rte_ring *r, void **obj_table, unsigned n)

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#1387: FILE: lib/librte_ring/rte_ring.h:402:
+rte_ring_dequeue_bulk(struct rte_ring *r, void **obj_table, unsigned n)

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#1552: FILE: lib/librte_ring/rte_ring.h:567:
+static inline unsigned __attribute__((always_inline))

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#1554: FILE: lib/librte_ring/rte_ring.h:569:
+			 unsigned n)

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#1571: FILE: lib/librte_ring/rte_ring.h:586:
+static inline unsigned __attribute__((always_inline))

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#1573: FILE: lib/librte_ring/rte_ring.h:588:
+			 unsigned n)

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#1594: FILE: lib/librte_ring/rte_ring.h:609:
+static inline unsigned __attribute__((always_inline))

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#1596: FILE: lib/librte_ring/rte_ring.h:611:
+		      unsigned n)

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#1621: FILE: lib/librte_ring/rte_ring.h:636:
+static inline unsigned __attribute__((always_inline))

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#1622: FILE: lib/librte_ring/rte_ring.h:637:
+rte_ring_mc_dequeue_burst(struct rte_ring *r, void **obj_table, unsigned n)

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#1641: FILE: lib/librte_ring/rte_ring.h:656:
+static inline unsigned __attribute__((always_inline))

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#1642: FILE: lib/librte_ring/rte_ring.h:657:
+rte_ring_sc_dequeue_burst(struct rte_ring *r, void **obj_table, unsigned n)

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#1663: FILE: lib/librte_ring/rte_ring.h:678:
+static inline unsigned __attribute__((always_inline))

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#1664: FILE: lib/librte_ring/rte_ring.h:679:
+rte_ring_dequeue_burst(struct rte_ring *r, void **obj_table, unsigned n)

total: 1 errors, 40 warnings, 1553 lines checked


More information about the test-report mailing list