From: Jerin Jacob <jerinjacobk@gmail.com>
To: skori@marvell.com
Cc: Nithin Dabilpuram <ndabilpuram@marvell.com>,
Kiran Kumar K <kirankumark@marvell.com>,
Satha Rao <skoteshwar@marvell.com>,
dev@dpdk.org, stable@dpdk.org
Subject: Re: [PATCH] common/cnxk: reduce channel count per LMAC
Date: Fri, 13 Jan 2023 10:53:58 +0530 [thread overview]
Message-ID: <CALBAE1OQfOqSm9rv-4NJsprYW7j+sx2iHnUTUWGx=ZsopNF9Yw@mail.gmail.com> (raw)
In-Reply-To: <20230104060712.283128-1-skori@marvell.com>
On Wed, Jan 4, 2023 at 11:37 AM <skori@marvell.com> wrote:
>
> From: Sunil Kumar Kori <skori@marvell.com>
>
> Due to limitation, imposed by latest kernel, maximum number of
> channel supported per LMAC is reduced to 8. Because of this change
> application gets failed to initialize for more channels.
>
> Also this limitation impacts PFC functional behaviour.
>
> So patch just aligns the supported number of channel with kernel.
>
> Fixes: 20d02329cdc2 ("common/cnxk: support priority flow control")
> Cc: stable@dpdk.org
>
> Signed-off-by: Sunil Kumar Kori <skori@marvell.com>
Updated the git commit as follows and applied to
dpdk-next-net-mrvl/for-next-net. Thanks
common/cnxk: reduce channel count per LMAC
Due to limitation, imposed by latest AF kernel driver, maximum number of
channel supported per LMAC is reduced to 8. Because of this change
application gets failed to initialize for more channels.
Also this limitation impacts PFC functional behaviour.
So patch just aligns the supported number of channel with AF kernel
driver.
Fixes: 20d02329cdc2 ("common/cnxk: support priority flow control")
Cc: stable@dpdk.org
Signed-off-by: Sunil Kumar Kori <skori@marvell.com>
> ---
> drivers/common/cnxk/roc_mbox.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/common/cnxk/roc_mbox.h b/drivers/common/cnxk/roc_mbox.h
> index 8b0384c737..fd9d3e73cd 100644
> --- a/drivers/common/cnxk/roc_mbox.h
> +++ b/drivers/common/cnxk/roc_mbox.h
> @@ -1169,7 +1169,7 @@ struct nix_bp_cfg_req {
> * so maximum 256 channels are possible.
> */
> #define NIX_MAX_CHAN 256
> -#define NIX_CGX_MAX_CHAN 16
> +#define NIX_CGX_MAX_CHAN 8
> #define NIX_LBK_MAX_CHAN 1
> struct nix_bp_cfg_rsp {
> struct mbox_msghdr hdr;
> --
> 2.25.1
>
next prev parent reply other threads:[~2023-01-13 5:24 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-01-04 6:07 skori
2023-01-13 5:23 ` Jerin Jacob [this message]
2023-01-13 6:29 ` [EXT] " Sunil Kumar Kori
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='CALBAE1OQfOqSm9rv-4NJsprYW7j+sx2iHnUTUWGx=ZsopNF9Yw@mail.gmail.com' \
--to=jerinjacobk@gmail.com \
--cc=dev@dpdk.org \
--cc=kirankumark@marvell.com \
--cc=ndabilpuram@marvell.com \
--cc=skori@marvell.com \
--cc=skoteshwar@marvell.com \
--cc=stable@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).