[dpdk-test-report] |WARNING| pw37507 [PATCH v5 2/3] app/testpmd: add command for queue setup

checkpatch at dpdk.org checkpatch at dpdk.org
Sat Apr 7 21:42:04 CEST 2018


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

_coding style issues_


WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line)
#21: 
Cc: dev at dpdk.org; Xing, Beilei <beilei.xing at intel.com>; Wu, Jingjing <jingjing.wu at intel.com>; Lu, Wenzhuo <wenzhuo.lu at intel.com>; Zhang, Qi Z <qi.z.zhang at intel.com>

ERROR:TRAILING_WHITESPACE: trailing whitespace
#67: FILE: app/test-pmd/cmdline.c:16038:
+/* Common result structure for queue setup */ struct $

ERROR:TRAILING_WHITESPACE: trailing whitespace
#78: FILE: app/test-pmd/cmdline.c:16049:
+/* Common CLI fields for queue setup */ cmdline_parse_token_string_t $

ERROR:TRAILING_WHITESPACE: trailing whitespace
#80: FILE: app/test-pmd/cmdline.c:16051:
+^ITOKEN_STRING_INITIALIZER(struct cmd_queue_setup_result, queue, $

ERROR:TRAILING_WHITESPACE: trailing whitespace
#82: FILE: app/test-pmd/cmdline.c:16053:
+^ITOKEN_STRING_INITIALIZER(struct cmd_queue_setup_result, setup, $

ERROR:TRAILING_WHITESPACE: trailing whitespace
#84: FILE: app/test-pmd/cmdline.c:16055:
+^ITOKEN_STRING_INITIALIZER(struct cmd_queue_setup_result, rxtx, $

ERROR:TRAILING_WHITESPACE: trailing whitespace
#86: FILE: app/test-pmd/cmdline.c:16057:
+^ITOKEN_NUM_INITIALIZER(struct cmd_queue_setup_result, port_id, UINT16); $

ERROR:TRAILING_WHITESPACE: trailing whitespace
#88: FILE: app/test-pmd/cmdline.c:16059:
+^ITOKEN_NUM_INITIALIZER(struct cmd_queue_setup_result, queue_idx, $

ERROR:TRAILING_WHITESPACE: trailing whitespace
#90: FILE: app/test-pmd/cmdline.c:16061:
+^ITOKEN_NUM_INITIALIZER(struct cmd_queue_setup_result, ring_size, $

ERROR:TRAILING_WHITESPACE: trailing whitespace
#92: FILE: app/test-pmd/cmdline.c:16063:
+^ITOKEN_NUM_INITIALIZER(struct cmd_queue_setup_result, offloads, $

WARNING:SUSPECT_CODE_INDENT: suspect code indent for conditional statements (8, 0)
#114: FILE: app/test-pmd/cmdline.c:16085:
+	if (rx && res->ring_size <= rx_free_thresh) {
[...]


ERROR:CORRUPTED_PATCH: patch seems to be corrupt (line wrapped?)
#117: FILE: app/test-pmd/cmdline.c:16087:
[Rami Rosen] Nitpick: shouldn't it be: must > rx_free_thresh:  ? 

ERROR:TRAILING_WHITESPACE: trailing whitespace
#219: FILE: doc/guides/testpmd_app_ug/testpmd_funcs.rst:1452:
+   testpmd> queue setup (rx|tx) (port_id) (queue_idx) (ring_size) $

total: 11 errors, 2 warnings, 161 lines checked


More information about the test-report mailing list