[dpdk-dev] [PATCH v2] net/i40e: fix incorrect hash look up table

Wang, ShougangX shougangx.wang at intel.com
Wed Jul 22 08:20:41 CEST 2020


Hi, Qi

> -----Original Message-----
> From: Zhang, Qi Z <qi.z.zhang at intel.com>
> Sent: Wednesday, July 22, 2020 1:31 PM
> To: Wang, ShougangX <shougangx.wang at intel.com>; dev at dpdk.org
> Cc: Xing, Beilei <beilei.xing at intel.com>; Guo, Jia <jia.guo at intel.com>; Wang,
> ShougangX <shougangx.wang at intel.com>; stable at dpdk.org
> Subject: RE: [dpdk-dev] [PATCH v2] net/i40e: fix incorrect hash look up table
> 
> 
> 
> > -----Original Message-----
> > From: dev <dev-bounces at dpdk.org> On Behalf Of Shougang Wang
> > Sent: Tuesday, July 21, 2020 1:49 PM
> > To: dev at dpdk.org
> > Cc: Xing, Beilei <beilei.xing at intel.com>; Guo, Jia
> > <jia.guo at intel.com>; Wang, ShougangX <shougangx.wang at intel.com>;
> > stable at dpdk.org
> > Subject: [dpdk-dev] [PATCH v2] net/i40e: fix incorrect hash look up
> > table
> >
> > The hash look up table(LUT) will not be initializing when starting
> > testpmd with --disable-rss. So that some invalid queue indexes may
> > still in the LUT. When enable RSS by creating RSS rule, some packets will not
> be into the valid queues.
> > This patch fixes this issue by initializing the LUT when creating an RSS rule.
> 
> Could you explain why you only initialize the LUT when creating an RSS rule
> but not at dev_init or dev_start?
It is good to initialize the LUT at dev_start, I will fix it.

> What if user configure LUT table before create a RSS rule? Does that mean
> the LUT table will be flushed?
Yes.

Thanks.
Shougang


More information about the dev mailing list