[dts] [PATCH V3 1/3] ethtool_stats: update test plan
Wang, Yinan
yinan.wang at intel.com
Tue Nov 12 16:34:38 CET 2019
Hi Yufeng,
The test plan you upstreamed can’t work as missing important parameters. Please take a look.
--latencystats=N: enable latency and jitter statistcs monitoring on forwarding lcore id N.
--bitrate-stats=N: set the logical core N to perform bit-rate calculation.
Br,
yinan
> -----Original Message-----
> From: Mo, YufengX <yufengx.mo at intel.com>
> Sent: 2019年11月1日 15:07
> To: dts at dpdk.org; Wang, Yinan <yinan.wang at intel.com>
> Cc: Mo, YufengX <yufengx.mo at intel.com>
> Subject: [dts][PATCH V3 1/3] ethtool_stats: update test plan
>
>
> 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
More information about the dts
mailing list