Re: [PATCH v5 5/5] doc: update release notes for 24.07

[David Marchand <david.marchand@redhat.com>]

On Wed, Jun 19, 2024 at 11:02 PM Abdullah Sevincer
<abdullah.sevincer@intel.com> wrote:
>
> Update release notes for new DLB features.
>
> Signed-off-by: Abdullah Sevincer <abdullah.sevincer@intel.com>
> ---
>  doc/guides/rel_notes/release_24_07.rst | 32 ++++++++++++++++++++++++++
>  1 file changed, 32 insertions(+)
>
> diff --git a/doc/guides/rel_notes/release_24_07.rst b/doc/guides/rel_notes/release_24_07.rst
> index 7c88de381b..b4eb819503 100644
> --- a/doc/guides/rel_notes/release_24_07.rst
> +++ b/doc/guides/rel_notes/release_24_07.rst
> @@ -144,6 +144,38 @@ New Features
>
>    Added an API that allows the user to reclaim the defer queue with RCU.
>
> +* **Added API to support HW delayed token feature for DLB 2.5 device.**
> +
> +  * Added API ``rte_pmd_dlb2_set_port_params`` to support delayed token
> +    feature for DLB 2.5 device. The feature will resume CQ scheduling
> +    when the number of pending completions fall below a configured
> +    threshold.
> +
> +* **Introduced dynamic HL (History List) feature for DLB device.**
> +
> +  * Users can configure history list entries dynamically by passing
> +    parameters ``use_default_hl`` and ``alloc_hl_entries``.
> +
> +  * When 'use_default_hl = 1', Per port HL is set to
> +    DLB2_FIXED_CQ_HL_SIZE (32) and command line parameter
> +    alloc_hl_entries is ignored.
> +
> +  * When 'use_default_hl = 0', Per LDB port HL = 2 * CQ depth and per
> +    port HL is set to 2 * DLB2_FIXED_CQ_HL_SIZE.
> +
> +* **DLB credit handling scenario improvements.**
> +
> +  * When ports hold on to credits but can't release them due to insufficient
> +    accumulation (less than 2 * credit quanta) deadlocks may occur.
> +    Improvement made for worker ports to release all accumulated credits when
> +    back-to-back zero poll count reaches preset threshold and producer ports
> +    release all accumulated credits if enqueue fails for a consecutive number
> +    of retries.
> +
> +  * New meson options are provided for handling credits. Valid options
> +    are ``bypass_fence``, ``hw_credits_checks``, ``sw_credits_checks`` and
> +    ``type_check``. These options need to be provided in meson in comma
> +    separated form.
>

Those 3 entries can be gathered under a single item about the dlb2 driver.
Like:

* **Updated dlb2 eventdev driver.**

  * Added API ``rte_pmd_dlb2_set_port_params`` to support delayed token...
    ...

  * Introduced dynamic HL (History List) feature for DLB device...
    ...
etc...


Besides, those doc updates should be split and go with the patches
that introduce the features.
This comment applies to the previous doc patch too.

Thanks.


-- 
David Marchand