On 4/5/2020 4:12 PM, Ori Kam wrote: > Hi Xiao, > > > >> -----Original Message----- >> From: Xiao Zhang <xiao.zh...@intel.com> >> Sent: Tuesday, March 31, 2020 4:30 PM >> To: dev@dpdk.org >> Cc: Ori Kam <or...@mellanox.com>; qi.z.zh...@intel.com; >> wei.zh...@intel.com; Xiao Zhang <xiao.zh...@intel.com>; sta...@dpdk.org >> Subject: [v2] app/testpmd: fix PPPOES flow API >> >> The command line to create RTE flow for specific proto_id of PPPOES can >> not work. >> >> It was: >> testpmd> flow create 0 ingress pattern proto_id >> proto_id [TOKEN]: match PPPoE session protocol identifier >> testpmd> flow create 0 ingress pattern proto_id proto_id >> proto_id [TOKEN]: match PPPoE session protocol identifier >> testpmd> flow create 0 ingress pattern proto_id proto_id proto_id >> proto_id [TOKEN]: match PPPoE session protocol identifier >> >> The proto_id can not be set with previous implementation. >> >> This patch is to fix this issue, and change the command line to: >> testpmd> flow create 0 pattern pppoe_proto_id is xxxx >> >> Fixes: 226c6e60c35b ("ethdev: add PPPoE to flow API") >> Cc: sta...@dpdk.org >> >> Signed-off-by: Xiao Zhang <xiao.zh...@intel.com> >> --- >> v2 change the command line according to review comments. >> --- > > Acked-by: Ori Kam <or...@mellanox.com>
Applied to dpdk-next-net/master, thanks.