From: Jerin Jacob <jerinjacobk@gmail.com>
To: Pavan Nikhilesh <pbhagavatula@marvell.com>
Cc: Jerin Jacob <jerinj@marvell.com>,
Nithin Dabilpuram <ndabilpuram@marvell.com>,
Kiran Kumar K <kirankumark@marvell.com>,
Sunil Kumar Kori <skori@marvell.com>,
Satha Rao <skoteshwar@marvell.com>, dpdk-dev <dev@dpdk.org>
Subject: Re: [PATCH] common/cnxk: use XAQ create API for inline device
Date: Mon, 24 Jan 2022 14:37:07 +0530 [thread overview]
Message-ID: <CALBAE1P6-iepLRkBVUL8OtSEQTcyKYW32kKpeQdpQwk8ndL2GQ@mail.gmail.com> (raw)
In-Reply-To: <20211213205633.5724-1-pbhagavatula@marvell.com>
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
>
prev parent reply other threads:[~2022-01-24 9:07 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-12-13 20:56 pbhagavatula
2022-01-24 9:07 ` Jerin Jacob [this message]
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=CALBAE1P6-iepLRkBVUL8OtSEQTcyKYW32kKpeQdpQwk8ndL2GQ@mail.gmail.com \
--to=jerinjacobk@gmail.com \
--cc=dev@dpdk.org \
--cc=jerinj@marvell.com \
--cc=kirankumark@marvell.com \
--cc=ndabilpuram@marvell.com \
--cc=pbhagavatula@marvell.com \
--cc=skori@marvell.com \
--cc=skoteshwar@marvell.com \
/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).