Hi Thomas, This is my reply.
> -----Original Message----- > From: Thomas Monjalon [mailto:[email protected]] > Sent: Friday, July 27, 2018 1:42 > To: Xu, Rosen <[email protected]> > Cc: [email protected]; Yigit, Ferruh <[email protected]>; > [email protected]; Gilmore, Walter E <[email protected]>; > Zhang, Qi Z <[email protected]>; [email protected] > Subject: Re: [dpdk-dev] [PATCH v3] examples/flow_filtering: add > rte_fdir_conf initialization > > 22/07/2018 12:39, Rosen Xu: > > Rte_fdir_conf of rte_eth_conf should be initialized before port > > initialization. It is a workaround solution when working with Intel > > I40e. > [...] > > + /* > > + * Initialize fdir_conf of ete_eth_conf > > Typo and lack of punctuation. Fixed. > > + * it is a workaround solution when working with Intel I40e > > + * and it is not the normal way > > It is not said why it is needed, > and what are we waiting to remove the workaround. Added. > > + */ > > + .fdir_conf = { > > + .mode = RTE_FDIR_MODE_PERFECT, > > + .pballoc = RTE_FDIR_PBALLOC_64K, > > + .status = RTE_FDIR_REPORT_STATUS, > > + .drop_queue = 127, > > + }, > > Please work on a v4 with better explanations. >
