Hi Thomas, > -----Original Message----- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Tuesday, April 5, 2016 4:48 PM > To: Iremonger, Bernard <bernard.iremonger at intel.com> > Cc: dev at dpdk.org; Lu, Wenzhuo <wenzhuo.lu at intel.com> > Subject: Re: [dpdk-dev] [PATCH v2] ixgbe: fix occasional timeouts when > starting VF > > 2016-04-05 15:55, Bernard Iremonger: > > - poll_ms = 10; > > + poll_ms = RTE_IXGBE_REGISTER_POLL_WAIT_15_MS; > [...] > > #define RTE_IXGBE_REGISTER_POLL_WAIT_10_MS 10 > > +#define RTE_IXGBE_REGISTER_POLL_WAIT_15_MS 15 > > #define RTE_IXGBE_WAIT_100_US 100 > > I don't understand why these constants are needed. > There is no semantic, just an arbitrary number.
I am seeing intermittent failures in the following test at line 4969 in ixgbe_rxtx.c: while (--poll_ms && !(txdctl & IXGBE_TXDCTL_ENABLE)) Increasing the value of poll_ms does not address the root cause of this failure. This needs more investigation. Self NAK.