[dpdk-test-report] |WARNING| pw95997 [RFC v2 8/8] test: support ethdev

checkpatch at dpdk.org checkpatch at dpdk.org
Fri Jul 16 16:29:18 CEST 2021


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

_coding style issues_


ERROR:C99_COMMENTS: do not use C99 // comments
#1040: FILE: app/test/test_ethdev.c:889:
+	// rss_hf src_only and dst_only

ERROR:C99_COMMENTS: do not use C99 // comments
#1041: FILE: app/test/test_ethdev.c:890:
+	// eth_dev_tx_queue_config

ERROR:C99_COMMENTS: do not use C99 // comments
#1042: FILE: app/test/test_ethdev.c:891:
+	// eth_dev_rx_queue_config

ERROR:C99_COMMENTS: do not use C99 // comments
#1043: FILE: app/test/test_ethdev.c:892:
+	// RTE_ETHDEV_PROFILE_WITH_VTUNE

ERROR:C99_COMMENTS: do not use C99 // comments
#1044: FILE: app/test/test_ethdev.c:893:
+	// eth_dev_validate_offloads

ERROR:C99_COMMENTS: do not use C99 // comments
#1045: FILE: app/test/test_ethdev.c:894:
+	// restore config

ERROR:C99_COMMENTS: do not use C99 // comments
#1046: FILE: app/test/test_ethdev.c:895:
+	// restore mtu

ERROR:C99_COMMENTS: do not use C99 // comments
#1373: FILE: app/test/test_ethdev.c:1222:
+		//TODO: when rxseg.split.length is 0, API fails, check it

ERROR:C99_COMMENTS: do not use C99 // comments
#1394: FILE: app/test/test_ethdev.c:1243:
+		//TODO: Add more segment Rx tests based on other capabilities

WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#1507: FILE: app/test/test_ethdev.c:1356:
+		 * no rx_queue_release dev_ops */

WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#1540: FILE: app/test/test_ethdev.c:1389:
+		 * enabled in the port configure */

ERROR:C99_COMMENTS: do not use C99 // comments
#1549: FILE: app/test/test_ethdev.c:1398:
+		//TODO: LRO

WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#1747: FILE: lib/ethdev/ethdev_driver.h:38:
+ * */

total: 10 errors, 3 warnings, 1713 lines checked
Warning in app/test/test_ethdev.c:
Declaring a variable inside for()


More information about the test-report mailing list