Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 11 Sep 2009 00:00:23 +0000 (UTC)
From:      Jack F Vogel <jfv@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r197084 - head/sys/dev/ixgbe
Message-ID:  <200909110000.n8B00N5C035175@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: jfv
Date: Fri Sep 11 00:00:23 2009
New Revision: 197084
URL: http://svn.freebsd.org/changeset/base/197084

Log:
  Stats missed packet handling was still not quite
  right, thanks to Dmitrij Tejblum for the correction,
  need a variable with scope only within the for loop
  for all queues.
  
  MFC: 3 days

Modified:
  head/sys/dev/ixgbe/ixgbe.c

Modified: head/sys/dev/ixgbe/ixgbe.c
==============================================================================
--- head/sys/dev/ixgbe/ixgbe.c	Thu Sep 10 23:30:13 2009	(r197083)
+++ head/sys/dev/ixgbe/ixgbe.c	Fri Sep 11 00:00:23 2009	(r197084)
@@ -4480,9 +4480,11 @@ ixgbe_update_stats_counters(struct adapt
 	adapter->stats.crcerrs += IXGBE_READ_REG(hw, IXGBE_CRCERRS);
 
 	for (int i = 0; i < 8; i++) {
+		u32 mp;
+		mp = IXGBE_READ_REG(hw, IXGBE_MPC(i));
 		/* missed_rx tallies misses for the gprc workaround */
-		missed_rx += IXGBE_READ_REG(hw, IXGBE_MPC(i));
-        	adapter->stats.mpc[i] += missed_rx;
+		missed_rx += mp;
+        	adapter->stats.mpc[i] += mp;
 		/* Running comprehensive total for stats display */
 		total_missed_rx += adapter->stats.mpc[i];
 		if (hw->mac.type == ixgbe_mac_82598EB)



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200909110000.n8B00N5C035175>