[V2] pmd/checksum_checking: supplement the test plan

Message ID 20221206073659.385898-1-hongbox.li@intel.com (mailing list archive)
State Accepted
Headers
Series [V2] pmd/checksum_checking: supplement the test plan |

Checks

Context Check Description
ci/Intel-dts-format-test success Testing OK
ci/Intel-dts-pylama-test success Testing OK
ci/Intel-dts-suite-test warning SKIPPED

Commit Message

Li, HongboX Dec. 6, 2022, 7:36 a.m. UTC
  the case of pmd/checksum_checking only cover in testcase not in test plan. so supplement the test plan.

Signed-off-by: Hongbo Li <hongbox.li@intel.com>
---
v2:
-Expected results are added and formatted

 test_plans/pmd_test_plan.rst | 37 ++++++++++++++++++++++++++++++++++++
 1 file changed, 37 insertions(+)
  

Comments

Huang, ZhiminX Dec. 6, 2022, 10:33 a.m. UTC | #1
> -----Original Message-----
> From: Hongbo Li <hongbox.li@intel.com>
> Sent: Tuesday, December 6, 2022 3:37 PM
> To: dts@dpdk.org
> Cc: Li, HongboX <hongbox.li@intel.com>
> Subject: [dts][PATCH V2] pmd/checksum_checking: supplement the test plan
> 
> the case of pmd/checksum_checking only cover in testcase not in test plan. so
> supplement the test plan.
> 
> Signed-off-by: Hongbo Li <hongbox.li@intel.com>
> ---
> v2:
> -Expected results are added and formatted
> 
Acked-by: Zhimin Huang <zhiminx.huang@intel.com >
  
Tu, Lijuan Dec. 26, 2022, 5:01 a.m. UTC | #2
On Tue,  6 Dec 2022 07:36:59 +0000, Hongbo Li <hongbox.li@intel.com> wrote:
> the case of pmd/checksum_checking only cover in testcase not in test plan. so supplement the test plan.
> 
> Signed-off-by: Hongbo Li <hongbox.li@intel.com>

Acked-by: Lijuan Tu <lijuan.tu@intel.com>
Applied, thanks
  

Patch

diff --git a/test_plans/pmd_test_plan.rst b/test_plans/pmd_test_plan.rst
index c371edb1..ca2709a3 100644
--- a/test_plans/pmd_test_plan.rst
+++ b/test_plans/pmd_test_plan.rst
@@ -73,6 +73,43 @@  Test Case: Packet Checking
    which will be forwarded by the DUT. The test checks if the packets are correctly forwarded and
    if both RX and TX packet sizes match by `show port all stats`
 
+Test Case: Checksum checking
+============================
+
+1. Start testpmd with rxfreet=0 and set forward mode to csum::
+
+    ./x86_64-native-linuxapp-gcc/app/dpdk-testpmd -l 1-2 -n 4 -a 0000:18:00.0 -a 0000:1b:00.0 --file-prefix=dpdk_2179994_20221121150432    -- -i --portmask=0x3 --enable-rx-cksum --disable-rss --rxd=1024 --txd=1024 --rxfreet=0
+    testpmd> set fwd csum
+    testpmd> start
+
+2. The tester sends 4 messages with error checksum, and check whether the message is correctly forwarded by DUT::
+
+    sendp([Ether(dst=00:11:22:33:44:55, src="52:00:00:00:00:00")/IP(len=46)/UDP(chksum=0x1)/Raw(load="P"*18)], iface="ens9", count=4)
+
+3. Stop testpmd and check whether the l4 checksum error reported by the Rx port is equal to 4.
+
+    ---------------------- Forward statistics for port 0  ----------------------
+    RX-packets: 0              RX-dropped: 0             RX-total: 0
+    Bad-ipcsum: 0              Bad-l4csum: 0             Bad-outer-l4csum: 0
+    Bad-outer-ipcsum: 0
+    TX-packets: 4              TX-dropped: 0             TX-total: 4
+    ----------------------------------------------------------------------------
+
+    ---------------------- Forward statistics for port 1  ----------------------
+    RX-packets: 4              RX-dropped: 0             RX-total: 4
+    Bad-ipcsum: 0              Bad-l4csum: 4             Bad-outer-l4csum: 0
+    Bad-outer-ipcsum: 0
+    TX-packets: 0              TX-dropped: 0             TX-total: 0
+    ----------------------------------------------------------------------------
+
+    +++++++++++++++ Accumulated forward statistics for all ports+++++++++++++++
+    RX-packets: 4              RX-dropped: 0             RX-total: 4
+    TX-packets: 4              TX-dropped: 0             TX-total: 4
+    ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
+
+4. change the value of "rxfreet"(0, 8, 16, 32, 64, 128), and start testpmd, repeat step 1-3, check the result is same.
+
+
 Test Case: Packet Checking in scalar mode
 =========================================