test suite reviews and discussions
 help / color / Atom feed
* [dts] [PATCH V1] framework/pktgen_trex: check the port status before start transmission
@ 2020-06-04  1:25 lihong
  2020-06-05  7:46 ` Xie, WeiX
  2020-06-11 12:16 ` Tu, Lijuan
  0 siblings, 2 replies; 3+ messages in thread
From: lihong @ 2020-06-04  1:25 UTC (permalink / raw)
  To: dts, weix.xie; +Cc: lihong

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

diff --git a/framework/pktgen_trex.py b/framework/pktgen_trex.py
index 202c778..8da0c84 100644
--- a/framework/pktgen_trex.py
+++ b/framework/pktgen_trex.py
@@ -790,6 +790,21 @@ class TrexPacketGenerator(PacketGenerator):
         rate_percent = "{0}%".format(options.get('rate') or
                                      self._traffic_opt.get('rate') or
                                      '100')
+        # check the link status before transmission
+        self.logger.info('check the trex port link status')
+        for port in self._traffic_ports:
+            try_times = 0
+            port_attr = self._conn.get_port_attr(port)
+            while try_times < 5:
+                self.logger.info(pformat(port_attr))
+                if 'link' in port_attr.keys() and port_attr['link'].lower() == 'down':
+                    time.sleep(2)
+                    try_times = try_times + 1
+                    port_attr = self._conn.get_port_attr(port)
+                else:
+                    break
+                if try_times == 5 and port_attr['link'].lower() == 'down':
+                    self.logger.error('the port: %d link status is down, the transmission can not work right' % port)
         try:
             # clear the stats before injecting
             self._conn.clear_stats()
-- 
2.7.4


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

* Re: [dts] [PATCH V1] framework/pktgen_trex: check the port status before start transmission
  2020-06-04  1:25 [dts] [PATCH V1] framework/pktgen_trex: check the port status before start transmission lihong
@ 2020-06-05  7:46 ` Xie, WeiX
  2020-06-11 12:16 ` Tu, Lijuan
  1 sibling, 0 replies; 3+ messages in thread
From: Xie, WeiX @ 2020-06-05  7:46 UTC (permalink / raw)
  To: Ma, LihongX, dts

[-- Attachment #1: Type: text/plain, Size: 1825 bytes --]

Tested-by:  Xie,WeiX < weix.xie@intel.com>

Regards,
Xie Wei

-----Original Message-----
From: Ma, LihongX 
Sent: Thursday, June 4, 2020 9:26 AM
To: dts@dpdk.org; Xie, WeiX <weix.xie@intel.com>
Cc: Ma, LihongX <lihongx.ma@intel.com>
Subject: [dts][PATCH V1] framework/pktgen_trex: check the port status before start transmission

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

diff --git a/framework/pktgen_trex.py b/framework/pktgen_trex.py index 202c778..8da0c84 100644
--- a/framework/pktgen_trex.py
+++ b/framework/pktgen_trex.py
@@ -790,6 +790,21 @@ class TrexPacketGenerator(PacketGenerator):
         rate_percent = "{0}%".format(options.get('rate') or
                                      self._traffic_opt.get('rate') or
                                      '100')
+        # check the link status before transmission
+        self.logger.info('check the trex port link status')
+        for port in self._traffic_ports:
+            try_times = 0
+            port_attr = self._conn.get_port_attr(port)
+            while try_times < 5:
+                self.logger.info(pformat(port_attr))
+                if 'link' in port_attr.keys() and port_attr['link'].lower() == 'down':
+                    time.sleep(2)
+                    try_times = try_times + 1
+                    port_attr = self._conn.get_port_attr(port)
+                else:
+                    break
+                if try_times == 5 and port_attr['link'].lower() == 'down':
+                    self.logger.error('the port: %d link status is 
+ down, the transmission can not work right' % port)
         try:
             # clear the stats before injecting
             self._conn.clear_stats()
--
2.7.4


[-- Attachment #2: TestVmHotplug.log --]
[-- Type: application/octet-stream, Size: 7263124 bytes --]

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

* Re: [dts] [PATCH V1] framework/pktgen_trex: check the port status before start transmission
  2020-06-04  1:25 [dts] [PATCH V1] framework/pktgen_trex: check the port status before start transmission lihong
  2020-06-05  7:46 ` Xie, WeiX
@ 2020-06-11 12:16 ` Tu, Lijuan
  1 sibling, 0 replies; 3+ messages in thread
From: Tu, Lijuan @ 2020-06-11 12:16 UTC (permalink / raw)
  To: Ma, LihongX, dts, Xie, WeiX; +Cc: Ma, LihongX

Applied, thanks

-----Original Message-----
From: dts <dts-bounces@dpdk.org> On Behalf Of lihong
Sent: 2020^[$BG/^[(B6^[$B7n^[(B4^[$BF|^[(B 9:26
To: dts@dpdk.org; Xie, WeiX <weix.xie@intel.com>
Cc: Ma, LihongX <lihongx.ma@intel.com>
Subject: [dts] [PATCH V1] framework/pktgen_trex: check the port status before start transmission

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

diff --git a/framework/pktgen_trex.py b/framework/pktgen_trex.py index 202c778..8da0c84 100644
--- a/framework/pktgen_trex.py
+++ b/framework/pktgen_trex.py
@@ -790,6 +790,21 @@ class TrexPacketGenerator(PacketGenerator):
         rate_percent = "{0}%".format(options.get('rate') or
                                      self._traffic_opt.get('rate') or
                                      '100')
+        # check the link status before transmission
+        self.logger.info('check the trex port link status')
+        for port in self._traffic_ports:
+            try_times = 0
+            port_attr = self._conn.get_port_attr(port)
+            while try_times < 5:
+                self.logger.info(pformat(port_attr))
+                if 'link' in port_attr.keys() and port_attr['link'].lower() == 'down':
+                    time.sleep(2)
+                    try_times = try_times + 1
+                    port_attr = self._conn.get_port_attr(port)
+                else:
+                    break
+                if try_times == 5 and port_attr['link'].lower() == 'down':
+                    self.logger.error('the port: %d link status is 
+ down, the transmission can not work right' % port)
         try:
             # clear the stats before injecting
             self._conn.clear_stats()
--
2.7.4


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

end of thread, back to index

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-06-04  1:25 [dts] [PATCH V1] framework/pktgen_trex: check the port status before start transmission lihong
2020-06-05  7:46 ` Xie, WeiX
2020-06-11 12:16 ` 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