> -----Original Message----- > From: Venugopal Busireddy [mailto:[email protected]] > Sent: Tuesday, September 20, 2011 9:37 AM > To: [email protected] > Subject: [E1000-devel] [ixgbevf]: patch to fix incorrect PF NACK > assertion... > > diff -r 194e296c8d2e ixgbevf_main.c > --- a/ixgbevf_main.c Thu Sep 15 10:46:23 2011 -0500 > +++ b/ixgbevf_main.c Thu Sep 15 10:47:08 2011 -0500 > @@ -997,15 +997,18 @@ > if (!hw->mbx.ops.check_for_ack(hw, 0)) > got_ack = true; > > - hw->mbx.ops.read(hw, &msg, 1, 0); > - > - if ((msg & IXGBE_MBVFICR_VFREQ_MASK) == IXGBE_PF_CONTROL_MSG) > - mod_timer(&adapter->watchdog_timer, > - round_jiffies(jiffies + 1)); > - > - if (msg & IXGBE_VT_MSGTYPE_NACK) > - DPRINTK(DRV, ERR, "Last Request of type %2.2x to PF > Nacked\n", > - msg & 0xFF); > + if (!hw->mbx.ops.check_for_msg(hw, 0)) { > + hw->mbx.ops.read(hw, &msg, 1, 0); > + > + if ((msg & IXGBE_MBVFICR_VFREQ_MASK) == > IXGBE_PF_CONTROL_MSG) > + mod_timer(&adapter->watchdog_timer, > + round_jiffies(jiffies + 1)); > + > + if (msg & IXGBE_VT_MSGTYPE_NACK) > + DPRINTK(DRV, ERR, > + "Last Request of type %2.2x to PF > Nacked\n", > + msg & 0xFF); > + } > > /* > * checking for the ack clears the PFACK bit. Place
Thanks for submitting the patch. I'm curious what steps you were taking to cause a problem with this code. In our own testing we haven't been seeing an issue with this. If you could provide some information on the steps to reproduce the problem I'd appreciate it. Thanks, - Greg ------------------------------------------------------------------------------ All the data continuously generated in your IT infrastructure contains a definitive record of customers, application performance, security threats, fraudulent activity and more. Splunk takes this data and makes sense of it. Business sense. IT sense. Common sense. http://p.sf.net/sfu/splunk-d2dcopy1 _______________________________________________ E1000-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/e1000-devel To learn more about Intel® Ethernet, visit http://communities.intel.com/community/wired
