test suite reviews and discussions
 help / color / mirror / Atom feed
* [dts] [PATCH V1] framework/dut: fix issue of ports config in eal params
@ 2019-12-05 21:54 lihong
  2019-12-06  5:21 ` Ma, LihongX
  2019-12-17  7:08 ` Tu, Lijuan
  0 siblings, 2 replies; 3+ messages in thread
From: lihong @ 2019-12-05 21:54 UTC (permalink / raw)
  To: dts; +Cc: lihong

Signed-off-by: lihong <lihongx.ma@intel.com>
---
 framework/dut.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/framework/dut.py b/framework/dut.py
index f26b081..6e4b1e5 100644
--- a/framework/dut.py
+++ b/framework/dut.py
@@ -143,9 +143,9 @@ class Dut(Crb):
                     else:
                         if config.has_key('port_options') and port in config['port_options'].keys():
                             port_option = config['port_options'][port]
-                            w_pci_list.append('-w %s,%s' % (self.ports_info[config['ports'].index(port)]['pci'], port_option))
+                            w_pci_list.append('-w %s,%s' % (port, port_option))
                         else:
-                            w_pci_list.append('-w %s' % self.ports_info[config['ports'].index(port)]['pci'])
+                            w_pci_list.append('-w %s' % port)
             w_pci_str = ' '.join(w_pci_list)
 
             # deal with black ports
-- 
2.7.4


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

* Re: [dts] [PATCH V1] framework/dut: fix issue of ports config in eal params
  2019-12-05 21:54 [dts] [PATCH V1] framework/dut: fix issue of ports config in eal params lihong
@ 2019-12-06  5:21 ` Ma, LihongX
  2019-12-17  7:08 ` Tu, Lijuan
  1 sibling, 0 replies; 3+ messages in thread
From: Ma, LihongX @ 2019-12-06  5:21 UTC (permalink / raw)
  To: dts

Tested-by: ma,lihong<lihongx.ma@intel.com>

-----Original Message-----
From: Ma, LihongX 
Sent: Friday, December 6, 2019 5:54 AM
To: dts@dpdk.org
Cc: Ma, LihongX <lihongx.ma@intel.com>
Subject: [dts][PATCH V1] framework/dut: fix issue of ports config in eal params

Signed-off-by: lihong <lihongx.ma@intel.com>
---
 framework/dut.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/framework/dut.py b/framework/dut.py index f26b081..6e4b1e5 100644
--- a/framework/dut.py
+++ b/framework/dut.py
@@ -143,9 +143,9 @@ class Dut(Crb):
                     else:
                         if config.has_key('port_options') and port in config['port_options'].keys():
                             port_option = config['port_options'][port]
-                            w_pci_list.append('-w %s,%s' % (self.ports_info[config['ports'].index(port)]['pci'], port_option))
+                            w_pci_list.append('-w %s,%s' % (port, 
+ port_option))
                         else:
-                            w_pci_list.append('-w %s' % self.ports_info[config['ports'].index(port)]['pci'])
+                            w_pci_list.append('-w %s' % port)
             w_pci_str = ' '.join(w_pci_list)
 
             # deal with black ports
--
2.7.4


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

* Re: [dts] [PATCH V1] framework/dut: fix issue of ports config in eal params
  2019-12-05 21:54 [dts] [PATCH V1] framework/dut: fix issue of ports config in eal params lihong
  2019-12-06  5:21 ` Ma, LihongX
@ 2019-12-17  7:08 ` Tu, Lijuan
  1 sibling, 0 replies; 3+ messages in thread
From: Tu, Lijuan @ 2019-12-17  7:08 UTC (permalink / raw)
  To: Ma, LihongX, dts; +Cc: Ma, LihongX

Applied, thanks

> -----Original Message-----
> From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of lihong
> Sent: Friday, December 6, 2019 5:54 AM
> To: dts@dpdk.org
> Cc: Ma, LihongX <lihongx.ma@intel.com>
> Subject: [dts] [PATCH V1] framework/dut: fix issue of ports config in eal
> params
> 
> Signed-off-by: lihong <lihongx.ma@intel.com>
> ---
>  framework/dut.py | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/framework/dut.py b/framework/dut.py index f26b081..6e4b1e5
> 100644
> --- a/framework/dut.py
> +++ b/framework/dut.py
> @@ -143,9 +143,9 @@ class Dut(Crb):
>                      else:
>                          if config.has_key('port_options') and port in
> config['port_options'].keys():
>                              port_option = config['port_options'][port]
> -                            w_pci_list.append('-w %s,%s' %
> (self.ports_info[config['ports'].index(port)]['pci'], port_option))
> +                            w_pci_list.append('-w %s,%s' % (port,
> + port_option))
>                          else:
> -                            w_pci_list.append('-w %s' %
> self.ports_info[config['ports'].index(port)]['pci'])
> +                            w_pci_list.append('-w %s' % port)
>              w_pci_str = ' '.join(w_pci_list)
> 
>              # deal with black ports
> --
> 2.7.4


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

end of thread, other threads:[~2019-12-17  7:08 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-12-05 21:54 [dts] [PATCH V1] framework/dut: fix issue of ports config in eal params lihong
2019-12-06  5:21 ` Ma, LihongX
2019-12-17  7:08 ` Tu, Lijuan

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