test suite reviews and discussions
 help / color / mirror / Atom feed
* [dts] [PATCH V2 0/2] ethtool_stats: upload test plan
@ 2019-08-05  6:16 yufengmx
  2019-08-05  6:16 ` [dts] [PATCH V2 1/2] " yufengmx
                   ` (2 more replies)
  0 siblings, 3 replies; 7+ messages in thread
From: yufengmx @ 2019-08-05  6:16 UTC (permalink / raw)
  To: dts; +Cc: yufengmx

 upload ethtool_stats test plan 
 update test plan index list 

v2: 
 - Follow Wang,yinan suggestion 
 -  *. remove --tx-offloads=0xXXXX 
 -  *. fix typo 

yufengmx (2):
  ethtool_stats: upload test plan
  ethtool_stats: add ethtool_stats_test_plan label

 test_plans/ethtool_stats_test_plan.rst | 161 +++++++++++++++++++++++++++++++++
 test_plans/index.rst                   |   1 +
 2 files changed, 162 insertions(+)
 create mode 100644 test_plans/ethtool_stats_test_plan.rst

-- 
1.9.3


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

* [dts] [PATCH V2 1/2] ethtool_stats: upload test plan
  2019-08-05  6:16 [dts] [PATCH V2 0/2] ethtool_stats: upload test plan yufengmx
