test suite reviews and discussions
 help / color / mirror / Atom feed
From: "Tu, Lijuan" <lijuan.tu@intel.com>
To: "dts@dpdk.org" <dts@dpdk.org>
Subject: Re: [dts] [PATCH] test_plan: fix doc build warning
Date: Wed, 1 Apr 2020 05:55:00 +0000	[thread overview]
Message-ID: <8CE3E05A3F976642AAB0F4675D0AD20E0BC04E4B@SHSMSX101.ccr.corp.intel.com> (raw)
In-Reply-To: <1585733796-87928-1-git-send-email-lijuan.tu@intel.com>

Applied, thanks

> -----Original Message-----
> From: Tu, Lijuan
> Sent: Wednesday, April 1, 2020 5:37 PM
> To: dts@dpdk.org
> Cc: Tu, Lijuan <lijuan.tu@intel.com>
> Subject: [PATCH] test_plan: fix doc build warning
> 
> * fix format
> * fix index
> 
> Signed-off-by: Lijuan Tu <lijuan.tu@intel.com>
> ---
>  test_plans/ABI_stable_test.rst      | 64 +++++++++++++++++++------------------
>  test_plans/cvl_fdir_test_plan.rst   | 14 ++++----
>  test_plans/hotplug_mp_test_plan.rst |  4 +--
>  test_plans/index.rst                |  2 +-
>  4 files changed, 44 insertions(+), 40 deletions(-)
> 
> diff --git a/test_plans/ABI_stable_test.rst b/test_plans/ABI_stable_test.rst
> index ab2fd7e..47ef787 100644
> --- a/test_plans/ABI_stable_test.rst
> +++ b/test_plans/ABI_stable_test.rst
> @@ -87,66 +87,68 @@ Reuse our existing test suites for ABI stable checking.
>  Execute Test Suites
>  ===================
> 
> +.. table::
> +
> +  +-------------------------------+------------------------+
> +  |       Test Suites             |          Type          |
>    +===============================+========================+
> -  |       Test Suites              |          Type           |
> -  +===============================+========================+
> -  |   unit_tests_cmdline           |     functional          |
> +  |   unit_tests_cmdline          |     functional         |
>    +-------------------------------+------------------------+
> -  |   unit_tests_crc               |     functional          |
> +  |   unit_tests_crc              |     functional         |
>    +-------------------------------+------------------------+
> -  |   unit_tests_dump              |     functional          |
> +  |   unit_tests_dump             |     functional         |
>    +-------------------------------+------------------------+
> -  |   unit_tests_eal               |     functional          |
> +  |   unit_tests_eal              |     functional         |
>    +-------------------------------+------------------------+
> -  |   unit_tests_kni               |     functional          |
> +  |   unit_tests_kni              |     functional         |
>    +-------------------------------+------------------------+
> -  |   unit_tests_loopback          |     functional          |
> +  |   unit_tests_loopback         |     functional         |
>    +-------------------------------+------------------------+
> -  |   unit_tests_lpm               |     functional          |
> +  |   unit_tests_lpm              |     functional         |
>    +-------------------------------+------------------------+
> -  |   unit_tests_mbuf              |     functional          |
> +  |   unit_tests_mbuf             |     functional         |
>    +-------------------------------+------------------------+
> -  |   unit_tests_mempool           |     functional          |
> +  |   unit_tests_mempool          |     functional         |
>    +-------------------------------+------------------------+
> -  |   unit_tests_pmd_perf          |     functional          |
> +  |   unit_tests_pmd_perf         |     functional         |
>    +-------------------------------+------------------------+
> -  |   unit_tests_power             |     functional          |
> +  |   unit_tests_power            |     functional         |
>    +-------------------------------+------------------------+
> -  |   unit_tests_qos               |     functional          |
> +  |   unit_tests_qos              |     functional         |
>    +-------------------------------+------------------------+
> -  |   unit_tests_ringpmd           |     functional          |
> +  |   unit_tests_ringpmd          |     functional         |
>    +-------------------------------+------------------------+
> -  |   unit_tests_ring              |     functional          |
> +  |   unit_tests_ring             |     functional         |
>    +-------------------------------+------------------------+
> -  |   unit_tests_timer             |     functional          |
> +  |   unit_tests_timer            |     functional         |
>    +-------------------------------+------------------------+
> -  |   vhost_1024_ethports          |     functional          |
> +  |   vhost_1024_ethports         |     functional         |
>    +-------------------------------+------------------------+
> -  |   vhost_dequeue_zero_copy      |     functional          |
> +  |   vhost_dequeue_zero_copy     |     functional         |
>    +-------------------------------+------------------------+
> -  |   vhost_enqueue_interrupt      |     functional          |
> +  |   vhost_enqueue_interrupt     |     functional         |
>    +-------------------------------+------------------------+
> -  |   vhost_event_idx_interrupt    |     functional          |
> +  |   vhost_event_idx_interrupt   |     functional         |
>    +-------------------------------+------------------------+
> -  |   vhost_multi_queue_qemu       |     functional          |
> +  |   vhost_multi_queue_qemu      |     functional         |
>    +-------------------------------+------------------------+
> -  |   vhost_pmd_xstats             |     functional          |
> +  |   vhost_pmd_xstats            |     functional         |
>    +-------------------------------+------------------------+
> -  |   vhost_virtio_user_interrupt  |     functional          |
> +  |   vhost_virtio_user_interrupt |     functional         |
>    +-------------------------------+------------------------+
> -  |   vhost_user_live_migration    |     functional          |
> +  |   vhost_user_live_migration   |     functional         |
>    +-------------------------------+------------------------+
> -  |   flow_classify                |     functional          |
> +  |   flow_classify               |     functional         |
>    +-------------------------------+------------------------+
> -  |   flow_classify_softnic        |     functional          |
> +  |   flow_classify_softnic       |     functional         |
>    +-------------------------------+------------------------+
> -  |   vhost_virtio_pmd_interrupt   |     functional          |
> +  |   vhost_virtio_pmd_interrupt  |     functional         |
>    +-------------------------------+------------------------+
> -  |   l2fwd                        |     performance         |
> +  |   l2fwd                       |     performance        |
>    +-------------------------------+------------------------+
> -  |   nic_single_core_perf         |     performance         |
> +  |   nic_single_core_perf        |     performance        |
>    +-------------------------------+------------------------+
> -  |   l3fwd                        |     performance         |
> +  |   l3fwd                       |     performance        |
>    +-------------------------------+------------------------+
> 
> 
> diff --git a/test_plans/cvl_fdir_test_plan.rst
> b/test_plans/cvl_fdir_test_plan.rst
> index 08e8f21..a7eee0b 100644
> --- a/test_plans/cvl_fdir_test_plan.rst
> +++ b/test_plans/cvl_fdir_test_plan.rst
> @@ -82,10 +82,12 @@ Pattern and input set
>      |                              | MAC_IPV4_GTPU_EH           | [TEID], [QFI]
> |
>      +------------------------------+----------------------------+----------------------------------
> ---------------------------------------------+
> 
> -Notes: 1. Enable fdir filter for UDP tunnel: Vxlan / NVGRE (OS default
> package) , share code not
> -          support outer header as inputset, so Out Dest IP and VNI/GRE_KEY
> may not able to be implemented.
> -       2. For VXLAN case MAC_IPV4_TUN_*** means
> MAC_IPV4_UDP_VXLAN_***
> -       3. For Dest MAC, there is package /sharecode limitation on multicast dst
> mac support for FDIR
> +.. note::
> +
> +   1. Enable fdir filter for UDP tunnel: Vxlan / NVGRE (OS default package) ,
> share code not support
> +      outer header as inputset, so Out Dest IP and VNI/GRE_KEY may not able
> to be implemented.
> +   2. For VXLAN case MAC_IPV4_TUN_*** means
> MAC_IPV4_UDP_VXLAN_***
> +   3. For Dest MAC, there is package /sharecode limitation on multicast
> + dst mac support for FDIR
> 
>  Function type
>  -------------
> @@ -573,7 +575,7 @@ Test case: negative validation
> 
>  10. invalid port::
> 
> -    flow validate 2 ingress pattern eth / ipv4 src is 192.168.0.20 dst is
> 192.168.0.21 proto is 255 ttl is 2 tos is 4 / end actions queue index 1 / end
> +     flow validate 2 ingress pattern eth / ipv4 src is 192.168.0.20 dst
> + is 192.168.0.21 proto is 255 ttl is 2 tos is 4 / end actions queue
> + index 1 / end
> 
>     get the message::
> 
> @@ -581,7 +583,7 @@ Test case: negative validation
> 
>  11. check the flow list::
> 
> -    testpmd> flow list 0
> +     testpmd> flow list 0
> 
>     there is no rule listed.
> 
> diff --git a/test_plans/hotplug_mp_test_plan.rst
> b/test_plans/hotplug_mp_test_plan.rst
> index a69ea6f..c4f149d 100644
> --- a/test_plans/hotplug_mp_test_plan.rst
> +++ b/test_plans/hotplug_mp_test_plan.rst
> @@ -275,7 +275,7 @@ Repeat above attach and detach for 2 times  Quit
> primary and secondary processes
> 
>  Test Case: Repeat to attach/detach virtio-user from primary or secondary -
> ================================================================
> =======
> +===============================================================
> ========
> +=
>  Start sample code as primary then secondary::
> 
>       ./hotplug_mp --proc-type=auto
> @@ -308,4 +308,4 @@ secondary processes fail to detach same device again
> 
>  Repeat above attach and detach for 2 times
> 
> -Quit primary and secondary processes
> \ No newline at end of file
> +Quit primary and secondary processes
> diff --git a/test_plans/index.rst b/test_plans/index.rst index
> 542d473..132e957 100644
> --- a/test_plans/index.rst
> +++ b/test_plans/index.rst
> @@ -164,7 +164,7 @@ The following are the test plans for the DPDK DTS
> automated test system.
>      vhost_multi_queue_qemu_test_plan
>      vhost_qemu_mtu_test_plan
>      vhost_user_live_migration_test_plan
> -    vhost_xstats_test_plan
> +    vhost_pmd_xstats_test_plan
>      vm_power_manager_test_plan
>      vm_pw_mgmt_policy_test_plan
>      power_bidirection_channel_test_plan
> --
> 1.8.3.1


      reply	other threads:[~2020-04-01  5:55 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-04-01  9:36 Lijuan Tu
2020-04-01  5:55 ` Tu, Lijuan [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=8CE3E05A3F976642AAB0F4675D0AD20E0BC04E4B@SHSMSX101.ccr.corp.intel.com \
    --to=lijuan.tu@intel.com \
    --cc=dts@dpdk.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).