automatic DPDK test reports
 help / color / mirror / Atom feed
From: checkpatch@dpdk.org
To: test-report@dpdk.org
Cc: "Min Hu (Connor)" <humin29@huawei.com>
Subject: |WARNING| pw111007 [PATCH v4] app/testpmd: fix port status of slave device
Date: Wed, 11 May 2022 04:16:06 +0200 (CEST)	[thread overview]
Message-ID: <20220511021606.34F191242ED@dpdk.org> (raw)
In-Reply-To: <20220511021434.40565-1-humin29@huawei.com>

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

_coding style issues_


WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#33: 
Subject: [PATCH v4] app/testpmd: fix port status of slave device

WARNING:TYPO_SPELLING: 'slaves' may be misspelled - perhaps 'secondaries'?
#59: 
Starting or stopping a bonded port also starts or stops all active slaves

WARNING:TYPO_SPELLING: 'slaves' may be misspelled - perhaps 'secondaries'?
#61: 
the port status of all slaves.

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#109: FILE: app/test-pmd/testpmd.c:604:
+change_bonding_slave_port_status(portid_t bond_pid, bool is_stop)

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#113: FILE: app/test-pmd/testpmd.c:608:
+	portid_t slave_pids[RTE_MAX_ETHPORTS];

WARNING:TYPO_SPELLING: 'slaves' may be misspelled - perhaps 'secondaries'?
#115: FILE: app/test-pmd/testpmd.c:610:
+	int num_slaves;

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#116: FILE: app/test-pmd/testpmd.c:611:
+	portid_t slave_pid;

WARNING:TYPO_SPELLING: 'slaves' may be misspelled - perhaps 'secondaries'?
#119: FILE: app/test-pmd/testpmd.c:614:
+	num_slaves = rte_eth_bond_slaves_get(bond_pid, slave_pids,

WARNING:TYPO_SPELLING: 'slaves' may be misspelled - perhaps 'secondaries'?
#119: FILE: app/test-pmd/testpmd.c:614:
+	num_slaves = rte_eth_bond_slaves_get(bond_pid, slave_pids,

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#119: FILE: app/test-pmd/testpmd.c:614:
+	num_slaves = rte_eth_bond_slaves_get(bond_pid, slave_pids,

WARNING:TYPO_SPELLING: 'slaves' may be misspelled - perhaps 'secondaries'?
#121: FILE: app/test-pmd/testpmd.c:616:
+	if (num_slaves < 0) {

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#122: FILE: app/test-pmd/testpmd.c:617:
+		fprintf(stderr, "Failed to get slave list for port = %u
",

WARNING:TYPO_SPELLING: 'slaves' may be misspelled - perhaps 'secondaries'?
#124: FILE: app/test-pmd/testpmd.c:619:
+		return num_slaves;

WARNING:TYPO_SPELLING: 'slaves' may be misspelled - perhaps 'secondaries'?
#127: FILE: app/test-pmd/testpmd.c:622:
+	for (i = 0; i < num_slaves; i++) {

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#128: FILE: app/test-pmd/testpmd.c:623:
+		slave_pid = slave_pids[i];

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#128: FILE: app/test-pmd/testpmd.c:623:
+		slave_pid = slave_pids[i];

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#129: FILE: app/test-pmd/testpmd.c:624:
+		port = &ports[slave_pid];

WARNING:TYPO_SPELLING: 'slaves' may be misspelled - perhaps 'secondaries'?
#155: FILE: app/test-pmd/testpmd.c:648:
+		 * Starting a bonded port also starts all slaves under the bonded

WARNING:TYPO_SPELLING: 'slaves' may be misspelled - perhaps 'secondaries'?
#157: FILE: app/test-pmd/testpmd.c:650:
+		 * port status of these slaves.

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#160: FILE: app/test-pmd/testpmd.c:653:
+			return change_bonding_slave_port_status(port_id, false);

WARNING:TYPO_SPELLING: 'slaves' may be misspelled - perhaps 'secondaries'?
#181: FILE: app/test-pmd/testpmd.c:672:
+		 * Stopping a bonded port also stops all slaves under the bonded

WARNING:TYPO_SPELLING: 'slaves' may be misspelled - perhaps 'secondaries'?
#183: FILE: app/test-pmd/testpmd.c:674:
+		 * port status of these slaves.

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#186: FILE: app/test-pmd/testpmd.c:677:
+			return change_bonding_slave_port_status(port_id, true);

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#200: FILE: app/test-pmd/testpmd.h:269:
+	uint8_t                 slave_flag : 1, /**< bonding slave port */

WARNING:TYPO_SPELLING: 'slave' may be misspelled - perhaps 'secondary'?
#200: FILE: app/test-pmd/testpmd.h:269:
+	uint8_t                 slave_flag : 1, /**< bonding slave port */

total: 0 errors, 25 warnings, 110 lines checked

       reply	other threads:[~2022-05-11  2:16 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20220511021434.40565-1-humin29@huawei.com>
2022-05-11  2:16 ` checkpatch [this message]
2022-05-11  3:59 ` |SUCCESS| " 0-day Robot

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20220511021606.34F191242ED@dpdk.org \
    --to=checkpatch@dpdk.org \
    --cc=humin29@huawei.com \
    --cc=test-report@dpdk.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).