automatic DPDK test reports
 help / color / mirror / Atom feed
* [dpdk-test-report] |WARNING| pw66354 [PATCH 4/4] if_proxy: add example application
       [not found] <20200306164104.15528-5-aostruszka@marvell.com>
@ 2020-03-06 16:42 ` checkpatch
  2020-03-06 18:06 ` [dpdk-test-report] |WARNING| pw66354 " 0-day Robot
  1 sibling, 0 replies; 2+ messages in thread
From: checkpatch @ 2020-03-06 16:42 UTC (permalink / raw)
  To: test-report; +Cc: Andrzej Ostruszka

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

_coding style issues_


WARNING:AVOID_EXTERNS: externs should be avoided in .c files
#296: FILE: examples/l3fwd-ifpx/l3fwd.c:84:
+RTE_STD_C11

WARNING:IF_0: Consider removing the code enclosed by this #if 0 and its #endif
#398: FILE: examples/l3fwd-ifpx/l3fwd.c:186:
+#if 0 // FIXME - either remove this or add IP fragmentation

WARNING:IF_0: Consider removing the code enclosed by this #if 0 and its #endif
#611: FILE: examples/l3fwd-ifpx/l3fwd.c:399:
+#if 0

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#632: FILE: examples/l3fwd-ifpx/l3fwd.c:420:
+	unsigned i;

WARNING:IF_0: Consider removing the code enclosed by this #if 0 and its #endif
#634: FILE: examples/l3fwd-ifpx/l3fwd.c:422:
+#if 0

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#683: FILE: examples/l3fwd-ifpx/l3fwd.c:471:
+	unsigned i;

ERROR:POINTER_LOCATION: "(foo**)" should be "(foo **)"
#1062: FILE: examples/l3fwd-ifpx/l3fwd.c:850:
+	while (rte_ring_dequeue(lconf->ev_queue, (void**)&ev) == 0) {

ERROR:POINTER_LOCATION: "(foo*)" should be "(foo *)"
#1110: FILE: examples/l3fwd-ifpx/l3fwd.c:898:
+	return rte_jhash_1word(*(const uint32_t*)key, init_val);

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#1247: FILE: examples/l3fwd-ifpx/l3fwd.c:1035:
+	unsigned lcore_id;

ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#1595: FILE: examples/l3fwd-ifpx/main.c:150:
+		if ((socketid = rte_lcore_to_socket_id(lcore)) != 0) {

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#1669: FILE: examples/l3fwd-ifpx/main.c:224:
+				(unsigned)nb_rx_queue + 1, (unsigned)lcore);

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#1669: FILE: examples/l3fwd-ifpx/main.c:224:
+				(unsigned)nb_rx_queue + 1, (unsigned)lcore);

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#1739: FILE: examples/l3fwd-ifpx/main.c:294:
+	unsigned size;

ERROR:SPACING: space required after that ',' (ctx:VxV)
#1743: FILE: examples/l3fwd-ifpx/main.c:298:
+	while ((p = strchr(p0,'(')) != NULL) {
 	                     ^

ERROR:SPACING: space required after that ',' (ctx:VxV)
#1745: FILE: examples/l3fwd-ifpx/main.c:300:
+		if((p0 = strchr(p,')')) == NULL)
 		                 ^

ERROR:SPACING: space required before the open parenthesis '('
#1745: FILE: examples/l3fwd-ifpx/main.c:300:
+		if((p0 = strchr(p,')')) == NULL)

ERROR:ASSIGN_IN_IF: do not use assignment in if condition
#1745: FILE: examples/l3fwd-ifpx/main.c:300:
+		if((p0 = strchr(p,')')) == NULL)

ERROR:SPACING: space required before the open parenthesis '('
#1749: FILE: examples/l3fwd-ifpx/main.c:304:
+		if(size >= sizeof(s))

ERROR:SPACING: space required before the open brace '{'
#1755: FILE: examples/l3fwd-ifpx/main.c:310:
+		for (i = 0; i < _NUM_FLD; i++){

WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#1794: FILE: examples/l3fwd-ifpx/main.c:349:
+	 * conflict with short options */

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#1818: FILE: examples/l3fwd-ifpx/main.c:373:
+	(unsigned)8192)

ERROR:CODE_INDENT: code indent should use tabs where possible
#1890: FILE: examples/l3fwd-ifpx/main.c:445:
+^I^I        "Signal %d received, preparing to exit...
", signum);$

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#1926: FILE: examples/l3fwd-ifpx/main.c:481:
+	unsigned nb_ports;

WARNING:UNSPECIFIED_INT: Prefer 'unsigned int' to bare use of 'unsigned'
#1929: FILE: examples/l3fwd-ifpx/main.c:484:
+	unsigned lcore_id;

ERROR:SPACING: space prohibited before that close parenthesis ')'
#2072: FILE: examples/l3fwd-ifpx/main.c:627:
+			lcore_id );

ERROR:SPACING: space required before the open parenthesis '('
#2074: FILE: examples/l3fwd-ifpx/main.c:629:
+		for(queue = 0; queue < lconf->n_rx_queue; ++queue) {

ERROR:CODE_INDENT: code indent should use tabs where possible
#2091: FILE: examples/l3fwd-ifpx/main.c:646:
+^I^I^I                             nb_rxd, SOCKET_ID_ANY,$

ERROR:CODE_INDENT: code indent should use tabs where possible
#2092: FILE: examples/l3fwd-ifpx/main.c:647:
+^I^I^I                             &rxq_conf,$

ERROR:CODE_INDENT: code indent should use tabs where possible
#2093: FILE: examples/l3fwd-ifpx/main.c:648:
+^I^I^I                             pktmbuf_pool);$

WARNING:BRACES: braces {} are not necessary for single statement blocks
#2105: FILE: examples/l3fwd-ifpx/main.c:660:
+		if ((enabled_port_mask & (1 << portid)) == 0) {
+			continue;
+		}

total: 15 errors, 15 warnings, 2043 lines checked

^ permalink raw reply	[flat|nested] 2+ messages in thread

* [dpdk-test-report] |WARNING| pw66354 if_proxy: add example application
       [not found] <20200306164104.15528-5-aostruszka@marvell.com>
  2020-03-06 16:42 ` [dpdk-test-report] |WARNING| pw66354 [PATCH 4/4] if_proxy: add example application checkpatch
@ 2020-03-06 18:06 ` 0-day Robot
  1 sibling, 0 replies; 2+ messages in thread
From: 0-day Robot @ 2020-03-06 18:06 UTC (permalink / raw)
  To: test-report; +Cc: aostruszka, robot

From: robot@bytheb.org

Test-Label: travis-robot
Test-Status: WARNING
http://dpdk.org/patch/66354

_Travis build: failed_
Build URL: https://travis-ci.com/ovsrobot/dpdk/builds/152184372

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2020-03-06 18:08 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <20200306164104.15528-5-aostruszka@marvell.com>
2020-03-06 16:42 ` [dpdk-test-report] |WARNING| pw66354 [PATCH 4/4] if_proxy: add example application checkpatch
2020-03-06 18:06 ` [dpdk-test-report] |WARNING| pw66354 " 0-day Robot

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).