Module Name:    src
Committed By:   thorpej
Date:           Thu Mar 19 14:10:56 UTC 2020

Modified Files:
        src/sys/dev/ic: i82586.c

Log Message:
Don't bother with IFF_OACTIVE.  Just keep processing so long as
sc->xmit_busy is less than NTXBUF.


To generate a diff of this commit:
cvs rdiff -u -r1.88 -r1.89 src/sys/dev/ic/i82586.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/sys/dev/ic/i82586.c
diff -u src/sys/dev/ic/i82586.c:1.88 src/sys/dev/ic/i82586.c:1.89
--- src/sys/dev/ic/i82586.c:1.88	Wed Jan 29 14:49:44 2020
+++ src/sys/dev/ic/i82586.c	Thu Mar 19 14:10:56 2020
@@ -1,4 +1,4 @@
-/*	$NetBSD: i82586.c,v 1.88 2020/01/29 14:49:44 thorpej Exp $	*/
+/*	$NetBSD: i82586.c,v 1.89 2020/03/19 14:10:56 thorpej Exp $	*/
 
 /*-
  * Copyright (c) 1998 The NetBSD Foundation, Inc.
@@ -137,7 +137,7 @@ Mode of operation:
 */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: i82586.c,v 1.88 2020/01/29 14:49:44 thorpej Exp $");
+__KERNEL_RCSID(0, "$NetBSD: i82586.c,v 1.89 2020/03/19 14:10:56 thorpej Exp $");
 
 
 #include <sys/param.h>
@@ -663,7 +663,6 @@ i82586_tint(struct ie_softc *sc, int scb
 	int status;
 
 	ifp->if_timer = 0;
-	ifp->if_flags &= ~IFF_OACTIVE;
 
 #if I82586_DEBUG
 	if (sc->xmit_busy <= 0) {
@@ -1129,15 +1128,10 @@ i82586_start(struct ifnet *ifp)
 	u_short	len;
 	int	s;
 
-	if ((ifp->if_flags & (IFF_RUNNING | IFF_OACTIVE)) != IFF_RUNNING)
+	if ((ifp->if_flags & IFF_RUNNING) != IFF_RUNNING)
 		return;
 
-	for (;;) {
-		if (sc->xmit_busy == NTXBUF) {
-			ifp->if_flags |= IFF_OACTIVE;
-			break;
-		}
-
+	while (sc->xmit_busy < NTXBUF) {
 		head = sc->xchead;
 		xbase = sc->xbds;
 
@@ -1244,9 +1238,7 @@ i82586_reset(struct ie_softc *sc, int ha
 	if (hard)
 		printf("%s: reset\n", device_xname(sc->sc_dev));
 
-	/* Clear OACTIVE in case we're called from watchdog (frozen xmit). */
 	sc->sc_ethercom.ec_if.if_timer = 0;
-	sc->sc_ethercom.ec_if.if_flags &= ~IFF_OACTIVE;
 
 	/*
 	 * Stop i82586 dead in its tracks.
@@ -1657,7 +1649,6 @@ i82586_init(struct ifnet *ifp)
 		(sc->hwinit)(sc);
 
 	ifp->if_flags |= IFF_RUNNING;
-	ifp->if_flags &= ~IFF_OACTIVE;
 
 	if (NTXBUF < 2)
 		sc->do_xmitnopchain = 0;

Reply via email to