On 7/23/2019 7:39 AM, Raslan Darawsheh wrote: > Tested-by: Raslan Darawsheh <rasl...@mellanox.com> > Seems OK for me, > > Kindest regards, > Raslan Darawsheh > >> -----Original Message----- >> From: Stephen Hemminger <step...@networkplumber.org> >> Sent: Monday, July 22, 2019 8:20 PM >> To: Ferruh Yigit <ferruh.yi...@intel.com> >> Cc: Adrien Mazarguil <adrien.mazarg...@6wind.com>; Wenzhuo Lu >> <wenzhuo...@intel.com>; Jingjing Wu <jingjing...@intel.com>; Bernard >> Iremonger <bernard.iremon...@intel.com>; dev@dpdk.org; Raslan >> Darawsheh <rasl...@mellanox.com> >> Subject: Re: [dpdk-dev] [PATCH] app/testpmd: fix MAC addr parser for flow >> >> On Mon, 22 Jul 2019 18:02:09 +0100 >> Ferruh Yigit <ferruh.yi...@intel.com> wrote: >> >>> On 7/22/2019 5:58 PM, Ferruh Yigit wrote: >>>> MAC address parsing was causing failure [1], this patch partially >>>> reverts the commit commit b5ddce8959b2 ("app/testpmd: use new >>>> ethernet address parser") >>>> >>>> [1] >>>> testpmd> flow validate 0 priority 2 ingress group 0 pattern eth >>>> testpmd> dst >>>> is 98:03:9B:5C:D9:00 / end actions queue index 0 / end Bad >>>> arguments >>>> >>>> Fixes: b5ddce8959b2 ("app/testpmd: use new ethernet address parser") >>>> >>>> Reported-by: Raslan Darawsheh <rasl...@mellanox.com> >>>> Signed-off-by: Ferruh Yigit <ferruh.yi...@intel.com> >>>> --- >>>> Cc: Stephen Hemminger <step...@networkplumber.org> >>> >>> If there is no objection, or more proper fix, till tomorrow morning >>> (europe time), I am planning to merge this one, fyi. >> >> Works for me, testpmd always has the cmdline library anyway.
Applied to dpdk-next-net/master, thanks.