DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] app/testpmd: fix Rx offload search error
@ 2018-11-07  6:14 Wei Zhao
  2018-11-07  7:58 ` Peng, Yuan
  0 siblings, 1 reply; 5+ messages in thread
From: Wei Zhao @ 2018-11-07  6:14 UTC (permalink / raw)
  To: dev; +Cc: stable, yuan.peng, wenzhuo.lu, Wei Zhao

There is an error in function in function search_rx_offload(),
it will break when get unexpected return value from function
rte_eth_dev_rx_offload_name(), but rte_eth_dev_rx_offload_name()
will return some unexpected value indead.

Fixes: c73a9071877a ("app/testpmd: add commands to test new offload API")
Signed-off-by: Wei Zhao <wei.zhao1@intel.com>
---
 app/test-pmd/cmdline.c | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/app/test-pmd/cmdline.c b/app/test-pmd/cmdline.c
index 5e08a1b..1275074 100644
--- a/app/test-pmd/cmdline.c
+++ b/app/test-pmd/cmdline.c
@@ -17805,10 +17805,7 @@ search_rx_offload(const char *name)
 		if (!strcasecmp(single_name, name)) {
 			found = 1;
 			break;
-		} else if (!strcasecmp(single_name, "UNKNOWN"))
-			break;
-		else if (single_name == NULL)
-			break;
+		}
 		single_offload <<= 1;
 	}
 
-- 
2.7.5

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

* Re: [dpdk-dev] [PATCH] app/testpmd: fix Rx offload search error
  2018-11-07  6:14 [dpdk-dev] [PATCH] app/testpmd: fix Rx offload search error Wei Zhao
@ 2018-11-07  7:58 ` Peng, Yuan
  2018-11-07 17:09   ` Iremonger, Bernard
  2018-11-07 19:02   ` [dpdk-dev] [dpdk-stable] " Ferruh Yigit
  0 siblings, 2 replies; 5+ messages in thread
From: Peng, Yuan @ 2018-11-07  7:58 UTC (permalink / raw)
  To: Zhao1, Wei, dev; +Cc: stable, Lu, Wenzhuo

Tested-by: Peng, Yuan <yuan.peng@intel.com>

- Tested Branch: master
- Tested Commit: c59b06294fb4531792a4c74ca63fa79a4cb53457
- OS: 4.5.5-300.fc24.x86_64
- GCC: gcc (GCC) 5.3.1 20151207 (Red Hat 5.3.1-2)
- CPU: Intel(R) Xeon(R) CPU E5-2699 v4 @ 2.20GHz
- NIC: Intel Corporation 82599ES 10-Gigabit SFI/SFP+ Network Connection [8086:10fb] (rev 01)
     Intel Corporation Ethernet Controller X710 for 10GbE SFP+ [8086:1572] (rev 01)
- Default x86_64-native-linuxapp-gcc configuration
- Prerequisites:
- Total 1 cases, 1 passed, 0 failed

- Case:
./usertools/dpdk-devbind.py -b igb_uio 07:00.0 07:00.1
./x86_64-native-linuxapp-gcc/app/testpmd -c f -n 4 - -i --rxq=4 --txq=4
 testpmd> show port 0 rx_offload capabilities
 Rx Offloading Capabilities of port 0 :
Per Queue : VLAN_STRIP
Per Port : IPV4_CKSUM UDP_CKSUM TCP_CKSUM TCP_LRO MACSEC_STRIP VLAN_FILTER VLAN_EXTEND JUMBO_FRAME SCATTER SECURITY KEEP_CRC
 testpmd> show port 0 rx_offload configuration
 Rx Offloading Configuration of port 0 :
  Port :
  Queue[ 0] :
  Queue[ 1] :
  Queue[ 2] :
  Queue[ 3] :
