test suite reviews and discussions
 help / color / mirror / Atom feed
* [dts] [PATCH V3] checksum_offload/l4_tx:remove check sctp test step
@ 2021-12-20 13:58 Zhimin Huang
  2022-01-05  5:47 ` Tu, Lijuan
  0 siblings, 1 reply; 2+ messages in thread
From: Zhimin Huang @ 2021-12-20 13:58 UTC (permalink / raw)
  To: dts; +Cc: Zhimin Huang

cause root commit is b2a9e4a8,it affects the tx sctp checksum.
but customers only require UDP and TCP,so remove check sctp checksum test step. 

Signed-off-by: Zhimin Huang <zhiminx.huang@intel.com>
---
 test_plans/checksum_offload_test_plan.rst | 2 +-
 tests/TestSuite_checksum_offload.py       | 1 -
 2 files changed, 1 insertion(+), 2 deletions(-)

diff --git a/test_plans/checksum_offload_test_plan.rst b/test_plans/checksum_offload_test_plan.rst
index 6748aa34..47f35236 100644
--- a/test_plans/checksum_offload_test_plan.rst
+++ b/test_plans/checksum_offload_test_plan.rst
@@ -297,7 +297,7 @@ Test Case: Hardware Checksum Check L4 TX
 ===========================================
 This test involves testing many different scenarios with a L4 checksum.
 A variety of tunneling protocols, L3 protocols and L4 protocols are combined
-to test as many scenarios as possible. Currently, UDP, TCP and SCTP are used
+to test as many scenarios as possible. Currently, UDP and TCP are used
 as L4 protocols, with IP and IPv6 being used at level 3. The tested tunneling
 protocols are VXLAN and GRE. This test is used to determine whether the
 hardware offloading of checksums works properly.
diff --git a/tests/TestSuite_checksum_offload.py b/tests/TestSuite_checksum_offload.py
index 6e2b59c6..8f47c546 100644
--- a/tests/TestSuite_checksum_offload.py
+++ b/tests/TestSuite_checksum_offload.py
@@ -69,7 +69,6 @@ l3_proto_classes = [
 l4_proto_classes = [
     UDP,
     TCP,
-    SCTP
 ]
 
 tunnelling_proto_classes = [
-- 
2.17.1


^ permalink raw reply	[flat|nested] 2+ messages in thread

* RE: [dts] [PATCH V3] checksum_offload/l4_tx:remove check sctp test step
  2021-12-20 13:58 [dts] [PATCH V3] checksum_offload/l4_tx:remove check sctp test step Zhimin Huang
@ 2022-01-05  5:47 ` Tu, Lijuan
  0 siblings, 0 replies; 2+ messages in thread
From: Tu, Lijuan @ 2022-01-05  5:47 UTC (permalink / raw)
  To: Huang, ZhiminX, dts; +Cc: Huang, ZhiminX

> -----Original Message-----
> From: Zhimin Huang <zhiminx.huang@intel.com>
> Sent: 2021年12月20日 21:58
> To: dts@dpdk.org
> Cc: Huang, ZhiminX <zhiminx.huang@intel.com>
> Subject: [dts] [PATCH V3] checksum_offload/l4_tx:remove check sctp test step
> 
> cause root commit is b2a9e4a8,it affects the tx sctp checksum.
> but customers only require UDP and TCP,so remove check sctp checksum test
> step.
> 
> Signed-off-by: Zhimin Huang <zhiminx.huang@intel.com>

Applied.

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2022-01-05  5:47 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-12-20 13:58 [dts] [PATCH V3] checksum_offload/l4_tx:remove check sctp test step Zhimin Huang
2022-01-05  5:47 ` Tu, Lijuan

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).