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

test suite reviews and discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://inbox.dpdk.org/dts/0 dts/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 dts dts/ http://inbox.dpdk.org/dts \
		dts@dpdk.org
	public-inbox-index dts

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://inbox.dpdk.org/inbox.dpdk.dts


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git