DPDK patches and discussions
 help / color / mirror / Atom feed
* [PATCH] common/cnxk: use XAQ create API for inline device
@ 2021-12-13 20:56 pbhagavatula
  2022-01-24  9:07 ` Jerin Jacob
  0 siblings, 1 reply; 2+ messages in thread
From: pbhagavatula @ 2021-12-13 20:56 UTC (permalink / raw)
  To: jerinj, Nithin Dabilpuram, Kiran Kumar K, Sunil Kumar Kori, Satha Rao
  Cc: dev, Pavan Nikhilesh

From: Pavan Nikhilesh <pbhagavatula@marvell.com>

Use the XAQ aura create and free API while initializing the
inline device.

Signed-off-by: Pavan Nikhilesh <pbhagavatula@marvell.com>
---
 drivers/common/cnxk/roc_nix_debug.c    |  4 +-
 drivers/common/cnxk/roc_nix_inl_dev.c  | 53 ++++++--------------------
 drivers/common/cnxk/roc_nix_inl_priv.h |  3 +-
 3 files changed, 15 insertions(+), 45 deletions(-)

diff --git a/drivers/common/cnxk/roc_nix_debug.c b/drivers/common/cnxk/roc_nix_debug.c
index 266935a6c5..5886650d6e 100644
--- a/drivers/common/cnxk/roc_nix_debug.c
+++ b/drivers/common/cnxk/roc_nix_debug.c
@@ -1257,8 +1257,8 @@ roc_nix_inl_dev_dump(struct roc_nix_inl_dev *roc_inl_dev)
 	nix_dump("  \txaq_buf_size = %u", inl_dev->xaq_buf_size);
 	nix_dump("  \txae_waes = %u", inl_dev->xae_waes);
 	nix_dump("  \tiue = %u", inl_dev->iue);
-	nix_dump("  \txaq_aura = 0x%" PRIx64, inl_dev->xaq_aura);
-	nix_dump("  \txaq_mem = 0x%p", inl_dev->xaq_mem);
+	nix_dump("  \txaq_aura = 0x%" PRIx64, inl_dev->xaq.aura_handle);
+	nix_dump("  \txaq_mem = 0x%p", inl_dev->xaq.mem);
 
 	nix_dump("  \tinl_dev_rq:");
 	roc_nix_rq_dump(&inl_dev->rq);
diff --git a/drivers/common/cnxk/roc_nix_inl_dev.c b/drivers/common/cnxk/roc_nix_inl_dev.c
index 10912a6c93..dd93765a2b 100644
--- a/drivers/common/cnxk/roc_nix_inl_dev.c
+++ b/drivers/common/cnxk/roc_nix_inl_dev.c
@@ -5,8 +5,6 @@
 #include "roc_api.h"
 #include "roc_priv.h"
 
-#define XAQ_CACHE_CNT 0x7
-
 /* Default Rx Config for Inline NIX LF */
 #define NIX_INL_LF_RX_CFG                                                      \
 	(ROC_NIX_LF_RX_CFG_DROP_RE | ROC_NIX_LF_RX_CFG_L2_LEN_ERR |            \
@@ -220,10 +218,8 @@ nix_inl_sso_setup(struct nix_inl_dev *inl_dev)
 {
 	struct sso_lf_alloc_rsp *sso_rsp;
 	struct dev *dev = &inl_dev->dev;
-	uint32_t xaq_cnt, count, aura;
 	uint16_t hwgrp[1] = {0};
-	struct npa_pool_s pool;
-	uintptr_t iova;
+	uint32_t xae_cnt;
 	int rc;
 
 	/* Alloc SSOW LF */
@@ -244,41 +240,17 @@ nix_inl_sso_setup(struct nix_inl_dev *inl_dev)
 	inl_dev->xae_waes = sso_rsp->xaq_wq_entries;
 	inl_dev->iue = sso_rsp->in_unit_entries;
 
-	/* Create XAQ pool */
-	xaq_cnt = XAQ_CACHE_CNT;
-	xaq_cnt += inl_dev->iue / inl_dev->xae_waes;
-	plt_sso_dbg("Configuring %d xaq buffers", xaq_cnt);
-
-	inl_dev->xaq_mem = plt_zmalloc(inl_dev->xaq_buf_size * xaq_cnt,
-				       inl_dev->xaq_buf_size);
-	if (!inl_dev->xaq_mem) {
-		rc = NIX_ERR_NO_MEM;
-		plt_err("Failed to alloc xaq buf mem");
-		goto free_sso;
-	}
-
-	memset(&pool, 0, sizeof(struct npa_pool_s));
-	pool.nat_align = 1;
-	rc = roc_npa_pool_create(&inl_dev->xaq_aura, inl_dev->xaq_buf_size,
-				 xaq_cnt, NULL, &pool);
+	xae_cnt = inl_dev->iue;
+	rc = sso_hwgrp_init_xaq_aura(dev, &inl_dev->xaq, xae_cnt,
+				     inl_dev->xae_waes, inl_dev->xaq_buf_size,
+				     1);
 	if (rc) {
-		plt_err("Failed to alloc aura for XAQ, rc=%d", rc);
-		goto free_mem;
-	}
-
-	/* Fill the XAQ buffers */
-	iova = (uint64_t)inl_dev->xaq_mem;
-	for (count = 0; count < xaq_cnt; count++) {
-		roc_npa_aura_op_free(inl_dev->xaq_aura, 0, iova);
-		iova += inl_dev->xaq_buf_size;
+		plt_err("Failed to alloc SSO XAQ aura, rc=%d", rc);
+		goto free_sso;
 	}
-	roc_npa_aura_op_range_set(inl_dev->xaq_aura, (uint64_t)inl_dev->xaq_mem,
-				  iova);
-
-	aura = roc_npa_aura_handle_to_aura(inl_dev->xaq_aura);
 
 	/* Setup xaq for hwgrps */
-	rc = sso_hwgrp_alloc_xaq(dev, aura, 1);
+	rc = sso_hwgrp_alloc_xaq(dev, inl_dev->xaq.aura_handle, 1);
 	if (rc) {
 		plt_err("Failed to setup hwgrp xaq aura, rc=%d", rc);
 		goto destroy_pool;
@@ -302,11 +274,7 @@ nix_inl_sso_setup(struct nix_inl_dev *inl_dev)
 release_xaq:
 	sso_hwgrp_release_xaq(&inl_dev->dev, 1);
 destroy_pool:
-	roc_npa_pool_destroy(inl_dev->xaq_aura);
-	inl_dev->xaq_aura = 0;
-free_mem:
-	plt_free(inl_dev->xaq_mem);
-	inl_dev->xaq_mem = NULL;
+	sso_hwgrp_free_xaq_aura(dev, &inl_dev->xaq, 0);
 free_sso:
 	sso_lf_free(dev, SSO_LF_TYPE_HWGRP, 1);
 free_ssow:
@@ -335,6 +303,9 @@ nix_inl_sso_release(struct nix_inl_dev *inl_dev)
 	sso_lf_free(&inl_dev->dev, SSO_LF_TYPE_HWS, 1);
 	sso_lf_free(&inl_dev->dev, SSO_LF_TYPE_HWGRP, 1);
 
+	/* Free the XAQ aura */
+	sso_hwgrp_free_xaq_aura(&inl_dev->dev, &inl_dev->xaq, 0);
+
 	return 0;
 }
 
diff --git a/drivers/common/cnxk/roc_nix_inl_priv.h b/drivers/common/cnxk/roc_nix_inl_priv.h
index be53a3fa81..2cdab6dc7a 100644
--- a/drivers/common/cnxk/roc_nix_inl_priv.h
+++ b/drivers/common/cnxk/roc_nix_inl_priv.h
@@ -27,8 +27,7 @@ struct nix_inl_dev {
 	uint32_t xaq_buf_size;
 	uint32_t xae_waes;
 	uint32_t iue;
-	uint64_t xaq_aura;
-	void *xaq_mem;
+	struct roc_sso_xaq_data xaq;
 	roc_nix_inl_sso_work_cb_t work_cb;
 	void *cb_args;
 
-- 
2.17.1


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

* Re: [PATCH] common/cnxk: use XAQ create API for inline device
  2021-12-13 20:56 [PATCH] common/cnxk: use XAQ create API for inline device pbhagavatula
@ 2022-01-24  9:07 ` Jerin Jacob
  0 siblings, 0 replies; 2+ messages in thread
