test suite reviews and discussions
 help / color / mirror / Atom feed
* [dts] [PATCH] test_plans: set max_vfs to 4 when creating 4 VFs in sriov_kvm test plan.
@ 2017-01-05 20:59 Rami Rosen
  2017-01-11  8:11 ` Liu, Yong
  0 siblings, 1 reply; 2+ messages in thread
From: Rami Rosen @ 2017-01-05 20:59 UTC (permalink / raw)
  To: yong.liu; +Cc: dts, Rami Rosen

When creating 4 VFs, we should set max_vfs to 4 and not to 2. This patch fixes 
this assignment.

Signed-off-by: Rami Rosen <rami.rosen@intel.com>
---
 test_plans/sriov_kvm_test_plan.rst | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/test_plans/sriov_kvm_test_plan.rst b/test_plans/sriov_kvm_test_plan.rst
index 5317d10..713c217 100644
--- a/test_plans/sriov_kvm_test_plan.rst
+++ b/test_plans/sriov_kvm_test_plan.rst
@@ -465,7 +465,7 @@ them to VM0, VM1, VM2 and VM3.Start PF driver on the Host and skip the VF
 driver will has been already attached to VMs::
     
     On PF ./tools/pci_unbind.py --bind=igb_uio 0000:08:00.0
-    echo 2 > /sys/bus/pci/devices/0000\:08\:00.0/max_vfs
+    echo 4 > /sys/bus/pci/devices/0000\:08\:00.0/max_vfs
     ./x86_64-default-linuxapp-gcc/app/testpmd -c f -n 4 -b 0000:08:10.0 -b 0000:08:10.2 -b 0000:08:10.4 -b 0000:08:10.6 --  -i
  
 If you want to run all common 4VM cases, please run testpmd on VM0, VM1, VM2
-- 
1.9.1

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

* Re: [dts] [PATCH] test_plans: set max_vfs to 4 when creating 4 VFs in sriov_kvm test plan.
  2017-01-05 20:59 [dts] [PATCH] test_plans: set max_vfs to 4 when creating 4 VFs in sriov_kvm test plan Rami Rosen
@ 2017-01-11  8:11 ` Liu, Yong
  0 siblings, 0 replies; 2+ messages in thread
From: Liu, Yong @ 2017-01-11  8:11 UTC (permalink / raw)
  To: Rami Rosen; +Cc: dts

Thanks Remi, applied.

On 01/06/2017 04:59 AM, Rami Rosen wrote:
> When creating 4 VFs, we should set max_vfs to 4 and not to 2. This patch fixes
> this assignment.
>
> Signed-off-by: Rami Rosen <rami.rosen@intel.com>
> ---
>   test_plans/sriov_kvm_test_plan.rst | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/test_plans/sriov_kvm_test_plan.rst b/test_plans/sriov_kvm_test_plan.rst
> index 5317d10..713c217 100644
> --- a/test_plans/sriov_kvm_test_plan.rst
> +++ b/test_plans/sriov_kvm_test_plan.rst
> @@ -465,7 +465,7 @@ them to VM0, VM1, VM2 and VM3.Start PF driver on the Host and skip the VF
>   driver will has been already attached to VMs::
>       
>       On PF ./tools/pci_unbind.py --bind=igb_uio 0000:08:00.0
> -    echo 2 > /sys/bus/pci/devices/0000\:08\:00.0/max_vfs
> +    echo 4 > /sys/bus/pci/devices/0000\:08\:00.0/max_vfs
>       ./x86_64-default-linuxapp-gcc/app/testpmd -c f -n 4 -b 0000:08:10.0 -b 0000:08:10.2 -b 0000:08:10.4 -b 0000:08:10.6 --  -i
>    
>   If you want to run all common 4VM cases, please run testpmd on VM0, VM1, VM2

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

end of thread, other threads:[~2017-01-11  8:15 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-01-05 20:59 [dts] [PATCH] test_plans: set max_vfs to 4 when creating 4 VFs in sriov_kvm test plan Rami Rosen
2017-01-11  8:11 ` Liu, Yong

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