As a minor optimization, do suspended checking more like it was origi…

…nally in the PR - before the while loop.
keulraesik · Sep 17, 2000 · 46ce32e · 46ce32e
1 parent 683279a
commit 46ce32e
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/sys/dev/fxp/if_fxp.c b/sys/dev/fxp/if_fxp.c
@@ -1218,7 +1218,12 @@ fxp_intr(arg)
 
 	FXP_LOCK(sc, s);
 
-	while (!sc->suspended && (statack = CSR_READ_1(sc, FXP_CSR_SCB_STATACK)) != 0) {
+	if (sc->suspended) {
+		FXP_UNLOCK(sc, s);
+		return;
+	}
+
+	while ((statack = CSR_READ_1(sc, FXP_CSR_SCB_STATACK)) != 0) {
 #if defined(__NetBSD__)
 		claimed = 1;
 #endif

diff --git a/sys/pci/if_fxp.c b/sys/pci/if_fxp.c
@@ -1218,7 +1218,12 @@ fxp_intr(arg)
 
 	FXP_LOCK(sc, s);
 
-	while (!sc->suspended && (statack = CSR_READ_1(sc, FXP_CSR_SCB_STATACK)) != 0) {
+	if (sc->suspended) {
+		FXP_UNLOCK(sc, s);
+		return;
+	}
+
+	while ((statack = CSR_READ_1(sc, FXP_CSR_SCB_STATACK)) != 0) {
 #if defined(__NetBSD__)
 		claimed = 1;
 #endif