Re: [dpdk-dev] [PATCH v10 0/3] add actions to modify header fields

[Adrien Mazarguil <adrien.mazarguil@6wind.com>]

On Tue, Jul 02, 2019 at 05:44:25PM +0300, Dekel Peled wrote:
> Patch [1] implemented set of header modification actions in MLX PMD, based on ethdev and testpmd updates included in [2].
> This series implements support of additional header modification actions, in ethdev, testpmd, and MLX5 PMD.
> 
> Original work by Xiaoyu Min.
> 
> [1] http://patches.dpdk.org/patch/49310/
> [2] http://mails.dpdk.org/archives/dev/2018-August/109672.html
> 
> ---
> v2: apply code review comments.
> v3: apply additional code review comments.
>     - Update documentation of new commands.
>     - Use common general struct for all commands.
> v4: apply checkpatch comments.
> v5: apply additional code review comments.
>     - Add 8, 16, 32 bit types to union.
>     - Update struct name and documentation.
> v6: expand description of new struct in h file and commit log.
> v7: - Remove the common general struct with union added in v3 & v5.
>     - Commands take a simple integer value, not enclosed in a structure.
>     - Use separate commands for INC and DEC with 32 bit unsigned value
>       of type rte_be32_t.
> v8: clean redundant comments refering to removed structure.
> v9: - Send the announcement of new approach (use action with single
>       argument configuration) in separate patch before this series,
>       see http://patches.dpdk.org/patch/55773/.
>     - Add PMD release notes update.
> v10: - Reorder release notes update properly.
>      - Update comments for doxygen.

Phew!

Acked-by: Adrien Mazarguil <adrien.mazarguil@6wind.com>

-- 
Adrien Mazarguil
6WIND