* [PATCH] common/cnxk: reduce channel count per LMAC
@ 2023-01-04 6:07 skori
2023-01-13 5:23 ` Jerin Jacob
0 siblings, 1 reply; 3+ messages in thread
From: skori @ 2023-01-04 6:07 UTC (permalink / raw)
To: Nithin Dabilpuram, Kiran Kumar K, Sunil Kumar Kori, Satha Rao; +Cc: dev, stable
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>
---
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
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] common/cnxk: reduce channel count per LMAC
2023-01-04 6:07 [PATCH] common/cnxk: reduce channel count per LMAC skori
@ 2023-01-13 5:23 ` Jerin Jacob
2023-01-13 6:29 ` [EXT] " Sunil Kumar Kori
0 siblings, 1 reply; 3+ messages in thread
From: Jerin Jacob @ 2023-01-13 5:23 UTC (permalink / raw)
To: skori; +Cc: Nithin Dabilpuram, Kiran Kumar K, Satha Rao, dev, stable
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
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* RE: [EXT] Re: [PATCH] common/cnxk: reduce channel count per LMAC
2023-01-13 5:23 ` Jerin Jacob
@ 2023-01-13 6:29 ` Sunil Kumar Kori
0 siblings, 0 replies; 3+ messages in thread
From: Sunil Kumar Kori @ 2023-01-13 6:29 UTC (permalink / raw)
To: Jerin Jacob
Cc: Nithin Kumar Dabilpuram, Kiran Kumar Kokkilagadda,
Satha Koteswara Rao Kottidi, dev, stable
Ack.
Regards
Sunil Kumar Kori
> -----Original Message-----
> From: Jerin Jacob <jerinjacobk@gmail.com>
> Sent: Friday, January 13, 2023 10:54 AM
> To: Sunil Kumar Kori <skori@marvell.com>
> Cc: Nithin Kumar Dabilpuram <ndabilpuram@marvell.com>; Kiran Kumar
> Kokkilagadda <kirankumark@marvell.com>; Satha Koteswara Rao Kottidi
> <skoteshwar@marvell.com>; dev@dpdk.org; stable@dpdk.org
> Subject: [EXT] Re: [PATCH] common/cnxk: reduce channel count per LMAC
>
> External Email
>
> ----------------------------------------------------------------------
> 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
> >
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2023-01-13 6:29 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-01-04 6:07 [PATCH] common/cnxk: reduce channel count per LMAC skori
2023-01-13 5:23 ` Jerin Jacob
2023-01-13 6:29 ` [EXT] " Sunil Kumar Kori
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).