@ 2019-08-05  6:16 ` yufengmx
  2019-08-06  6:49   ` Wang, Yinan
  2019-08-08  5:05   ` Tu, Lijuan
  2019-08-05  6:16 ` [dts] [PATCH V2 2/2] ethtool_stats: add ethtool_stats_test_plan label yufengmx
  2019-08-06  6:49 ` [dts] [PATCH V2 0/2] ethtool_stats: upload test plan Wang, Yinan
  2 siblings, 2 replies; 7+ messages in thread
From: yufengmx @ 2019-08-05  6:16 UTC (permalink / raw)
  To: dts; +Cc: yufengmx


Currently Ethtool supports a more complete list of stats for the same drivers
that DPDK supports. The idea behind this epic is two fold as following:
1. To achieve metric equivalence with what linux's ethtool provides.
2. To extend the functionality of the xstats API to enable the following options:
   ## 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.

Signed-off-by: yufengmx <yufengx.mo@intel.com>
---
 test_plans/ethtool_stats_test_plan.rst | 161 +++++++++++++++++++++++++++++++++
 1 file changed, 161 insertions(+)
 create mode 100644 test_plans/ethtool_stats_test_plan.rst

diff --git a/test_plans/ethtool_stats_test_plan.rst b/test_plans/ethtool_stats_test_plan.rst
new file mode 100644
index 0000000..13b5909
--- /dev/null
+++ b/test_plans/ethtool_stats_test_plan.rst
@@ -0,0 +1,161 @@
+.. Copyright (c) <2010-2019>, Intel Corporation
+   All rights reserved.
+
+   Redistribution and use in source and binary forms, with or without
+   modification, are permitted provided that the following conditions
+   are met:
+
+   - Redistributions of source code must retain the above copyright
+     notice, this list of conditions and the following disclaimer.
+
+   - Redistributions in binary form must reproduce the above copyright
+     notice, this list of conditions and the following disclaimer in
+     the documentation and/or other materials provided with the
+     distribution.
+
+   - Neither the name of Intel Corporation nor the names of its
+     contributors may be used to endorse or promote products derived
+     from this software without specific prior written permission.
+
+   THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+   "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+   LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS
+   FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE
+   COPYRIGHT OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT,
+   INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES
+   (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR
+   SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
+   HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT,
+   STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
+   ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED
+   OF THE POSSIBILITY OF SUCH DAMAGE.
+
+=============
+ethtool stats
+=============
+
+This document provides test plan for ethtool stats . This is a simple example
+app featuring packet processing, ethtool is a utility for Linux kernel-based
+operating system for displaying and modifying some parameters of network
+interface controllers (NICs) and their device drivers.
+
+Currently Ethtool supports a more complete list of stats for the same drivers
+that DPDK supports. The idea behind this epic is two fold as following.
+1. To achieve metric equivalence with what linux's ethtool provides.
+2. To extend the functionality of the xstats API to enable the following options::
+
+    - 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.
+
+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.
+
+Test cases
+==========
+
+Check port extended statistics parameter after sending packet from peer link port.
+
+bind two ports::
+
+    ./usertools/dpdk-devbind.py --bind=igb_uio <pci address 1> <pci address 2>
+
+Test Case: xstat command set intergrity
+---------------------------------------
+
+check ``dpdk-procinfo`` tool support ``xstats`` command options.
+
+These options should be included::
+
+   ``xstats``
+   ``xstats-name``
+   ``xstats-id``
+   ``xstats-reset``
+
+steps:
+
+#. boot up ``testpmd``::
+
+    ./<target name>/app/testpmd -c 0x600006 -n 4  -- -i --port-topology=loop
+
+#. run ``dpdk-procinfo`` tool::
+
+    ./<target name>/app/dpdk-procinfo
+
+#. check ``dpdk-procinfo`` tool output should contain upper options.
+
+Test Case: xstat command
+------------------------
+
+check if port extended statistics can get right data.
+
+steps:
+
+#. boot up ``testpmd``::
+
+    ./<target name>/app/testpmd -c 0x600006 -n 4  -- -i --port-topology=loop
+
+#. 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::
+
+    ./<target name>/app/dpdk-procinfo -- -p 3 --xstats
+
+Test Case: xstat-reset command
+------------------------------
+
+check if port extended statistics can be cleared.
+
+steps:
+
+#. boot up ``testpmd``::
+
+    ./<target name>/app/testpmd -c 0x600006 -n 4  -- -i --port-topology=loop
+
+#. 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-reset`` option and check if all port
+   statistics have been cleared::
+
+    ./<target name>/app/dpdk-procinfo -- -p 3 --xstats-reset
+
+Test Case: xstat single statistic
+---------------------------------
+
+check if port extended statistic name can be get by statistic id and check
+related data's correctness.
+
+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/dpdk-procinfo -- -p 3 --xstats
+
+#. 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-id`` option to get the statistic
+   name corresponding with the index id::
+
+    ./<target name>/app/dpdk-procinfo -- -p 3 --xstats-id 0,1,...N
+
+#. 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
-- 
1.9.3


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

* [dts] [PATCH V2 2/2] ethtool_stats: add ethtool_stats_test_plan label
  2019-08-05  6:16 [dts] [PATCH V2 0/2] ethtool_stats: upload test plan yufengmx
  2019-08-05  6:16 ` [dts] [PATCH V2 1/2] " yufengmx
@ 2019-08-05  6:16 ` yufengmx
  2019-08-06  6:49   ` Wang, Yinan
  2019-08-06  6:49 ` [dts] [PATCH V2 0/2] ethtool_stats: upload test plan Wang, Yinan
  2 siblings, 1 reply; 7+ messages in thread
From: yufengmx @ 2019-08-05  6:16 UTC (permalink / raw)
  To: dts; +Cc: yufengmx


add ethtool_stats_test_plan label.

Signed-off-by: yufengmx <yufengx.mo@intel.com>
---
 test_plans/index.rst | 1 +
 1 file changed, 1 insertion(+)

diff --git a/test_plans/index.rst b/test_plans/index.rst
index 52d4e55..117e131 100644
--- a/test_plans/index.rst
+++ b/test_plans/index.rst
@@ -217,3 +217,4 @@ The following are the test plans for the DPDK DTS automated test system.
     efd_test_plan
     example_build_test_plan
     flow_classify_test_plan
+    ethtool_stats_test_plan
-- 
1.9.3


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

* Re: [dts] [PATCH V2 0/2] ethtool_stats: upload test plan
  2019-08-05  6:16 [dts] [PATCH V2 0/2] ethtool_stats: upload test plan yufengmx
  2019-08-05  6:16 ` [dts] [PATCH V2 1/2] " yufengmx
  2019-08-05  6:16 ` [dts] [PATCH V2 2/2] ethtool_stats: add ethtool_stats_test_plan label yufengmx
@ 2019-08-06  6:49 ` Wang, Yinan
  2 siblings, 0 replies; 7+ messages in thread
