DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] mlx5: spelling fixes
@ 2018-08-03 20:31 Stephen Hemminger
  2018-08-03 22:23 ` Yongseok Koh
  0 siblings, 1 reply; 2+ messages in thread
From: Stephen Hemminger @ 2018-08-03 20:31 UTC (permalink / raw)
  To: yskoh, shahafs; +Cc: dev, Stephen Hemminger

Fix spelling errors in messages and comments.

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
 drivers/net/mlx5/mlx5_ethdev.c |  2 +-
 drivers/net/mlx5/mlx5_flow.c   |  4 ++--
 drivers/net/mlx5/mlx5_mr.c     |  8 ++++----
 drivers/net/mlx5/mlx5_rxq.c    | 20 ++++++++++----------
 drivers/net/mlx5/mlx5_rxtx.c   |  2 +-
 5 files changed, 18 insertions(+), 18 deletions(-)

diff --git a/drivers/net/mlx5/mlx5_ethdev.c b/drivers/net/mlx5/mlx5_ethdev.c
index 34c5b95ee6d2..2c838e6539b6 100644
--- a/drivers/net/mlx5/mlx5_ethdev.c
+++ b/drivers/net/mlx5/mlx5_ethdev.c
@@ -1138,7 +1138,7 @@ mlx5_dev_interrupt_handler_install(struct rte_eth_dev *dev)
 	}
 	ret = mlx5_socket_init(dev);
 	if (ret)
-		DRV_LOG(ERR, "port %u cannot initialise socket: %s",
+		DRV_LOG(ERR, "port %u cannot initialize socket: %s",
 			dev->data->port_id, strerror(rte_errno));
 	else if (priv->primary_socket) {
 		priv->intr_handle_socket.fd = priv->primary_socket;
diff --git a/drivers/net/mlx5/mlx5_flow.c b/drivers/net/mlx5/mlx5_flow.c
index b94c442ec4e6..d13178be6ba1 100644
--- a/drivers/net/mlx5/mlx5_flow.c
+++ b/drivers/net/mlx5/mlx5_flow.c
@@ -1090,7 +1090,7 @@ mlx5_flow_item_ipv6(const struct rte_flow_item *item, struct rte_flow *flow,
 					  item,
 					  "L3 cannot follow an L4 layer.");
 	/*
-	 * IPv6 is not recognised by the NIC inside a GRE tunnel.
+	 * IPv6 is not recognized by the NIC inside a GRE tunnel.
 	 * Such support has to be disabled as the rule will be
 	 * accepted.  Issue reproduced with Mellanox OFED 4.3-3.0.2.1 and
 	 * Mellanox OFED 4.4-1.0.0.0.
@@ -1100,7 +1100,7 @@ mlx5_flow_item_ipv6(const struct rte_flow_item *item, struct rte_flow *flow,
 					  RTE_FLOW_ERROR_TYPE_ITEM,
 					  item,
 					  "IPv6 inside a GRE tunnel is"
-					  " not recognised.");
+					  " not recognized.");
 	if (!mask)
 		mask = &rte_flow_item_ipv6_mask;
 	ret = mlx5_flow_item_acceptable
diff --git a/drivers/net/mlx5/mlx5_mr.c b/drivers/net/mlx5/mlx5_mr.c
index 1d1bcb5fe028..aa7ca355b4d8 100644
--- a/drivers/net/mlx5/mlx5_mr.c
+++ b/drivers/net/mlx5/mlx5_mr.c
@@ -446,7 +446,7 @@ mr_free(struct mlx5_mr *mr)
 }
 
 /**
- * Releass resources of detached MR having no online entry.
+ * Release resources of detached MR having no online entry.
  *
  * @param dev
  *   Pointer to Ethernet device.
@@ -496,7 +496,7 @@ mr_find_contig_memsegs_cb(const struct rte_memseg_list *msl,
 }
 
 /**
- * Create a new global Memroy Region (MR) for a missing virtual address.
+ * Create a new global Memory Region (MR) for a missing virtual address.
  * Register entire virtually contiguous memory chunk around the address.
  *
  * @param dev
@@ -553,7 +553,7 @@ mlx5_mr_create(struct rte_eth_dev *dev, struct mlx5_mr_cache *entry,
 	 * Find out a contiguous virtual address chunk in use, to which the
 	 * given address belongs, in order to register maximum range. In the
 	 * best case where mempools are not dynamically recreated and
-	 * '--socket-mem' is speicified as an EAL option, it is very likely to
+	 * '--socket-mem' is specified as an EAL option, it is very likely to
 	 * have only one MR(LKey) per a socket and per a hugepage-size even
 	 * though the system memory is highly fragmented.
 	 */
@@ -604,7 +604,7 @@ mlx5_mr_create(struct rte_eth_dev *dev, struct mlx5_mr_cache *entry,
 	bmp_mem = RTE_PTR_ALIGN_CEIL(mr + 1, RTE_CACHE_LINE_SIZE);
 	mr->ms_bmp = rte_bitmap_init(ms_n, bmp_mem, bmp_size);
 	if (mr->ms_bmp == NULL) {
-		DEBUG("port %u unable to initialize bitamp for a new MR of"
+		DEBUG("port %u unable to initialize bitmap for a new MR of"
 		      " address (%p).",
 		      dev->data->port_id, (void *)addr);
 		rte_errno = EINVAL;
diff --git a/drivers/net/mlx5/mlx5_rxq.c b/drivers/net/mlx5/mlx5_rxq.c
index 16e1641d00bc..7c2d65ff2007 100644
--- a/drivers/net/mlx5/mlx5_rxq.c
+++ b/drivers/net/mlx5/mlx5_rxq.c
@@ -751,7 +751,7 @@ mlx5_rx_intr_disable(struct rte_eth_dev *dev, uint16_t rx_queue_id)
  *   Queue index in DPDK Rx queue array
  *
  * @return
- *   The Verbs object initialised, NULL otherwise and rte_errno is set.
+ *   The Verbs object initialized, NULL otherwise and rte_errno is set.
  */
 struct mlx5_rxq_ibv *
 mlx5_rxq_ibv_new(struct rte_eth_dev *dev, uint16_t idx)
@@ -1179,7 +1179,7 @@ mlx5_mprq_free_mp(struct rte_eth_dev *dev)
 
 /**
  * Allocate a mempool for Multi-Packet RQ. All configured Rx queues share the
- * mempool. If already allocated, reuse it if there're enough elements.
+ * mempool. If already allocated, reuse it if there are enough elements.
  * Otherwise, resize it.
  *
  * @param dev
@@ -1234,7 +1234,7 @@ mlx5_mprq_alloc_mp(struct rte_eth_dev *dev)
 	 */
 	desc *= 4;
 	obj_num = desc + MLX5_MPRQ_MP_CACHE_SZ * priv->rxqs_n;
-	/* Check a mempool is already allocated and if it can be resued. */
+	/* Check a mempool is already allocated and if it can be reused. */
 	if (mp != NULL && mp->elt_size >= obj_size && mp->size >= obj_num) {
 		DRV_LOG(DEBUG, "port %u mempool %s is being reused",
 			dev->data->port_id, mp->name);
@@ -1583,7 +1583,7 @@ mlx5_rxq_verify(struct rte_eth_dev *dev)
  *   Number of queues in the array.
  *
  * @return
- *   The Verbs object initialised, NULL otherwise and rte_errno is set.
+ *   The Verbs object initialized, NULL otherwise and rte_errno is set.
  */
 struct mlx5_ind_table_ibv *
 mlx5_ind_table_ibv_new(struct rte_eth_dev *dev, const uint16_t *queues,
@@ -1613,7 +1613,7 @@ mlx5_ind_table_ibv_new(struct rte_eth_dev *dev, const uint16_t *queues,
 		ind_tbl->queues[i] = queues[i];
 	}
 	ind_tbl->queues_n = queues_n;
-	/* Finalise indirection table. */
+	/* Finalize indirection table. */
 	for (j = 0; i != (unsigned int)(1 << wq_n); ++i, ++j)
 		wq[i] = wq[j];
 	ind_tbl->ind_table = mlx5_glue->create_rwq_ind_table
@@ -1746,7 +1746,7 @@ mlx5_ind_table_ibv_verify(struct rte_eth_dev *dev)
  *   Number of queues.
  *
  * @return
- *   The Verbs object initialised, NULL otherwise and rte_errno is set.
+ *   The Verbs object initialized, NULL otherwise and rte_errno is set.
  */
 struct mlx5_hrxq *
 mlx5_hrxq_new(struct rte_eth_dev *dev,
@@ -1950,7 +1950,7 @@ mlx5_hrxq_ibv_verify(struct rte_eth_dev *dev)
  *   Pointer to Ethernet device.
  *
  * @return
- *   The Verbs object initialised, NULL otherwise and rte_errno is set.
+ *   The Verbs object initialized, NULL otherwise and rte_errno is set.
  */
 struct mlx5_rxq_ibv *
 mlx5_rxq_ibv_drop_new(struct rte_eth_dev *dev)
@@ -2009,7 +2009,7 @@ mlx5_rxq_ibv_drop_new(struct rte_eth_dev *dev)
  *   Pointer to Ethernet device.
  *
  * @return
- *   The Verbs object initialised, NULL otherwise and rte_errno is set.
+ *   The Verbs object initialized, NULL otherwise and rte_errno is set.
  */
 void
 mlx5_rxq_ibv_drop_release(struct rte_eth_dev *dev)
@@ -2032,7 +2032,7 @@ mlx5_rxq_ibv_drop_release(struct rte_eth_dev *dev)
  *   Pointer to Ethernet device.
  *
  * @return
- *   The Verbs object initialised, NULL otherwise and rte_errno is set.
+ *   The Verbs object initialized, NULL otherwise and rte_errno is set.
  */
 struct mlx5_ind_table_ibv *
 mlx5_ind_table_ibv_drop_new(struct rte_eth_dev *dev)
@@ -2096,7 +2096,7 @@ mlx5_ind_table_ibv_drop_release(struct rte_eth_dev *dev)
  *   Pointer to Ethernet device.
  *
  * @return
- *   The Verbs object initialised, NULL otherwise and rte_errno is set.
+ *   The Verbs object initialized, NULL otherwise and rte_errno is set.
  */
 struct mlx5_hrxq *
 mlx5_hrxq_drop_new(struct rte_eth_dev *dev)
diff --git a/drivers/net/mlx5/mlx5_rxtx.c b/drivers/net/mlx5/mlx5_rxtx.c
index 2d14f8a6edbd..95cbc513ca7b 100644
--- a/drivers/net/mlx5/mlx5_rxtx.c
+++ b/drivers/net/mlx5/mlx5_rxtx.c
@@ -1507,7 +1507,7 @@ txq_burst_empw(struct mlx5_txq_data *txq, struct rte_mbuf **pkts,
 			mpw.wqe->eseg.cs_flags = cs_flags;
 		} else {
 			/* Evaluate whether the next packet can be inlined.
-			 * Inlininig is possible when:
+			 * Inlining is possible when:
 			 * - length is less than configured value
 			 * - length fits for remaining space
 			 * - not required to fill the title WQEBB with dsegs
-- 
2.18.0

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [dpdk-dev] [PATCH] mlx5: spelling fixes
  2018-08-03 20:31 [dpdk-dev] [PATCH] mlx5: spelling fixes Stephen Hemminger
@ 2018-08-03 22:23 ` Yongseok Koh
  0 siblings, 0 replies; 2+ messages in thread