testpmd> port stop 0
testpmd> port config 0 rx_offload security on
testpmd> port config 0 rx_offload scatter on
testpmd> port config 0 rx_offload keep_crc on
testpmd> port config 0 rx_offload jumbo_frame on
testpmd> port config 0 rx_offload vlan_extend on
testpmd> port config 0 rx_offload vlan_filter on
testpmd> port config 0 rx_offload macsec_strip on
testpmd> port config 0 rx_offload tcp_lro on
testpmd> port config 0 rx_offload tcp_cksum on
testpmd> port config 0 rx_offload udp_cksum on
testpmd> port config 0 rx_offload ipv4_cksum on
testpmd>
testpmd> show port 0 rx_offload configuration
Rx Offloading Configuration of port 0 :
  Port : IPV4_CKSUM UDP_CKSUM TCP_CKSUM TCP_LRO MACSEC_STRIP VLAN_FILTER VLAN_EXTEND JUMBO_FRAME SCATTER SECURITY KEEP_CRC
  Queue[ 0] : IPV4_CKSUM UDP_CKSUM TCP_CKSUM TCP_LRO MACSEC_STRIP VLAN_FILTER VLAN_EXTEND JUMBO_FRAME SCATTER SECURITY KEEP_CRC
  Queue[ 1] : IPV4_CKSUM UDP_CKSUM TCP_CKSUM TCP_LRO MACSEC_STRIP VLAN_FILTER VLAN_EXTEND JUMBO_FRAME SCATTER SECURITY KEEP_CRC
  Queue[ 2] : IPV4_CKSUM UDP_CKSUM TCP_CKSUM TCP_LRO MACSEC_STRIP VLAN_FILTER VLAN_EXTEND JUMBO_FRAME SCATTER SECURITY KEEP_CRC
  Queue[ 3] : IPV4_CKSUM UDP_CKSUM TCP_CKSUM TCP_LRO MACSEC_STRIP VLAN_FILTER VLAN_EXTEND JUMBO_FRAME SCATTER SECURITY KEEP_CRC
testpmd> port config 0 rx_offload ipv4_cksum off
testpmd> port config 0 rx_offload udp_cksum off
testpmd> port config 0 rx_offload tcp_cksum off
testpmd> port config 0 rx_offload tcp_lro off
testpmd> port config 0 rx_offload macsec_strip off
testpmd> port config 0 rx_offload vlan_filter off
testpmd> port config 0 rx_offload vlan_extend off
testpmd> port config 0 rx_offload jumbo_frame off
testpmd> port config 0 rx_offload keep_crc off
testpmd> port config 0 rx_offload scatter off
testpmd> port config 0 rx_offload security off
testpmd> show port 0 rx_offload configuration
Rx Offloading Configuration of port 0 :
  Port :
  Queue[ 0] :
  Queue[ 1] :
  Queue[ 2] :
  Queue[ 3] :

All the capabilities can be configured successfully.
The case can pass with i40e and ixgbe card.


-----Original Message-----
From: Zhao1, Wei 
Sent: Wednesday, November 7, 2018 2:14 PM
To: dev@dpdk.org
Cc: stable@dpdk.org; Peng, Yuan <yuan.peng@intel.com>; Lu, Wenzhuo <wenzhuo.lu@intel.com>; Zhao1, Wei <wei.zhao1@intel.com>
Subject: [PATCH] app/testpmd: fix Rx offload search error

There is an error in function in function search_rx_offload(), it will break when get unexpected return value from function rte_eth_dev_rx_offload_name(), but rte_eth_dev_rx_offload_name() will return some unexpected value indead.

Fixes: c73a9071877a ("app/testpmd: add commands to test new offload API")
Signed-off-by: Wei Zhao <wei.zhao1@intel.com>
---
 app/test-pmd/cmdline.c | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/app/test-pmd/cmdline.c b/app/test-pmd/cmdline.c index 5e08a1b..1275074 100644
