[dpdk-test-report] |WARNING| pw32467 [PATCH RFC v2 11/23] eal: replace memseg with memseg lists

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


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

_coding style issues_


ERROR:OPEN_BRACE: open brace '{' following function definitions go on the next line
#148: FILE: drivers/net/virtio/virtio_user/vhost_kernel.c:103:
+static int
+add_memory_region(struct vhost_memory_region *mr, const struct rte_fbarray *arr,
+		int reg_start_idx, int max) {

ERROR:SPACING: space required before the open brace '{'
#162: FILE: drivers/net/virtio/virtio_user/vhost_kernel.c:117:
+	while (idx < max){

CHECK:BRACES: braces {} should be used on all arms of this statement
#164: FILE: drivers/net/virtio/virtio_user/vhost_kernel.c:119:
+		if (expected_addr == NULL) {
[...]
+		} else if (ms->addr != expected_addr)
[...]

CHECK:SPACING: No space is necessary after a cast
#173: FILE: drivers/net/virtio/virtio_user/vhost_kernel.c:128:
+	mr->guest_phys_addr = (uint64_t)(uintptr_t) start_addr;

CHECK:SPACING: No space is necessary after a cast
#174: FILE: drivers/net/virtio/virtio_user/vhost_kernel.c:129:
+	mr->userspace_addr = (uint64_t)(uintptr_t) start_addr;

ERROR:OPEN_BRACE: open brace '{' following function definitions go on the next line
#301: FILE: lib/librte_eal/common/eal_common_memory.c:123:
+static uint64_t
+get_mem_amount(uint64_t page_sz) {

ERROR:OPEN_BRACE: open brace '{' following function definitions go on the next line
#312: FILE: lib/librte_eal/common/eal_common_memory.c:134:
+static int
+get_max_num_pages(uint64_t page_sz, uint64_t mem_amount) {

ERROR:OPEN_BRACE: open brace '{' following function definitions go on the next line
#317: FILE: lib/librte_eal/common/eal_common_memory.c:139:
+static int
+get_min_num_pages(int max_pages) {

ERROR:OPEN_BRACE: open brace '{' following function definitions go on the next line
#322: FILE: lib/librte_eal/common/eal_common_memory.c:144:
+static int
+alloc_memseg_list(struct rte_memseg_list *msl, uint64_t page_sz,
+		int socket_id) {

ERROR:OPEN_BRACE: open brace '{' following function definitions go on the next line
#361: FILE: lib/librte_eal/common/eal_common_memory.c:183:
+static int
+memseg_init(void) {

WARNING:BRACES: braces {} are not necessary for single statement blocks
#390: FILE: lib/librte_eal/common/eal_common_memory.c:212:
+			if (alloc_memseg_list(msl, hugepage_sz, socket_id)) {
+				return -1;
+			}

ERROR:OPEN_BRACE: open brace '{' following function definitions go on the next line
#398: FILE: lib/librte_eal/common/eal_common_memory.c:220:
+static const struct rte_memseg *
+virt2memseg(const void *addr, const struct rte_memseg_list *msl) {

ERROR:OPEN_BRACE: open brace '{' following function definitions go on the next line
#437: FILE: lib/librte_eal/common/eal_common_memory.c:259:
+static const struct rte_memseg *
+virt2memseg_legacy(const void *addr) {

ERROR:OPEN_BRACE: open brace '{' following function definitions go on the next line
#463: FILE: lib/librte_eal/common/eal_common_memory.c:285:
+const struct rte_memseg *
+rte_mem_virt2memseg(const void *addr, const struct rte_memseg_list *msl) {

WARNING:BRACES: braces {} are not necessary for any arm of this statement
#466: FILE: lib/librte_eal/common/eal_common_memory.c:288:
+	if (internal_config.legacy_mem) {
[...]
+	} else {
[...]

ERROR:OPEN_BRACE: open brace '{' following function definitions go on the next line
#960: FILE: lib/librte_eal/linuxapp/eal/eal_memory.c:932:
+static struct rte_memseg_list *
+get_memseg_list(int socket, uint64_t page_sz) {

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#1115: FILE: lib/librte_eal/linuxapp/eal/eal_memory.c:1349:
+	unsigned i;

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#1117: FILE: lib/librte_eal/linuxapp/eal/eal_memory.c:1351:
+	unsigned cur_seg, max_seg;

WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#1285: FILE: lib/librte_eal/linuxapp/eal/eal_memory.c:1468:
+			 * entries before it did the second mmap of them */

ERROR:SPACING: space required before the open brace '{'
#1286: FILE: lib/librte_eal/linuxapp/eal/eal_memory.c:1469:
+			for (i = 0; i < num_hp && offset < ms->len; i++){

WARNING:LONG_LINE: line over 90 characters
#1297: FILE: lib/librte_eal/linuxapp/eal/eal_memory.c:1480:
+							mapping_size, PROT_READ | PROT_WRITE,

WARNING:LONG_LINE_COMMENT: line over 90 characters
#1299: FILE: lib/librte_eal/linuxapp/eal/eal_memory.c:1482:
+					close(fd); /* close file both on success and on failure */

WARNING:LONG_LINE: line over 90 characters
#1301: FILE: lib/librte_eal/linuxapp/eal/eal_memory.c:1484:
+							addr != RTE_PTR_ADD(base_addr, offset)) {

ERROR:SPACING: spaces required around that '+=' (ctx:VxV)
#1306: FILE: lib/librte_eal/linuxapp/eal/eal_memory.c:1489:
+					offset+=mapping_size;
 					      ^

WARNING:BRACES: braces {} are not necessary for single statement blocks
#1690: FILE: test/test/test_memory.c:104:
+			for (j = 0; j < ms->len; j++) {
+				*((volatile uint8_t *) ms->addr + j);
+			}

total: 13 errors, 9 warnings, 1536 lines checked


More information about the test-report mailing list