From: Yongseok Koh @ 2018-08-03 22:23 UTC (permalink / raw)
  To: Stephen Hemminger; +Cc: shahafs, dev

On Fri, Aug 03, 2018 at 01:31:48PM -0700, Stephen Hemminger wrote:
> Fix spelling errors in messages and comments.
> 
> Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
> ---

Hi Stephen,

'initialise' is accepted, not a typo. AFAIK, it is 'British' way.
Same for 'recognised' and 'finalise'. I don't want to enforce people to use
'non-Britain' English for mlx5 PMD. :-)

Other than those, thanks for correcting typos.

Please send out v2.

Thanks,
Yongseok

>  drivers/net/mlx5/mlx5_ethdev.c |  2 +-
>  drivers/net/mlx5/mlx5_flow.c   |  4 ++--
>  drivers/net/mlx5/mlx5_mr.c     |  8 ++++----
>  drivers/net/mlx5/mlx5_rxq.c    | 20 ++++++++++----------
>  drivers/net/mlx5/mlx5_rxtx.c   |  2 +-
>  5 files changed, 18 insertions(+), 18 deletions(-)
> 
> diff --git a/drivers/net/mlx5/mlx5_ethdev.c b/drivers/net/mlx5/mlx5_ethdev.c
> index 34c5b95ee6d2..2c838e6539b6 100644
> --- a/drivers/net/mlx5/mlx5_ethdev.c
> +++ b/drivers/net/mlx5/mlx5_ethdev.c
> @@ -1138,7 +1138,7 @@ mlx5_dev_interrupt_handler_install(struct rte_eth_dev *dev)
>  	}
>  	ret = mlx5_socket_init(dev);
>  	if (ret)
> -		DRV_LOG(ERR, "port %u cannot initialise socket: %s",
> +		DRV_LOG(ERR, "port %u cannot initialize socket: %s",
>  			dev->data->port_id, strerror(rte_errno));
>  	else if (priv->primary_socket) {
>  		priv->intr_handle_socket.fd = priv->primary_socket;
> diff --git a/drivers/net/mlx5/mlx5_flow.c b/drivers/net/mlx5/mlx5_flow.c
> index b94c442ec4e6..d13178be6ba1 100644
> --- a/drivers/net/mlx5/mlx5_flow.c
> +++ b/drivers/net/mlx5/mlx5_flow.c
> @@ -1090,7 +1090,7 @@ mlx5_flow_item_ipv6(const struct rte_flow_item *item, struct rte_flow *flow,
>  					  item,
>  					  "L3 cannot follow an L4 layer.");
>  	/*
> -	 * IPv6 is not recognised by the NIC inside a GRE tunnel.
> +	 * IPv6 is not recognized by the NIC inside a GRE tunnel.
>  	 * Such support has to be disabled as the rule will be
>  	 * accepted.  Issue reproduced with Mellanox OFED 4.3-3.0.2.1 and
>  	 * Mellanox OFED 4.4-1.0.0.0.
> @@ -1100,7 +1100,7 @@ mlx5_flow_item_ipv6(const struct rte_flow_item *item, struct rte_flow *flow,
>  					  RTE_FLOW_ERROR_TYPE_ITEM,
>  					  item,
>  					  "IPv6 inside a GRE tunnel is"
> -					  " not recognised.");
> +					  " not recognized.");
>  	if (!mask)
>  		mask = &rte_flow_item_ipv6_mask;
>  	ret = mlx5_flow_item_acceptable
> diff --git a/drivers/net/mlx5/mlx5_mr.c b/drivers/net/mlx5/mlx5_mr.c
> index 1d1bcb5fe028..aa7ca355b4d8 100644
> --- a/drivers/net/mlx5/mlx5_mr.c
> +++ b/drivers/net/mlx5/mlx5_mr.c
> @@ -446,7 +446,7 @@ mr_free(struct mlx5_mr *mr)
>  }
>  
>  /**
> - * Releass resources of detached MR having no online entry.
> + * Release resources of detached MR having no online entry.
>   *
>   * @param dev
>   *   Pointer to Ethernet device.
> @@ -496,7 +496,7 @@ mr_find_contig_memsegs_cb(const struct rte_memseg_list *msl,
>  }
>  
>  /**
> - * Create a new global Memroy Region (MR) for a missing virtual address.
> + * Create a new global Memory Region (MR) for a missing virtual address.
>   * Register entire virtually contiguous memory chunk around the address.
>   *
>   * @param dev
> @@ -553,7 +553,7 @@ mlx5_mr_create(struct rte_eth_dev *dev, struct mlx5_mr_cache *entry,
>  	 * Find out a contiguous virtual address chunk in use, to which the
>  	 * given address belongs, in order to register maximum range. In the
>  	 * best case where mempools are not dynamically recreated and
> -	 * '--socket-mem' is speicified as an EAL option, it is very likely to
> +	 * '--socket-mem' is specified as an EAL option, it is very likely to
>  	 * have only one MR(LKey) per a socket and per a hugepage-size even
>  	 * though the system memory is highly fragmented.
>  	 */
> @@ -604,7 +604,7 @@ mlx5_mr_create(struct rte_eth_dev *dev, struct mlx5_mr_cache *entry,
>  	bmp_mem = RTE_PTR_ALIGN_CEIL(mr + 1, RTE_CACHE_LINE_SIZE);
>  	mr->ms_bmp = rte_bitmap_init(ms_n, bmp_mem, bmp_size);
>  	if (mr->ms_bmp == NULL) {
> -		DEBUG("port %u unable to initialize bitamp for a new MR of"
> +		DEBUG("port %u unable to initialize bitmap for a new MR of"
>  		      " address (%p).",
>  		      dev->data->port_id, (void *)addr);
>  		rte_errno = EINVAL;
> diff --git a/drivers/net/mlx5/mlx5_rxq.c b/drivers/net/mlx5/mlx5_rxq.c
> index 16e1641d00bc..7c2d65ff2007 100644
> --- a/drivers/net/mlx5/mlx5_rxq.c
> +++ b/drivers/net/mlx5/mlx5_rxq.c
> @@ -751,7 +751,7 @@ mlx5_rx_intr_disable(struct rte_eth_dev *dev, uint16_t rx_queue_id)
>   *   Queue index in DPDK Rx queue array
>   *
>   * @return
> - *   The Verbs object initialised, NULL otherwise and rte_errno is set.
> + *   The Verbs object initialized, NULL otherwise and rte_errno is set.
>   */
>  struct mlx5_rxq_ibv *
>  mlx5_rxq_ibv_new(struct rte_eth_dev *dev, uint16_t idx)
> @@ -1179,7 +1179,7 @@ mlx5_mprq_free_mp(struct rte_eth_dev *dev)
>  
>  /**
>   * Allocate a mempool for Multi-Packet RQ. All configured Rx queues share the
> - * mempool. If already allocated, reuse it if there're enough elements.
> + * mempool. If already allocated, reuse it if there are enough elements.
>   * Otherwise, resize it.
>   *
>   * @param dev
> @@ -1234,7 +1234,7 @@ mlx5_mprq_alloc_mp(struct rte_eth_dev *dev)
>  	 */
>  	desc *= 4;
>  	obj_num = desc + MLX5_MPRQ_MP_CACHE_SZ * priv->rxqs_n;
> -	/* Check a mempool is already allocated and if it can be resued. */
> +	/* Check a mempool is already allocated and if it can be reused. */
>  	if (mp != NULL && mp->elt_size >= obj_size && mp->size >= obj_num) {
>  		DRV_LOG(DEBUG, "port %u mempool %s is being reused",
>  			dev->data->port_id, mp->name);
> @@ -1583,7 +1583,7 @@ mlx5_rxq_verify(struct rte_eth_dev *dev)
>   *   Number of queues in the array.
>   *
>   * @return
> - *   The Verbs object initialised, NULL otherwise and rte_errno is set.
> + *   The Verbs object initialized, NULL otherwise and rte_errno is set.
>   */
>  struct mlx5_ind_table_ibv *
>  mlx5_ind_table_ibv_new(struct rte_eth_dev *dev, const uint16_t *queues,
> @@ -1613,7 +1613,7 @@ mlx5_ind_table_ibv_new(struct rte_eth_dev *dev, const uint16_t *queues,
>  		ind_tbl->queues[i] = queues[i];
>  	}
>  	ind_tbl->queues_n = queues_n;
> -	/* Finalise indirection table. */
> +	/* Finalize indirection table. */
>  	for (j = 0; i != (unsigned int)(1 << wq_n); ++i, ++j)
>  		wq[i] = wq[j];
>  	ind_tbl->ind_table = mlx5_glue->create_rwq_ind_table
> @@ -1746,7 +1746,7 @@ mlx5_ind_table_ibv_verify(struct rte_eth_dev *dev)
>   *   Number of queues.
>   *
>   * @return
> - *   The Verbs object initialised, NULL otherwise and rte_errno is set.
> + *   The Verbs object initialized, NULL otherwise and rte_errno is set.
>   */
>  struct mlx5_hrxq *
>  mlx5_hrxq_new(struct rte_eth_dev *dev,
> @@ -1950,7 +1950,7 @@ mlx5_hrxq_ibv_verify(struct rte_eth_dev *dev)
>   *   Pointer to Ethernet device.
>   *
>   * @return
> - *   The Verbs object initialised, NULL otherwise and rte_errno is set.
> + *   The Verbs object initialized, NULL otherwise and rte_errno is set.
>   */
>  struct mlx5_rxq_ibv *
>  mlx5_rxq_ibv_drop_new(struct rte_eth_dev *dev)
> @@ -2009,7 +2009,7 @@ mlx5_rxq_ibv_drop_new(struct rte_eth_dev *dev)
>   *   Pointer to Ethernet device.
>   *
>   * @return
> - *   The Verbs object initialised, NULL otherwise and rte_errno is set.
> + *   The Verbs object initialized, NULL otherwise and rte_errno is set.
>   */
>  void
>  mlx5_rxq_ibv_drop_release(struct rte_eth_dev *dev)
> @@ -2032,7 +2032,7 @@ mlx5_rxq_ibv_drop_release(struct rte_eth_dev *dev)
>   *   Pointer to Ethernet device.
>   *
>   * @return
> - *   The Verbs object initialised, NULL otherwise and rte_errno is set.
> + *   The Verbs object initialized, NULL otherwise and rte_errno is set.
>   */
>  struct mlx5_ind_table_ibv *
>  mlx5_ind_table_ibv_drop_new(struct rte_eth_dev *dev)
> @@ -2096,7 +2096,7 @@ mlx5_ind_table_ibv_drop_release(struct rte_eth_dev *dev)
>   *   Pointer to Ethernet device.
>   *
>   * @return
> - *   The Verbs object initialised, NULL otherwise and rte_errno is set.
> + *   The Verbs object initialized, NULL otherwise and rte_errno is set.
>   */
>  struct mlx5_hrxq *
>  mlx5_hrxq_drop_new(struct rte_eth_dev *dev)
> diff --git a/drivers/net/mlx5/mlx5_rxtx.c b/drivers/net/mlx5/mlx5_rxtx.c
> index 2d14f8a6edbd..95cbc513ca7b 100644
> --- a/drivers/net/mlx5/mlx5_rxtx.c
> +++ b/drivers/net/mlx5/mlx5_rxtx.c
> @@ -1507,7 +1507,7 @@ txq_burst_empw(struct mlx5_txq_data *txq, struct rte_mbuf **pkts,
>  			mpw.wqe->eseg.cs_flags = cs_flags;
>  		} else {
>  			/* Evaluate whether the next packet can be inlined.
> -			 * Inlininig is possible when:
> +			 * Inlining is possible when:
>  			 * - length is less than configured value
>  			 * - length fits for remaining space
>  			 * - not required to fill the title WQEBB with dsegs
> -- 
> 2.18.0
> 

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2018-08-03 22:23 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-08-03 20:31 [dpdk-dev] [PATCH] mlx5: spelling fixes Stephen Hemminger
2018-08-03 22:23 ` Yongseok Koh

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).