[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