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.mazarg...@6wind.com> -- Adrien Mazarguil 6WIND