test suite reviews and discussions
 help / color / Atom feed
* [dts] [PATCH V3 0/2] dts/pktgen: fix missing parameter setting
@ 2019-09-20  2:19 yufengmx
  2019-09-20  2:19 ` [dts] [PATCH V3 1/2] framework/pktgen_base: fix measure_throughput missed option yufengmx
  2019-09-20  2:19 ` [dts] [PATCH V3 2/2] framework/pktgen: fix PacketGeneratorHelper missed rate yufengmx
  0 siblings, 2 replies; 5+ messages in thread
From: yufengmx @ 2019-09-20  2:19 UTC (permalink / raw)
  To: dts, lihongx.ma, lijuan.tu; +Cc: yufengmx

 These commits are internal bugs fixed, which are reported by ma,lihong 
 
 *. add missing rate percent key of stream config option in pktgen module. 
 *. add missing start traffic option input in measure_throughput method. 
 

v3: 
 -  rebase source code. 
 -  

yufengmx (2):
  framework/pktgen_base: fix measure_throughput missed option input
  framework/pktgen: fix PacketGeneratorHelper missed rate setting

 framework/pktgen.py      | 1 +
 framework/pktgen_base.py | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)

-- 
2.21.0


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

* [dts] [PATCH V3 1/2] framework/pktgen_base: fix measure_throughput missed option
  2019-09-20  2:19 [dts] [PATCH V3 0/2] dts/pktgen: fix missing parameter setting yufengmx
@ 2019-09-20  2:19 ` yufengmx
  2019-09-20  6:09   ` Tu, Lijuan
  2019-09-20  2:19 ` [dts] [PATCH V3 2/2] framework/pktgen: fix PacketGeneratorHelper missed rate yufengmx
  1 sibling, 1 reply; 5+ messages in thread
From: yufengmx @ 2019-09-20  2:19 UTC (permalink / raw)
  To: dts, lihongx.ma, lijuan.tu; +Cc: yufengmx

 input

add missing _start_transmission options input in measure_throughput method.

Signed-off-by: yufengmx <yufengx.mo@intel.com>
---
 framework/pktgen_base.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/framework/pktgen_base.py b/framework/pktgen_base.py
index 06ad738..4cb5447 100644
--- a/framework/pktgen_base.py
+++ b/framework/pktgen_base.py
@@ -170,7 +170,7 @@ class PacketGenerator(object):
         bps_rx = []
         pps_rx = []
         self._prepare_transmission(stream_ids=stream_ids)
-        self._start_transmission(stream_ids)
+        self._start_transmission(stream_ids, options)
 
         delay = options.get('delay') or 5
         time.sleep(delay)
-- 
2.21.0


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

* [dts] [PATCH V3 2/2] framework/pktgen: fix PacketGeneratorHelper missed rate
  2019-09-20  2:19 [dts] [PATCH V3 0/2] dts/pktgen: fix missing parameter setting yufengmx
  2019-09-20  2:19 ` [dts] [PATCH V3 1/2] framework/pktgen_base: fix measure_throughput missed option yufengmx
@ 2019-09-20  2:19 ` yufengmx
  2019-09-20  6:08   ` Tu, Lijuan
  1 sibling, 1 reply; 5+ messages in thread
From: yufengmx @ 2019-09-20  2:19 UTC (permalink / raw)
  To: dts, lihongx.ma, lijuan.tu; +Cc: yufengmx

 setting

add missing stream rate option in prepare_stream_from_tginput method.

Signed-off-by: yufengmx <yufengx.mo@intel.com>
---
 framework/pktgen.py | 1 +
 1 file changed, 1 insertion(+)

diff --git a/framework/pktgen.py b/framework/pktgen.py
index 1c8acac..2c6cf7f 100644
--- a/framework/pktgen.py
+++ b/framework/pktgen.py
@@ -194,6 +194,7 @@ class PacketGeneratorHelper(object):
             pcap = config[2]
             _options = deepcopy(self.default_opt)
             _options['pcap'] = pcap
+            _options['stream_config']['rate'] = ratePercent
             # if vm is set
             if vm_config:
                 _options['fields_config'] = \
-- 
2.21.0


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

* Re: [dts] [PATCH V3 2/2] framework/pktgen: fix PacketGeneratorHelper missed rate
  2019-09-20  2:19 ` [dts] [PATCH V3 2/2] framework/pktgen: fix PacketGeneratorHelper missed rate yufengmx
@ 2019-09-20  6:08   ` Tu, Lijuan
  0 siblings, 0 replies; 5+ messages in thread
