[dpdk-test-report] |WARNING| [PATCH 5/7] test/eventdev: unit and functional tests

checkpatch at dpdk.org checkpatch at dpdk.org
Wed Nov 16 19:01:34 CET 2016


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

_coding style issues_


WARNING:BLOCK_COMMENT_STYLE: Block comments should align the * on each line
#131: FILE: app/test/test_eventdev_func.c:70:
+	/*
+	* len = 14 + 46

WARNING:LONG_LINE: line over 90 characters
#192: FILE: app/test/test_eventdev_func.c:131:
+		printf("%d: Eventdev %s not found - quitting.
", __LINE__, eventdev_name);

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#351: FILE: app/test/test_eventdev_func.c:290:
+	unsigned i;

ERROR:SPACING: space required before the open parenthesis '('
#352: FILE: app/test/test_eventdev_func.c:291:
+	for(i = 0; i < RTE_DIM(MAGIC_SEQN); i++) {

ERROR:SPACING: space required before the open parenthesis '('
#398: FILE: app/test/test_eventdev_func.c:337:
+	if(ev.mbuf->seqn != MAGIC_SEQN[1]) {

ERROR:SPACING: space required before the open parenthesis '('
#411: FILE: app/test/test_eventdev_func.c:350:
+	if(ev2.mbuf->seqn != MAGIC_SEQN[0]) {

WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#537: FILE: app/test/test_eventdev_func.c:476:
+	 * iter 512 (the 513th pkt) is the first refused NEW packet */

ERROR:SPACING: space required before the open parenthesis '('
#547: FILE: app/test/test_eventdev_func.c:486:
+		if(i == THRES-1) {

ERROR:SPACING: space required before the open parenthesis '('
#548: FILE: app/test/test_eventdev_func.c:487:
+			if(err != -ENOSPC) {

WARNING:LONG_LINE: line over 90 characters
#549: FILE: app/test/test_eventdev_func.c:488:
+				printf("%d: overload trip didn't cause NEW pkt enq fail
", __LINE__);

ERROR:ELSE_AFTER_BRACE: else should follow close brace '}'
#552: FILE: app/test/test_eventdev_func.c:491:
+			}
+			else {

WARNING:LONG_LINE_COMMENT: line over 90 characters
#553: FILE: app/test/test_eventdev_func.c:492:
+				//printf("iter %d -ENOSPC returned for new enq as expected.
", i);

WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#574: FILE: app/test/test_eventdev_func.c:513:
+		 * when enqueue is attempted in overload mode */

WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#608: FILE: app/test/test_eventdev_func.c:547:
+	 * iter 512 (the 513th pkt) is the first refused NEW packet */

ERROR:SPACING: space required before the open parenthesis '('
#618: FILE: app/test/test_eventdev_func.c:557:
+		if(i == THRES-1) {

ERROR:SPACING: space required before the open parenthesis '('
#619: FILE: app/test/test_eventdev_func.c:558:
+			if(err != -ENOSPC) {

WARNING:LONG_LINE: line over 90 characters
#620: FILE: app/test/test_eventdev_func.c:559:
+				printf("%d: overload trip didn't cause NEW pkt enq fail
", __LINE__);

ERROR:ELSE_AFTER_BRACE: else should follow close brace '}'
#623: FILE: app/test/test_eventdev_func.c:562:
+			}
+			else {

WARNING:LONG_LINE_COMMENT: line over 90 characters
#624: FILE: app/test/test_eventdev_func.c:563:
+				//printf("iter %d -ENOSPC returned for new enq as expected.
", i);

WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#651: FILE: app/test/test_eventdev_func.c:590:
+		 * when enqueue is attempted in overload mode */

WARNING:LONG_LINE_STRING: line over 90 characters
#656: FILE: app/test/test_eventdev_func.c:595:
+			printf("%d: warning failed to deq (iter = %d), ret %d. Dumping stats
",

ERROR:SPACING: space prohibited before that ',' (ctx:WxW)
#805: FILE: app/test/test_eventdev_func.c:744:
+			printf("%d: error generating pkt
" , __LINE__);
 			                                    ^

WARNING:ARRAY_SIZE: Prefer ARRAY_SIZE(flows)
#910: FILE: app/test/test_eventdev_func.c:849:
+#define PKT_NUM (sizeof(flows) / sizeof(flows[0]))

ERROR:SPACING: space required before the open parenthesis '('
#974: FILE: app/test/test_eventdev_func.c:913:
+	for(i = 0; i < 4; i++) {

WARNING:ARRAY_SIZE: Prefer ARRAY_SIZE(flows1)
#997: FILE: app/test/test_eventdev_func.c:936:
+#define PKT_NUM1 (sizeof(flows1) / sizeof(flows1[0]))

ERROR:SPACING: space required before the open parenthesis '('
#1105: FILE: app/test/test_eventdev_func.c:1044:
+			if(err != 1) {

ERROR:SPACING: space required before the open parenthesis '('
#1168: FILE: app/test/test_eventdev_func.c:1107:
+	for(i = 0; i < 8; i++) {

WARNING:LONG_LINE: line over 90 characters
#1211: FILE: app/test/test_eventdev_func.c:1150:
+				printf("%d: 
No schedules for seconds, deadlock
", __LINE__);

WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline
#1211: FILE: app/test/test_eventdev_func.c:1150:
+				printf("%d: 
No schedules for seconds, deadlock
", __LINE__);

WARNING:QUOTED_WHITESPACE_BEFORE_NEWLINE: unnecessary whitespace before a quoted newline
#1319: FILE: app/test/test_eventdev_func.c:1258:
+		printf("### Not enough cores for worker loopback test. 
");

WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#1323: FILE: app/test/test_eventdev_func.c:1262:
+	 * in the static eventdev_func_mempool variable. It is re-used on re-runs */

WARNING:UNNECESSARY_ELSE: else is not generally useful after a break or return
#1432: FILE: app/test/test_eventdev_unit.c:97:
+		return 0;
+	} else {

WARNING:TYPO_SPELLING: 'aquisition' may be misspelled - perhaps 'acquisition'?
#1864: FILE: app/test/test_eventdev_unit.c:529:
+		/* device aquisition and config */

total: 13 errors, 20 warnings, 1825 lines checked


More information about the test-report mailing list