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. Is there any chance driver select the FDIR config automatically based on rte_flow rule, unless explicitly a FDIR config set by user? [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
