Ok, makes sense now, both flow query and flow list work with this patch, yeah.
Tested-by: John Daley <[email protected]> Remember that https://patches.dpdk.org/patch/46221/ still needs to be reverted out of dpdk-next-net/master otherwise we are still broken. The patch was squished into e5b652ea34. -johnd > -----Original Message----- > From: dev <[email protected]> On Behalf Of Mordechay Haimovsky > Sent: Wednesday, October 10, 2018 10:05 AM > To: Adrien Mazarguil <[email protected]>; Shahaf Shuler > <[email protected]>; Ori Kam <[email protected]> > Cc: [email protected]; Mordechay Haimovsky <[email protected]> > Subject: [dpdk-dev] [PATCH] app/testpmd: fix flow query failure > > This patch fixes a bug found in port_flow_query routine which caused flow > query command to fail with the following error "Caught error type 1 (cause > unspecified): unknown object type to retrieve the name > of: Invalid argument". > > Fixes: f7ba5e7a0f8c ("app/testpmd: rely on flow API conversion function") > > Signed-off-by: Moti Haimovsky <[email protected]> > --- > app/test-pmd/config.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/app/test-pmd/config.c b/app/test-pmd/config.c index > 009c92c..b11317b 100644 > --- a/app/test-pmd/config.c > +++ b/app/test-pmd/config.c > @@ -1239,7 +1239,8 @@ void print_valid_ports(void) > return -ENOENT; > } > ret = rte_flow_conv(RTE_FLOW_CONV_OP_ACTION_NAME_PTR, > - &name, sizeof(name), action, &error); > + &name, sizeof(name), > + (void *)(uintptr_t)action->type, &error); > if (ret < 0) > return port_flow_complain(&error); > switch (action->type) { > -- > 1.8.3.1

