* [dts] [PATCH V1] Add a function used to verify keep packets' order feature
@ 2017-11-02 14:53 Yuwei Zhang
2017-11-10 5:31 ` Liu, Yong
0 siblings, 1 reply; 2+ messages in thread
From: Yuwei Zhang @ 2017-11-02 14:53 UTC (permalink / raw)
To: dts; +Cc: Yuwei Zhang
use ixia to send and check packets' order captured which could be used to verify keep packets' order feature
Signed-off-by: Yuwei Zhang <yuwei1.zhang@intel.com>
---
framework/etgen.py | 20 ++++++++++++++++++++
framework/tester.py | 7 +++++++
2 files changed, 27 insertions(+)
diff --git a/framework/etgen.py b/framework/etgen.py
index b19ae84..3892252 100644
--- a/framework/etgen.py
+++ b/framework/etgen.py
@@ -558,6 +558,26 @@ class IxiaPacketGenerator(SSHConnection):
rxPortlist, txPortlist = self._configure_everything(port_list, rate_percent)
return self.get_transmission_results(rxPortlist, txPortlist, delay)
+ def is_packet_ordered(self, port_list, rate_percent=100, delay=5, latency=False):
+ self.send_expect("port setFactoryDefaults %d %d %d" % (self.chasId, self.ports[0]['card'], self.ports[0]['port']), '%')
+ self.send_expect('port config -receiveMode [expr $::portCapture|$::portRxFirstTimeStamp|$::portRxSequenceChecking|$::portRxModeWidePacketGroup]', '%')
+ self.send_expect('port config -autonegotiate true', '%')
+ self.send_expect('lappend portTxList [list %d %d %d]' % (self.chasId, self.ports[0]['card'], self.ports[0]['port']), '%')
+ self.send_expect('ixWritePortsToHardware portTxList', '%')
+ self.send_expect('set streamId 1', '%')
+ self.send_expect('stream setDefault', '%')
+ self.send_expect('ixStartPortPacketGroups %d %d %d' % (self.chasId, self.ports[0]['card'], self.ports[0]['port']), '%')
+ self.send_expect('ixStartTransmit portTxList', '%')
+ self.send_expect('after 1000', '%')
+ self.send_expect('ixStopTransmit portTxList', '%')
+ self.send_expect('ixStopPortPacketGroups %d %d %d' % (self.chasId, self.ports[0]['card'], self.ports[0]['port']), '%')
+ self.send_expect('after 5000', '%')
+ self.send_expect('packetGroupStats get %d %d %d 1 1' % (self.chasId, self.ports[0]['card'], self.ports[0]['port']), '%')
+ self.send_expect('packetroupStats getGroup 1', '%')
+ self.send_expect('set reverseSequenceError [packetGroupStats cget -reverseSequenceError]]', '%')
+ output = self.send_expect('puts $reverseSequenceError', '%')
+ return int(output[:-2])
+
def _configure_everything(self, port_list, rate_percent, latency=False):
"""
Prepare and configure IXIA ports for performance test.
diff --git a/framework/tester.py b/framework/tester.py
index 9208b0e..7062356 100644
--- a/framework/tester.py
+++ b/framework/tester.py
@@ -502,6 +502,13 @@ class Tester(Crb):
return None
return self.packet_gen.throughput(portList, rate_percent)
+ def verify_packet_order(self, portList, rate_percent=100, delay=5):
+ if self.check_port_list(portList, 'ixia'):
+ return self.ixia_packet_gen.is_packet_ordered(portList, rate_percent, delay)
+ else:
+ self.logger.warning("Only ixia port support check verify packet order function")
+ return False
+
def run_rfc2544(self, portlist, delay=120, permit_loss_rate=0):
"""
test_rate: the line rate we are going to test.
--
2.14.1.windows.1
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [dts] [PATCH V1] Add a function used to verify keep packets' order feature
2017-11-02 14:53 [dts] [PATCH V1] Add a function used to verify keep packets' order feature Yuwei Zhang
@ 2017-11-10 5:31 ` Liu, Yong
0 siblings, 0 replies; 2+ messages in thread
From: Liu, Yong @ 2017-11-10 5:31 UTC (permalink / raw)
To: Zhang, Yuwei1, dts; +Cc: Zhang, Yuwei1
Yuwei,
Look like we can't change packet content for sequence check. Is it possible to do that like latency function?
And another two comments are in inline.
Thanks,
Marvin
> -----Original Message-----
> From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of Yuwei Zhang
> Sent: Thursday, November 02, 2017 10:53 PM
> To: dts@dpdk.org
> Cc: Zhang, Yuwei1 <yuwei1.zhang@intel.com>
> Subject: [dts] [PATCH V1] Add a function used to verify keep packets'
> order feature
>
> use ixia to send and check packets' order captured which could be used to
> verify keep packets' order feature
>
> Signed-off-by: Yuwei Zhang <yuwei1.zhang@intel.com>
> ---
> framework/etgen.py | 20 ++++++++++++++++++++
> framework/tester.py | 7 +++++++
> 2 files changed, 27 insertions(+)
>
> diff --git a/framework/etgen.py b/framework/etgen.py
> index b19ae84..3892252 100644
> --- a/framework/etgen.py
> +++ b/framework/etgen.py
> @@ -558,6 +558,26 @@ class IxiaPacketGenerator(SSHConnection):
> rxPortlist, txPortlist = self._configure_everything(port_list,
> rate_percent)
> return self.get_transmission_results(rxPortlist, txPortlist,
> delay)
>
> + def is_packet_ordered(self, port_list, rate_percent=100, delay=5,
> latency=False):
If argument "rate_percent" is useful, please implement rate setting in this function. Otherwise, please remove it.
> + self.send_expect("port setFactoryDefaults %d %d %d" %
> (self.chasId, self.ports[0]['card'], self.ports[0]['port']), '%')
> + self.send_expect('port config -receiveMode [expr
> $::portCapture|$::portRxFirstTimeStamp|$::portRxSequenceChecking|$::portRx
> ModeWidePacketGroup]', '%')
> + self.send_expect('port config -autonegotiate true', '%')
> + self.send_expect('lappend portTxList [list %d %d %d]' %
> (self.chasId, self.ports[0]['card'], self.ports[0]['port']), '%')
> + self.send_expect('ixWritePortsToHardware portTxList', '%')
> + self.send_expect('set streamId 1', '%')
> + self.send_expect('stream setDefault', '%')
> + self.send_expect('ixStartPortPacketGroups %d %d %d' %
> (self.chasId, self.ports[0]['card'], self.ports[0]['port']), '%')
> + self.send_expect('ixStartTransmit portTxList', '%')
> + self.send_expect('after 1000', '%')
1000 here should be delay * 1000.
> + self.send_expect('ixStopTransmit portTxList', '%')
> + self.send_expect('ixStopPortPacketGroups %d %d %d' % (self.chasId,
> self.ports[0]['card'], self.ports[0]['port']), '%')
> + self.send_expect('after 5000', '%')
> + self.send_expect('packetGroupStats get %d %d %d 1 1' %
> (self.chasId, self.ports[0]['card'], self.ports[0]['port']), '%')
> + self.send_expect('packetroupStats getGroup 1', '%')
> + self.send_expect('set reverseSequenceError [packetGroupStats cget
> -reverseSequenceError]]', '%')
> + output = self.send_expect('puts $reverseSequenceError', '%')
> + return int(output[:-2])
> +
> def _configure_everything(self, port_list, rate_percent,
> latency=False):
> """
> Prepare and configure IXIA ports for performance test.
> diff --git a/framework/tester.py b/framework/tester.py
> index 9208b0e..7062356 100644
> --- a/framework/tester.py
> +++ b/framework/tester.py
> @@ -502,6 +502,13 @@ class Tester(Crb):
> return None
> return self.packet_gen.throughput(portList, rate_percent)
>
> + def verify_packet_order(self, portList, rate_percent=100, delay=5):
> + if self.check_port_list(portList, 'ixia'):
> + return self.ixia_packet_gen.is_packet_ordered(portList,
> rate_percent, delay)
> + else:
> + self.logger.warning("Only ixia port support check verify
> packet order function")
> + return False
> +
> def run_rfc2544(self, portlist, delay=120, permit_loss_rate=0):
> """
> test_rate: the line rate we are going to test.
> --
> 2.14.1.windows.1
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2017-11-10 5:31 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-11-02 14:53 [dts] [PATCH V1] Add a function used to verify keep packets' order feature Yuwei Zhang
2017-11-10 5:31 ` 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).