app/testpmd: fix missing geneve flow item in raw encap

Message ID 1571300024-23720-1-git-send-email-wisamm@mellanox.com (mailing list archive)
State Accepted, archived
Delegated to: Ferruh Yigit
Headers
Series app/testpmd: fix missing geneve flow item in raw encap |

Checks

Context Check Description
ci/checkpatch warning coding style issues
ci/Intel-compilation success Compilation OK
ci/iol-compilation success Compile Testing PASS
ci/iol-mellanox-Performance success Performance Testing PASS
ci/iol-intel-Performance success Performance Testing PASS

Commit Message

Wisam Jaddo Oct. 17, 2019, 8:13 a.m. UTC
  Fixes: 0f4203fe9d18 ("app/testpmd: support GENEVE pattern item in flow rules")
Cc: roman.zhukov@oktetlabs.ru
Cc: stable@dpdk.org

Signed-off-by: Wisam Jaddo <wisamm@mellanox.com>
---
 app/test-pmd/cmdline_flow.c | 3 +++
 1 file changed, 3 insertions(+)
  

Comments

Ori Kam Oct. 23, 2019, 1:46 p.m. UTC | #1
> -----Original Message-----
> From: dev <dev-bounces@dpdk.org> On Behalf Of Wisam Monther
> Sent: Thursday, October 17, 2019 11:14 AM
> To: wenzhuo.lu@intel.com; dev@dpdk.org
> Cc: Wisam Monther <wisamm@mellanox.com>; roman.zhukov@oktetlabs.ru;
> stable@dpdk.org
> Subject: [dpdk-dev] [PATCH] app/testpmd: fix missing geneve flow item in raw
> encap
> 
> Fixes: 0f4203fe9d18 ("app/testpmd: support GENEVE pattern item in flow
> rules")
> Cc: roman.zhukov@oktetlabs.ru
> Cc: stable@dpdk.org
> 
> Signed-off-by: Wisam Jaddo <wisamm@mellanox.com>
> ---

Acked-by: Ori Kam <orika@mellanox.com>
Thanks,
Ori
  
Ferruh Yigit Oct. 23, 2019, 5:56 p.m. UTC | #2
On 10/23/2019 2:46 PM, Ori Kam wrote:
> 
> 
>> -----Original Message-----
>> From: dev <dev-bounces@dpdk.org> On Behalf Of Wisam Monther
>> Sent: Thursday, October 17, 2019 11:14 AM
>> To: wenzhuo.lu@intel.com; dev@dpdk.org
>> Cc: Wisam Monther <wisamm@mellanox.com>; roman.zhukov@oktetlabs.ru;
>> stable@dpdk.org
>> Subject: [dpdk-dev] [PATCH] app/testpmd: fix missing geneve flow item in raw
>> encap
>>
>> Fixes: 0f4203fe9d18 ("app/testpmd: support GENEVE pattern item in flow
>> rules")
>> Cc: roman.zhukov@oktetlabs.ru
>> Cc: stable@dpdk.org
>>
>> Signed-off-by: Wisam Jaddo <wisamm@mellanox.com>
>> ---
> 
> Acked-by: Ori Kam <orika@mellanox.com>

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

Patch

diff --git a/app/test-pmd/cmdline_flow.c b/app/test-pmd/cmdline_flow.c
index b26b8bf..84d4ae2 100644
--- a/app/test-pmd/cmdline_flow.c
+++ b/app/test-pmd/cmdline_flow.c
@@ -5944,6 +5944,9 @@  cmd_set_raw_parsed(const struct buffer *in)
 			size = sizeof(struct rte_flow_item_nvgre);
 			proto = 0x2F;
 			break;
+		case RTE_FLOW_ITEM_TYPE_GENEVE:
+			size = sizeof(struct rte_flow_item_geneve);
+			break;
 		default:
 			printf("Error - Not supported item\n");
 			*total_size = 0;