[dpdk-test-report] |WARNING| pw17960 [PATCH 2/2] hyperv: VMBUS support infrastucture

checkpatch at dpdk.org checkpatch at dpdk.org
Thu Dec 15 01:00:03 CET 2016


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

_coding style issues_


WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#203: FILE: lib/librte_eal/common/eal_internal_cfg.h:72:
+					   * instead of native TSC */

ERROR:TRAILING_WHITESPACE: trailing whitespace
#350: FILE: lib/librte_eal/common/include/rte_vmbus.h:71:
+^I$

WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#472: FILE: lib/librte_eal/common/include/rte_vmbus.h:193:
+ * ^IThe VMBUS device uuid.$

WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#487: FILE: lib/librte_eal/common/include/rte_vmbus.h:208:
+ * ^IThe VMBUS device uuid.$

ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#688: FILE: lib/librte_eal/linuxapp/eal/eal_vmbus.c:96:
+	if (buf[0] == '{' && (cp = strchr(buf, '}')))

ERROR:CODE_INDENT: code indent should use tabs where possible
#1082: FILE: lib/librte_eal/linuxapp/eal/eal_vmbus.c:490:
+ ^I^I^I^Ivmbus_unmap_resource(uio_res->maps[i].addr,$

WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#1082: FILE: lib/librte_eal/linuxapp/eal/eal_vmbus.c:490:
+ ^I^I^I^Ivmbus_unmap_resource(uio_res->maps[i].addr,$

WARNING:LEADING_SPACE: please, no spaces at the start of a line
#1082: FILE: lib/librte_eal/linuxapp/eal/eal_vmbus.c:490:
+ ^I^I^I^Ivmbus_unmap_resource(uio_res->maps[i].addr,$

WARNING:UNNECESSARY_ELSE: else is not generally useful after a break or return
#1249: FILE: lib/librte_eal/linuxapp/eal/eal_vmbus.c:657:
+			return 0;
+		else {

total: 3 errors, 6 warnings, 1492 lines checked


More information about the test-report mailing list