--- a/app/test-pmd/cmdline.c
+++ b/app/test-pmd/cmdline.c
@@ -17805,10 +17805,7 @@ search_rx_offload(const char *name)
 		if (!strcasecmp(single_name, name)) {
 			found = 1;
 			break;
-		} else if (!strcasecmp(single_name, "UNKNOWN"))
-			break;
-		else if (single_name == NULL)
-			break;
+		}
 		single_offload <<= 1;
 	}
 
--
2.7.5

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

* Re: [dpdk-dev] [PATCH] app/testpmd: fix Rx offload search error
  2018-11-07  7:58 ` Peng, Yuan
@ 2018-11-07 17:09   ` Iremonger, Bernard
  2018-11-07 19:02   ` [dpdk-dev] [dpdk-stable] " Ferruh Yigit
  1 sibling, 0 replies; 5+ messages in thread
From: Iremonger, Bernard @ 2018-11-07 17:09 UTC (permalink / raw)
  To: Peng, Yuan, Zhao1, Wei, dev; +Cc: stable, Lu, Wenzhuo

Hi Wei,

> -----Original Message-----
> From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Peng, Yuan
> Sent: Wednesday, November 7, 2018 7:58 AM
> To: Zhao1, Wei <wei.zhao1@intel.com>; dev@dpdk.org
> Cc: stable@dpdk.org; Lu, Wenzhuo <wenzhuo.lu@intel.com>
> Subject: Re: [dpdk-dev] [PATCH] app/testpmd: fix Rx offload search error
> 
> Tested-by: Peng, Yuan <yuan.peng@intel.com>

./devtools/check-git-log.sh -1
Wrong tag:
        Tested-by: Peng, Yuan <yuan.peng@intel.com>

Checkpatches.pl is detecting a duplicate signoff
### app/testpmd: fix Rx offload search error

WARNING:BAD_SIGN_OFF: Duplicate signature
#26: 
Signed-off-by: Wei Zhao <wei.zhao1@intel.com>

<snip>

> Subject: [PATCH] app/testpmd: fix Rx offload search error
> 
> There is an error in function in function search_rx_offload(), it will break when
> get unexpected return value from function rte_eth_dev_rx_offload_name(), but
> rte_eth_dev_rx_offload_name() will return some unexpected value indead.
> 
> Fixes: c73a9071877a ("app/testpmd: add commands to test new offload API")
> Signed-off-by: Wei Zhao <wei.zhao1@intel.com>
> ---
>  app/test-pmd/cmdline.c | 5 +----
>  1 file changed, 1 insertion(+), 4 deletions(-)
> 
> diff --git a/app/test-pmd/cmdline.c b/app/test-pmd/cmdline.c index
> 5e08a1b..1275074 100644
> --- a/app/test-pmd/cmdline.c
> +++ b/app/test-pmd/cmdline.c
> @@ -17805,10 +17805,7 @@ search_rx_offload(const char *name)
>  		if (!strcasecmp(single_name, name)) {
>  			found = 1;
>  			break;
> -		} else if (!strcasecmp(single_name, "UNKNOWN"))
> -			break;
> -		else if (single_name == NULL)
> -			break;
> +		}
>  		single_offload <<= 1;
>  	}
> 
> --
> 2.7.5

Regards,

Bernard.

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

* Re: [dpdk-dev] [dpdk-stable] [PATCH] app/testpmd: fix Rx offload search error
  2018-11-07  7:58 ` Peng, Yuan
  2018-11-07 17:09   ` Iremonger, Bernard
@ 2018-11-07 19:02   ` Ferruh Yigit
  2018-11-08  1:30     ` Zhao1, Wei
  1 sibling, 1 reply; 5+ messages in thread
From: Ferruh Yigit @ 2018-11-07 19:02 UTC (permalink / raw)
  To: Peng, Yuan, Zhao1, Wei, dev; +Cc: stable, Lu, Wenzhuo, Bernard Iremonger

On 11/7/2018 7:58 AM, Peng, Yuan wrote:

<...>

> From: Zhao1, Wei 
> Sent: Wednesday, November 7, 2018 2:14 PM
> To: dev@dpdk.org
> Cc: stable@dpdk.org; Peng, Yuan <yuan.peng@intel.com>; Lu, Wenzhuo <wenzhuo.lu@intel.com>; Zhao1, Wei <wei.zhao1@intel.com>
> Subject: [PATCH] app/testpmd: fix Rx offload search error
> 
> There is an error in function in function search_rx_offload(), it will break when get unexpected return value from function rte_eth_dev_rx_offload_name(), but rte_eth_dev_rx_offload_name() will return some unexpected value indead.
> 
> Fixes: c73a9071877a ("app/testpmd: add commands to test new offload API")
> Signed-off-by: Wei Zhao <wei.zhao1@intel.com>
>
> Tested-by: Peng, Yuan <yuan.peng@intel.com>

Applied to dpdk-next-net/master, thanks.

(checkpatch issues reported by Bernard fixed while merging.)

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

* Re: [dpdk-dev] [dpdk-stable] [PATCH] app/testpmd: fix Rx offload search error
  2018-11-07 19:02   ` [dpdk-dev] [dpdk-stable] " Ferruh Yigit