From: Tu, Lijuan @ 2019-09-20  6:08 UTC (permalink / raw)
  To: Mo, YufengX, dts, Ma,  LihongX

Applied, thanks

> -----Original Message-----
> From: Mo, YufengX
> Sent: Friday, September 20, 2019 10:19 AM
> To: dts@dpdk.org; Ma, LihongX <lihongx.ma@intel.com>; Tu, Lijuan
> <lijuan.tu@intel.com>
> Cc: Mo, YufengX <yufengx.mo@intel.com>
> Subject: [dts][PATCH V3 2/2] framework/pktgen: fix PacketGeneratorHelper
> missed rate
> 
>  setting
> 
> add missing stream rate option in prepare_stream_from_tginput method.
> 
> Signed-off-by: yufengmx <yufengx.mo@intel.com>
> ---
>  framework/pktgen.py | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/framework/pktgen.py b/framework/pktgen.py index
> 1c8acac..2c6cf7f 100644
> --- a/framework/pktgen.py
> +++ b/framework/pktgen.py
> @@ -194,6 +194,7 @@ class PacketGeneratorHelper(object):
>              pcap = config[2]
>              _options = deepcopy(self.default_opt)
>              _options['pcap'] = pcap
> +            _options['stream_config']['rate'] = ratePercent
>              # if vm is set
>              if vm_config:
>                  _options['fields_config'] = \
> --
> 2.21.0


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

* Re: [dts] [PATCH V3 1/2] framework/pktgen_base: fix measure_throughput missed option
  2019-09-20  2:19 ` [dts] [PATCH V3 1/2] framework/pktgen_base: fix measure_throughput missed option yufengmx
@ 2019-09-20  6:09   ` Tu, Lijuan
  0 siblings, 0 replies; 5+ messages in thread
From: Tu, Lijuan @ 2019-09-20  6:09 UTC (permalink / raw)
  To: Mo, YufengX, dts, Ma,  LihongX

Applied, thanks

> -----Original Message-----
> From: Mo, YufengX
> Sent: Friday, September 20, 2019 10:19 AM
> To: dts@dpdk.org; Ma, LihongX <lihongx.ma@intel.com>; Tu, Lijuan
> <lijuan.tu@intel.com>
> Cc: Mo, YufengX <yufengx.mo@intel.com>
> Subject: [dts][PATCH V3 1/2] framework/pktgen_base: fix
> measure_throughput missed option
> 
>  input
> 
> add missing _start_transmission options input in measure_throughput
> method.
> 
> Signed-off-by: yufengmx <yufengx.mo@intel.com>
> ---
>  framework/pktgen_base.py | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/framework/pktgen_base.py b/framework/pktgen_base.py index
> 06ad738..4cb5447 100644
> --- a/framework/pktgen_base.py
> +++ b/framework/pktgen_base.py
> @@ -170,7 +170,7 @@ class PacketGenerator(object):
>          bps_rx = []
>          pps_rx = []
>          self._prepare_transmission(stream_ids=stream_ids)
> -        self._start_transmission(stream_ids)
> +        self._start_transmission(stream_ids, options)
> 
>          delay = options.get('delay') or 5
>          time.sleep(delay)
> --
> 2.21.0


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

end of thread, back to index

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-09-20  2:19 [dts] [PATCH V3 0/2] dts/pktgen: fix missing parameter setting yufengmx
2019-09-20  2:19 ` [dts] [PATCH V3 1/2] framework/pktgen_base: fix measure_throughput missed option yufengmx
2019-09-20  6:09   ` Tu, Lijuan
2019-09-20  2:19 ` [dts] [PATCH V3 2/2] framework/pktgen: fix PacketGeneratorHelper missed rate yufengmx
2019-09-20  6:08   ` Tu, Lijuan

test suite reviews and discussions

Archives are clonable:
	git clone --mirror http://inbox.dpdk.org/dts/0 dts/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 dts dts/ http://inbox.dpdk.org/dts \
		dts@dpdk.org
	public-inbox-index dts


Newsgroup available over NNTP:
	nntp://inbox.dpdk.org/inbox.dpdk.dts


AGPL code for this site: git clone https://public-inbox.org/ public-inbox