From: Jerin Jacob @ 2022-01-24  9:07 UTC (permalink / raw)
  To: Pavan Nikhilesh
  Cc: Jerin Jacob, Nithin Dabilpuram, Kiran Kumar K, Sunil Kumar Kori,
	Satha Rao, dpdk-dev

On Tue, Dec 14, 2021 at 2:26 AM <pbhagavatula@marvell.com> wrote:
>
> From: Pavan Nikhilesh <pbhagavatula@marvell.com>
>
> Use the XAQ aura create and free API while initializing the

aura-> AURA

> inline device.
>
> Signed-off-by: Pavan Nikhilesh <pbhagavatula@marvell.com>

Acked-by: Jerin Jacob <jerinj@marvell.com>
Applied to dpdk-next-net-eventdev/for-main. Thanks

> ---
>  drivers/common/cnxk/roc_nix_debug.c    |  4 +-
>  drivers/common/cnxk/roc_nix_inl_dev.c  | 53 ++++++--------------------
>  drivers/common/cnxk/roc_nix_inl_priv.h |  3 +-
>  3 files changed, 15 insertions(+), 45 deletions(-)
>
> diff --git a/drivers/common/cnxk/roc_nix_debug.c b/drivers/common/cnxk/roc_nix_debug.c
> index 266935a6c5..5886650d6e 100644
> --- a/drivers/common/cnxk/roc_nix_debug.c
> +++ b/drivers/common/cnxk/roc_nix_debug.c
> @@ -1257,8 +1257,8 @@ roc_nix_inl_dev_dump(struct roc_nix_inl_dev *roc_inl_dev)
>         nix_dump("  \txaq_buf_size = %u", inl_dev->xaq_buf_size);
>         nix_dump("  \txae_waes = %u", inl_dev->xae_waes);
>         nix_dump("  \tiue = %u", inl_dev->iue);
> -       nix_dump("  \txaq_aura = 0x%" PRIx64, inl_dev->xaq_aura);
> -       nix_dump("  \txaq_mem = 0x%p", inl_dev->xaq_mem);
> +       nix_dump("  \txaq_aura = 0x%" PRIx64, inl_dev->xaq.aura_handle);
> +       nix_dump("  \txaq_mem = 0x%p", inl_dev->xaq.mem);
>
>         nix_dump("  \tinl_dev_rq:");
>         roc_nix_rq_dump(&inl_dev->rq);
> diff --git a/drivers/common/cnxk/roc_nix_inl_dev.c b/drivers/common/cnxk/roc_nix_inl_dev.c
> index 10912a6c93..dd93765a2b 100644
> --- a/drivers/common/cnxk/roc_nix_inl_dev.c
> +++ b/drivers/common/cnxk/roc_nix_inl_dev.c
> @@ -5,8 +5,6 @@
>  #include "roc_api.h"
>  #include "roc_priv.h"
>
> -#define XAQ_CACHE_CNT 0x7
> -
>  /* Default Rx Config for Inline NIX LF */
>  #define NIX_INL_LF_RX_CFG                                                      \
>         (ROC_NIX_LF_RX_CFG_DROP_RE | ROC_NIX_LF_RX_CFG_L2_LEN_ERR |            \
> @@ -220,10 +218,8 @@ nix_inl_sso_setup(struct nix_inl_dev *inl_dev)
>  {
>         struct sso_lf_alloc_rsp *sso_rsp;
>         struct dev *dev = &inl_dev->dev;
> -       uint32_t xaq_cnt, count, aura;
>         uint16_t hwgrp[1] = {0};
> -       struct npa_pool_s pool;
> -       uintptr_t iova;
> +       uint32_t xae_cnt;
>         int rc;
>
>         /* Alloc SSOW LF */
> @@ -244,41 +240,17 @@ nix_inl_sso_setup(struct nix_inl_dev *inl_dev)
>         inl_dev->xae_waes = sso_rsp->xaq_wq_entries;
>         inl_dev->iue = sso_rsp->in_unit_entries;
>
> -       /* Create XAQ pool */
> -       xaq_cnt = XAQ_CACHE_CNT;
> -       xaq_cnt += inl_dev->iue / inl_dev->xae_waes;
> -       plt_sso_dbg("Configuring %d xaq buffers", xaq_cnt);
> -
> -       inl_dev->xaq_mem = plt_zmalloc(inl_dev->xaq_buf_size * xaq_cnt,
> -                                      inl_dev->xaq_buf_size);
> -       if (!inl_dev->xaq_mem) {
> -               rc = NIX_ERR_NO_MEM;
> -               plt_err("Failed to alloc xaq buf mem");
> -               goto free_sso;
> -       }
> -
> -       memset(&pool, 0, sizeof(struct npa_pool_s));
> -       pool.nat_align = 1;
> -       rc = roc_npa_pool_create(&inl_dev->xaq_aura, inl_dev->xaq_buf_size,
> -                                xaq_cnt, NULL, &pool);
> +       xae_cnt = inl_dev->iue;
> +       rc = sso_hwgrp_init_xaq_aura(dev, &inl_dev->xaq, xae_cnt,
> +                                    inl_dev->xae_waes, inl_dev->xaq_buf_size,
> +                                    1);
>         if (rc) {
> -               plt_err("Failed to alloc aura for XAQ, rc=%d", rc);
> -               goto free_mem;
> -       }
> -
> -       /* Fill the XAQ buffers */
> -       iova = (uint64_t)inl_dev->xaq_mem;
> -       for (count = 0; count < xaq_cnt; count++) {
> -               roc_npa_aura_op_free(inl_dev->xaq_aura, 0, iova);
> -               iova += inl_dev->xaq_buf_size;
> +               plt_err("Failed to alloc SSO XAQ aura, rc=%d", rc);
> +               goto free_sso;
>         }
> -       roc_npa_aura_op_range_set(inl_dev->xaq_aura, (uint64_t)inl_dev->xaq_mem,
> -                                 iova);
> -
> -       aura = roc_npa_aura_handle_to_aura(inl_dev->xaq_aura);
>
>         /* Setup xaq for hwgrps */
> -       rc = sso_hwgrp_alloc_xaq(dev, aura, 1);
> +       rc = sso_hwgrp_alloc_xaq(dev, inl_dev->xaq.aura_handle, 1);
>         if (rc) {
>                 plt_err("Failed to setup hwgrp xaq aura, rc=%d", rc);
>                 goto destroy_pool;
> @@ -302,11 +274,7 @@ nix_inl_sso_setup(struct nix_inl_dev *inl_dev)
>  release_xaq:
>         sso_hwgrp_release_xaq(&inl_dev->dev, 1);
>  destroy_pool:
> -       roc_npa_pool_destroy(inl_dev->xaq_aura);
> -       inl_dev->xaq_aura = 0;
> -free_mem:
> -       plt_free(inl_dev->xaq_mem);
> -       inl_dev->xaq_mem = NULL;
> +       sso_hwgrp_free_xaq_aura(dev, &inl_dev->xaq, 0);
>  free_sso:
>         sso_lf_free(dev, SSO_LF_TYPE_HWGRP, 1);
>  free_ssow:
> @@ -335,6 +303,9 @@ nix_inl_sso_release(struct nix_inl_dev *inl_dev)
>         sso_lf_free(&inl_dev->dev, SSO_LF_TYPE_HWS, 1);
>         sso_lf_free(&inl_dev->dev, SSO_LF_TYPE_HWGRP, 1);
>
> +       /* Free the XAQ aura */
> +       sso_hwgrp_free_xaq_aura(&inl_dev->dev, &inl_dev->xaq, 0);
> +
>         return 0;
>  }
>
> diff --git a/drivers/common/cnxk/roc_nix_inl_priv.h b/drivers/common/cnxk/roc_nix_inl_priv.h
> index be53a3fa81..2cdab6dc7a 100644
> --- a/drivers/common/cnxk/roc_nix_inl_priv.h
> +++ b/drivers/common/cnxk/roc_nix_inl_priv.h
> @@ -27,8 +27,7 @@ struct nix_inl_dev {
>         uint32_t xaq_buf_size;
>         uint32_t xae_waes;
>         uint32_t iue;
> -       uint64_t xaq_aura;
> -       void *xaq_mem;
> +       struct roc_sso_xaq_data xaq;
>         roc_nix_inl_sso_work_cb_t work_cb;
>         void *cb_args;
>
> --
> 2.17.1
>

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

end of thread, other threads:[~2022-01-24  9:07 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-12-13 20:56 [PATCH] common/cnxk: use XAQ create API for inline device pbhagavatula
2022-01-24  9:07 ` Jerin Jacob

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