@ 2018-11-08  1:30     ` Zhao1, Wei
  0 siblings, 0 replies; 5+ messages in thread
From: Zhao1, Wei @ 2018-11-08  1:30 UTC (permalink / raw)
  To: Yigit, Ferruh, Peng, Yuan, dev; +Cc: stable, Lu, Wenzhuo, Iremonger, Bernard



> -----Original Message-----
> From: Yigit, Ferruh
> Sent: Thursday, November 8, 2018 3:02 AM
> To: Peng, Yuan <yuan.peng@intel.com>; Zhao1, Wei <wei.zhao1@intel.com>;
> dev@dpdk.org
> Cc: stable@dpdk.org; Lu, Wenzhuo <wenzhuo.lu@intel.com>; Iremonger,
> Bernard <bernard.iremonger@intel.com>
> Subject: Re: [dpdk-stable] [PATCH] app/testpmd: fix Rx offload search error
> 
> On 11/7/2018 7:58 AM, Peng, Yuan wrote:
> 
> <...>
> 
> > From: Zhao1, Wei
> > Sent: Wednesday, November 7, 2018 2:14 PM
> > To: dev@dpdk.org
> > Cc: stable@dpdk.org; Peng, Yuan <yuan.peng@intel.com>; Lu, Wenzhuo
> <wenzhuo.lu@intel.com>; Zhao1, Wei <wei.zhao1@intel.com>
> > Subject: [PATCH] app/testpmd: fix Rx offload search error
> >
> > There is an error in function in function search_rx_offload(), it will break
> when get unexpected return value from function
> rte_eth_dev_rx_offload_name(), but rte_eth_dev_rx_offload_name() will
> return some unexpected value indead.
> >
> > Fixes: c73a9071877a ("app/testpmd: add commands to test new offload
> API")
> > Signed-off-by: Wei Zhao <wei.zhao1@intel.com>
> >
> > Tested-by: Peng, Yuan <yuan.peng@intel.com>
> 
> Applied to dpdk-next-net/master, thanks.
> 
> (checkpatch issues reported by Bernard fixed while merging.)

Thanks!

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

end of thread, other threads:[~2018-11-08  1:30 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-11-07  6:14 [dpdk-dev] [PATCH] app/testpmd: fix Rx offload search error Wei Zhao
2018-11-07  7:58 ` Peng, Yuan
2018-11-07 17:09   ` Iremonger, Bernard
2018-11-07 19:02   ` [dpdk-dev] [dpdk-stable] " Ferruh Yigit
2018-11-08  1:30     ` Zhao1, Wei

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