* [dts] [PATCH V1] fix userspace failed case
@ 2017-08-23 9:01 xu,gang
2017-08-24 3:50 ` Liu, Yong
0 siblings, 1 reply; 9+ messages in thread
From: xu,gang @ 2017-08-23 9:01 UTC (permalink / raw)
To: dts; +Cc: xu,huilong
From: "xu,huilong" <huilongx.xu@intel.com>
update list:
1. update userspace example path for setup app
2. add sleep time for app update link status with fortville nic in port config case
3. dpdk userspace tools dump eeprom file size different kernel ethtools dump, so only compare same size bytes
4. use scapy tool send packet replace packet model, because app can't received packet by use packet model
Signed-off-by: xu,huilong <huilongx.xu@intel.com>
---
tests/TestSuite_userspace_ethtool.py | 16 ++++++++++++----
1 file changed, 12 insertions(+), 4 deletions(-)
diff --git a/tests/TestSuite_userspace_ethtool.py b/tests/TestSuite_userspace_ethtool.py
index 937a9e5..1b234c1 100644
--- a/tests/TestSuite_userspace_ethtool.py
+++ b/tests/TestSuite_userspace_ethtool.py
@@ -61,7 +61,7 @@ class TestUserspaceEthtool(TestCase, IxiaPacketGenerator):
self.verify("Error" not in out, "compilation error 1")
self.verify("No such file" not in out, "compilation error 2")
- path = "./examples/ethtool/ethtool-app/ethtool-app/%s/ethtool" % self.target
+ path = "./examples/ethtool/ethtool-app/%s/ethtool" % self.target
self.cmd = "%s -c f -n %d" % (path, self.dut.get_memory_channels())
# pause frame basic configuration
@@ -370,7 +370,10 @@ class TestUserspaceEthtool(TestCase, IxiaPacketGenerator):
self.dut.send_expect("ethtool --eeprom-dump %s raw on > %s" % (intf, ethtool_eeprom), "# ")
# wait for file ready
time.sleep(2)
- portinfo['ethtool_eeprom'] = ethtool_eeprom
+ # dpdk userspace dump eeprom-dump size different with kernel ethtool dump eeprom-dump, so only compare same size bytes
+ dpdk_eeprom_size = int(self.dut.send_expect('ls -all %s' % portinfo['eeprom_file'], '# ').split(' ')[4].strip())
+ self.dut.send_expect('dd if=%s of=%s bs=%d count=1' % (ethtool_eeprom, "ethtool_eeprom_%d_cat.bin" % index, dpdk_eeprom_size), "#")
+ portinfo['ethtool_eeprom'] = "ethtool_eeprom_%d_cat.bin" % index
# bind to original driver
portinfo['net_dev'].bind_driver(portinfo['ori_driver'])
@@ -394,10 +397,12 @@ class TestUserspaceEthtool(TestCase, IxiaPacketGenerator):
rx_ring, _, tx_ring, _ = self.strip_ringparam(index)
self.verify(rx_ring == rx_max, "Userspace tool failed to set Rx ring parameter")
self.verify(tx_ring == tx_max, "Userspace tool failed to set Tx ring parameter")
- pkt = Packet()
tester_port = self.tester.get_local_port(port)
intf = self.tester.get_interface(tester_port)
- pkt.send_pkt(tx_port=intf)
+ # use scapy send packet, because example can't receive packet by packet model
+ packet = r'sendp([Ether(dst="ff:ff:ff:ff:ff:ff")/IP()/UDP()/Raw()], iface="%s")' % intf
+ self.tester.scapy_append(packet)
+ self.tester.scapy_execute()
rx_pkts, tx_pkts = self.strip_portstats(index)
self.verify(rx_pkts == ori_rx_pkts + 1, "Failed to forward after ring parameter changed")
self.dut.send_expect("quit", "# ")
@@ -499,8 +504,11 @@ class TestUserspaceEthtool(TestCase, IxiaPacketGenerator):
for index in range(len(self.ports)):
port = self.ports[index]
ori_rx_pkts, _ = self.strip_portstats(index)
+ # add sleep time for example update port link status
+ time.sleep(10)
# stop port
self.dut.send_expect("stop %d" % index, "EthApp>")
+ time.sleep(10)
# check packet not forwarded when port is stop
pkt = Packet()
tester_port = self.tester.get_local_port(port)
--
1.9.3
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [dts] [PATCH V1] fix userspace failed case
2017-08-23 9:01 [dts] [PATCH V1] fix userspace failed case xu,gang
@ 2017-08-24 3:50 ` Liu, Yong
2017-08-24 3:57 ` Xu, HuilongX
0 siblings, 1 reply; 9+ messages in thread
From: Liu, Yong @ 2017-08-24 3:50 UTC (permalink / raw)
To: Xu, GangX, dts; +Cc: Xu, HuilongX
Thanks gang, few comments below.
> -----Original Message-----
> From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of xu,gang
> Sent: Wednesday, August 23, 2017 5:02 PM
> To: dts@dpdk.org
> Cc: Xu, HuilongX <huilongx.xu@intel.com>
> Subject: [dts] [PATCH V1] fix userspace failed case
>
> From: "xu,huilong" <huilongx.xu@intel.com>
>
> update list:
> 1. update userspace example path for setup app
> 2. add sleep time for app update link status with fortville nic in port
> config case
> 3. dpdk userspace tools dump eeprom file size different kernel ethtools
> dump, so only compare same size bytes
> 4. use scapy tool send packet replace packet model, because app can't
> received packet by use packet model
>
> Signed-off-by: xu,huilong <huilongx.xu@intel.com>
> ---
> tests/TestSuite_userspace_ethtool.py | 16 ++++++++++++----
> 1 file changed, 12 insertions(+), 4 deletions(-)
>
> diff --git a/tests/TestSuite_userspace_ethtool.py
> b/tests/TestSuite_userspace_ethtool.py
> index 937a9e5..1b234c1 100644
> --- a/tests/TestSuite_userspace_ethtool.py
> +++ b/tests/TestSuite_userspace_ethtool.py
> @@ -61,7 +61,7 @@ class TestUserspaceEthtool(TestCase,
> IxiaPacketGenerator):
> self.verify("Error" not in out, "compilation error 1")
> self.verify("No such file" not in out, "compilation error 2")
>
> - path = "./examples/ethtool/ethtool-app/ethtool-app/%s/ethtool" %
> self.target
> + path = "./examples/ethtool/ethtool-app/%s/ethtool" % self.target
> self.cmd = "%s -c f -n %d" % (path,
> self.dut.get_memory_channels())
>
> # pause frame basic configuration
> @@ -370,7 +370,10 @@ class TestUserspaceEthtool(TestCase,
> IxiaPacketGenerator):
> self.dut.send_expect("ethtool --eeprom-dump %s raw on > %s" %
> (intf, ethtool_eeprom), "# ")
> # wait for file ready
> time.sleep(2)
> - portinfo['ethtool_eeprom'] = ethtool_eeprom
> + # dpdk userspace dump eeprom-dump size different with kernel
> ethtool dump eeprom-dump, so only compare same size bytes
> + dpdk_eeprom_size = int(self.dut.send_expect('ls -all %s' %
> portinfo['eeprom_file'], '# ').split(' ')[4].strip())
There is one more simple command to get file size.
"stat -c %s"
> + self.dut.send_expect('dd if=%s of=%s bs=%d count=1' %
> (ethtool_eeprom, "ethtool_eeprom_%d_cat.bin" % index, dpdk_eeprom_size),
> "#")
> + portinfo['ethtool_eeprom'] = "ethtool_eeprom_%d_cat.bin" %
> index
> # bind to original driver
> portinfo['net_dev'].bind_driver(portinfo['ori_driver'])
>
> @@ -394,10 +397,12 @@ class TestUserspaceEthtool(TestCase,
> IxiaPacketGenerator):
> rx_ring, _, tx_ring, _ = self.strip_ringparam(index)
> self.verify(rx_ring == rx_max, "Userspace tool failed to set
> Rx ring parameter")
> self.verify(tx_ring == tx_max, "Userspace tool failed to set
> Tx ring parameter")
> - pkt = Packet()
> tester_port = self.tester.get_local_port(port)
> intf = self.tester.get_interface(tester_port)
> - pkt.send_pkt(tx_port=intf)
> + # use scapy send packet, because example can't receive packet
> by packet model
> + packet =
> r'sendp([Ether(dst="ff:ff:ff:ff:ff:ff")/IP()/UDP()/Raw()], iface="%s")' %
> intf
> + self.tester.scapy_append(packet)
> + self.tester.scapy_execute()
The only difference I can see is the destination mac address.
Could you please try again with below code?
pkt.config_layer('ether', {'dst': 'ff:ff:ff:ff:ff:ff'})
> rx_pkts, tx_pkts = self.strip_portstats(index)
> self.verify(rx_pkts == ori_rx_pkts + 1, "Failed to forward
> after ring parameter changed")
> self.dut.send_expect("quit", "# ")
> @@ -499,8 +504,11 @@ class TestUserspaceEthtool(TestCase,
> IxiaPacketGenerator):
> for index in range(len(self.ports)):
> port = self.ports[index]
> ori_rx_pkts, _ = self.strip_portstats(index)
> + # add sleep time for example update port link status
> + time.sleep(10)
> # stop port
> self.dut.send_expect("stop %d" % index, "EthApp>")
> + time.sleep(10)
> # check packet not forwarded when port is stop
> pkt = Packet()
> tester_port = self.tester.get_local_port(port)
> --
> 1.9.3
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [dts] [PATCH V1] fix userspace failed case
2017-08-24 3:50 ` Liu, Yong
@ 2017-08-24 3:57 ` Xu, HuilongX
2017-08-24 5:00 ` Liu, Yong
0 siblings, 1 reply; 9+ messages in thread
From: Xu, HuilongX @ 2017-08-24 3:57 UTC (permalink / raw)
To: Liu, Yong, Xu, GangX, dts
Hi yong,
My comments as below. Could you check it?
-----Original Message-----
From: Liu, Yong
Sent: Thursday, August 24, 2017 11:51 AM
To: Xu, GangX <gangx.xu@intel.com>; dts@dpdk.org
Cc: Xu, HuilongX <huilongx.xu@intel.com>
Subject: RE: [dts] [PATCH V1] fix userspace failed case
Thanks gang, few comments below.
> -----Original Message-----
> From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of xu,gang
> Sent: Wednesday, August 23, 2017 5:02 PM
> To: dts@dpdk.org
> Cc: Xu, HuilongX <huilongx.xu@intel.com>
> Subject: [dts] [PATCH V1] fix userspace failed case
>
> From: "xu,huilong" <huilongx.xu@intel.com>
>
> update list:
> 1. update userspace example path for setup app 2. add sleep time for
> app update link status with fortville nic in port config case 3. dpdk
> userspace tools dump eeprom file size different kernel ethtools dump,
> so only compare same size bytes 4. use scapy tool send packet replace
> packet model, because app can't received packet by use packet model
>
> Signed-off-by: xu,huilong <huilongx.xu@intel.com>
> ---
> tests/TestSuite_userspace_ethtool.py | 16 ++++++++++++----
> 1 file changed, 12 insertions(+), 4 deletions(-)
>
> diff --git a/tests/TestSuite_userspace_ethtool.py
> b/tests/TestSuite_userspace_ethtool.py
> index 937a9e5..1b234c1 100644
> --- a/tests/TestSuite_userspace_ethtool.py
> +++ b/tests/TestSuite_userspace_ethtool.py
> @@ -61,7 +61,7 @@ class TestUserspaceEthtool(TestCase,
> IxiaPacketGenerator):
> self.verify("Error" not in out, "compilation error 1")
> self.verify("No such file" not in out, "compilation error 2")
>
> - path = "./examples/ethtool/ethtool-app/ethtool-app/%s/ethtool" %
> self.target
> + path = "./examples/ethtool/ethtool-app/%s/ethtool" %
> + self.target
> self.cmd = "%s -c f -n %d" % (path,
> self.dut.get_memory_channels())
>
> # pause frame basic configuration @@ -370,7 +370,10 @@ class
> TestUserspaceEthtool(TestCase,
> IxiaPacketGenerator):
> self.dut.send_expect("ethtool --eeprom-dump %s raw on >
> %s" % (intf, ethtool_eeprom), "# ")
> # wait for file ready
> time.sleep(2)
> - portinfo['ethtool_eeprom'] = ethtool_eeprom
> + # dpdk userspace dump eeprom-dump size different with
> + kernel
> ethtool dump eeprom-dump, so only compare same size bytes
> + dpdk_eeprom_size = int(self.dut.send_expect('ls -all %s'
> + %
> portinfo['eeprom_file'], '# ').split(' ')[4].strip())
There is one more simple command to get file size.
"stat -c %s"
Stat command is default tools in all linux os?, if yes, I think can use stat command replace ls.
> + self.dut.send_expect('dd if=%s of=%s bs=%d count=1' %
> (ethtool_eeprom, "ethtool_eeprom_%d_cat.bin" % index,
> dpdk_eeprom_size),
> "#")
> + portinfo['ethtool_eeprom'] = "ethtool_eeprom_%d_cat.bin"
> + %
> index
> # bind to original driver
> portinfo['net_dev'].bind_driver(portinfo['ori_driver'])
>
> @@ -394,10 +397,12 @@ class TestUserspaceEthtool(TestCase,
> IxiaPacketGenerator):
> rx_ring, _, tx_ring, _ = self.strip_ringparam(index)
> self.verify(rx_ring == rx_max, "Userspace tool failed to
> set Rx ring parameter")
> self.verify(tx_ring == tx_max, "Userspace tool failed to
> set Tx ring parameter")
> - pkt = Packet()
> tester_port = self.tester.get_local_port(port)
> intf = self.tester.get_interface(tester_port)
> - pkt.send_pkt(tx_port=intf)
> + # use scapy send packet, because example can't receive
> + packet
> by packet model
> + packet =
> r'sendp([Ether(dst="ff:ff:ff:ff:ff:ff")/IP()/UDP()/Raw()],
> iface="%s")' % intf
> + self.tester.scapy_append(packet)
> + self.tester.scapy_execute()
The only difference I can see is the destination mac address.
Could you please try again with below code?
I review packet mode, if destination mac address not set in packet mode.
Packet model will set default destination mac address is "ff:ff:ff:ff:ff:ff",
So I think not try again pkt with "ff:ff:ff:ff:ff:ff" destination mac address.
pkt.config_layer('ether', {'dst': 'ff:ff:ff:ff:ff:ff'})
> rx_pkts, tx_pkts = self.strip_portstats(index)
> self.verify(rx_pkts == ori_rx_pkts + 1, "Failed to
> forward after ring parameter changed")
> self.dut.send_expect("quit", "# ") @@ -499,8 +504,11 @@
> class TestUserspaceEthtool(TestCase,
> IxiaPacketGenerator):
> for index in range(len(self.ports)):
> port = self.ports[index]
> ori_rx_pkts, _ = self.strip_portstats(index)
> + # add sleep time for example update port link status
> + time.sleep(10)
> # stop port
> self.dut.send_expect("stop %d" % index, "EthApp>")
> + time.sleep(10)
> # check packet not forwarded when port is stop
> pkt = Packet()
> tester_port = self.tester.get_local_port(port)
> --
> 1.9.3
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [dts] [PATCH V1] fix userspace failed case
2017-08-24 3:57 ` Xu, HuilongX
@ 2017-08-24 5:00 ` Liu, Yong
2017-08-24 5:41 ` Xu, HuilongX
0 siblings, 1 reply; 9+ messages in thread
From: Liu, Yong @ 2017-08-24 5:00 UTC (permalink / raw)
To: Xu, HuilongX, Xu, GangX, dts
Huilong,
Comments inline. Thanks.
> -----Original Message-----
> From: Xu, HuilongX
> Sent: Thursday, August 24, 2017 11:57 AM
> To: Liu, Yong <yong.liu@intel.com>; Xu, GangX <gangx.xu@intel.com>;
> dts@dpdk.org
> Subject: RE: [dts] [PATCH V1] fix userspace failed case
>
> Hi yong,
> My comments as below. Could you check it?
>
> -----Original Message-----
> From: Liu, Yong
> Sent: Thursday, August 24, 2017 11:51 AM
> To: Xu, GangX <gangx.xu@intel.com>; dts@dpdk.org
> Cc: Xu, HuilongX <huilongx.xu@intel.com>
> Subject: RE: [dts] [PATCH V1] fix userspace failed case
>
> Thanks gang, few comments below.
>
> > -----Original Message-----
> > From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of xu,gang
> > Sent: Wednesday, August 23, 2017 5:02 PM
> > To: dts@dpdk.org
> > Cc: Xu, HuilongX <huilongx.xu@intel.com>
> > Subject: [dts] [PATCH V1] fix userspace failed case
> >
> > From: "xu,huilong" <huilongx.xu@intel.com>
> >
> > update list:
> > 1. update userspace example path for setup app 2. add sleep time for
> > app update link status with fortville nic in port config case 3. dpdk
> > userspace tools dump eeprom file size different kernel ethtools dump,
> > so only compare same size bytes 4. use scapy tool send packet replace
> > packet model, because app can't received packet by use packet model
> >
> > Signed-off-by: xu,huilong <huilongx.xu@intel.com>
> > ---
> > tests/TestSuite_userspace_ethtool.py | 16 ++++++++++++----
> > 1 file changed, 12 insertions(+), 4 deletions(-)
> >
> > diff --git a/tests/TestSuite_userspace_ethtool.py
> > b/tests/TestSuite_userspace_ethtool.py
> > index 937a9e5..1b234c1 100644
> > --- a/tests/TestSuite_userspace_ethtool.py
> > +++ b/tests/TestSuite_userspace_ethtool.py
> > @@ -61,7 +61,7 @@ class TestUserspaceEthtool(TestCase,
> > IxiaPacketGenerator):
> > self.verify("Error" not in out, "compilation error 1")
> > self.verify("No such file" not in out, "compilation error 2")
> >
> > - path = "./examples/ethtool/ethtool-app/ethtool-app/%s/ethtool" %
> > self.target
> > + path = "./examples/ethtool/ethtool-app/%s/ethtool" %
> > + self.target
> > self.cmd = "%s -c f -n %d" % (path,
> > self.dut.get_memory_channels())
> >
> > # pause frame basic configuration @@ -370,7 +370,10 @@ class
> > TestUserspaceEthtool(TestCase,
> > IxiaPacketGenerator):
> > self.dut.send_expect("ethtool --eeprom-dump %s raw on >
> > %s" % (intf, ethtool_eeprom), "# ")
> > # wait for file ready
> > time.sleep(2)
> > - portinfo['ethtool_eeprom'] = ethtool_eeprom
> > + # dpdk userspace dump eeprom-dump size different with
> > + kernel
> > ethtool dump eeprom-dump, so only compare same size bytes
> > + dpdk_eeprom_size = int(self.dut.send_expect('ls -all %s'
> > + %
> > portinfo['eeprom_file'], '# ').split(' ')[4].strip())
>
> There is one more simple command to get file size.
> "stat -c %s"
> Stat command is default tools in all linux os?, if yes, I think can use
> stat command replace ls.
Stat command is belong to GNU coreutils. "ls" also in the same package.
It will be fine to utilize stat command.
> > + self.dut.send_expect('dd if=%s of=%s bs=%d count=1' %
> > (ethtool_eeprom, "ethtool_eeprom_%d_cat.bin" % index,
> > dpdk_eeprom_size),
> > "#")
> > + portinfo['ethtool_eeprom'] = "ethtool_eeprom_%d_cat.bin"
> > + %
> > index
> > # bind to original driver
> > portinfo['net_dev'].bind_driver(portinfo['ori_driver'])
> >
> > @@ -394,10 +397,12 @@ class TestUserspaceEthtool(TestCase,
> > IxiaPacketGenerator):
> > rx_ring, _, tx_ring, _ = self.strip_ringparam(index)
> > self.verify(rx_ring == rx_max, "Userspace tool failed to
> > set Rx ring parameter")
> > self.verify(tx_ring == tx_max, "Userspace tool failed to
> > set Tx ring parameter")
> > - pkt = Packet()
> > tester_port = self.tester.get_local_port(port)
> > intf = self.tester.get_interface(tester_port)
> > - pkt.send_pkt(tx_port=intf)
> > + # use scapy send packet, because example can't receive
> > + packet
> > by packet model
> > + packet =
> > r'sendp([Ether(dst="ff:ff:ff:ff:ff:ff")/IP()/UDP()/Raw()],
> > iface="%s")' % intf
> > + self.tester.scapy_append(packet)
> > + self.tester.scapy_execute()
>
> The only difference I can see is the destination mac address.
> Could you please try again with below code?
> I review packet mode, if destination mac address not set in packet mode.
> Packet model will set default destination mac address is
> "ff:ff:ff:ff:ff:ff",
> So I think not try again pkt with "ff:ff:ff:ff:ff:ff" destination mac
> address.
>
I'm wondering that if any bug in packet module which can cause this issue.
packet module will call the same API as scapy cli.
> pkt.config_layer('ether', {'dst': 'ff:ff:ff:ff:ff:ff'})
>
> > rx_pkts, tx_pkts = self.strip_portstats(index)
> > self.verify(rx_pkts == ori_rx_pkts + 1, "Failed to
> > forward after ring parameter changed")
> > self.dut.send_expect("quit", "# ") @@ -499,8 +504,11 @@
> > class TestUserspaceEthtool(TestCase,
> > IxiaPacketGenerator):
> > for index in range(len(self.ports)):
> > port = self.ports[index]
> > ori_rx_pkts, _ = self.strip_portstats(index)
> > + # add sleep time for example update port link status
> > + time.sleep(10)
> > # stop port
> > self.dut.send_expect("stop %d" % index, "EthApp>")
> > + time.sleep(10)
> > # check packet not forwarded when port is stop
> > pkt = Packet()
> > tester_port = self.tester.get_local_port(port)
> > --
> > 1.9.3
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [dts] [PATCH V1] fix userspace failed case
2017-08-24 5:00 ` Liu, Yong
@ 2017-08-24 5:41 ` Xu, HuilongX
2017-08-24 5:49 ` Liu, Yong
0 siblings, 1 reply; 9+ messages in thread
From: Xu, HuilongX @ 2017-08-24 5:41 UTC (permalink / raw)
To: Liu, Yong, Xu, GangX, dts
Hi, yong,
Two comments as below,
1. use stat tools replace ls in next version
2. for use scapy replace packet mode.
In this case, send packet very simple. We should use scapy default interface for send packet.
Because it very steady, the log is clearness and friendly, the function interface in tester mode very simple also.
Thanks a lot
-----Original Message-----
From: Liu, Yong
Sent: Thursday, August 24, 2017 1:00 PM
To: Xu, HuilongX <huilongx.xu@intel.com>; Xu, GangX <gangx.xu@intel.com>; dts@dpdk.org
Subject: RE: [dts] [PATCH V1] fix userspace failed case
Huilong,
Comments inline. Thanks.
> -----Original Message-----
> From: Xu, HuilongX
> Sent: Thursday, August 24, 2017 11:57 AM
> To: Liu, Yong <yong.liu@intel.com>; Xu, GangX <gangx.xu@intel.com>;
> dts@dpdk.org
> Subject: RE: [dts] [PATCH V1] fix userspace failed case
>
> Hi yong,
> My comments as below. Could you check it?
>
> -----Original Message-----
> From: Liu, Yong
> Sent: Thursday, August 24, 2017 11:51 AM
> To: Xu, GangX <gangx.xu@intel.com>; dts@dpdk.org
> Cc: Xu, HuilongX <huilongx.xu@intel.com>
> Subject: RE: [dts] [PATCH V1] fix userspace failed case
>
> Thanks gang, few comments below.
>
> > -----Original Message-----
> > From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of xu,gang
> > Sent: Wednesday, August 23, 2017 5:02 PM
> > To: dts@dpdk.org
> > Cc: Xu, HuilongX <huilongx.xu@intel.com>
> > Subject: [dts] [PATCH V1] fix userspace failed case
> >
> > From: "xu,huilong" <huilongx.xu@intel.com>
> >
> > update list:
> > 1. update userspace example path for setup app 2. add sleep time for
> > app update link status with fortville nic in port config case 3.
> > dpdk userspace tools dump eeprom file size different kernel ethtools
> > dump, so only compare same size bytes 4. use scapy tool send packet
> > replace packet model, because app can't received packet by use
> > packet model
> >
> > Signed-off-by: xu,huilong <huilongx.xu@intel.com>
> > ---
> > tests/TestSuite_userspace_ethtool.py | 16 ++++++++++++----
> > 1 file changed, 12 insertions(+), 4 deletions(-)
> >
> > diff --git a/tests/TestSuite_userspace_ethtool.py
> > b/tests/TestSuite_userspace_ethtool.py
> > index 937a9e5..1b234c1 100644
> > --- a/tests/TestSuite_userspace_ethtool.py
> > +++ b/tests/TestSuite_userspace_ethtool.py
> > @@ -61,7 +61,7 @@ class TestUserspaceEthtool(TestCase,
> > IxiaPacketGenerator):
> > self.verify("Error" not in out, "compilation error 1")
> > self.verify("No such file" not in out, "compilation error
> > 2")
> >
> > - path = "./examples/ethtool/ethtool-app/ethtool-app/%s/ethtool" %
> > self.target
> > + path = "./examples/ethtool/ethtool-app/%s/ethtool" %
> > + self.target
> > self.cmd = "%s -c f -n %d" % (path,
> > self.dut.get_memory_channels())
> >
> > # pause frame basic configuration @@ -370,7 +370,10 @@
> > class TestUserspaceEthtool(TestCase,
> > IxiaPacketGenerator):
> > self.dut.send_expect("ethtool --eeprom-dump %s raw on >
> > %s" % (intf, ethtool_eeprom), "# ")
> > # wait for file ready
> > time.sleep(2)
> > - portinfo['ethtool_eeprom'] = ethtool_eeprom
> > + # dpdk userspace dump eeprom-dump size different with
> > + kernel
> > ethtool dump eeprom-dump, so only compare same size bytes
> > + dpdk_eeprom_size = int(self.dut.send_expect('ls -all %s'
> > + %
> > portinfo['eeprom_file'], '# ').split(' ')[4].strip())
>
> There is one more simple command to get file size.
> "stat -c %s"
> Stat command is default tools in all linux os?, if yes, I think can
> use stat command replace ls.
Stat command is belong to GNU coreutils. "ls" also in the same package.
It will be fine to utilize stat command.
> > + self.dut.send_expect('dd if=%s of=%s bs=%d count=1' %
> > (ethtool_eeprom, "ethtool_eeprom_%d_cat.bin" % index,
> > dpdk_eeprom_size),
> > "#")
> > + portinfo['ethtool_eeprom'] = "ethtool_eeprom_%d_cat.bin"
> > + %
> > index
> > # bind to original driver
> > portinfo['net_dev'].bind_driver(portinfo['ori_driver'])
> >
> > @@ -394,10 +397,12 @@ class TestUserspaceEthtool(TestCase,
> > IxiaPacketGenerator):
> > rx_ring, _, tx_ring, _ = self.strip_ringparam(index)
> > self.verify(rx_ring == rx_max, "Userspace tool failed
> > to set Rx ring parameter")
> > self.verify(tx_ring == tx_max, "Userspace tool failed
> > to set Tx ring parameter")
> > - pkt = Packet()
> > tester_port = self.tester.get_local_port(port)
> > intf = self.tester.get_interface(tester_port)
> > - pkt.send_pkt(tx_port=intf)
> > + # use scapy send packet, because example can't receive
> > + packet
> > by packet model
> > + packet =
> > r'sendp([Ether(dst="ff:ff:ff:ff:ff:ff")/IP()/UDP()/Raw()],
> > iface="%s")' % intf
> > + self.tester.scapy_append(packet)
> > + self.tester.scapy_execute()
>
> The only difference I can see is the destination mac address.
> Could you please try again with below code?
> I review packet mode, if destination mac address not set in packet mode.
> Packet model will set default destination mac address is
> "ff:ff:ff:ff:ff:ff", So I think not try again pkt with
> "ff:ff:ff:ff:ff:ff" destination mac address.
>
I'm wondering that if any bug in packet module which can cause this issue.
packet module will call the same API as scapy cli.
> pkt.config_layer('ether', {'dst': 'ff:ff:ff:ff:ff:ff'})
>
> > rx_pkts, tx_pkts = self.strip_portstats(index)
> > self.verify(rx_pkts == ori_rx_pkts + 1, "Failed to
> > forward after ring parameter changed")
> > self.dut.send_expect("quit", "# ") @@ -499,8 +504,11 @@
> > class TestUserspaceEthtool(TestCase,
> > IxiaPacketGenerator):
> > for index in range(len(self.ports)):
> > port = self.ports[index]
> > ori_rx_pkts, _ = self.strip_portstats(index)
> > + # add sleep time for example update port link status
> > + time.sleep(10)
> > # stop port
> > self.dut.send_expect("stop %d" % index, "EthApp>")
> > + time.sleep(10)
> > # check packet not forwarded when port is stop
> > pkt = Packet()
> > tester_port = self.tester.get_local_port(port)
> > --
> > 1.9.3
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [dts] [PATCH V1] fix userspace failed case
2017-08-24 5:41 ` Xu, HuilongX
@ 2017-08-24 5:49 ` Liu, Yong
2017-08-24 6:17 ` Xu, HuilongX
0 siblings, 1 reply; 9+ messages in thread
From: Liu, Yong @ 2017-08-24 5:49 UTC (permalink / raw)
To: Xu, HuilongX, Xu, GangX, dts
> -----Original Message-----
> From: Xu, HuilongX
> Sent: Thursday, August 24, 2017 1:42 PM
> To: Liu, Yong <yong.liu@intel.com>; Xu, GangX <gangx.xu@intel.com>;
> dts@dpdk.org
> Subject: RE: [dts] [PATCH V1] fix userspace failed case
>
> Hi, yong,
> Two comments as below,
> 1. use stat tools replace ls in next version
> 2. for use scapy replace packet mode.
> In this case, send packet very simple. We should use scapy default
> interface for send packet.
> Because it very steady, the log is clearness and friendly, the
> function interface in tester mode very simple also.
Agree with you, send/analyze packets by scapy is very simple.
But packet module is also very easy to use in transmission side.
Just two lines of code is needed. My option is that we should fix issue in the module rather than just skip it.
And I prefer that packet module to be the default method. The reason is that we can used it for other kinds generator like Trex/Ixia later.
> Thanks a lot
> -----Original Message-----
> From: Liu, Yong
> Sent: Thursday, August 24, 2017 1:00 PM
> To: Xu, HuilongX <huilongx.xu@intel.com>; Xu, GangX <gangx.xu@intel.com>;
> dts@dpdk.org
> Subject: RE: [dts] [PATCH V1] fix userspace failed case
>
> Huilong,
> Comments inline. Thanks.
>
> > -----Original Message-----
> > From: Xu, HuilongX
> > Sent: Thursday, August 24, 2017 11:57 AM
> > To: Liu, Yong <yong.liu@intel.com>; Xu, GangX <gangx.xu@intel.com>;
> > dts@dpdk.org
> > Subject: RE: [dts] [PATCH V1] fix userspace failed case
> >
> > Hi yong,
> > My comments as below. Could you check it?
> >
> > -----Original Message-----
> > From: Liu, Yong
> > Sent: Thursday, August 24, 2017 11:51 AM
> > To: Xu, GangX <gangx.xu@intel.com>; dts@dpdk.org
> > Cc: Xu, HuilongX <huilongx.xu@intel.com>
> > Subject: RE: [dts] [PATCH V1] fix userspace failed case
> >
> > Thanks gang, few comments below.
> >
> > > -----Original Message-----
> > > From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of xu,gang
> > > Sent: Wednesday, August 23, 2017 5:02 PM
> > > To: dts@dpdk.org
> > > Cc: Xu, HuilongX <huilongx.xu@intel.com>
> > > Subject: [dts] [PATCH V1] fix userspace failed case
> > >
> > > From: "xu,huilong" <huilongx.xu@intel.com>
> > >
> > > update list:
> > > 1. update userspace example path for setup app 2. add sleep time for
> > > app update link status with fortville nic in port config case 3.
> > > dpdk userspace tools dump eeprom file size different kernel ethtools
> > > dump, so only compare same size bytes 4. use scapy tool send packet
> > > replace packet model, because app can't received packet by use
> > > packet model
> > >
> > > Signed-off-by: xu,huilong <huilongx.xu@intel.com>
> > > ---
> > > tests/TestSuite_userspace_ethtool.py | 16 ++++++++++++----
> > > 1 file changed, 12 insertions(+), 4 deletions(-)
> > >
> > > diff --git a/tests/TestSuite_userspace_ethtool.py
> > > b/tests/TestSuite_userspace_ethtool.py
> > > index 937a9e5..1b234c1 100644
> > > --- a/tests/TestSuite_userspace_ethtool.py
> > > +++ b/tests/TestSuite_userspace_ethtool.py
> > > @@ -61,7 +61,7 @@ class TestUserspaceEthtool(TestCase,
> > > IxiaPacketGenerator):
> > > self.verify("Error" not in out, "compilation error 1")
> > > self.verify("No such file" not in out, "compilation error
> > > 2")
> > >
> > > - path = "./examples/ethtool/ethtool-app/ethtool-
> app/%s/ethtool" %
> > > self.target
> > > + path = "./examples/ethtool/ethtool-app/%s/ethtool" %
> > > + self.target
> > > self.cmd = "%s -c f -n %d" % (path,
> > > self.dut.get_memory_channels())
> > >
> > > # pause frame basic configuration @@ -370,7 +370,10 @@
> > > class TestUserspaceEthtool(TestCase,
> > > IxiaPacketGenerator):
> > > self.dut.send_expect("ethtool --eeprom-dump %s raw on >
> > > %s" % (intf, ethtool_eeprom), "# ")
> > > # wait for file ready
> > > time.sleep(2)
> > > - portinfo['ethtool_eeprom'] = ethtool_eeprom
> > > + # dpdk userspace dump eeprom-dump size different with
> > > + kernel
> > > ethtool dump eeprom-dump, so only compare same size bytes
> > > + dpdk_eeprom_size = int(self.dut.send_expect('ls -all %s'
> > > + %
> > > portinfo['eeprom_file'], '# ').split(' ')[4].strip())
> >
> > There is one more simple command to get file size.
> > "stat -c %s"
> > Stat command is default tools in all linux os?, if yes, I think can
> > use stat command replace ls.
>
> Stat command is belong to GNU coreutils. "ls" also in the same package.
> It will be fine to utilize stat command.
>
> > > + self.dut.send_expect('dd if=%s of=%s bs=%d count=1' %
> > > (ethtool_eeprom, "ethtool_eeprom_%d_cat.bin" % index,
> > > dpdk_eeprom_size),
> > > "#")
> > > + portinfo['ethtool_eeprom'] = "ethtool_eeprom_%d_cat.bin"
> > > + %
> > > index
> > > # bind to original driver
> > > portinfo['net_dev'].bind_driver(portinfo['ori_driver'])
> > >
> > > @@ -394,10 +397,12 @@ class TestUserspaceEthtool(TestCase,
> > > IxiaPacketGenerator):
> > > rx_ring, _, tx_ring, _ = self.strip_ringparam(index)
> > > self.verify(rx_ring == rx_max, "Userspace tool failed
> > > to set Rx ring parameter")
> > > self.verify(tx_ring == tx_max, "Userspace tool failed
> > > to set Tx ring parameter")
> > > - pkt = Packet()
> > > tester_port = self.tester.get_local_port(port)
> > > intf = self.tester.get_interface(tester_port)
> > > - pkt.send_pkt(tx_port=intf)
> > > + # use scapy send packet, because example can't receive
> > > + packet
> > > by packet model
> > > + packet =
> > > r'sendp([Ether(dst="ff:ff:ff:ff:ff:ff")/IP()/UDP()/Raw()],
> > > iface="%s")' % intf
> > > + self.tester.scapy_append(packet)
> > > + self.tester.scapy_execute()
> >
> > The only difference I can see is the destination mac address.
> > Could you please try again with below code?
> > I review packet mode, if destination mac address not set in packet mode.
> > Packet model will set default destination mac address is
> > "ff:ff:ff:ff:ff:ff", So I think not try again pkt with
> > "ff:ff:ff:ff:ff:ff" destination mac address.
> >
>
> I'm wondering that if any bug in packet module which can cause this issue.
> packet module will call the same API as scapy cli.
>
> > pkt.config_layer('ether', {'dst': 'ff:ff:ff:ff:ff:ff'})
> >
> > > rx_pkts, tx_pkts = self.strip_portstats(index)
> > > self.verify(rx_pkts == ori_rx_pkts + 1, "Failed to
> > > forward after ring parameter changed")
> > > self.dut.send_expect("quit", "# ") @@ -499,8 +504,11 @@
> > > class TestUserspaceEthtool(TestCase,
> > > IxiaPacketGenerator):
> > > for index in range(len(self.ports)):
> > > port = self.ports[index]
> > > ori_rx_pkts, _ = self.strip_portstats(index)
> > > + # add sleep time for example update port link status
> > > + time.sleep(10)
> > > # stop port
> > > self.dut.send_expect("stop %d" % index, "EthApp>")
> > > + time.sleep(10)
> > > # check packet not forwarded when port is stop
> > > pkt = Packet()
> > > tester_port = self.tester.get_local_port(port)
> > > --
> > > 1.9.3
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [dts] [PATCH V1] fix userspace failed case
2017-08-24 5:49 ` Liu, Yong
@ 2017-08-24 6:17 ` Xu, HuilongX
2017-08-24 6:25 ` Liu, Yong
0 siblings, 1 reply; 9+ messages in thread
From: Xu, HuilongX @ 2017-08-24 6:17 UTC (permalink / raw)
To: Liu, Yong, Xu, GangX, dts
Hi yong,
Yes, you are right, but for function test, we only should send/analyze some packets(most of the time, we only send one packet)
For performance test, we need send packet stream and set send packet line rate by ixia/trex or other tools. But whether use any tools, we should
Use unify interface, and the unify interface must use send packet tools supplied default interface, not develop a new interface. Eg, set line rate function,
In our unify interface maye is set_line_rate, but set_line_rate will use ixia_set_line_rate or trex_set_line_rate, not develop a new ixia_set_line_rate or trex_line_rate. Now packet mode in framework we use scapy internal function for build packet and send packet, it is the root case why packet mode not stable.
If we need use packet mode send packet for function, I think we can re-develop packet model. In new packet mode, we can use send_packet for send packet.
But send_packet shoud call sendp function from scpay, when use scapy, use ixia_send_packet when use ixia.
Thanks a lot
-----Original Message-----
From: Liu, Yong
Sent: Thursday, August 24, 2017 1:50 PM
To: Xu, HuilongX <huilongx.xu@intel.com>; Xu, GangX <gangx.xu@intel.com>; dts@dpdk.org
Subject: RE: [dts] [PATCH V1] fix userspace failed case
> -----Original Message-----
> From: Xu, HuilongX
> Sent: Thursday, August 24, 2017 1:42 PM
> To: Liu, Yong <yong.liu@intel.com>; Xu, GangX <gangx.xu@intel.com>;
> dts@dpdk.org
> Subject: RE: [dts] [PATCH V1] fix userspace failed case
>
> Hi, yong,
> Two comments as below,
> 1. use stat tools replace ls in next version 2. for use scapy replace
> packet mode.
> In this case, send packet very simple. We should use scapy default
> interface for send packet.
> Because it very steady, the log is clearness and friendly, the
> function interface in tester mode very simple also.
Agree with you, send/analyze packets by scapy is very simple.
But packet module is also very easy to use in transmission side.
Just two lines of code is needed. My option is that we should fix issue in the module rather than just skip it.
And I prefer that packet module to be the default method. The reason is that we can used it for other kinds generator like Trex/Ixia later.
> Thanks a lot
> -----Original Message-----
> From: Liu, Yong
> Sent: Thursday, August 24, 2017 1:00 PM
> To: Xu, HuilongX <huilongx.xu@intel.com>; Xu, GangX
> <gangx.xu@intel.com>; dts@dpdk.org
> Subject: RE: [dts] [PATCH V1] fix userspace failed case
>
> Huilong,
> Comments inline. Thanks.
>
> > -----Original Message-----
> > From: Xu, HuilongX
> > Sent: Thursday, August 24, 2017 11:57 AM
> > To: Liu, Yong <yong.liu@intel.com>; Xu, GangX <gangx.xu@intel.com>;
> > dts@dpdk.org
> > Subject: RE: [dts] [PATCH V1] fix userspace failed case
> >
> > Hi yong,
> > My comments as below. Could you check it?
> >
> > -----Original Message-----
> > From: Liu, Yong
> > Sent: Thursday, August 24, 2017 11:51 AM
> > To: Xu, GangX <gangx.xu@intel.com>; dts@dpdk.org
> > Cc: Xu, HuilongX <huilongx.xu@intel.com>
> > Subject: RE: [dts] [PATCH V1] fix userspace failed case
> >
> > Thanks gang, few comments below.
> >
> > > -----Original Message-----
> > > From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of xu,gang
> > > Sent: Wednesday, August 23, 2017 5:02 PM
> > > To: dts@dpdk.org
> > > Cc: Xu, HuilongX <huilongx.xu@intel.com>
> > > Subject: [dts] [PATCH V1] fix userspace failed case
> > >
> > > From: "xu,huilong" <huilongx.xu@intel.com>
> > >
> > > update list:
> > > 1. update userspace example path for setup app 2. add sleep time
> > > for app update link status with fortville nic in port config case 3.
> > > dpdk userspace tools dump eeprom file size different kernel
> > > ethtools dump, so only compare same size bytes 4. use scapy tool
> > > send packet replace packet model, because app can't received
> > > packet by use packet model
> > >
> > > Signed-off-by: xu,huilong <huilongx.xu@intel.com>
> > > ---
> > > tests/TestSuite_userspace_ethtool.py | 16 ++++++++++++----
> > > 1 file changed, 12 insertions(+), 4 deletions(-)
> > >
> > > diff --git a/tests/TestSuite_userspace_ethtool.py
> > > b/tests/TestSuite_userspace_ethtool.py
> > > index 937a9e5..1b234c1 100644
> > > --- a/tests/TestSuite_userspace_ethtool.py
> > > +++ b/tests/TestSuite_userspace_ethtool.py
> > > @@ -61,7 +61,7 @@ class TestUserspaceEthtool(TestCase,
> > > IxiaPacketGenerator):
> > > self.verify("Error" not in out, "compilation error 1")
> > > self.verify("No such file" not in out, "compilation error
> > > 2")
> > >
> > > - path = "./examples/ethtool/ethtool-app/ethtool-
> app/%s/ethtool" %
> > > self.target
> > > + path = "./examples/ethtool/ethtool-app/%s/ethtool" %
> > > + self.target
> > > self.cmd = "%s -c f -n %d" % (path,
> > > self.dut.get_memory_channels())
> > >
> > > # pause frame basic configuration @@ -370,7 +370,10 @@
> > > class TestUserspaceEthtool(TestCase,
> > > IxiaPacketGenerator):
> > > self.dut.send_expect("ethtool --eeprom-dump %s raw on
> > > > %s" % (intf, ethtool_eeprom), "# ")
> > > # wait for file ready
> > > time.sleep(2)
> > > - portinfo['ethtool_eeprom'] = ethtool_eeprom
> > > + # dpdk userspace dump eeprom-dump size different with
> > > + kernel
> > > ethtool dump eeprom-dump, so only compare same size bytes
> > > + dpdk_eeprom_size = int(self.dut.send_expect('ls -all %s'
> > > + %
> > > portinfo['eeprom_file'], '# ').split(' ')[4].strip())
> >
> > There is one more simple command to get file size.
> > "stat -c %s"
> > Stat command is default tools in all linux os?, if yes, I think can
> > use stat command replace ls.
>
> Stat command is belong to GNU coreutils. "ls" also in the same package.
> It will be fine to utilize stat command.
>
> > > + self.dut.send_expect('dd if=%s of=%s bs=%d count=1' %
> > > (ethtool_eeprom, "ethtool_eeprom_%d_cat.bin" % index,
> > > dpdk_eeprom_size),
> > > "#")
> > > + portinfo['ethtool_eeprom'] = "ethtool_eeprom_%d_cat.bin"
> > > + %
> > > index
> > > # bind to original driver
> > >
> > > portinfo['net_dev'].bind_driver(portinfo['ori_driver'])
> > >
> > > @@ -394,10 +397,12 @@ class TestUserspaceEthtool(TestCase,
> > > IxiaPacketGenerator):
> > > rx_ring, _, tx_ring, _ = self.strip_ringparam(index)
> > > self.verify(rx_ring == rx_max, "Userspace tool failed
> > > to set Rx ring parameter")
> > > self.verify(tx_ring == tx_max, "Userspace tool failed
> > > to set Tx ring parameter")
> > > - pkt = Packet()
> > > tester_port = self.tester.get_local_port(port)
> > > intf = self.tester.get_interface(tester_port)
> > > - pkt.send_pkt(tx_port=intf)
> > > + # use scapy send packet, because example can't
> > > + receive packet
> > > by packet model
> > > + packet =
> > > r'sendp([Ether(dst="ff:ff:ff:ff:ff:ff")/IP()/UDP()/Raw()],
> > > iface="%s")' % intf
> > > + self.tester.scapy_append(packet)
> > > + self.tester.scapy_execute()
> >
> > The only difference I can see is the destination mac address.
> > Could you please try again with below code?
> > I review packet mode, if destination mac address not set in packet mode.
> > Packet model will set default destination mac address is
> > "ff:ff:ff:ff:ff:ff", So I think not try again pkt with
> > "ff:ff:ff:ff:ff:ff" destination mac address.
> >
>
> I'm wondering that if any bug in packet module which can cause this issue.
> packet module will call the same API as scapy cli.
>
> > pkt.config_layer('ether', {'dst': 'ff:ff:ff:ff:ff:ff'})
> >
> > > rx_pkts, tx_pkts = self.strip_portstats(index)
> > > self.verify(rx_pkts == ori_rx_pkts + 1, "Failed to
> > > forward after ring parameter changed")
> > > self.dut.send_expect("quit", "# ") @@ -499,8 +504,11
> > > @@ class TestUserspaceEthtool(TestCase,
> > > IxiaPacketGenerator):
> > > for index in range(len(self.ports)):
> > > port = self.ports[index]
> > > ori_rx_pkts, _ = self.strip_portstats(index)
> > > + # add sleep time for example update port link status
> > > + time.sleep(10)
> > > # stop port
> > > self.dut.send_expect("stop %d" % index, "EthApp>")
> > > + time.sleep(10)
> > > # check packet not forwarded when port is stop
> > > pkt = Packet()
> > > tester_port = self.tester.get_local_port(port)
> > > --
> > > 1.9.3
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [dts] [PATCH V1] fix userspace failed case
2017-08-24 6:17 ` Xu, HuilongX
@ 2017-08-24 6:25 ` Liu, Yong
2017-08-24 6:47 ` Xu, HuilongX
0 siblings, 1 reply; 9+ messages in thread
From: Liu, Yong @ 2017-08-24 6:25 UTC (permalink / raw)
To: Xu, HuilongX, Xu, GangX, dts
Huilong,
First, "scapy internal function for build packet and send packet" is not the root cause for stable issue.
Scapy command line will call the same function.
Second, IXIA and Trex can do function test as they can transmit/sniff packets. There's one requirement that do function/performance cases in the same setup.
At last, let's close this discussion for it can't help us figure out what's the root cause.
Thanks,
Marvin
> -----Original Message-----
> From: Xu, HuilongX
> Sent: Thursday, August 24, 2017 2:18 PM
> To: Liu, Yong <yong.liu@intel.com>; Xu, GangX <gangx.xu@intel.com>;
> dts@dpdk.org
> Subject: RE: [dts] [PATCH V1] fix userspace failed case
>
> Hi yong,
> Yes, you are right, but for function test, we only should send/analyze
> some packets(most of the time, we only send one packet)
> For performance test, we need send packet stream and set send packet line
> rate by ixia/trex or other tools. But whether use any tools, we should
> Use unify interface, and the unify interface must use send packet tools
> supplied default interface, not develop a new interface. Eg, set line rate
> function,
> In our unify interface maye is set_line_rate, but set_line_rate will use
> ixia_set_line_rate or trex_set_line_rate, not develop a new
> ixia_set_line_rate or trex_line_rate. Now packet mode in framework we use
> scapy internal function for build packet and send packet, it is the root
> case why packet mode not stable.
> If we need use packet mode send packet for function, I think we can re-
> develop packet model. In new packet mode, we can use send_packet for send
> packet.
> But send_packet shoud call sendp function from scpay, when use scapy,
> use ixia_send_packet when use ixia.
>
> Thanks a lot
> -----Original Message-----
> From: Liu, Yong
> Sent: Thursday, August 24, 2017 1:50 PM
> To: Xu, HuilongX <huilongx.xu@intel.com>; Xu, GangX <gangx.xu@intel.com>;
> dts@dpdk.org
> Subject: RE: [dts] [PATCH V1] fix userspace failed case
>
> > -----Original Message-----
> > From: Xu, HuilongX
> > Sent: Thursday, August 24, 2017 1:42 PM
> > To: Liu, Yong <yong.liu@intel.com>; Xu, GangX <gangx.xu@intel.com>;
> > dts@dpdk.org
> > Subject: RE: [dts] [PATCH V1] fix userspace failed case
> >
> > Hi, yong,
> > Two comments as below,
> > 1. use stat tools replace ls in next version 2. for use scapy replace
> > packet mode.
> > In this case, send packet very simple. We should use scapy default
> > interface for send packet.
> > Because it very steady, the log is clearness and friendly, the
> > function interface in tester mode very simple also.
>
> Agree with you, send/analyze packets by scapy is very simple.
> But packet module is also very easy to use in transmission side.
> Just two lines of code is needed. My option is that we should fix issue in
> the module rather than just skip it.
> And I prefer that packet module to be the default method. The reason is
> that we can used it for other kinds generator like Trex/Ixia later.
>
> > Thanks a lot
> > -----Original Message-----
> > From: Liu, Yong
> > Sent: Thursday, August 24, 2017 1:00 PM
> > To: Xu, HuilongX <huilongx.xu@intel.com>; Xu, GangX
> > <gangx.xu@intel.com>; dts@dpdk.org
> > Subject: RE: [dts] [PATCH V1] fix userspace failed case
> >
> > Huilong,
> > Comments inline. Thanks.
> >
> > > -----Original Message-----
> > > From: Xu, HuilongX
> > > Sent: Thursday, August 24, 2017 11:57 AM
> > > To: Liu, Yong <yong.liu@intel.com>; Xu, GangX <gangx.xu@intel.com>;
> > > dts@dpdk.org
> > > Subject: RE: [dts] [PATCH V1] fix userspace failed case
> > >
> > > Hi yong,
> > > My comments as below. Could you check it?
> > >
> > > -----Original Message-----
> > > From: Liu, Yong
> > > Sent: Thursday, August 24, 2017 11:51 AM
> > > To: Xu, GangX <gangx.xu@intel.com>; dts@dpdk.org
> > > Cc: Xu, HuilongX <huilongx.xu@intel.com>
> > > Subject: RE: [dts] [PATCH V1] fix userspace failed case
> > >
> > > Thanks gang, few comments below.
> > >
> > > > -----Original Message-----
> > > > From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of xu,gang
> > > > Sent: Wednesday, August 23, 2017 5:02 PM
> > > > To: dts@dpdk.org
> > > > Cc: Xu, HuilongX <huilongx.xu@intel.com>
> > > > Subject: [dts] [PATCH V1] fix userspace failed case
> > > >
> > > > From: "xu,huilong" <huilongx.xu@intel.com>
> > > >
> > > > update list:
> > > > 1. update userspace example path for setup app 2. add sleep time
> > > > for app update link status with fortville nic in port config case 3.
> > > > dpdk userspace tools dump eeprom file size different kernel
> > > > ethtools dump, so only compare same size bytes 4. use scapy tool
> > > > send packet replace packet model, because app can't received
> > > > packet by use packet model
> > > >
> > > > Signed-off-by: xu,huilong <huilongx.xu@intel.com>
> > > > ---
> > > > tests/TestSuite_userspace_ethtool.py | 16 ++++++++++++----
> > > > 1 file changed, 12 insertions(+), 4 deletions(-)
> > > >
> > > > diff --git a/tests/TestSuite_userspace_ethtool.py
> > > > b/tests/TestSuite_userspace_ethtool.py
> > > > index 937a9e5..1b234c1 100644
> > > > --- a/tests/TestSuite_userspace_ethtool.py
> > > > +++ b/tests/TestSuite_userspace_ethtool.py
> > > > @@ -61,7 +61,7 @@ class TestUserspaceEthtool(TestCase,
> > > > IxiaPacketGenerator):
> > > > self.verify("Error" not in out, "compilation error 1")
> > > > self.verify("No such file" not in out, "compilation error
> > > > 2")
> > > >
> > > > - path = "./examples/ethtool/ethtool-app/ethtool-
> > app/%s/ethtool" %
> > > > self.target
> > > > + path = "./examples/ethtool/ethtool-app/%s/ethtool" %
> > > > + self.target
> > > > self.cmd = "%s -c f -n %d" % (path,
> > > > self.dut.get_memory_channels())
> > > >
> > > > # pause frame basic configuration @@ -370,7 +370,10 @@
> > > > class TestUserspaceEthtool(TestCase,
> > > > IxiaPacketGenerator):
> > > > self.dut.send_expect("ethtool --eeprom-dump %s raw on
> > > > > %s" % (intf, ethtool_eeprom), "# ")
> > > > # wait for file ready
> > > > time.sleep(2)
> > > > - portinfo['ethtool_eeprom'] = ethtool_eeprom
> > > > + # dpdk userspace dump eeprom-dump size different with
> > > > + kernel
> > > > ethtool dump eeprom-dump, so only compare same size bytes
> > > > + dpdk_eeprom_size = int(self.dut.send_expect('ls -
> all %s'
> > > > + %
> > > > portinfo['eeprom_file'], '# ').split(' ')[4].strip())
> > >
> > > There is one more simple command to get file size.
> > > "stat -c %s"
> > > Stat command is default tools in all linux os?, if yes, I think can
> > > use stat command replace ls.
> >
> > Stat command is belong to GNU coreutils. "ls" also in the same package.
> > It will be fine to utilize stat command.
> >
> > > > + self.dut.send_expect('dd if=%s of=%s bs=%d count=1' %
> > > > (ethtool_eeprom, "ethtool_eeprom_%d_cat.bin" % index,
> > > > dpdk_eeprom_size),
> > > > "#")
> > > > + portinfo['ethtool_eeprom'] =
> "ethtool_eeprom_%d_cat.bin"
> > > > + %
> > > > index
> > > > # bind to original driver
> > > >
> > > > portinfo['net_dev'].bind_driver(portinfo['ori_driver'])
> > > >
> > > > @@ -394,10 +397,12 @@ class TestUserspaceEthtool(TestCase,
> > > > IxiaPacketGenerator):
> > > > rx_ring, _, tx_ring, _ = self.strip_ringparam(index)
> > > > self.verify(rx_ring == rx_max, "Userspace tool failed
> > > > to set Rx ring parameter")
> > > > self.verify(tx_ring == tx_max, "Userspace tool failed
> > > > to set Tx ring parameter")
> > > > - pkt = Packet()
> > > > tester_port = self.tester.get_local_port(port)
> > > > intf = self.tester.get_interface(tester_port)
> > > > - pkt.send_pkt(tx_port=intf)
> > > > + # use scapy send packet, because example can't
> > > > + receive packet
> > > > by packet model
> > > > + packet =
> > > > r'sendp([Ether(dst="ff:ff:ff:ff:ff:ff")/IP()/UDP()/Raw()],
> > > > iface="%s")' % intf
> > > > + self.tester.scapy_append(packet)
> > > > + self.tester.scapy_execute()
> > >
> > > The only difference I can see is the destination mac address.
> > > Could you please try again with below code?
> > > I review packet mode, if destination mac address not set in packet
> mode.
> > > Packet model will set default destination mac address is
> > > "ff:ff:ff:ff:ff:ff", So I think not try again pkt with
> > > "ff:ff:ff:ff:ff:ff" destination mac address.
> > >
> >
> > I'm wondering that if any bug in packet module which can cause this
> issue.
> > packet module will call the same API as scapy cli.
> >
> > > pkt.config_layer('ether', {'dst': 'ff:ff:ff:ff:ff:ff'})
> > >
> > > > rx_pkts, tx_pkts = self.strip_portstats(index)
> > > > self.verify(rx_pkts == ori_rx_pkts + 1, "Failed to
> > > > forward after ring parameter changed")
> > > > self.dut.send_expect("quit", "# ") @@ -499,8 +504,11
> > > > @@ class TestUserspaceEthtool(TestCase,
> > > > IxiaPacketGenerator):
> > > > for index in range(len(self.ports)):
> > > > port = self.ports[index]
> > > > ori_rx_pkts, _ = self.strip_portstats(index)
> > > > + # add sleep time for example update port link status
> > > > + time.sleep(10)
> > > > # stop port
> > > > self.dut.send_expect("stop %d" % index, "EthApp>")
> > > > + time.sleep(10)
> > > > # check packet not forwarded when port is stop
> > > > pkt = Packet()
> > > > tester_port = self.tester.get_local_port(port)
> > > > --
> > > > 1.9.3
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [dts] [PATCH V1] fix userspace failed case
2017-08-24 6:25 ` Liu, Yong
@ 2017-08-24 6:47 ` Xu, HuilongX
0 siblings, 0 replies; 9+ messages in thread
From: Xu, HuilongX @ 2017-08-24 6:47 UTC (permalink / raw)
To: Liu, Yong, Xu, GangX, dts
Ok for this patch, I will use stat replace ls in V2.
I will not fix send packet error in this patch.
-----Original Message-----
From: Liu, Yong
Sent: Thursday, August 24, 2017 2:26 PM
To: Xu, HuilongX <huilongx.xu@intel.com>; Xu, GangX <gangx.xu@intel.com>; dts@dpdk.org
Subject: RE: [dts] [PATCH V1] fix userspace failed case
Huilong,
First, "scapy internal function for build packet and send packet" is not the root cause for stable issue.
Scapy command line will call the same function.
Second, IXIA and Trex can do function test as they can transmit/sniff packets. There's one requirement that do function/performance cases in the same setup.
At last, let's close this discussion for it can't help us figure out what's the root cause.
Thanks,
Marvin
> -----Original Message-----
> From: Xu, HuilongX
> Sent: Thursday, August 24, 2017 2:18 PM
> To: Liu, Yong <yong.liu@intel.com>; Xu, GangX <gangx.xu@intel.com>;
> dts@dpdk.org
> Subject: RE: [dts] [PATCH V1] fix userspace failed case
>
> Hi yong,
> Yes, you are right, but for function test, we only should send/analyze
> some packets(most of the time, we only send one packet) For
> performance test, we need send packet stream and set send packet line
> rate by ixia/trex or other tools. But whether use any tools, we should
> Use unify interface, and the unify interface must use send packet
> tools supplied default interface, not develop a new interface. Eg, set
> line rate function, In our unify interface maye is set_line_rate, but
> set_line_rate will use ixia_set_line_rate or trex_set_line_rate, not
> develop a new ixia_set_line_rate or trex_line_rate. Now packet mode in
> framework we use scapy internal function for build packet and send
> packet, it is the root case why packet mode not stable.
> If we need use packet mode send packet for function, I think we can
> re- develop packet model. In new packet mode, we can use send_packet
> for send packet.
> But send_packet shoud call sendp function from scpay, when use scapy,
> use ixia_send_packet when use ixia.
>
> Thanks a lot
> -----Original Message-----
> From: Liu, Yong
> Sent: Thursday, August 24, 2017 1:50 PM
> To: Xu, HuilongX <huilongx.xu@intel.com>; Xu, GangX
> <gangx.xu@intel.com>; dts@dpdk.org
> Subject: RE: [dts] [PATCH V1] fix userspace failed case
>
> > -----Original Message-----
> > From: Xu, HuilongX
> > Sent: Thursday, August 24, 2017 1:42 PM
> > To: Liu, Yong <yong.liu@intel.com>; Xu, GangX <gangx.xu@intel.com>;
> > dts@dpdk.org
> > Subject: RE: [dts] [PATCH V1] fix userspace failed case
> >
> > Hi, yong,
> > Two comments as below,
> > 1. use stat tools replace ls in next version 2. for use scapy
> > replace packet mode.
> > In this case, send packet very simple. We should use scapy
> > default interface for send packet.
> > Because it very steady, the log is clearness and friendly, the
> > function interface in tester mode very simple also.
>
> Agree with you, send/analyze packets by scapy is very simple.
> But packet module is also very easy to use in transmission side.
> Just two lines of code is needed. My option is that we should fix
> issue in the module rather than just skip it.
> And I prefer that packet module to be the default method. The reason
> is that we can used it for other kinds generator like Trex/Ixia later.
>
> > Thanks a lot
> > -----Original Message-----
> > From: Liu, Yong
> > Sent: Thursday, August 24, 2017 1:00 PM
> > To: Xu, HuilongX <huilongx.xu@intel.com>; Xu, GangX
> > <gangx.xu@intel.com>; dts@dpdk.org
> > Subject: RE: [dts] [PATCH V1] fix userspace failed case
> >
> > Huilong,
> > Comments inline. Thanks.
> >
> > > -----Original Message-----
> > > From: Xu, HuilongX
> > > Sent: Thursday, August 24, 2017 11:57 AM
> > > To: Liu, Yong <yong.liu@intel.com>; Xu, GangX
> > > <gangx.xu@intel.com>; dts@dpdk.org
> > > Subject: RE: [dts] [PATCH V1] fix userspace failed case
> > >
> > > Hi yong,
> > > My comments as below. Could you check it?
> > >
> > > -----Original Message-----
> > > From: Liu, Yong
> > > Sent: Thursday, August 24, 2017 11:51 AM
> > > To: Xu, GangX <gangx.xu@intel.com>; dts@dpdk.org
> > > Cc: Xu, HuilongX <huilongx.xu@intel.com>
> > > Subject: RE: [dts] [PATCH V1] fix userspace failed case
> > >
> > > Thanks gang, few comments below.
> > >
> > > > -----Original Message-----
> > > > From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of xu,gang
> > > > Sent: Wednesday, August 23, 2017 5:02 PM
> > > > To: dts@dpdk.org
> > > > Cc: Xu, HuilongX <huilongx.xu@intel.com>
> > > > Subject: [dts] [PATCH V1] fix userspace failed case
> > > >
> > > > From: "xu,huilong" <huilongx.xu@intel.com>
> > > >
> > > > update list:
> > > > 1. update userspace example path for setup app 2. add sleep time
> > > > for app update link status with fortville nic in port config case 3.
> > > > dpdk userspace tools dump eeprom file size different kernel
> > > > ethtools dump, so only compare same size bytes 4. use scapy tool
> > > > send packet replace packet model, because app can't received
> > > > packet by use packet model
> > > >
> > > > Signed-off-by: xu,huilong <huilongx.xu@intel.com>
> > > > ---
> > > > tests/TestSuite_userspace_ethtool.py | 16 ++++++++++++----
> > > > 1 file changed, 12 insertions(+), 4 deletions(-)
> > > >
> > > > diff --git a/tests/TestSuite_userspace_ethtool.py
> > > > b/tests/TestSuite_userspace_ethtool.py
> > > > index 937a9e5..1b234c1 100644
> > > > --- a/tests/TestSuite_userspace_ethtool.py
> > > > +++ b/tests/TestSuite_userspace_ethtool.py
> > > > @@ -61,7 +61,7 @@ class TestUserspaceEthtool(TestCase,
> > > > IxiaPacketGenerator):
> > > > self.verify("Error" not in out, "compilation error 1")
> > > > self.verify("No such file" not in out, "compilation
> > > > error
> > > > 2")
> > > >
> > > > - path = "./examples/ethtool/ethtool-app/ethtool-
> > app/%s/ethtool" %
> > > > self.target
> > > > + path = "./examples/ethtool/ethtool-app/%s/ethtool" %
> > > > + self.target
> > > > self.cmd = "%s -c f -n %d" % (path,
> > > > self.dut.get_memory_channels())
> > > >
> > > > # pause frame basic configuration @@ -370,7 +370,10 @@
> > > > class TestUserspaceEthtool(TestCase,
> > > > IxiaPacketGenerator):
> > > > self.dut.send_expect("ethtool --eeprom-dump %s raw
> > > > on
> > > > > %s" % (intf, ethtool_eeprom), "# ")
> > > > # wait for file ready
> > > > time.sleep(2)
> > > > - portinfo['ethtool_eeprom'] = ethtool_eeprom
> > > > + # dpdk userspace dump eeprom-dump size different
> > > > + with kernel
> > > > ethtool dump eeprom-dump, so only compare same size bytes
> > > > + dpdk_eeprom_size = int(self.dut.send_expect('ls -
> all %s'
> > > > + %
> > > > portinfo['eeprom_file'], '# ').split(' ')[4].strip())
> > >
> > > There is one more simple command to get file size.
> > > "stat -c %s"
> > > Stat command is default tools in all linux os?, if yes, I think
> > > can use stat command replace ls.
> >
> > Stat command is belong to GNU coreutils. "ls" also in the same package.
> > It will be fine to utilize stat command.
> >
> > > > + self.dut.send_expect('dd if=%s of=%s bs=%d count=1'
> > > > + %
> > > > (ethtool_eeprom, "ethtool_eeprom_%d_cat.bin" % index,
> > > > dpdk_eeprom_size),
> > > > "#")
> > > > + portinfo['ethtool_eeprom'] =
> "ethtool_eeprom_%d_cat.bin"
> > > > + %
> > > > index
> > > > # bind to original driver
> > > >
> > > > portinfo['net_dev'].bind_driver(portinfo['ori_driver'])
> > > >
> > > > @@ -394,10 +397,12 @@ class TestUserspaceEthtool(TestCase,
> > > > IxiaPacketGenerator):
> > > > rx_ring, _, tx_ring, _ = self.strip_ringparam(index)
> > > > self.verify(rx_ring == rx_max, "Userspace tool
> > > > failed to set Rx ring parameter")
> > > > self.verify(tx_ring == tx_max, "Userspace tool
> > > > failed to set Tx ring parameter")
> > > > - pkt = Packet()
> > > > tester_port = self.tester.get_local_port(port)
> > > > intf = self.tester.get_interface(tester_port)
> > > > - pkt.send_pkt(tx_port=intf)
> > > > + # use scapy send packet, because example can't
> > > > + receive packet
> > > > by packet model
> > > > + packet =
> > > > r'sendp([Ether(dst="ff:ff:ff:ff:ff:ff")/IP()/UDP()/Raw()],
> > > > iface="%s")' % intf
> > > > + self.tester.scapy_append(packet)
> > > > + self.tester.scapy_execute()
> > >
> > > The only difference I can see is the destination mac address.
> > > Could you please try again with below code?
> > > I review packet mode, if destination mac address not set in packet
> mode.
> > > Packet model will set default destination mac address is
> > > "ff:ff:ff:ff:ff:ff", So I think not try again pkt with
> > > "ff:ff:ff:ff:ff:ff" destination mac address.
> > >
> >
> > I'm wondering that if any bug in packet module which can cause this
> issue.
> > packet module will call the same API as scapy cli.
> >
> > > pkt.config_layer('ether', {'dst': 'ff:ff:ff:ff:ff:ff'})
> > >
> > > > rx_pkts, tx_pkts = self.strip_portstats(index)
> > > > self.verify(rx_pkts == ori_rx_pkts + 1, "Failed to
> > > > forward after ring parameter changed")
> > > > self.dut.send_expect("quit", "# ") @@ -499,8
> > > > +504,11 @@ class TestUserspaceEthtool(TestCase,
> > > > IxiaPacketGenerator):
> > > > for index in range(len(self.ports)):
> > > > port = self.ports[index]
> > > > ori_rx_pkts, _ = self.strip_portstats(index)
> > > > + # add sleep time for example update port link status
> > > > + time.sleep(10)
> > > > # stop port
> > > > self.dut.send_expect("stop %d" % index, "EthApp>")
> > > > + time.sleep(10)
> > > > # check packet not forwarded when port is stop
> > > > pkt = Packet()
> > > > tester_port = self.tester.get_local_port(port)
> > > > --
> > > > 1.9.3
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2017-08-24 6:47 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-08-23 9:01 [dts] [PATCH V1] fix userspace failed case xu,gang
2017-08-24 3:50 ` Liu, Yong
2017-08-24 3:57 ` Xu, HuilongX
2017-08-24 5:00 ` Liu, Yong
2017-08-24 5:41 ` Xu, HuilongX
2017-08-24 5:49 ` Liu, Yong
2017-08-24 6:17 ` Xu, HuilongX
2017-08-24 6:25 ` Liu, Yong
2017-08-24 6:47 ` Xu, HuilongX
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).