From: checkpatch@dpdk.org
To: test-report@dpdk.org
Cc: Harry van Haaren <harry.van.haaren@intel.com>
Subject: [dpdk-test-report] |WARNING| [PATCH 5/7] test/eventdev: unit and functional tests
Date: Wed, 16 Nov 2016 19:01:34 +0100 (CET) [thread overview]
Message-ID: <20161116180134.5CA416AE0@dpdk.org> (raw)
In-Reply-To: <1479319207-130646-6-git-send-email-harry.van.haaren@intel.com>
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
parent reply other threads:[~2016-11-16 18:01 UTC|newest]
Thread overview: expand[flat|nested] mbox.gz Atom feed
[parent not found: <1479319207-130646-6-git-send-email-harry.van.haaren@intel.com>]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20161116180134.5CA416AE0@dpdk.org \
--to=checkpatch@dpdk.org \
--cc=harry.van.haaren@intel.com \
--cc=test-report@dpdk.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).