On Wed, Nov 3, 2021 at 6:24 AM Ferruh Yigit <ferruh.yi...@intel.com> wrote:
>
> On 11/3/2021 12:52 AM, Ajit Khaparde wrote:
> > From: Kishore Padmanabha <kishore.padmana...@broadcom.com>
> >
> > The accumulation of flow counters is not determined by the
> > application device arguments anymore. Instead it is now dictated by
> > the platform capabilities whether to do software based accumulation or not.
> >
> > Signed-off-by: Kishore Padmanabha <kishore.padmana...@broadcom.com>
> > Signed-off-by: Venkat Duvvuru <venkatkumar.duvv...@broadcom.com>
> > Reviewed-by: Mike Baucom <michael.bau...@broadcom.com>
> > Reviewed-by: Ajit Khaparde <ajit.khapa...@broadcom.com>
>
> <...>
>
> > @@ -87,7 +87,6 @@ static const struct rte_pci_id bnxt_pci_id_map[] = {
> > { .vendor_id = 0, /* sentinel */ },
> > };
> >
> > -#define BNXT_DEVARG_ACCUM_STATS "accum-stats"
> > #define BNXT_DEVARG_FLOW_XSTAT "flow-xstat"
> > #define BNXT_DEVARG_MAX_NUM_KFLOWS "max-num-kflows"
> > #define BNXT_DEVARG_REPRESENTOR "representor"
>
> Hi Ajit,
>
> Not for this patch, but I recognized that runtime devargs for the driver
> is not documented at all.
>
> Can you please have a separate patch to document them in bnxt.rst?
Hi Freeuh,
Yes. I am working on that.