From: Wang, Yinan @ 2019-08-06  6:49 UTC (permalink / raw)
  To: Mo, YufengX, dts; +Cc: Mo, YufengX

Acked-by: Wang, Yinan <yinan.wang@intel.com>

> -----Original Message-----
> From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of yufengmx
> Sent: 2019年8月5日 14:16
> To: dts@dpdk.org
> Cc: Mo, YufengX <yufengx.mo@intel.com>
> Subject: [dts] [PATCH V2 0/2] ethtool_stats: upload test plan
> 
>  upload ethtool_stats test plan
>  update test plan index list
> 
> v2:
>  - Follow Wang,yinan suggestion
>  -  *. remove --tx-offloads=0xXXXX
>  -  *. fix typo
> 
> yufengmx (2):
>   ethtool_stats: upload test plan
>   ethtool_stats: add ethtool_stats_test_plan label
> 
>  test_plans/ethtool_stats_test_plan.rst | 161
> +++++++++++++++++++++++++++++++++
>  test_plans/index.rst                   |   1 +
>  2 files changed, 162 insertions(+)
>  create mode 100644 test_plans/ethtool_stats_test_plan.rst
> 
> --
> 1.9.3


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

* Re: [dts] [PATCH V2 2/2] ethtool_stats: add ethtool_stats_test_plan label
  2019-08-05  6:16 ` [dts] [PATCH V2 2/2] ethtool_stats: add ethtool_stats_test_plan label yufengmx
@ 2019-08-06  6:49   ` Wang, Yinan
  0 siblings, 0 replies; 7+ messages in thread
From: Wang, Yinan @ 2019-08-06  6:49 UTC (permalink / raw)
  To: Mo, YufengX, dts; +Cc: Mo, YufengX

Acked-by: Wang, Yinan <yinan.wang@intel.com>

> -----Original Message-----
> From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of yufengmx
> Sent: 2019年8月5日 14:16
> To: dts@dpdk.org
> Cc: Mo, YufengX <yufengx.mo@intel.com>
> Subject: [dts] [PATCH V2 2/2] ethtool_stats: add ethtool_stats_test_plan label
> 
> 
> add ethtool_stats_test_plan label.
> 
> Signed-off-by: yufengmx <yufengx.mo@intel.com>
> ---
>  test_plans/index.rst | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/test_plans/index.rst b/test_plans/index.rst index 52d4e55..117e131
> 100644
> --- a/test_plans/index.rst
> +++ b/test_plans/index.rst
> @@ -217,3 +217,4 @@ The following are the test plans for the DPDK DTS
> automated test system.
>      efd_test_plan
>      example_build_test_plan
>      flow_classify_test_plan
> +    ethtool_stats_test_plan
> --
> 1.9.3


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

