On 10/13/2021 7:45 PM, Viacheslav Ovsiienko wrote:
The generic modify field flow action introduced in [1] has some issues related to the immediate source operand:- immediate source can be presented either as an unsigned 64-bit integer or pointer to data pattern in memory. There was no explicit pointer field defined in the union - the byte ordering for 64-bit integer was not specified. Many fields have lesser lengths and byte ordering is crucial. - how the bit offset is applied to the immediate source field was not defined and documented - 64-bit integer size is not enough to provide MAC and IPv6 addresses In order to cover the issues and exclude any ambiguities the following is done: - introduce the explicit pointer field in rte_flow_action_modify_data structure - replace the 64-bit unsigned integer with 16-byte array - update the modify field flow action documentation Appropriate commit message has been removed. [1] commit 73b68f4c54a0 ("ethdev: introduce generic modify flow action") [2] RFC: http://patches.dpdk.org/project/dpdk/patch/[email protected]/ [3] Deprecation notice: http://patches.dpdk.org/project/dpdk/patch/[email protected]/ [4] v1 - http://patches.dpdk.org/project/dpdk/cover/[email protected]/ [5] v2 - http://patches.dpdk.org/project/dpdk/patch/[email protected]/ [6] v3 - http://patches.dpdk.org/project/dpdk/cover/[email protected]/ [7] v4 - http://patches.dpdk.org/project/dpdk/cover/[email protected]/ [8] v5 - http://patches.dpdk.org/project/dpdk/patch/[email protected]/ v2: - comments addressed - documentation updated - typos fixed - mlx5 PMD updated v3: - comments addressed - documentation updated - typos fixed v4: - removed errorneously added Ack by Ori K. for mlx5 patch - mlx5 patch updated - bug fixes and cleanup v5: - fix compilation issue with unused variable in mlx5 v6: - fix compilation issue with unused variable in mlx5 Viacheslav Ovsiienko (5): ethdev: update modify field flow action ethdev: fix missed experimental tag for modify field action app/testpmd: update modify field flow action support app/testpmd: fix hex string parser in flow commands net/mlx5: update modify field action
Series applied to dpdk-next-net/main, thanks.
