[V1] tests/vm2vm_virtio_user_cbdma: add port=[] in create_eal_parameters API

Message ID 20220527034129.530032-1-weix.ling@intel.com (mailing list archive)
State Accepted
Headers
Series [V1] tests/vm2vm_virtio_user_cbdma: add port=[] in create_eal_parameters API |

Checks

Context Check Description
ci/Intel-dts-suite-test warning SKIPPED

Commit Message

Ling, WeiX May 27, 2022, 3:41 a.m. UTC
  When call the create_eal_parameters API to genrate the eal parameters,
if the default no ports parameter, the create_eal_parameters API will
use the PCI info in conf/ports.conf, so add port=[] parameter.

Signed-off-by: Wei Ling <weix.ling@intel.com>
---
 tests/TestSuite_vm2vm_virtio_user_cbdma.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
  

Comments

He, Xingguang May 27, 2022, 4:01 a.m. UTC | #1
> -----Original Message-----
> From: Wei Ling <weix.ling@intel.com>
> Sent: Friday, May 27, 2022 11:41 AM
> To: dts@dpdk.org
> Cc: Ling, WeiX <weix.ling@intel.com>
> Subject: [dts][PATCH V1] tests/vm2vm_virtio_user_cbdma: add port=[] in
> create_eal_parameters API
> 
> When call the create_eal_parameters API to genrate the eal parameters, if
> the default no ports parameter, the create_eal_parameters API will use the
> PCI info in conf/ports.conf, so add port=[] parameter.
> 
> Signed-off-by: Wei Ling <weix.ling@intel.com>
> ---
Acked-by: Xingguang He<xingguang.he@intel.com>
  
He, Xingguang May 27, 2022, 5:23 a.m. UTC | #2
> -----Original Message-----
> From: He, Xingguang <xingguang.he@intel.com>
> Sent: Friday, May 27, 2022 12:02 PM
> To: Ling, WeiX <weix.ling@intel.com>; dts@dpdk.org
> Cc: Ling, WeiX <weix.ling@intel.com>
> Subject: RE: [dts][PATCH V1] tests/vm2vm_virtio_user_cbdma: add port=[] in
> create_eal_parameters API
> 
> > -----Original Message-----
> > From: Wei Ling <weix.ling@intel.com>
> > Sent: Friday, May 27, 2022 11:41 AM
> > To: dts@dpdk.org
> > Cc: Ling, WeiX <weix.ling@intel.com>
> > Subject: [dts][PATCH V1] tests/vm2vm_virtio_user_cbdma: add port=[] in
> > create_eal_parameters API
> >
> > When call the create_eal_parameters API to genrate the eal parameters,
> > if the default no ports parameter, the create_eal_parameters API will
> > use the PCI info in conf/ports.conf, so add port=[] parameter.
> >
> > Signed-off-by: Wei Ling <weix.ling@intel.com>
> > ---
> Acked-by: Xingguang He<xingguang.he@intel.com>

Tested-by: Xingguang He<xingguang.he@intel.com>
  
Tu, Lijuan May 28, 2022, 8:53 a.m. UTC | #3
On Thu, 26 May 2022 23:41:29 -0400, Wei Ling <weix.ling@intel.com> wrote:
> When call the create_eal_parameters API to genrate the eal parameters,
> if the default no ports parameter, the create_eal_parameters API will
> use the PCI info in conf/ports.conf, so add port=[] parameter.
> 
> Signed-off-by: Wei Ling <weix.ling@intel.com>


Applied, thanks
  

Patch

diff --git a/tests/TestSuite_vm2vm_virtio_user_cbdma.py b/tests/TestSuite_vm2vm_virtio_user_cbdma.py
index 35d69325..0057047c 100644
--- a/tests/TestSuite_vm2vm_virtio_user_cbdma.py
+++ b/tests/TestSuite_vm2vm_virtio_user_cbdma.py
@@ -269,7 +269,7 @@  class TestVM2VMVirtioUserCbdma(TestCase):
         if dump the virtio-testpmd, the virtio-testpmd should started before launch pdump
         """
         eal_params = self.dut.create_eal_parameters(
-            cores="Default", prefix="virtio-user1", fixed_prefix=True
+            cores="Default", prefix="virtio-user1", fixed_prefix=True, ports=[]
         )
         command_line = (
             self.app_pdump