Re: [PATCH] app/testpmd: remove useless check

2022-10-17 Thread Andrew Rybchenko
On 10/14/22 12:52, Wang, YuanX wrote: Hi Singh, -Original Message- From: Singh, Aman Deep Sent: Friday, October 14, 2022 5:37 PM To: Wang, YuanX ; Zhang, Yuying Cc: dev@dpdk.org; Ding, Xuan ; Tang, Yaqi Subject: Re: [PATCH] app/testpmd: remove useless check Thanks for the patch

RE: [PATCH] app/testpmd: remove useless check

2022-10-14 Thread Wang, YuanX
Hi Singh, > -Original Message- > From: Singh, Aman Deep > Sent: Friday, October 14, 2022 5:37 PM > To: Wang, YuanX ; Zhang, Yuying > > Cc: dev@dpdk.org; Ding, Xuan ; Tang, Yaqi > > Subject: Re: [PATCH] app/testpmd: remove useless check > > Thanks for the

Re: [PATCH] app/testpmd: remove useless check

2022-10-14 Thread Singh, Aman Deep
Thanks for the patch Yuan. On 10/12/2022 7:59 PM, Yuan Wang wrote: Protocol header sequence checking is supported in the ethdev library, the application does not need to do it again. I would like to rephrase the patch title to make it more specific. Like "remove useless check" to "remove unuse

[PATCH] app/testpmd: remove useless check

2022-10-11 Thread Yuan Wang
Protocol header sequence checking is supported in the ethdev library, the application does not need to do it again. Coverity issue: 381396 Fixes: 52e2e7edcf48 ("app/testpmd: add protocol-based buffer split") Signed-off-by: Yuan Wang --- app/test-pmd/cmdline.c| 6 ++ app/test-pmd/paramet