[dpdk-test-report] [Patchwork]|ERROR| pw16868 mempool: Add sanity check when secondary link in less mempools than primary

sys_stv at intel.com sys_stv at intel.com
Fri Oct 28 20:57:02 CEST 2016


Test-Label: Intel Niantic on Fedora
Test-Status: ERROR

Patchwork ID: 16868
http://www.dpdk.org/dev/patchwork/patch/16868/
Submitter: Jean Tourrilhes <jt at labs.hpe.com>
Date: Fri, 28 Oct 2016 11:37:05 -0700
DPDK git baseline: ca41215c051a1f4f6f96e947ccb671dab3509cbf

Check patch error:
16868: 
ERROR: space required before the open parenthesis '('
#56: FILE: lib/librte_mempool/rte_mempool.c:1287:
+	if(mp->ops_index >= (int32_t) rte_mempool_ops_table.num_ops) {

WARNING: braces {} are not necessary for single statement blocks
#59: FILE: lib/librte_mempool/rte_mempool.c:1290:
+		for (i = 0; i < rte_mempool_ops_table.num_ops; i++) {
+			RTE_LOG(ERR, EAL, "Registered mempool[%d] is %s\n", i, rte_mempool_ops_table.ops[i].name);
+		}

WARNING: line over 80 characters
#60: FILE: lib/librte_mempool/rte_mempool.c:1291:
+			RTE_LOG(ERR, EAL, "Registered mempool[%d] is %s\n", i, rte_mempool_ops_table.ops[i].name);

WARNING: line over 80 characters
#63: FILE: lib/librte_mempool/rte_mempool.c:1294:
+		rte_panic("Cannot find ops for mempool, ops_index %d, num_ops %d - maybe due to build process or linker configuration\n", mp->ops_index, rte_mempool_ops_table.num_ops);

total: 1 errors, 3 warnings, 25 lines checked

/home/patchWorkOrg/patches/dpdk-dev-v2-1-1-mempool-Add-sanity-check-when-secondary-link-in-less-mempools-than-primary.patch has style problems, please review.

If any of these errors are false positives, please report
them to the maintainer, see CHECKPATCH in MAINTAINERS.


Compilation:
OS: fedora
Nic: niantic
GCC: gcc_x86-64, 4.8.3
ICC:16.0.2
i686-native-linuxapp-icc: compile pass
x86_64-native-linuxapp-gcc-combined: compile pass
i686-native-linuxapp-gcc: compile pass
x86_64-native-linuxapp-gcc: compile pass
x86_64-native-linuxapp-icc: compile pass
x86_64-native-linuxapp-gcc-debug: compile pass
x86_64-native-linuxapp-gcc-shared: compile pass
x86_64-native-linuxapp-clang: compile pass



DPDK STV team 


More information about the test-report mailing list