Do not call mii_pollstat() from within device tick routines; the status

information is updated by mii_tick().

Pointed out by: wpaul (a while back)

3 files changed, 11 insertions, 17 deletions

diff --git a/sys/dev/dc/if_dc.c b/sys/dev/dc/if_dc.c
index 8cf4276..0d22729 100644
--- a/sys/dev/dc/if_dc.c
+++ b/sys/dev/dc/if_dc.c
@@ -2708,14 +2708,11 @@ static void dc_tick(xsc)
 	 * that time, packets will stay in the send queue, and once the
 	 * link comes up, they will be flushed out to the wire.
 	 */
-	if (!sc->dc_link) {
-		mii_pollstat(mii);
-		if (mii->mii_media_status & IFM_ACTIVE &&
-		    IFM_SUBTYPE(mii->mii_media_active) != IFM_NONE) {
-			sc->dc_link++;
-			if (ifp->if_snd.ifq_head != NULL)
-				dc_start(ifp);
-		}
+	if (!sc->dc_link && mii->mii_media_status & IFM_ACTIVE &&
+	    IFM_SUBTYPE(mii->mii_media_active) != IFM_NONE) {
+		sc->dc_link++;
+		if (ifp->if_snd.ifq_head != NULL)
+			dc_start(ifp);
 	}
 
 	if (sc->dc_flags & DC_21143_NWAY && !sc->dc_link)
diff --git a/sys/dev/sf/if_sf.c b/sys/dev/sf/if_sf.c
index c0c05d5..487b48f 100644
--- a/sys/dev/sf/if_sf.c
+++ b/sys/dev/sf/if_sf.c
@@ -1489,13 +1489,12 @@ static void sf_stats_update(xsc)
 	    stats.sf_tx_multi_colls + stats.sf_tx_excess_colls;
 
 	mii_tick(mii);
-	if (!sc->sf_link) {
-		mii_pollstat(mii);
-		if (mii->mii_media_status & IFM_ACTIVE &&
-		    IFM_SUBTYPE(mii->mii_media_active) != IFM_NONE)
-			sc->sf_link++;
-			if (ifp->if_snd.ifq_head != NULL)
-				sf_start(ifp);
+
+	if (!sc->sf_link && mii->mii_media_status & IFM_ACTIVE &&
+	    IFM_SUBTYPE(mii->mii_media_active) != IFM_NONE) {
+		sc->sf_link++;
+		if (ifp->if_snd.ifq_head != NULL)
+			sf_start(ifp);
 	}
 
 	sc->sf_stat_ch = timeout(sf_stats_update, sc, hz);
diff --git a/sys/dev/sk/if_sk.c b/sys/dev/sk/if_sk.c
index 9c842d5..8604209 100644
--- a/sys/dev/sk/if_sk.c
+++ b/sys/dev/sk/if_sk.c
@@ -1691,7 +1691,6 @@ static void sk_tick(xsc_if)
 	SK_XM_CLRBIT_2(sc_if, XM_IMR, XM_IMR_GP0_SET);
 	SK_XM_READ_2(sc_if, XM_ISR);
 	mii_tick(mii);
-	mii_pollstat(mii);
 	untimeout(sk_tick, sc_if, sc_if->sk_tick_ch);
 
 	SK_IF_UNLOCK(sc_if);
@@ -1744,7 +1743,6 @@ static void sk_intr_bcom(sc_if)
 			SK_IF_WRITE_1(sc_if, 0, SK_LINKLED1_CTL,
 			    SK_LINKLED_ON|SK_LINKLED_LINKSYNC_OFF|
 			    SK_LINKLED_BLINK_OFF);
-			mii_pollstat(mii);
 		} else {
 			mii_tick(mii);
 			sc_if->sk_tick_ch = timeout(sk_tick, sc_if, hz);