Hi, PSB
> -----Original Message----- > From: Xu, Rosen [mailto:rosen...@intel.com] > Sent: Thursday, July 12, 2018 8:27 AM > To: Ori Kam <or...@mellanox.com>; dev@dpdk.org > Cc: Yigit, Ferruh <ferruh.yi...@intel.com>; sta...@dpdk.org > Subject: RE: [dpdk-dev] [PATCH] examples/flow_filtering: add rte_fdir_conf > initialization > > Hi Ori, > > examples/flow_filtering sample app fails on i40e [1] because i40e requires > explicit FDIR configuration. > > But rte_flow in and hardware independent ways of describing flow-action, it > shouldn't require specific config options for specific hardware. > I don't understand why using rte flow require the use of fdir. it doesn't make sense to me, that new API will need old one. > Is there any chance driver select the FDIR config automatically based on > rte_flow rule, unless explicitly a FDIR config set by user? I don't know how the i40e driver is implemented but I know that Mellanox convert the other way around, if fdir is given it is converted to rte_flow. > > [1] > Flow can't be created 1 message: Check the mode in fdir_conf. > EAL: Error - exiting with code: 1 > > > -----Original Message----- > > From: Ori Kam [mailto:or...@mellanox.com] > > Sent: Thursday, July 12, 2018 13:17 > > To: Xu, Rosen <rosen...@intel.com>; dev@dpdk.org > > Cc: Yigit, Ferruh <ferruh.yi...@intel.com>; sta...@dpdk.org; Ori Kam > > <or...@mellanox.com> > > Subject: RE: [dpdk-dev] [PATCH] examples/flow_filtering: add > rte_fdir_conf > > initialization > > > > Hi Rosen, > > > > Why do the fdir_conf must be initialized? > > > > What is the issue you are seeing? > > > > Best, > > Ori > > > > > -----Original Message----- > > > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Rosen Xu > > > Sent: Thursday, July 12, 2018 5:10 AM > > > To: dev@dpdk.org > > > Cc: rosen...@intel.com; ferruh.yi...@intel.com; Ori Kam > > > <or...@mellanox.com>; sta...@dpdk.org > > > Subject: [dpdk-dev] [PATCH] examples/flow_filtering: add rte_fdir_conf > > > initialization > > > > > > Rte_fdir_conf of rte_eth_conf should be initialized before port > > > initialization. > > > > > > Fixes: 4a3ef59a10c8 ("examples/flow_filtering: add simple demo of flow > > > API") > > > Cc: sta...@dpdk.org > > > > > > Signed-off-by: Rosen Xu <rosen...@intel.com> > > > --- > > > examples/flow_filtering/main.c | 6 ++++++ > > > 1 file changed, 6 insertions(+) > > > > > > diff --git a/examples/flow_filtering/main.c > > > b/examples/flow_filtering/main.c index f595034..aa03e23 100644 > > > --- a/examples/flow_filtering/main.c > > > +++ b/examples/flow_filtering/main.c > > > @@ -132,6 +132,12 @@ > > > DEV_TX_OFFLOAD_SCTP_CKSUM | > > > DEV_TX_OFFLOAD_TCP_TSO, > > > }, > > > + .fdir_conf = { > > > + .mode = RTE_FDIR_MODE_PERFECT, > > > + .pballoc = RTE_FDIR_PBALLOC_64K, > > > + .status = RTE_FDIR_REPORT_STATUS, > > > + .drop_queue = 127, > > > + }, > > > }; > > > struct rte_eth_txconf txq_conf; > > > struct rte_eth_rxconf rxq_conf; > > > -- > > > 1.8.3.1