* Re: [dts] [PATCH V2 1/2] ethtool_stats: upload test plan
  2019-08-05  6:16 ` [dts] [PATCH V2 1/2] " yufengmx
@ 2019-08-06  6:49   ` Wang, Yinan
  2019-08-08  5:05   ` Tu, Lijuan
  1 sibling, 0 replies; 7+ messages in thread
From: Wang, Yinan @ 2019-08-06  6:49 UTC (permalink / raw)
  To: Mo, YufengX, dts; +Cc: Mo, YufengX

Acked-by: Wang, Yinan <yinan.wang@intel.com>

> -----Original Message-----
> From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of yufengmx
> Sent: 2019年8月5日 14:16
> To: dts@dpdk.org
> Cc: Mo, YufengX <yufengx.mo@intel.com>
> Subject: [dts] [PATCH V2 1/2] ethtool_stats: upload test plan
> 
> 
> Currently Ethtool supports a more complete list of stats for the same drivers that
> DPDK supports. The idea behind this epic is two fold as following:
> 1. To achieve metric equivalence with what linux's ethtool provides.
> 2. To extend the functionality of the xstats API to enable the following options:
>    ## 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.
> 
> Signed-off-by: yufengmx <yufengx.mo@intel.com>
> ---
>  test_plans/ethtool_stats_test_plan.rst | 161
> +++++++++++++++++++++++++++++++++
>  1 file changed, 161 insertions(+)
>  create mode 100644 test_plans/ethtool_stats_test_plan.rst
> 
> diff --git a/test_plans/ethtool_stats_test_plan.rst
> b/test_plans/ethtool_stats_test_plan.rst
> new file mode 100644
> index 0000000..13b5909
> --- /dev/null
> +++ b/test_plans/ethtool_stats_test_plan.rst
> @@ -0,0 +1,161 @@
> +.. Copyright (c) <2010-2019>, Intel Corporation
> +   All rights reserved.
> +
> +   Redistribution and use in source and binary forms, with or without
> +   modification, are permitted provided that the following conditions
> +   are met:
> +
> +   - Redistributions of source code must retain the above copyright
> +     notice, this list of conditions and the following disclaimer.
> +
> +   - Redistributions in binary form must reproduce the above copyright
> +     notice, this list of conditions and the following disclaimer in
> +     the documentation and/or other materials provided with the
> +     distribution.
> +
> +   - Neither the name of Intel Corporation nor the names of its
> +     contributors may be used to endorse or promote products derived
> +     from this software without specific prior written permission.
> +
> +   THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND
> CONTRIBUTORS
> +   "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
> +   LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS
> +   FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE
> +   COPYRIGHT OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT,
> INDIRECT,
> +   INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES
> +   (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
> OR
> +   SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
> +   HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
> CONTRACT,
> +   STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
> +   ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED
> +   OF THE POSSIBILITY OF SUCH DAMAGE.
> +
> +=============
> +ethtool stats
> +=============
> +
> +This document provides test plan for ethtool stats . This is a simple
> +example app featuring packet processing, ethtool is a utility for Linux
> +kernel-based operating system for displaying and modifying some
> +parameters of network interface controllers (NICs) and their device drivers.
> +
> +Currently Ethtool supports a more complete list of stats for the same
> +drivers that DPDK supports. The idea behind this epic is two fold as following.
> +1. To achieve metric equivalence with what linux's ethtool provides.
> +2. To extend the functionality of the xstats API to enable the following options::
> +
> +    - 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.
> +
> +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.
> +
> +Test cases
> +==========
> +
> +Check port extended statistics parameter after sending packet from peer link
> port.
> +
> +bind two ports::
> +
> +    ./usertools/dpdk-devbind.py --bind=igb_uio <pci address 1> <pci
> + address 2>
> +
> +Test Case: xstat command set intergrity
> +---------------------------------------
> +
> +check ``dpdk-procinfo`` tool support ``xstats`` command options.
> +
> +These options should be included::
> +
> +   ``xstats``
> +   ``xstats-name``
> +   ``xstats-id``
> +   ``xstats-reset``
> +
> +steps:
> +
> +#. boot up ``testpmd``::
> +
> +    ./<target name>/app/testpmd -c 0x600006 -n 4  -- -i
> + --port-topology=loop
> +
> +#. run ``dpdk-procinfo`` tool::
> +
> +    ./<target name>/app/dpdk-procinfo
> +
> +#. check ``dpdk-procinfo`` tool output should contain upper options.
> +
> +Test Case: xstat command
> +------------------------
> +
> +check if port extended statistics can get right data.
> +
> +steps:
> +
> +#. boot up ``testpmd``::
> +
> +    ./<target name>/app/testpmd -c 0x600006 -n 4  -- -i
> + --port-topology=loop
> +
> +#. 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::
> +
> +    ./<target name>/app/dpdk-procinfo -- -p 3 --xstats
> +
> +Test Case: xstat-reset command
> +------------------------------
> +
> +check if port extended statistics can be cleared.
> +
> +steps:
> +
> +#. boot up ``testpmd``::
> +
> +    ./<target name>/app/testpmd -c 0x600006 -n 4  -- -i
> + --port-topology=loop
> +
> +#. 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-reset`` option and check if all port
> +   statistics have been cleared::
> +
> +    ./<target name>/app/dpdk-procinfo -- -p 3 --xstats-reset
> +
> +Test Case: xstat single statistic
> +---------------------------------
> +
> +check if port extended statistic name can be get by statistic id and
> +check related data's correctness.
> +
> +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/dpdk-procinfo -- -p 3 --xstats
> +
> +#. 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-id`` option to get the statistic
> +   name corresponding with the index id::
> +
> +    ./<target name>/app/dpdk-procinfo -- -p 3 --xstats-id 0,1,...N
> +
> +#. 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
> --
> 1.9.3


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

* Re: [dts] [PATCH V2 1/2] ethtool_stats: upload test plan
  2019-08-05  6:16 ` [dts] [PATCH V2 1/2] " yufengmx
  2019-08-06  6:49   ` Wang, Yinan
@ 2019-08-08  5:05   ` Tu, Lijuan
  1 sibling, 0 replies; 7+ messages in thread
