Re: [dpdk-dev] [PATCH] app/testpmd: fix flow list command

2018-10-10 Thread Adrien Mazarguil
On Wed, Oct 10, 2018 at 04:27:59PM +, Mordechay Haimovsky wrote: > Hi Adrien, > You are correct, the bug is not where we thought it is, moreover the fix > breaks > the CLI and should be rejected. > We investigated more and found that the bug is in the port_flow_query routine > Which passes an

Re: [dpdk-dev] [PATCH] app/testpmd: fix flow list command

2018-10-10 Thread Mordechay Haimovsky
o: John Daley > Cc: dev@dpdk.org; Ferruh Yigit > Subject: Re: [dpdk-dev] [PATCH] app/testpmd: fix flow list command > > Hi John > > On Tue, Oct 09, 2018 at 03:51:24PM -0700, John Daley wrote: > > This patch fixes the 'flow list ' command which caused a > >

Re: [dpdk-dev] [PATCH] app/testpmd: fix flow list command

2018-10-10 Thread Adrien Mazarguil
Hi John On Tue, Oct 09, 2018 at 03:51:24PM -0700, John Daley wrote: > This patch fixes the 'flow list ' command which caused a > segfault when passing the action or item 'type' field instead > of the action or item struct pointer in the call to rte_flow_conv. > > Fixes: 7d94dcedf7ce ("app/testpmd

[dpdk-dev] [PATCH] app/testpmd: fix flow list command

2018-10-09 Thread John Daley
This patch fixes the 'flow list ' command which caused a segfault when passing the action or item 'type' field instead of the action or item struct pointer in the call to rte_flow_conv. Fixes: 7d94dcedf7ce ("app/testpmd: rely on flow API conversion function") Signed-off-by: John Daley --- app/t