[dpdk-test-report] |WARNING| pw86115 [PATCH v3 2/5] net/iavf_client: introduce iavf driver on vfio-user client

checkpatch at dpdk.org checkpatch at dpdk.org
Thu Jan 7 09:42:54 CET 2021


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

_coding style issues_


WARNING:STATIC_CONST_CHAR_ARRAY: static const char * array should probably be static const char * const
#192: FILE: drivers/net/iavf/iavf_client_ethdev.c:27:
+static const char *valid_args[] = {

CHECK:BRACES: Blank lines aren't necessary before a close brace '}'
#228: FILE: drivers/net/iavf/iavf_client_ethdev.c:63:
+
+	}

CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#295: FILE: drivers/net/iavf/iavf_client_ethdev.c:130:
+	eth_dev->data->mac_addrs = rte_zmalloc(

CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#307: FILE: drivers/net/iavf/iavf_client_ethdev.c:142:
+	if (!rte_is_valid_assigned_ether_addr(

WARNING:BRACES: braces {} are not necessary for single statement blocks
#409: FILE: drivers/net/iavf/iavf_client_ethdev.c:244:
+	if (ret) {
+		goto err;
+	}

WARNING:BRACES: braces {} are not necessary for single statement blocks
#414: FILE: drivers/net/iavf/iavf_client_ethdev.c:249:
+	if (ret) {
+		goto err;
+	}

CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#543: FILE: drivers/net/iavf/iavf_rxtx.c:604:
+		rxq->qrx_tail = client_vfio_user_get_bar_addr(

CHECK:OPEN_ENDED_LINE: Lines should not end with a '('
#559: FILE: drivers/net/iavf/iavf_rxtx.c:721:
+		txq->qtx_tail = client_vfio_user_get_bar_addr(

total: 0 errors, 3 warnings, 5 checks, 444 lines checked


More information about the test-report mailing list