[EXT] Re: [PATCH] common/cnxk: reduce channel count per LMAC

[Sunil Kumar Kori]

Ack.

Regards
Sunil Kumar Kori

> -----Original Message-----
> From: Jerin Jacob <jerinjacobk at gmail.com>
> Sent: Friday, January 13, 2023 10:54 AM
> To: Sunil Kumar Kori <skori at marvell.com>
> Cc: Nithin Kumar Dabilpuram <ndabilpuram at marvell.com>; Kiran Kumar
> Kokkilagadda <kirankumark at marvell.com>; Satha Koteswara Rao Kottidi
> <skoteshwar at marvell.com>; dev at dpdk.org; stable at 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 at marvell.com> wrote:
> >
> > From: Sunil Kumar Kori <skori at 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 at dpdk.org
> >
> > Signed-off-by: Sunil Kumar Kori <skori at 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 at dpdk.org
> 
>     Signed-off-by: Sunil Kumar Kori <skori at 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
> >