[dpdk-dev] [PATCH 01/10] i40e: spellin fixes

[dpdk-dev] [PATCH 01/10] i40e: spellin fixes

[Stephen Hemminger]

From: Stephen Hemminger <shemming@brocade.com>

Spelling errors found with code spell.

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
 lib/librte_pmd_i40e/i40e_ethdev.c | 8 ++++----
 lib/librte_pmd_i40e/i40e_ethdev.h | 2 +-
 lib/librte_pmd_i40e/i40e_rxtx.c   | 6 +++---
 3 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/lib/librte_pmd_i40e/i40e_ethdev.c b/lib/librte_pmd_i40e/i40e_ethdev.c
index b47a3d2..07efca5 100644
--- a/lib/librte_pmd_i40e/i40e_ethdev.c
+++ b/lib/librte_pmd_i40e/i40e_ethdev.c
@@ -3810,16 +3810,16 @@ i40e_dev_handle_vfr_event(struct rte_eth_dev *dev)
 		index = abs_vf_id / I40E_UINT32_BIT_SIZE;
 		offset = abs_vf_id % I40E_UINT32_BIT_SIZE;
 		val = I40E_READ_REG(hw, I40E_GLGEN_VFLRSTAT(index));
-		/* VFR event occured */
+		/* VFR event occurred */
 		if (val & (0x1 << offset)) {
 			int ret;
 
 			/* Clear the event first */
 			I40E_WRITE_REG(hw, I40E_GLGEN_VFLRSTAT(index),
 							(0x1 << offset));
-			PMD_DRV_LOG(INFO, "VF %u reset occured", abs_vf_id);
+			PMD_DRV_LOG(INFO, "VF %u reset occurred", abs_vf_id);
 			/**
-			 * Only notify a VF reset event occured,
+			 * Only notify a VF reset event occurred,
 			 * don't trigger another SW reset
 			 */
 			ret = i40e_pf_host_vf_reset(&pf->vfs[i], 0);
@@ -5007,7 +5007,7 @@ i40e_pf_config_rss(struct i40e_pf *pf)
 
 	/*
 	 * If both VMDQ and RSS enabled, not all of PF queues are configured.
-	 * It's necessary to calulate the actual PF queues that are configured.
+	 * It's necessary to calculate the actual PF queues that are configured.
 	 */
 	if (pf->dev_data->dev_conf.rxmode.mq_mode & ETH_MQ_RX_VMDQ_FLAG) {
 		num = i40e_pf_calc_configured_queues_num(pf);
diff --git a/lib/librte_pmd_i40e/i40e_ethdev.h b/lib/librte_pmd_i40e/i40e_ethdev.h
index f913ea9..dc870d2 100644
--- a/lib/librte_pmd_i40e/i40e_ethdev.h
+++ b/lib/librte_pmd_i40e/i40e_ethdev.h
@@ -273,7 +273,7 @@ enum I40E_VF_STATE {
 struct i40e_pf_vf {
 	struct i40e_pf *pf;
 	struct i40e_vsi *vsi;
-	enum I40E_VF_STATE state; /* The number of queue pairs availiable */
+	enum I40E_VF_STATE state; /* The number of queue pairs available */
 	uint16_t vf_idx; /* VF index in pf->vfs */
 	uint16_t lan_nb_qps; /* Actual queues allocated */
 	uint16_t reset_cnt; /* Total vf reset times */
diff --git a/lib/librte_pmd_i40e/i40e_rxtx.c b/lib/librte_pmd_i40e/i40e_rxtx.c
index 2beae3c..56125cf 100644
--- a/lib/librte_pmd_i40e/i40e_rxtx.c
+++ b/lib/librte_pmd_i40e/i40e_rxtx.c
@@ -1637,7 +1637,7 @@ i40e_dev_rx_queue_stop(struct rte_eth_dev *dev, uint16_t rx_queue_id)
 		rxq = dev->data->rx_queues[rx_queue_id];
 
 		/*
-		* rx_queue_id is queue id aplication refers to, while
+		* rx_queue_id is queue id application refers to, while
 		* rxq->reg_idx is the real queue index.
 		*/
 		err = i40e_switch_rx_queue(hw, rxq->reg_idx, FALSE);
@@ -1667,7 +1667,7 @@ i40e_dev_tx_queue_start(struct rte_eth_dev *dev, uint16_t tx_queue_id)
 		txq = dev->data->tx_queues[tx_queue_id];
 
 		/*
-		* tx_queue_id is queue id aplication refers to, while
+		* tx_queue_id is queue id application refers to, while
 		* rxq->reg_idx is the real queue index.
 		*/
 		err = i40e_switch_tx_queue(hw, txq->reg_idx, TRUE);
@@ -1690,7 +1690,7 @@ i40e_dev_tx_queue_stop(struct rte_eth_dev *dev, uint16_t tx_queue_id)
 		txq = dev->data->tx_queues[tx_queue_id];
 
 		/*
-		* tx_queue_id is queue id aplication refers to, while
+		* tx_queue_id is queue id application refers to, while
 		* txq->reg_idx is the real queue index.
 		*/
 		err = i40e_switch_tx_queue(hw, txq->reg_idx, FALSE);
-- 
2.1.4

[dpdk-dev] [PATCH 02/10] bond: fix spelling errors

[Stephen Hemminger]

From: Stephen Hemminger <shemming@brocade.com>

Bad spelling in comments found by codespell

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
 lib/librte_pmd_bond/rte_eth_bond_8023ad.c  | 2 +-
 lib/librte_pmd_bond/rte_eth_bond_private.h | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/lib/librte_pmd_bond/rte_eth_bond_8023ad.c b/lib/librte_pmd_bond/rte_eth_bond_8023ad.c
index f1cf81a..dbffc30 100644
--- a/lib/librte_pmd_bond/rte_eth_bond_8023ad.c
+++ b/lib/librte_pmd_bond/rte_eth_bond_8023ad.c
@@ -896,7 +896,7 @@ bond_mode_8023ad_activate_slave(struct rte_eth_dev *bond_dev, uint8_t slave_id)
 		sizeof(struct rte_pktmbuf_pool_private), rte_pktmbuf_pool_init,
 		NULL, rte_pktmbuf_init, NULL, socket_id, MEMPOOL_F_NO_SPREAD);
 
-	/* Any memory allocation failure in initalization is critical because
+	/* Any memory allocation failure in initialization is critical because
 	 * resources can't be free, so reinitialization is impossible. */
 	if (port->mbuf_pool == NULL) {
 		rte_panic("Slave %u: Failed to create memory pool '%s': %s\n",
diff --git a/lib/librte_pmd_bond/rte_eth_bond_private.h b/lib/librte_pmd_bond/rte_eth_bond_private.h
index e01e66b..b47c9a6 100644
--- a/lib/librte_pmd_bond/rte_eth_bond_private.h
+++ b/lib/librte_pmd_bond/rte_eth_bond_private.h
@@ -165,7 +165,7 @@ extern struct eth_dev_ops default_dev_ops;
 int
 valid_bonded_ethdev(struct rte_eth_dev *eth_dev);
 
-/* Search given slave array to find possition of given id.
+/* Search given slave array to find position of given id.
  * Return slave pos or slaves_count if not found. */
 static inline uint8_t
 find_slave_by_id(uint8_t *slaves, uint8_t slaves_count, uint8_t slave_id) {
-- 
2.1.4

[dpdk-dev] [PATCH 03/10] eal: fix spelling error

[Stephen Hemminger]

From: Stephen Hemminger <shemming@brocade.com>

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
 lib/librte_eal/common/include/rte_alarm.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/librte_eal/common/include/rte_alarm.h b/lib/librte_eal/common/include/rte_alarm.h
index 4012cd6..c275be1 100644
--- a/lib/librte_eal/common/include/rte_alarm.h
+++ b/lib/librte_eal/common/include/rte_alarm.h
@@ -91,7 +91,7 @@ int rte_eal_alarm_set(uint64_t us, rte_eal_alarm_callback cb, void *cb_arg);
  *      the number of canceled alarm callback functions
  *    - value greater or equal 0 and rte_errno set to EINPROGRESS, at least one
  *      alarm could not be canceled because cancellation was requested from alarm
- *      callback context. Returned value is the number of succesfuly canceled
+ *      callback context. Returned value is the number of successfully canceled
  *      alarm callbacks
  *    -  0 and rte_errno set to ENOENT - no alarm found
  *    - -1 and rte_errno set to EINVAL - invalid parameter (NULL callback)
-- 
2.1.4

[dpdk-dev] [PATCH 04/10] ip_frag: fix spelling errors

[Stephen Hemminger]

From: Stephen Hemminger <shemming@brocade.com>

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
 lib/librte_ip_frag/rte_ip_frag.h         | 4 ++--
 lib/librte_ip_frag/rte_ipv4_reassembly.c | 2 +-
 lib/librte_ip_frag/rte_ipv6_reassembly.c | 2 +-
 3 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/lib/librte_ip_frag/rte_ip_frag.h b/lib/librte_ip_frag/rte_ip_frag.h
index 1083d44..6a8f215 100644
--- a/lib/librte_ip_frag/rte_ip_frag.h
+++ b/lib/librte_ip_frag/rte_ip_frag.h
@@ -229,7 +229,7 @@ rte_ipv6_fragment_packet(struct rte_mbuf *pkt_in,
  *   Pointer to the IPv6 fragment extension header.
  * @return
  *   Pointer to mbuf for reassembled packet, or NULL if:
- *   - an error occured.
+ *   - an error occurred.
  *   - not all fragments of the packet are collected yet.
  */
 struct rte_mbuf *rte_ipv6_frag_reassemble_packet(struct rte_ip_frag_tbl *tbl,
@@ -305,7 +305,7 @@ int32_t rte_ipv4_fragment_packet(struct rte_mbuf *pkt_in,
  *   Pointer to the IPV4 header inside the fragment.
  * @return
  *   Pointer to mbuf for reassebled packet, or NULL if:
- *   - an error occured.
+ *   - an error occurred.
  *   - not all fragments of the packet are collected yet.
  */
 struct rte_mbuf * rte_ipv4_frag_reassemble_packet(struct rte_ip_frag_tbl *tbl,
diff --git a/lib/librte_ip_frag/rte_ipv4_reassembly.c b/lib/librte_ip_frag/rte_ipv4_reassembly.c
index 0b8ceeb..fe1d5c6 100644
--- a/lib/librte_ip_frag/rte_ipv4_reassembly.c
+++ b/lib/librte_ip_frag/rte_ipv4_reassembly.c
@@ -111,7 +111,7 @@ ipv4_frag_reassemble(const struct ip_frag_pkt *fp)
  *   Pointer to the IPV4 header inside the fragment.
  * @return
  *   Pointer to mbuf for reassebled packet, or NULL if:
- *   - an error occured.
+ *   - an error occurred.
  *   - not all fragments of the packet are collected yet.
  */
 struct rte_mbuf *
diff --git a/lib/librte_ip_frag/rte_ipv6_reassembly.c b/lib/librte_ip_frag/rte_ipv6_reassembly.c
index 71cf721..0b80dce 100644
--- a/lib/librte_ip_frag/rte_ipv6_reassembly.c
+++ b/lib/librte_ip_frag/rte_ipv6_reassembly.c
@@ -147,7 +147,7 @@ ipv6_frag_reassemble(const struct ip_frag_pkt *fp)
  *   Pointer to the IPV6 fragment extension header.
  * @return
  *   Pointer to mbuf for reassembled packet, or NULL if:
- *   - an error occured.
+ *   - an error occurred.
  *   - not all fragments of the packet are collected yet.
  */
 #define MORE_FRAGS(x) (((x) & 0x100) >> 8)
-- 
2.1.4

[dpdk-dev] [PATCH 05/10] app: fix spelling errors

[Stephen Hemminger]

From: Stephen Hemminger <shemming@brocade.com>

Errors from codespell

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
 app/test/test_cmdline_cirbuf.c  |  2 +-
 app/test/test_distributor.c     |  2 +-
 app/test/test_eal_flags.c       |  4 ++--
 app/test/test_func_reentrancy.c |  4 ++--
 app/test/test_hash.c            | 18 +++++++++---------
 app/test/test_interrupts.c      |  6 +++---
 app/test/test_ivshmem.c         |  2 +-
 app/test/test_link_bonding.c    |  2 +-
 app/test/test_malloc.c          |  2 +-
 app/test/test_mbuf.c            |  2 +-
 app/test/test_spinlock.c        |  2 +-
 11 files changed, 23 insertions(+), 23 deletions(-)

diff --git a/app/test/test_cmdline_cirbuf.c b/app/test/test_cmdline_cirbuf.c
index 87f83cc..2c32145 100644
--- a/app/test/test_cmdline_cirbuf.c
+++ b/app/test/test_cmdline_cirbuf.c
@@ -45,7 +45,7 @@
 #define CIRBUF_STR_HEAD " HEAD"
 #define CIRBUF_STR_TAIL "TAIL"
 
-/* miscelaneous tests - they make bullseye happy */
+/* miscellaneous tests - they make bullseye happy */
 static int
 test_cirbuf_string_misc(void)
 {
diff --git a/app/test/test_distributor.c b/app/test/test_distributor.c
index 9e8c06d..5c22f84 100644
--- a/app/test/test_distributor.c
+++ b/app/test/test_distributor.c
@@ -96,7 +96,7 @@ handle_work(void *arg)
 /* do basic sanity testing of the distributor. This test tests the following:
  * - send 32 packets through distributor with the same tag and ensure they
  *   all go to the one worker
- * - send 32 packets throught the distributor with two different tags and
+ * - send 32 packets through the distributor with two different tags and
  *   verify that they go equally to two different workers.
  * - send 32 packets with different tags through the distributors and
  *   just verify we get all packets back.
diff --git a/app/test/test_eal_flags.c b/app/test/test_eal_flags.c
index 0a8269c..738bbe4 100644
--- a/app/test/test_eal_flags.c
+++ b/app/test/test_eal_flags.c
@@ -747,7 +747,7 @@ test_dom0_misc_flags(void)
 
 	/* check that some general flags don't prevent things from working.
 	 * All cases, apart from the first, app should run.
-	 * No futher testing of output done.
+	 * No further testing of output done.
 	 */
 	/* sanity check - failure with invalid option */
 	const char *argv0[] = {prgname, prefix, mp_flag, "-c", "1", "--invalid-opt"};
@@ -855,7 +855,7 @@ test_misc_flags(void)
 
 	/* check that some general flags don't prevent things from working.
 	 * All cases, apart from the first, app should run.
-	 * No futher testing of output done.
+	 * No further testing of output done.
 	 */
 	/* sanity check - failure with invalid option */
 	const char *argv0[] = {prgname, prefix, mp_flag, "-c", "1", "--invalid-opt"};
diff --git a/app/test/test_func_reentrancy.c b/app/test/test_func_reentrancy.c
index 45928fa..87121da 100644
--- a/app/test/test_func_reentrancy.c
+++ b/app/test/test_func_reentrancy.c
@@ -137,7 +137,7 @@ ring_create_lookup(__attribute__((unused)) void *arg)
 			return -1;
 	}
 
-	/* verify all ring created sucessful */
+	/* verify all ring created successful */
 	for (i = 0; i < MAX_ITER_TIMES; i++) {
 		snprintf(ring_name, sizeof(ring_name), "fr_test_%d_%d", lcore_self, i);
 		if (rte_ring_lookup(ring_name) == NULL)
@@ -191,7 +191,7 @@ mempool_create_lookup(__attribute__((unused)) void *arg)
 			return -1;
 	}
 
-	/* verify all ring created sucessful */
+	/* verify all ring created successful */
 	for (i = 0; i < MAX_ITER_TIMES; i++) {
 		snprintf(mempool_name, sizeof(mempool_name), "fr_test_%d_%d", lcore_self, i);
 		if (rte_mempool_lookup(mempool_name) == NULL)
diff --git a/app/test/test_hash.c b/app/test/test_hash.c
index 178ec3f..ce7c533 100644
--- a/app/test/test_hash.c
+++ b/app/test/test_hash.c
@@ -953,7 +953,7 @@ static int test_hash_creation_with_bad_parameters(void)
 	handle = rte_hash_create(NULL);
 	if (handle != NULL) {
 		rte_hash_free(handle);
-		printf("Impossible creating hash sucessfully without any parameter\n");
+		printf("Impossible creating hash successfully without any parameter\n");
 		return -1;
 	}
 
@@ -963,7 +963,7 @@ static int test_hash_creation_with_bad_parameters(void)
 	handle = rte_hash_create(&params);
 	if (handle != NULL) {
 		rte_hash_free(handle);
-		printf("Impossible creating hash sucessfully with entries in parameter exceeded\n");
+		printf("Impossible creating hash successfully with entries in parameter exceeded\n");
 		return -1;
 	}
 
@@ -973,7 +973,7 @@ static int test_hash_creation_with_bad_parameters(void)
 	handle = rte_hash_create(&params);
 	if (handle != NULL) {
 		rte_hash_free(handle);
-		printf("Impossible creating hash sucessfully with bucket_entries in parameter exceeded\n");
+		printf("Impossible creating hash successfully with bucket_entries in parameter exceeded\n");
 		return -1;
 	}
 
@@ -983,7 +983,7 @@ static int test_hash_creation_with_bad_parameters(void)
 	handle = rte_hash_create(&params);
 	if (handle != NULL) {
 		rte_hash_free(handle);
-		printf("Impossible creating hash sucessfully if entries less than bucket_entries in parameter\n");
+		printf("Impossible creating hash successfully if entries less than bucket_entries in parameter\n");
 		return -1;
 	}
 
@@ -993,7 +993,7 @@ static int test_hash_creation_with_bad_parameters(void)
 	handle = rte_hash_create(&params);
 	if (handle != NULL) {
 		rte_hash_free(handle);
-		printf("Impossible creating hash sucessfully if entries in parameter is not power of 2\n");
+		printf("Impossible creating hash successfully if entries in parameter is not power of 2\n");
 		return -1;
 	}
 
@@ -1003,7 +1003,7 @@ static int test_hash_creation_with_bad_parameters(void)
 	handle = rte_hash_create(&params);
 	if (handle != NULL) {
 		rte_hash_free(handle);
-		printf("Impossible creating hash sucessfully if bucket_entries in parameter is not power of 2\n");
+		printf("Impossible creating hash successfully if bucket_entries in parameter is not power of 2\n");
 		return -1;
 	}
 
@@ -1013,7 +1013,7 @@ static int test_hash_creation_with_bad_parameters(void)
 	handle = rte_hash_create(&params);
 	if (handle != NULL) {
 		rte_hash_free(handle);
-		printf("Impossible creating hash sucessfully if key_len in parameter is zero\n");
+		printf("Impossible creating hash successfully if key_len in parameter is zero\n");
 		return -1;
 	}
 
@@ -1023,7 +1023,7 @@ static int test_hash_creation_with_bad_parameters(void)
 	handle = rte_hash_create(&params);
 	if (handle != NULL) {
 		rte_hash_free(handle);
-		printf("Impossible creating hash sucessfully if key_len is greater than the maximum\n");
+		printf("Impossible creating hash successfully if key_len is greater than the maximum\n");
 		return -1;
 	}
 
@@ -1033,7 +1033,7 @@ static int test_hash_creation_with_bad_parameters(void)
 	handle = rte_hash_create(&params);
 	if (handle != NULL) {
 		rte_hash_free(handle);
-		printf("Impossible creating hash sucessfully with invalid socket\n");
+		printf("Impossible creating hash successfully with invalid socket\n");
 		return -1;
 	}
 
diff --git a/app/test/test_interrupts.c b/app/test/test_interrupts.c
index 6e3dec3..e7f4599 100644
--- a/app/test/test_interrupts.c
+++ b/app/test/test_interrupts.c
@@ -408,7 +408,7 @@ test_interrupt(void)
 
 	printf("Check unknown valid interrupt full path\n");
 	if (test_interrupt_full_path_check(TEST_INTERRUPT_HANDLE_VALID) < 0) {
-		printf("failure occured during checking unknown valid "
+		printf("failure occurred during checking unknown valid "
 						"interrupt full path\n");
 		goto out;
 	}
@@ -416,7 +416,7 @@ test_interrupt(void)
 	printf("Check valid UIO interrupt full path\n");
 	if (test_interrupt_full_path_check(TEST_INTERRUPT_HANDLE_VALID_UIO)
 									< 0) {
-		printf("failure occured during checking valid UIO interrupt "
+		printf("failure occurred during checking valid UIO interrupt "
 								"full path\n");
 		goto out;
 	}
@@ -424,7 +424,7 @@ test_interrupt(void)
 	printf("Check valid alarm interrupt full path\n");
 	if (test_interrupt_full_path_check(TEST_INTERRUPT_HANDLE_VALID_ALARM)
 									< 0) {
-		printf("failure occured during checking valid alarm "
+		printf("failure occurred during checking valid alarm "
 						"interrupt full path\n");
 		goto out;
 	}
diff --git a/app/test/test_ivshmem.c b/app/test/test_ivshmem.c
index 4e61488..bc5417c 100644
--- a/app/test/test_ivshmem.c
+++ b/app/test/test_ivshmem.c
@@ -243,7 +243,7 @@ test_ivshmem_api_test(void)
 
 	ASSERT(rte_ivshmem_metadata_cmdline_generate(buf, sizeof(buf),
 			NONEXISTENT_METADATA) < 0,
-			"Writen command line for nonexistent metadata");
+			"Written command line for nonexistent metadata");
 	ASSERT(strnlen(buf, sizeof(buf)) == 0, "Buffer is not empty");
 
 	/* add stuff to config */
diff --git a/app/test/test_link_bonding.c b/app/test/test_link_bonding.c
index 4523de6..3e7633d 100644
--- a/app/test/test_link_bonding.c
+++ b/app/test/test_link_bonding.c
@@ -4366,7 +4366,7 @@ test_tlb_verify_mac_assignment(void)
 	/* Set explicit MAC address */
 	TEST_ASSERT_SUCCESS(rte_eth_bond_mac_address_set(
 			test_params->bonded_port_id, (struct ether_addr *)bonded_mac),
-			"failed to set MAC addres");
+			"failed to set MAC address");
 
 	rte_eth_macaddr_get(test_params->bonded_port_id, &read_mac_addr);
 	TEST_ASSERT_SUCCESS(memcmp(&bonded_mac, &read_mac_addr,
diff --git a/app/test/test_malloc.c b/app/test/test_malloc.c
index 8c53132..ebab582 100644
--- a/app/test/test_malloc.c
+++ b/app/test/test_malloc.c
@@ -813,7 +813,7 @@ err_return:
 	return -1;
 }
 
-/* Check if memory is avilable on a specific socket */
+/* Check if memory is available on a specific socket */
 static int
 is_mem_on_socket(int32_t socket)
 {
diff --git a/app/test/test_mbuf.c b/app/test/test_mbuf.c
index e86ba22..957119a 100644
--- a/app/test/test_mbuf.c
+++ b/app/test/test_mbuf.c
@@ -506,7 +506,7 @@ test_pktmbuf_free_segment(void)
  * Stress test for rte_mbuf atomic refcnt.
  * Implies that:
  * RTE_MBUF_REFCNT and RTE_MBUF_REFCNT_ATOMIC are both defined.
- * For more efficency, recomended to run with RTE_LIBRTE_MBUF_DEBUG defined.
+ * For more efficiency, recommended to run with RTE_LIBRTE_MBUF_DEBUG defined.
  */
 
 #if defined RTE_MBUF_REFCNT  && defined RTE_MBUF_REFCNT_ATOMIC
diff --git a/app/test/test_spinlock.c b/app/test/test_spinlock.c
index c95b0c0..29cc7aa 100644
--- a/app/test/test_spinlock.c
+++ b/app/test/test_spinlock.c
@@ -203,7 +203,7 @@ test_spinlock_perf(void)
 
 /*
  * Use rte_spinlock_trylock() to trylock a spinlock object,
- * If it could not lock the object sucessfully, it would
+ * If it could not lock the object successfully, it would
  * return immediately and the variable of "count" would be
  * increased by one per times. the value of "count" could be
  * checked as the result later.
-- 
2.1.4

[dpdk-dev] [PATCH 06/10] examples: fix spelling errors

[Stephen Hemminger]

From: Stephen Hemminger <shemming@brocade.com>

Run codespell over examples

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
 examples/dpdk_qat/crypto.c                 |  2 +-
 examples/l3fwd-power/main.c                |  2 +-
 examples/l3fwd/main.c                      |  2 +-
 examples/multi_process/l2fwd_fork/flib.h   |  2 +-
 examples/netmap_compat/lib/compat_netmap.c |  2 +-
 examples/vhost/main.c                      | 12 ++++++------
 examples/vhost_xen/main.c                  |  2 +-
 examples/vhost_xen/xenstore_parse.c        |  4 ++--
 8 files changed, 14 insertions(+), 14 deletions(-)

diff --git a/examples/dpdk_qat/crypto.c b/examples/dpdk_qat/crypto.c
index f0471e0..b3955f7 100644
--- a/examples/dpdk_qat/crypto.c
+++ b/examples/dpdk_qat/crypto.c
@@ -720,7 +720,7 @@ per_core_crypto_init(uint32_t lcore_id)
 	}
 
 	/*
-	 * Set the address translation callback for virtual to physcial address
+	 * Set the address translation callback for virtual to physical address
 	 * mapping. This will be called by the QAT driver during initialisation only.
 	 */
 	status = cpaCySetAddressTranslation(qaCoreConf[lcore_id].instanceHandle,
diff --git a/examples/l3fwd-power/main.c b/examples/l3fwd-power/main.c
index f6b55b9..cc8bc1e 100644
--- a/examples/l3fwd-power/main.c
+++ b/examples/l3fwd-power/main.c
@@ -935,7 +935,7 @@ main_loop(__attribute__((unused)) void *dummy)
  				 */
 				rte_delay_us(lcore_idle_hint);
 			else
-				/* long sleep force runing thread to suspend */
+				/* long sleep force running thread to suspend */
 				usleep(lcore_idle_hint);
 
 			stats[lcore_id].sleep_time += lcore_idle_hint;
diff --git a/examples/l3fwd/main.c b/examples/l3fwd/main.c
index 6f7d7d4..b06f0ba 100644
--- a/examples/l3fwd/main.c
+++ b/examples/l3fwd/main.c
@@ -1256,7 +1256,7 @@ processx4_step3(struct rte_mbuf *pkt[FWDSTEP], uint16_t dst_port[FWDSTEP])
  * Suppose we have array of destionation ports:
  * dst_port[] = {a, b, c, d,, e, ... }
  * dp1 should contain: <a, b, c, d>, dp2: <b, c, d, e>.
- * We doing 4 comparisions at once and the result is 4 bit mask.
+ * We doing 4 comparsions at once and the result is 4 bit mask.
  * This mask is used as an index into prebuild array of pnum values.
  */
 static inline uint16_t *
diff --git a/examples/multi_process/l2fwd_fork/flib.h b/examples/multi_process/l2fwd_fork/flib.h
index 711e3b6..1064c9b 100644
--- a/examples/multi_process/l2fwd_fork/flib.h
+++ b/examples/multi_process/l2fwd_fork/flib.h
@@ -120,7 +120,7 @@ int flib_register_slave_exit_notify(unsigned slave_id,
 /**
  * Assign a lcore ID to non-slave thread.  Non-slave thread refers to thread that
  * not created by function rte_eal_remote_launch or rte_eal_mp_remote_launch.
- * These threads can either bind lcore or float among differnt lcores.
+ * These threads can either bind lcore or float among different lcores.
  * This lcore ID will be unique in multi-thread or multi-process DPDK running
  * environment, then it can benefit from using the cache mechanism provided in
  * mempool library.
diff --git a/examples/netmap_compat/lib/compat_netmap.c b/examples/netmap_compat/lib/compat_netmap.c
index 1d86ef0..0e8e838 100644
--- a/examples/netmap_compat/lib/compat_netmap.c
+++ b/examples/netmap_compat/lib/compat_netmap.c
@@ -160,7 +160,7 @@ mbuf_to_slot(struct rte_mbuf *mbuf, struct netmap_ring *r, uint32_t index)
 /**
  * Given a Netmap ring and a slot index for that ring, construct a dpdk mbuf
  * from the data held in the buffer associated with the slot.
- * Allocation/deallocation of the dpdk mbuf are the responsability of the
+ * Allocation/deallocation of the dpdk mbuf are the responsibility of the
  * caller.
  * Note that mbuf chains are not supported.
  */
diff --git a/examples/vhost/main.c b/examples/vhost/main.c
index 3a35359..c42c9d1 100644
--- a/examples/vhost/main.c
+++ b/examples/vhost/main.c
@@ -1326,7 +1326,7 @@ switch_worker(__attribute__((unused)) void *arg)
 
 /*
  * This function gets available ring number for zero copy rx.
- * Only one thread will call this funciton for a paticular virtio device,
+ * Only one thread will call this funciton for a particular virtio device,
  * so, it is designed as non-thread-safe function.
  */
 static inline uint32_t __attribute__((always_inline))
@@ -1342,7 +1342,7 @@ get_available_ring_num_zcp(struct virtio_net *dev)
 /*
  * This function gets available ring index for zero copy rx,
  * it will retry 'burst_rx_retry_num' times till it get enough ring index.
- * Only one thread will call this funciton for a paticular virtio device,
+ * Only one thread will call this funciton for a particular virtio device,
  * so, it is designed as non-thread-safe function.
  */
 static inline uint32_t __attribute__((always_inline))
@@ -2413,8 +2413,8 @@ destroy_device (volatile struct virtio_net *dev)
 }
 
 /*
- * Calculate the region count of physical continous regions for one particular
- * region of whose vhost virtual address is continous. The particular region
+ * Calculate the region count of physical continuous regions for one particular
+ * region of whose vhost virtual address is continuous. The particular region
  * start from vva_start, with size of 'size' in argument.
  */
 static uint32_t
@@ -2462,9 +2462,9 @@ check_hpa_regions(uint64_t vva_start, uint64_t size)
 }
 
 /*
- * Divide each region whose vhost virtual address is continous into a few
+ * Divide each region whose vhost virtual address is continuous into a few
  * sub-regions, make sure the physical address within each sub-region are
- * continous. And fill offset(to GPA) and size etc. information of each
+ * continuous. And fill offset(to GPA) and size etc. information of each
  * sub-region into regions_hpa.
  */
 static uint32_t
diff --git a/examples/vhost_xen/main.c b/examples/vhost_xen/main.c
index b4a86e3..3e064d2 100644
--- a/examples/vhost_xen/main.c
+++ b/examples/vhost_xen/main.c
@@ -561,7 +561,7 @@ gpa_to_vva(struct virtio_net *dev, uint64_t guest_pa)
 /*
  * This function adds buffers to the virtio devices RX virtqueue. Buffers can
  * be received from the physical port or from another virtio device. A packet
- * count is returned to indicate the number of packets that were succesfully
+ * count is returned to indicate the number of packets that were successfully
  * added to the RX queue.
  */
 static inline uint32_t __attribute__((always_inline))
diff --git a/examples/vhost_xen/xenstore_parse.c b/examples/vhost_xen/xenstore_parse.c
index 9441639..c444d08 100644
--- a/examples/vhost_xen/xenstore_parse.c
+++ b/examples/vhost_xen/xenstore_parse.c
@@ -299,7 +299,7 @@ err:
 }
 
 /*
- * This function maps grant node of vring or mbuf pool to a continous virtual address space,
+ * This function maps grant node of vring or mbuf pool to a continuous virtual address space,
  * and returns mapped address, pfn array, index array
  * @param gntnode
  *  Pointer to grant node
@@ -468,7 +468,7 @@ cleanup_mempool(struct xen_mempool *mempool)
 
 /*
  * process mempool node idx#_mempool_gref, idx = 0, 1, 2...
- * untill we encounter a node that doesn't exist.
+ * until we encounter a node that doesn't exist.
  */
 int
 parse_mempoolnode(struct xen_guest *guest)
-- 
2.1.4

[dpdk-dev] [PATCH 07/10] vhost: fix spelling errors

[Stephen Hemminger]

From: Stephen Hemminger <shemming@brocade.com>

More codespell

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
 lib/librte_vhost/eventfd_link/eventfd_link.h | 2 +-
 lib/librte_vhost/rte_virtio_net.h            | 2 +-
 lib/librte_vhost/vhost_rxtx.c                | 4 ++--
 3 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/lib/librte_vhost/eventfd_link/eventfd_link.h b/lib/librte_vhost/eventfd_link/eventfd_link.h
index ea619ec..0711403 100644
--- a/lib/librte_vhost/eventfd_link/eventfd_link.h
+++ b/lib/librte_vhost/eventfd_link/eventfd_link.h
@@ -66,7 +66,7 @@
 #define EVENTFD_COPY 1
 
 /*
- * arguements for the EVENTFD_COPY ioctl
+ * arguments for the EVENTFD_COPY ioctl
  */
 struct eventfd_copy {
 	unsigned target_fd; /* fd in the target pid */
diff --git a/lib/librte_vhost/rte_virtio_net.h b/lib/librte_vhost/rte_virtio_net.h
index 0bf07c7..f4ebb07 100644
--- a/lib/librte_vhost/rte_virtio_net.h
+++ b/lib/librte_vhost/rte_virtio_net.h
@@ -188,7 +188,7 @@ int rte_vhost_driver_session_start(void);
 /**
  * This function adds buffers to the virtio devices RX virtqueue. Buffers can
  * be received from the physical port or from another virtual device. A packet
- * count is returned to indicate the number of packets that were succesfully
+ * count is returned to indicate the number of packets that were successfully
  * added to the RX queue.
  * @param queue_id
  *  virtio queue index in mq case
diff --git a/lib/librte_vhost/vhost_rxtx.c b/lib/librte_vhost/vhost_rxtx.c
index ccfd82f..7a2e935 100644
--- a/lib/librte_vhost/vhost_rxtx.c
+++ b/lib/librte_vhost/vhost_rxtx.c
@@ -45,7 +45,7 @@
 /**
  * This function adds buffers to the virtio devices RX virtqueue. Buffers can
  * be received from the physical port or from another virtio device. A packet
- * count is returned to indicate the number of packets that are succesfully
+ * count is returned to indicate the number of packets that are successfully
  * added to the RX queue. This function works when mergeable is disabled.
  */
 static inline uint32_t __attribute__((always_inline))
@@ -662,7 +662,7 @@ rte_vhost_dequeue_burst(struct virtio_net *dev, uint16_t queue_id,
 					if (seg_avail == 0) {
 						/*
 						 * The current segment hasn't
-						 * room to accomodate more
+						 * room to accommodate more
 						 * data.
 						 */
 						cur->data_len = seg_offset;
-- 
2.1.4

[dpdk-dev] [PATCH 08/10] malloc: spelling fix

[Stephen Hemminger]

From: Stephen Hemminger <shemming@brocade.com>

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
 lib/librte_malloc/rte_malloc.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/librte_malloc/rte_malloc.h b/lib/librte_malloc/rte_malloc.h
index 74bb78c..ba9080c 100644
--- a/lib/librte_malloc/rte_malloc.h
+++ b/lib/librte_malloc/rte_malloc.h
@@ -327,7 +327,7 @@ rte_malloc_set_limit(const char *type, size_t max);
  * rte_malloc
  *
  * @param addr
- *   Adress obtained from a previous rte_malloc call
+ *   Address obtained from a previous rte_malloc call
  * @return
  *   NULL on error
  *   otherwise return physical address of the buffer
-- 
2.1.4

[dpdk-dev] [PATCH 09/10] ring: fix spelling error

[Stephen Hemminger]

From: Stephen Hemminger <shemming@brocade.com>

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
 lib/librte_pmd_ring/rte_eth_ring.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/librte_pmd_ring/rte_eth_ring.c b/lib/librte_pmd_ring/rte_eth_ring.c
index f685f08..9e76c6b 100644
--- a/lib/librte_pmd_ring/rte_eth_ring.c
+++ b/lib/librte_pmd_ring/rte_eth_ring.c
@@ -421,7 +421,7 @@ static int parse_kvlist (const char *key __rte_unused, const char *value, void *
 	ret = -EINVAL;
 
 	if (!name) {
-		RTE_LOG(WARNING, PMD, "command line paramter is empty for ring pmd!\n");
+		RTE_LOG(WARNING, PMD, "command line parameter is empty for ring pmd!\n");
 		goto out;
 	}
 
-- 
2.1.4

[dpdk-dev] [PATCH 10/10] ixgbe: spelling fixes

[Stephen Hemminger]

From: Stephen Hemminger <shemming@brocade.com>

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>

---
This file comes from common code and probably has to be fixed upstream.
---
 lib/librte_pmd_ixgbe/ixgbe/ixgbe_type.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/lib/librte_pmd_ixgbe/ixgbe/ixgbe_type.h b/lib/librte_pmd_ixgbe/ixgbe/ixgbe_type.h
index c67d462..8aa956e 100644
--- a/lib/librte_pmd_ixgbe/ixgbe/ixgbe_type.h
+++ b/lib/librte_pmd_ixgbe/ixgbe/ixgbe_type.h
@@ -46,7 +46,7 @@ POSSIBILITY OF SUCH DAMAGE.
  *
  * - IXGBE_ERROR_POLLING
  * This category is for errors related to polling/timeout issues and should be
- * used in any case where the timeout occured, or a failure to obtain a lock, or
+ * used in any case where the timeout occurred, or a failure to obtain a lock, or
  * failure to receive data within the time limit.
  *
  * - IXGBE_ERROR_CAUTION
@@ -1911,7 +1911,7 @@ enum {
 #define IXGBE_MACC_FS		0x00040000
 #define IXGBE_MAC_RX2TX_LPBK	0x00000002
 
-/* Veto Bit definiton */
+/* Veto Bit definition */
 #define IXGBE_MMNGC_MNG_VETO	0x00000001
 
 /* LINKS Bit Masks */
-- 
2.1.4

Re: [dpdk-dev] [PATCH 01/10] i40e: spellin fixes

[Neil Horman]

On Sat, Jan 31, 2015 at 08:51:53PM +0000, Stephen Hemminger wrote:
> From: Stephen Hemminger <shemming@brocade.com>
> 
> Spelling errors found with code spell.
> 
> Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
> ---
>  lib/librte_pmd_i40e/i40e_ethdev.c | 8 ++++----
>  lib/librte_pmd_i40e/i40e_ethdev.h | 2 +-
>  lib/librte_pmd_i40e/i40e_rxtx.c   | 6 +++---
>  3 files changed, 8 insertions(+), 8 deletions(-)
> 
> diff --git a/lib/librte_pmd_i40e/i40e_ethdev.c b/lib/librte_pmd_i40e/i40e_ethdev.c
> index b47a3d2..07efca5 100644
> --- a/lib/librte_pmd_i40e/i40e_ethdev.c
> +++ b/lib/librte_pmd_i40e/i40e_ethdev.c
> @@ -3810,16 +3810,16 @@ i40e_dev_handle_vfr_event(struct rte_eth_dev *dev)
>  		index = abs_vf_id / I40E_UINT32_BIT_SIZE;
>  		offset = abs_vf_id % I40E_UINT32_BIT_SIZE;
>  		val = I40E_READ_REG(hw, I40E_GLGEN_VFLRSTAT(index));
> -		/* VFR event occured */
> +		/* VFR event occurred */
>  		if (val & (0x1 << offset)) {
>  			int ret;
>  
>  			/* Clear the event first */
>  			I40E_WRITE_REG(hw, I40E_GLGEN_VFLRSTAT(index),
>  							(0x1 << offset));
> -			PMD_DRV_LOG(INFO, "VF %u reset occured", abs_vf_id);
> +			PMD_DRV_LOG(INFO, "VF %u reset occurred", abs_vf_id);
>  			/**
> -			 * Only notify a VF reset event occured,
> +			 * Only notify a VF reset event occurred,
>  			 * don't trigger another SW reset
>  			 */
>  			ret = i40e_pf_host_vf_reset(&pf->vfs[i], 0);
> @@ -5007,7 +5007,7 @@ i40e_pf_config_rss(struct i40e_pf *pf)
>  
>  	/*
>  	 * If both VMDQ and RSS enabled, not all of PF queues are configured.
> -	 * It's necessary to calulate the actual PF queues that are configured.
> +	 * It's necessary to calculate the actual PF queues that are configured.
>  	 */
>  	if (pf->dev_data->dev_conf.rxmode.mq_mode & ETH_MQ_RX_VMDQ_FLAG) {
>  		num = i40e_pf_calc_configured_queues_num(pf);
> diff --git a/lib/librte_pmd_i40e/i40e_ethdev.h b/lib/librte_pmd_i40e/i40e_ethdev.h
> index f913ea9..dc870d2 100644
> --- a/lib/librte_pmd_i40e/i40e_ethdev.h
> +++ b/lib/librte_pmd_i40e/i40e_ethdev.h
> @@ -273,7 +273,7 @@ enum I40E_VF_STATE {
>  struct i40e_pf_vf {
>  	struct i40e_pf *pf;
>  	struct i40e_vsi *vsi;
> -	enum I40E_VF_STATE state; /* The number of queue pairs availiable */
> +	enum I40E_VF_STATE state; /* The number of queue pairs available */
>  	uint16_t vf_idx; /* VF index in pf->vfs */
>  	uint16_t lan_nb_qps; /* Actual queues allocated */
>  	uint16_t reset_cnt; /* Total vf reset times */
> diff --git a/lib/librte_pmd_i40e/i40e_rxtx.c b/lib/librte_pmd_i40e/i40e_rxtx.c
> index 2beae3c..56125cf 100644
> --- a/lib/librte_pmd_i40e/i40e_rxtx.c
> +++ b/lib/librte_pmd_i40e/i40e_rxtx.c
> @@ -1637,7 +1637,7 @@ i40e_dev_rx_queue_stop(struct rte_eth_dev *dev, uint16_t rx_queue_id)
>  		rxq = dev->data->rx_queues[rx_queue_id];
>  
>  		/*
> -		* rx_queue_id is queue id aplication refers to, while
> +		* rx_queue_id is queue id application refers to, while
>  		* rxq->reg_idx is the real queue index.
>  		*/
>  		err = i40e_switch_rx_queue(hw, rxq->reg_idx, FALSE);
> @@ -1667,7 +1667,7 @@ i40e_dev_tx_queue_start(struct rte_eth_dev *dev, uint16_t tx_queue_id)
>  		txq = dev->data->tx_queues[tx_queue_id];
>  
>  		/*
> -		* tx_queue_id is queue id aplication refers to, while
> +		* tx_queue_id is queue id application refers to, while
>  		* rxq->reg_idx is the real queue index.
>  		*/
>  		err = i40e_switch_tx_queue(hw, txq->reg_idx, TRUE);
> @@ -1690,7 +1690,7 @@ i40e_dev_tx_queue_stop(struct rte_eth_dev *dev, uint16_t tx_queue_id)
>  		txq = dev->data->tx_queues[tx_queue_id];
>  
>  		/*
> -		* tx_queue_id is queue id aplication refers to, while
> +		* tx_queue_id is queue id application refers to, while
>  		* txq->reg_idx is the real queue index.
>  		*/
>  		err = i40e_switch_tx_queue(hw, txq->reg_idx, FALSE);
> -- 
> 2.1.4
> 
> 
For the series
Acked-by: Neil Horman <nhorman@tuxdriver.com>