[dts] [PATCH V3 1/3] ethtool_stats: update test plan

[yufengmx]

use testpmd xstat data as reference data to check proc info tool xstat data.
update test plan content.

Signed-off-by: yufengmx <yufengx.mo at intel.com>
---
 test_plans/ethtool_stats_test_plan.rst | 66 ++++++++++++++++++--------
 1 file changed, 45 insertions(+), 21 deletions(-)

diff --git a/test_plans/ethtool_stats_test_plan.rst b/test_plans/ethtool_stats_test_plan.rst
index 13b5909..95f9e7a 100644
--- a/test_plans/ethtool_stats_test_plan.rst
+++ b/test_plans/ethtool_stats_test_plan.rst
@@ -47,13 +47,20 @@ that DPDK supports. The idea behind this epic is two fold as following.
     - the retrieval of aggregate stats upon request (Top level stats).
     - the retrieval of the extended NIC stats.
     - grouping of stats logically so they can be retrieved per logical grouping.
-    - the option to enable/disable the stats groups to retrieve similar to set private flags in ethtool.
+    - the option to enable/disable the stats groups to retrieve similar to set
+      private flags in ethtool.
 
 Prerequisites
 =============
 
 2xNICs (2 full duplex optical ports per NIC). One on dut, another one on tester,
-link them together. Update two nics' firmware to latest version.
+link them together. Update two nics' firmware to latest version::
+
+            Tester                          DUT
+          .-------.                      .-------.
+          | port0 | <------------------> | port0 |
+          | port1 | <------------------> | port1 |
+          '-------'                      '-------'
 
 Test cases
 ==========
@@ -64,8 +71,8 @@ bind two ports::
 
     ./usertools/dpdk-devbind.py --bind=igb_uio <pci address 1> <pci address 2>
 
-Test Case: xstat command set intergrity
----------------------------------------
+Test Case: xstat options
+------------------------
 
 check ``dpdk-procinfo`` tool support ``xstats`` command options.
 
@@ -80,7 +87,11 @@ steps:
 
 #. boot up ``testpmd``::
 
-    ./<target name>/app/testpmd -c 0x600006 -n 4  -- -i --port-topology=loop
+    ./<target name>/app/testpmd -c 0x3 -n 4  -- -i --port-topology=loop
+
+    testpmd> set fwd io
+    testpmd> clear port xstats all
+    testpmd> start
 
 #. run ``dpdk-procinfo`` tool::
 
@@ -88,25 +99,30 @@ steps:
 
 #. check ``dpdk-procinfo`` tool output should contain upper options.
 
-Test Case: xstat command
-------------------------
+Test Case: xstat statistic integrity
+------------------------------------
 
-check if port extended statistics can get right data.
+check if port extended statistics can access by xstat name or xstat id.
 
 steps:
 
 #. boot up ``testpmd``::
 
-    ./<target name>/app/testpmd -c 0x600006 -n 4  -- -i --port-topology=loop
+    ./<target name>/app/testpmd -c 0x3 -n 4  -- -i --port-topology=loop
+
+    testpmd> set fwd io
+    testpmd> clear port xstats all
+    testpmd> start
 
 #. send udp packet of 64/72/128/256/512/1024 size to port 0/1::
 
     sendp([Ether()/IP()/UDP()/Raw('\0'*60)], iface=<port 0 name>)
 
-#. run ``dpdk-procinfo`` tool with ``xstats`` option and check if all port have
-   a correct data of different size packet::
+#. run ``dpdk-procinfo`` tool with ``xstats`` option and check if all ports
+   extended statistics can access by xstat name or xstat id::
 
-    ./<target name>/app/dpdk-procinfo -- -p 3 --xstats
+    ./<target name>/app/dpdk-procinfo -- -p 3 --xstats-id <N>
+    ./<target name>/app/dpdk-procinfo -- -p 3 --xstats-name <statistic name>
 
 Test Case: xstat-reset command
 ------------------------------
@@ -117,7 +133,11 @@ steps:
 
 #. boot up ``testpmd``::
 
-    ./<target name>/app/testpmd -c 0x600006 -n 4  -- -i --port-topology=loop
+    ./<target name>/app/testpmd -c 0x3 -n 4  -- -i --port-topology=loop
+
+    testpmd> set fwd io
+    testpmd> clear port xstats all
+    testpmd> start
 
 #. send udp packet of 64/72/128/256/512/1024 size to port 0/1::
 
@@ -132,24 +152,26 @@ Test Case: xstat single statistic
 ---------------------------------
 
 check if port extended statistic name can be get by statistic id and check
-related data's correctness.
+related data's correctness with testpmd xstat data.
 
 steps:
 
 #. boot up ``testpmd``::
 
-    ./<target name>/app/testpmd -c 0x600006 -n 4  -- -i --port-topology=loop
-
-#. run ``dpdk-procinfo`` tool with ``xstats`` option to get all NIC extended
-   statistics parameters of port and use the output display sequence as parameter
-   index in the following search index::
+    ./<target name>/app/testpmd -c 0x3 -n 4  -- -i --port-topology=loop
 
-    ./<target name>/app/dpdk-procinfo -- -p 3 --xstats
+    testpmd> set fwd io
+    testpmd> clear port xstats all
+    testpmd> start
 
 #. send udp packet of 64/72/128/256/512/1024 size to port 0/1::
 
     sendp([Ether()/IP()/UDP()/Raw('\0'*60)], iface=<port 0 name>)
 
+#. run test pmd ``show port xstats all`` to get testpmd port xstat data:
+
+    testpmd> show port xstats all
+
 #. run ``dpdk-procinfo`` tool with ``xstats-id`` option to get the statistic
    name corresponding with the index id::
 
@@ -158,4 +180,6 @@ steps:
 #. run ``dpdk-procinfo`` tool with ``xstats-name`` option to get the statistic
    data corresponding with the statistic name::
 
-    ./<target name>/app/dpdk-procinfo -- -p 3 --xstats-name <statistic name>
\ No newline at end of file
+    ./<target name>/app/dpdk-procinfo -- -p 3 --xstats-name <statistic name>
+
+#. compare these proc info tool xstat values with testpmd xstat values.
\ No newline at end of file
-- 
2.21.0