[v2] doc/dlb2: update dlb2 documentation

  Number of direct credits, atomic inflight and history list are
updated to DLB2.0 supported sizes. As DLB2.0 does not provide
dev arg to override the default per-queue atomic inflight
allocation, it is removed from the documentation.

Signed-off-by: Rashmi Shetty <rashmi.shetty@intel.com>
---
 doc/guides/eventdevs/dlb2.rst | 19 ++++++-------------
 1 file changed, 6 insertions(+), 13 deletions(-)
  

> -----Original Message-----
> From: Shetty, Rashmi <rashmi.shetty@intel.com>
> Sent: Tuesday, December 7, 2021 5:02 PM
> To: dev@dpdk.org
> Cc: jerinj@marvell.com; Van Haaren, Harry <harry.van.haaren@intel.com>;
> Pathak, Pravin <pravin.pathak@intel.com>; Chen, Mike Ximing
> <mike.ximing.chen@intel.com>; McDaniel, Timothy
> <timothy.mcdaniel@intel.com>; Shetty, Rashmi <rashmi.shetty@intel.com>
> Subject: [PATCH v2] doc/dlb2: update dlb2 documentation
> 
> Number of direct credits, atomic inflight and history list are
> updated to DLB2.0 supported sizes. As DLB2.0 does not provide
> dev arg to override the default per-queue atomic inflight
> allocation, it is removed from the documentation.
> 
> Signed-off-by: Rashmi Shetty <rashmi.shetty@intel.com>
> ---
>  doc/guides/eventdevs/dlb2.rst | 19 ++++++-------------
>  1 file changed, 6 insertions(+), 13 deletions(-)
> 
> diff --git a/doc/guides/eventdevs/dlb2.rst b/doc/guides/eventdevs/dlb2.rst
> index bce984ca08..bc53618b53 100644
> --- a/doc/guides/eventdevs/dlb2.rst
> +++ b/doc/guides/eventdevs/dlb2.rst
> @@ -151,7 +151,7 @@ load-balanced queues, and directed credits are used for
> directed queues.
>  These pools' sizes are controlled by the nb_events_limit field in struct
>  rte_event_dev_config. The load-balanced pool is sized to contain
>  nb_events_limit credits, and the directed pool is sized to contain
> -nb_events_limit/4 credits. The directed pool size can be overridden with the
> +nb_events_limit/2 credits. The directed pool size can be overridden with the
>  num_dir_credits devargs argument, like so:
> 
>      .. code-block:: console
> @@ -239,8 +239,8 @@ queue A.
>  Due to this, workers should stop retrying after a time, release the events it
>  is attempting to enqueue, and dequeue more events. It is important that the
>  worker release the events and don't simply set them aside to retry the enqueue
> -again later, because the port has limited history list size (by default, twice
> -the port's dequeue_depth).
> +again later, because the port has limited history list size (by default, same
> +as port's dequeue_depth).
> 
>  Priority
>  ~~~~~~~~
> @@ -309,18 +309,11 @@ scheduled. The likelihood of this case depends on the
> eventdev configuration,
>  traffic behavior, event processing latency, potential for a worker to be
>  interrupted or otherwise delayed, etc.
> 
> -By default, the PMD allocates 16 buffer entries for each load-balanced queue,
> -which provides an even division across all 128 queues but potentially wastes
> +By default, the PMD allocates 64 buffer entries for each load-balanced queue,
> +which provides an even division across all 32 queues but potentially wastes
>  buffer space (e.g. if not all queues are used, or aren't used for atomic
>  scheduling).
> 
> -The PMD provides a dev arg to override the default per-queue allocation. To
> -increase per-queue atomic-inflight allocation to (for example) 64:
> -
> -    .. code-block:: console
> -
> -       --allow ea:00.0,atm_inflights=64
> -
>  QID Depth Threshold
>  ~~~~~~~~~~~~~~~~~~~
> 
> @@ -337,7 +330,7 @@ Per queue threshold metrics are tracked in the DLB
> xstats, and are also
>  returned in the impl_opaque field of each received event.
> 
>  The per qid threshold can be specified as part of the device args, and
> -can be applied to all queue, a range of queues, or a single queue, as
> +can be applied to all queues, a range of queues, or a single queue, as
>  shown below.
> 
>      .. code-block:: console
> --
> 2.25.1

    Reviewed-by: Timothy McDaniel <timothy.mcdaniel@intel.com>
  

On Tue, Dec 14, 2021 at 9:41 PM McDaniel, Timothy
<timothy.mcdaniel@intel.com> wrote:
>
> > -----Original Message-----
> > From: Shetty, Rashmi <rashmi.shetty@intel.com>
> > Sent: Tuesday, December 7, 2021 5:02 PM
> > To: dev@dpdk.org
> > Cc: jerinj@marvell.com; Van Haaren, Harry <harry.van.haaren@intel.com>;
> > Pathak, Pravin <pravin.pathak@intel.com>; Chen, Mike Ximing
> > <mike.ximing.chen@intel.com>; McDaniel, Timothy
> > <timothy.mcdaniel@intel.com>; Shetty, Rashmi <rashmi.shetty@intel.com>
> > Subject: [PATCH v2] doc/dlb2: update dlb2 documentation
> >
> > Number of direct credits, atomic inflight and history list are
> > updated to DLB2.0 supported sizes. As DLB2.0 does not provide
> > dev arg to override the default per-queue atomic inflight
> > allocation, it is removed from the documentation.
> >
> > Signed-off-by: Rashmi Shetty <rashmi.shetty@intel.com>
of queues, or a single queue, as
> >  shown below.
> >
> >      .. code-block:: console
> > --
> > 2.25.1
>
>     Reviewed-by: Timothy McDaniel <timothy.mcdaniel@intel.com>
Since it as fix and needs to update stable tree. Updated as follows
and applied to dpdk-next-net-eventdev/for-main. Thanks



   doc/dlb2: fix dlb2 documentation

    Number of direct credits, atomic inflight and history list are
    updated to DLB2.0 supported sizes. As DLB2.0 does not provide
    dev arg to override the default per-queue atomic inflight
    allocation, it is removed from the documentation.

    Fixes: f3cad285bb88 ("event/dlb2: add infos get and configure")
    Cc: stable@dpdk.org

    Signed-off-by: Rashmi Shetty <rashmi.shetty@intel.com>
    Reviewed-by: Timothy McDaniel <timothy.mcdaniel@intel.com>