21/07/2019 15:42, Raslan Darawsheh: > Hi Guys, > > We have a failure in testpmd commands when parsing rte_flow commands as > following which were interduce by this patch: > > This rule is a valid rule to be added to testpmd: > flow validate 0 priority 2 ingress group 0 pattern eth dst is > 98:03:9B:5C:D9:00 / end actions queue index 0 / end > > but, currently with latest master it will fail with Bad arguments > > the check for get_ether_addr6/3 is expecting that you reach the end of string > when it parse a MAC address but for rte_flow commands it's not the case, > since we provide the full string: > gdb) p s > $3 = 0x28225fa "98:03:9B:5C:D9:00 / end actions queue index 0 / end\n"
Ferruh, it seems this patch must be reverted. We probably need to adapt testpmd to the new function and properly test it. Let's target it for -rc3 or 19.11.
