summaryrefslogtreecommitdiffstats
path: root/sys/dev/en
diff options
context:
space:
mode:
authorscottl <scottl@FreeBSD.org>2004-02-29 09:26:01 +0000
committerscottl <scottl@FreeBSD.org>2004-02-29 09:26:01 +0000
commit28681e97b01c7595536de0338a8e8f65827477ae (patch)
treefd6e8d3b7da7419d53c635ba6dacaf83fe4c2400 /sys/dev/en
parent6becbe0dc56258a4e9f2945f223bec6080741c35 (diff)
downloadFreeBSD-src-28681e97b01c7595536de0338a8e8f65827477ae.zip
FreeBSD-src-28681e97b01c7595536de0338a8e8f65827477ae.tar.gz
All three of these drivers abused cv_waitq_empty in the same way by spinning
on it in hopes of making sure that the waitq was empty before going on. This wasn't needed and probably never would have worked as intended. Now that cv_waitq_empty() and friends are gone, the code in these drivers that spins on it can go away too. This should unbreak LINT. Discussed with: kan
Diffstat (limited to 'sys/dev/en')
-rw-r--r--sys/dev/en/midway.c5
1 files changed, 1 insertions, 4 deletions
diff --git a/sys/dev/en/midway.c b/sys/dev/en/midway.c
index 494a61e..3aa0016 100644
--- a/sys/dev/en/midway.c
+++ b/sys/dev/en/midway.c
@@ -1415,10 +1415,7 @@ en_reset_ul(struct en_softc *sc)
/*
* Unstop all waiters
*/
- while (!cv_waitq_empty(&sc->cv_close)) {
- cv_broadcast(&sc->cv_close);
- DELAY(100);
- }
+ cv_broadcast(&sc->cv_close);
}
/*
OpenPOWER on IntegriCloud