Hi
> -----Original Message-----
> From: Zhang, AlvinX <alvinx.zh...@intel.com>
> Sent: Saturday, September 18, 2021 11:07
> To: Li, Xiaoyun <xiaoyun...@intel.com>; Ananyev, Konstantin
> <konstantin.anan...@intel.com>
> Cc: dev@dpdk.org; Zhang, AlvinX <alvinx.zh...@intel.com>; sta...@dpdk.org
> Subject: [PATCH v3 2/2] app/testpmd: fix txonly forwording
>
> When random number of Tx segments is enabled, because the actual number of
> segments may be only one, the first segment of the Tx packets must
> accommodate a complete being sending Eth/IP/UDP packet.
>
> Besides, if multiple flow is enabled, the forwarding will update the IP and
> UDP
> header, these headers shouldn't cross segments.
> This also requires the first Tx segment can accommodate a complete
> Eth/IP/UDP packet.
>
> In addition, if time stamp is enabled, the forwarding need more Tx segment
needs
> space for time stamp information.
>
> This patch adds checks in beginning of forward engine to make sure all above
> conditions are meet.
are met
>
> Bugzilla ID: 797
> Fixes: 79bec05b32b7 ("app/testpmd: add ability to split outgoing packets")
> Cc: sta...@dpdk.org
This patch is based on the 1st patch of this patch-set. So if you only send
this patch to stable, there'll be incompatible issues.
Either you send both patches to stable or you just fix this for the future
(don't cc to stable).
But the patch looks good to me.
>
> Signed-off-by: Alvin Zhang <alvinx.zh...@intel.com>
> ---
> app/test-pmd/txonly.c | 67
> ++++++++++++++++++++++++++++++++++++++++++---------
> 1 file changed, 55 insertions(+), 12 deletions(-)
>
