> -----Original Message----- > From: David Miller [mailto:[email protected]] > Sent: Saturday, March 25, 2017 8:01 PM > To: Kirsher, Jeffrey T <[email protected]> > Cc: Wyborny, Carolyn <[email protected]>; > [email protected]; [email protected]; [email protected]; > [email protected] > Subject: Re: [net-next 05/11] i40e: fix for queue timing delays > > From: Jeff Kirsher <[email protected]> > Date: Sat, 25 Mar 2017 01:12:59 -0700 > > > From: Carolyn Wyborny <[email protected]> > > > > This patch adds a delay to Rx queue disables to accommodate HW needs. > > [..] > > + /* HW needs up to 50ms to finish RX queue disable*/ > > + mdelay(50); > > You just polled a register which indicates that the request is finished. > Needing futher time for the operation to complete doesn't make any > sense. > > If this is really needed, you need to explain in the commit message, in > detail, what is really the problem here. > > THanks.
It's a hw limitation, will submit an updated patch with more detail. Thanks, Carolyn Carolyn Wyborny Linux Development Networking Division Intel Corporation
