[dpdk-test-report] |WARNING| pw32470 [PATCH RFC v2 17/23] eal: add backend support for contiguous memory allocation

checkpatch at dpdk.org checkpatch at dpdk.org
Tue Dec 19 12:15:44 CET 2017


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

_coding style issues_


WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#39: FILE: lib/librte_eal/common/eal_common_memzone.c:130:
+		int socket_id, unsigned flags, unsigned align, unsigned bound,

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#39: FILE: lib/librte_eal/common/eal_common_memzone.c:130:
+		int socket_id, unsigned flags, unsigned align, unsigned bound,

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#39: FILE: lib/librte_eal/common/eal_common_memzone.c:130:
+		int socket_id, unsigned flags, unsigned align, unsigned bound,

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#58: FILE: lib/librte_eal/common/eal_common_memzone.c:255:
+				unsigned bound, bool contig)

ERROR:OPEN_BRACE: open brace '{' following function definitions go on the next line
#119: FILE: lib/librte_eal/common/malloc_elem.c:130:
+static bool
+elem_check_phys_contig(const struct rte_memseg_list *msl, void *start,
+		size_t size) {

ERROR:POINTER_LOCATION: "(foo*)" should be "(foo *)"
#201: FILE: lib/librte_eal/common/malloc_elem.c:195:
+					(void*) new_data_start, new_data_size)) {

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#272: FILE: lib/librte_eal/common/malloc_elem.h:152:
+		unsigned align, size_t bound, bool contig);

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#281: FILE: lib/librte_eal/common/malloc_elem.h:160:
+		unsigned align, size_t bound, bool contig);

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#303: FILE: lib/librte_eal/common/malloc_heap.c:126:
+		unsigned flags, size_t align, size_t bound, bool contig)

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#322: FILE: lib/librte_eal/common/malloc_heap.c:159:
+		unsigned flags, size_t align, size_t bound, bool contig)

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#394: FILE: lib/librte_eal/common/malloc_heap.c:263:
+		unsigned flags, size_t align, size_t bound, bool contig) {

ERROR:OPEN_BRACE: open brace '{' following function definitions go on the next line
#532: FILE: lib/librte_eal/common/rte_malloc.c:65:
+static void *
+malloc_socket(const char *type, size_t size, unsigned align, int socket_arg,
+		bool contig) {

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#533: FILE: lib/librte_eal/common/rte_malloc.c:66:
+malloc_socket(const char *type, size_t size, unsigned align, int socket_arg,

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#551: FILE: lib/librte_eal/common/rte_malloc.c:93:
+rte_malloc_socket(const char *type, size_t size, unsigned align, int socket_arg)

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#564: FILE: lib/librte_eal/common/rte_malloc.c:147:
+do_realloc(void *ptr, size_t size, unsigned align, bool contig)

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#591: FILE: lib/librte_eal/common/rte_malloc.c:178:
+rte_realloc(void *ptr, size_t size, unsigned align)

total: 3 errors, 13 warnings, 504 lines checked


More information about the test-report mailing list