Re: [dts] [PATCH V1 1/2] add dyanmic config bonding rss hash test plan

["Liu, Yong" <yong.liu@intel.com>]

Huilong,
Some comments below.

Thanks,
Marvin

> -----Original Message-----
> From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of xu,huilong
> Sent: Wednesday, March 01, 2017 4:37 PM
> To: dts@dpdk.org
> Cc: Xu, HuilongX <huilongx.xu@intel.com>
> Subject: [dts] [PATCH V1 1/2] add dyanmic config bonding rss hash test
> plan
> 
> Signed-off-by: xu,huilong <huilongx.xu@intel.com>
> ---
>  test_plans/pmdrss_hash_test_plan.rst | 24 ++++++++++++++++++++++++
>  1 file changed, 24 insertions(+)
> 
> diff --git a/test_plans/pmdrss_hash_test_plan.rst
> b/test_plans/pmdrss_hash_test_plan.rst
> index 4922d3b..15e6b3b 100644
> --- a/test_plans/pmdrss_hash_test_plan.rst
> +++ b/test_plans/pmdrss_hash_test_plan.rst
> @@ -150,3 +150,27 @@ Test Case:  test_simple_symmetric
>  The same as the above two test cases. Just pay attention to set the hash
> function to "simple xor"
> 
> 
> +Test Case:  test_dyanmic_rss_bond_config
> +=================================

Typo: should be dynamic_rss_bond_config

> +This case test bond slaves will auto sync rss hash config, so it only
> support fortville nic.
> +
> +#1. set up testpmd woth fortville NICs::
> +./testpmd -c f -n 4 -- -i --portmask 0x3 --txqflags=0
> +
> +#2 creat bond device with mode 3::
> + create bonded device 3 0
> +#3 add slave to bond device::
> + add bonding slave 0 2
> + add bonding slave 1 2
> +#4 get default hash algorithm on slave::
> + get_hash_global_config 0
> + get_hash_global_config 1
> +#5 set hash algorithm on slave 0::
> + set_hash_global_config 0 simple_xor ipv4-other  enable

Fortville not only support simple_xor, suggest test all kinds of hash algorithm supported by Fortville.
> +#6 get hash algorithm on slave 0 and 1::
> + get_hash_global_config 0
> + get_hash_global_config 1
> +#7 check slave 0 and 1 use same hash algorithm
> +
> +
> +
> --
> 1.9.3