Re: [dts] [PATCH V1] test_plans:fix build warnings

["Tu, Lijuan" <lijuan.tu@intel.com>]

Applied

> -----Original Message-----
> From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of Wenjie Li
> Sent: Sunday, December 22, 2019 1:21 PM
> To: dts@dpdk.org
> Cc: Li, WenjieX A <wenjiex.a.li@intel.com>
> Subject: [dts] [PATCH V1] test_plans:fix build warnings
> 
> fix build warnings in test plans.
> 
> Signed-off-by: Wenjie Li <wenjiex.a.li@intel.com>
> ---
>  test_plans/cbdma_test_plan.rst                        | 10 +++++-----
>  ...nable_package_download_in_ice_driver_test_plan.rst |  6 +++---
>  test_plans/index.rst                                  |  9 ++++++++-
>  test_plans/port_representor_test_plan.rst             | 11 ++---------
>  test_plans/vhost_dequeue_zero_copy_test_plan.rst      | 11 ++++++-----
>  5 files changed, 24 insertions(+), 23 deletions(-)
> 
> diff --git a/test_plans/cbdma_test_plan.rst b/test_plans/cbdma_test_plan.rst
> index 3cace07..aee05a6 100644
> --- a/test_plans/cbdma_test_plan.rst
> +++ b/test_plans/cbdma_test_plan.rst
> @@ -87,7 +87,7 @@ where,
>      or not
> 
>  Packet pipeline:
> -===============
> +================
>  NIC RX -> copy packet -> free original -> update mac addresses -> NIC TX
> 
>  Test Case1: CBDMA basic test with differnet size packets @@ -158,7 +158,7
> @@ Test Case5: CBDMA performance cmparison between mac-updating and
> no-mac-updating
> 
>  4. Check performance from ioat app::
> 
> -Total packets Tx:                   xxx [pps]
> +    Total packets Tx:                   xxx [pps]
> 
>  5.Launch ioatfwd app::
> 
> @@ -168,7 +168,7 @@ Total packets Tx:                   xxx [pps]
> 
>  7. Check performance from ioat app::
> 
> -Total packets Tx:                   xxx [pps]
> +    Total packets Tx:                   xxx [pps]
> 
>  Test Case6: CBDMA performance cmparison between HW copies and SW
> copies using different packet size
> ================================================================
> ===================================
> @@ -183,7 +183,7 @@ Test Case6: CBDMA performance cmparison
> between HW copies and SW copies using di
> 
>  4. Check performance from ioat app::
> 
> -Total packets Tx:                   xxx [pps]
> +    Total packets Tx:                   xxx [pps]
> 
>  5.Launch ioatfwd app with three cores::
> 
> @@ -193,4 +193,4 @@ Total packets Tx:                   xxx [pps]
> 
>  7. Check performance from ioat app and compare with hw copy test::
> 
> -Total packets Tx:                   xxx [pps]
> +    Total packets Tx:                   xxx [pps]
> diff --git a/test_plans/enable_package_download_in_ice_driver_test_plan.rst
> b/test_plans/enable_package_download_in_ice_driver_test_plan.rst
> index 55a83e2..4139191 100644
> --- a/test_plans/enable_package_download_in_ice_driver_test_plan.rst
> +++ b/test_plans/enable_package_download_in_ice_driver_test_plan.rst
> @@ -182,6 +182,6 @@ In this case, b1:00.0 interface is specific interface.
> 
>  Check the initial output log, it shows::
> 
> -EAL: PCI device 0000:b1:00.0 on NUMA socket 0
> -EAL:   probe driver: 8086:1593 net_ice
> -**ice_load_pkg(): pkg to be loaded: 1.2.100.0, ICE COMMS Package** \ No
> newline at end of file
> +  EAL: PCI device 0000:b1:00.0 on NUMA socket 0
> +  EAL:   probe driver: 8086:1593 net_ice
> +  **ice_load_pkg(): pkg to be loaded: 1.2.100.0, ICE COMMS Package**
> diff --git a/test_plans/index.rst b/test_plans/index.rst index
> eb506e4..0fdc039 100644
> --- a/test_plans/index.rst
> +++ b/test_plans/index.rst
> @@ -231,7 +231,6 @@ The following are the test plans for the DPDK DTS
> automated test system.
>      dpdk_hugetlbfs_mount_size_test_plan
>      nic_single_core_perf_test_plan
>      power_managerment_throughput_test_plan
> -    ethtool_stats_test_plan
>      iavf_test_plan
>      packet_capture_test_plan
>      packet_ordering_test_plan
> @@ -240,3 +239,11 @@ The following are the test plans for the DPDK DTS
> automated test system.
> 
>      fips_cryptodev_test_plan
>      flow_filtering_test_plan
> +    af_xdp_2_test_plan
> +    cbdma_test_plan
> +    flexible_rxd_test_plan
> +    ipsec_gw_and_library_test_plan
> +    port_control_test_plan
> +    port_representor_test_plan
> +    vm2vm_virtio_user_test_plan
> +    vmdq_dcb_test_plan
> diff --git a/test_plans/port_representor_test_plan.rst
> b/test_plans/port_representor_test_plan.rst
> index e7c1849..c54b7ec 100644
> --- a/test_plans/port_representor_test_plan.rst
> +++ b/test_plans/port_representor_test_plan.rst
> @@ -195,12 +195,5 @@ Description: use control testpmd to set vlan
>      scapy> sendp(pkts, iface="ens785f0")
> 
>  3. check port stats in 2 VF testpmd:
> -  expected result:
> -  2 VF testpmds should receive 10 packets separately.
> -
> -
> -
> -
> -
> -
> -
> +    expected result:
> +    2 VF testpmds should receive 10 packets separately.
> diff --git a/test_plans/vhost_dequeue_zero_copy_test_plan.rst
> b/test_plans/vhost_dequeue_zero_copy_test_plan.rst
> index 100afca..0c550d8 100644
> --- a/test_plans/vhost_dequeue_zero_copy_test_plan.rst
> +++ b/test_plans/vhost_dequeue_zero_copy_test_plan.rst
> @@ -37,11 +37,12 @@ vhost dequeue zero-copy test plan  Description
> ===========
> 
> -Vhost dequeue zero-copy is a performance optimization for vhost, the copy
> in the dequeue path is avoided in order to improve the performance. The
> test cases cover split ring and packed ring.
> +Vhost dequeue zero-copy is a performance optimization for vhost, the copy
> in the dequeue path is avoided in order to improve the performance. The
> test cases cover split ring and packed ring.
>  Notice:
> -*All packed ring case need special qemu version.
> -*In the PVP case, when packet size is 1518B, 10G nic could be the
> performance bottleneck, so we use 40G traffic genarator and 40G nic.
> -*Also as vhost zero copy mbufs should be consumed as soon as possible,
> don't start send packets at vhost side before VM and virtio-pmd launched.
> +
> +* All packed ring case need special qemu version.
> +* In the PVP case, when packet size is 1518B, 10G nic could be the
> performance bottleneck, so we use 40G traffic genarator and 40G nic.
> +* Also as vhost zero copy mbufs should be consumed as soon as possible,
> don't start send packets at vhost side before VM and virtio-pmd launched.
> 
>  Test flow
>  =========
> @@ -386,4 +387,4 @@ Test Case 8: pvp packed ring dequeue zero-copy test
> with driver reload test
> 
>  8. Check each queue's rx/tx packet numbers at vhost side::
> 
> -    testpmd>stop
> \ No newline at end of file
> +    testpmd>stop
> --
> 2.17.2