From: Tu, Lijuan @ 2019-08-08  5:05 UTC (permalink / raw)
  To: Mo, YufengX, dts; +Cc: Mo, YufengX

Applied, thanks

> -----Original Message-----
> From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of yufengmx
> Sent: Monday, August 5, 2019 2:16 PM
> To: dts@dpdk.org
> Cc: Mo, YufengX <yufengx.mo@intel.com>
> Subject: [dts] [PATCH V2 1/2] ethtool_stats: upload test plan
> 
> 
> Currently Ethtool supports a more complete list of stats for the same drivers
> that DPDK supports. The idea behind this epic is two fold as following:
> 1. To achieve metric equivalence with what linux's ethtool provides.
> 2. To extend the functionality of the xstats API to enable the following
> options:
>    ## 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.
> 
> Signed-off-by: yufengmx <yufengx.mo@intel.com>
> ---
>  test_plans/ethtool_stats_test_plan.rst | 161
> +++++++++++++++++++++++++++++++++
>  1 file changed, 161 insertions(+)
>  create mode 100644 test_plans/ethtool_stats_test_plan.rst
> 
> diff --git a/test_plans/ethtool_stats_test_plan.rst
> b/test_plans/ethtool_stats_test_plan.rst
> new file mode 100644
> index 0000000..13b5909
> --- /dev/null
> +++ b/test_plans/ethtool_stats_test_plan.rst
> @@ -0,0 +1,161 @@
> +.. Copyright (c) <2010-2019>, Intel Corporation
> +   All rights reserved.
> +
> +   Redistribution and use in source and binary forms, with or without
> +   modification, are permitted provided that the following conditions
> +   are met:
> +
> +   - Redistributions of source code must retain the above copyright
> +     notice, this list of conditions and the following disclaimer.
> +
> +   - Redistributions in binary form must reproduce the above copyright
> +     notice, this list of conditions and the following disclaimer in
> +     the documentation and/or other materials provided with the
> +     distribution.
> +
> +   - Neither the name of Intel Corporation nor the names of its
> +     contributors may be used to endorse or promote products derived
> +     from this software without specific prior written permission.
> +
> +   THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND
> CONTRIBUTORS
> +   "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
> +   LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND
> FITNESS
> +   FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE
> +   COPYRIGHT OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT,
> INDIRECT,
> +   INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES
> +   (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
> OR
> +   SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
> +   HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
> CONTRACT,
> +   STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
> +   ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF
> ADVISED
> +   OF THE POSSIBILITY OF SUCH DAMAGE.
> +
> +=============
> +ethtool stats
> +=============
> +
> +This document provides test plan for ethtool stats . This is a simple
> +example app featuring packet processing, ethtool is a utility for Linux
> +kernel-based operating system for displaying and modifying some
> +parameters of network interface controllers (NICs) and their device drivers.
> +
> +Currently Ethtool supports a more complete list of stats for the same
> +drivers that DPDK supports. The idea behind this epic is two fold as
> following.
> +1. To achieve metric equivalence with what linux's ethtool provides.
> +2. To extend the functionality of the xstats API to enable the following
> options::
> +
> +    - 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.
> +
> +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.
> +
> +Test cases
> +==========
> +
> +Check port extended statistics parameter after sending packet from peer
> link port.
> +
> +bind two ports::
> +
> +    ./usertools/dpdk-devbind.py --bind=igb_uio <pci address 1> <pci
> + address 2>
> +
> +Test Case: xstat command set intergrity
> +---------------------------------------
> +
> +check ``dpdk-procinfo`` tool support ``xstats`` command options.
> +
> +These options should be included::
> +
> +   ``xstats``
> +   ``xstats-name``
> +   ``xstats-id``
> +   ``xstats-reset``
> +
> +steps:
> +
> +#. boot up ``testpmd``::
> +
> +    ./<target name>/app/testpmd -c 0x600006 -n 4  -- -i
> + --port-topology=loop
> +
> +#. run ``dpdk-procinfo`` tool::
> +
> +    ./<target name>/app/dpdk-procinfo
> +
> +#. check ``dpdk-procinfo`` tool output should contain upper options.
> +
> +Test Case: xstat command
> +------------------------
> +
> +check if port extended statistics can get right data.
> +
> +steps:
> +
> +#. boot up ``testpmd``::
> +
> +    ./<target name>/app/testpmd -c 0x600006 -n 4  -- -i
> + --port-topology=loop
> +
> +#. 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::
> +
> +    ./<target name>/app/dpdk-procinfo -- -p 3 --xstats
> +
> +Test Case: xstat-reset command
> +------------------------------
> +
> +check if port extended statistics can be cleared.
> +
> +steps:
> +
> +#. boot up ``testpmd``::
> +
> +    ./<target name>/app/testpmd -c 0x600006 -n 4  -- -i
> + --port-topology=loop
> +
> +#. 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-reset`` option and check if all port
> +   statistics have been cleared::
> +
> +    ./<target name>/app/dpdk-procinfo -- -p 3 --xstats-reset
> +
> +Test Case: xstat single statistic
> +---------------------------------
> +
> +check if port extended statistic name can be get by statistic id and
> +check related data's correctness.
> +
> +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/dpdk-procinfo -- -p 3 --xstats
> +
> +#. 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-id`` option to get the statistic
> +   name corresponding with the index id::
> +
> +    ./<target name>/app/dpdk-procinfo -- -p 3 --xstats-id 0,1,...N
> +
> +#. 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
> --
> 1.9.3


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

end of thread, other threads:[~2019-08-08  5:05 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-08-05  6:16 [dts] [PATCH V2 0/2] ethtool_stats: upload test plan yufengmx
2019-08-05  6:16 ` [dts] [PATCH V2 1/2] " yufengmx
2019-08-06  6:49   ` Wang, Yinan
2019-08-08  5:05   ` Tu, Lijuan
2019-08-05  6:16 ` [dts] [PATCH V2 2/2] ethtool_stats: add ethtool_stats_test_plan label yufengmx
2019-08-06  6:49   ` Wang, Yinan
2019-08-06  6:49 ` [dts] [PATCH V2 0/2] ethtool_stats: upload test plan Wang, Yinan

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).