Aren/NetBSD
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

If a thread wake up spuriously, re-set l_mutex to make sure it's

Browse Source 
still at the original value and not the schedstate one.  This makes
select not miss wakeups in cases where there was a lot of selecting
going on (which is not all that common in a rump kernel).
This commit is contained in:
pooka 2010-07-23 19:14:14 +00:00
parent 4127064077
commit 8b9126ba6d
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
sys/rump/librump/rumpkern/sleepq.c
View File

@ -1,4 +1,4 @@
/* $NetBSD: sleepq.c,v 1.7 2010/07/22 21:00:07 pooka Exp $ */ /* $NetBSD: sleepq.c,v 1.8 2010/07/23 19:14:14 pooka Exp $ */
/* /*
* Copyright (c) 2008 Antti Kantee. All Rights Reserved. * Copyright (c) 2008 Antti Kantee. All Rights Reserved.
@ -26,7 +26,7 @@
*/ */
#include <sys/cdefs.h> #include <sys/cdefs.h>
__KERNEL_RCSID(0, "$NetBSD: sleepq.c,v 1.7 2010/07/22 21:00:07 pooka Exp $"); __KERNEL_RCSID(0, "$NetBSD: sleepq.c,v 1.8 2010/07/23 19:14:14 pooka Exp $");
#include <sys/param.h> #include <sys/param.h>
#include <sys/condvar.h> #include <sys/condvar.h>
@ -85,6 +85,7 @@ sleepq_block(int timo, bool catch)
int biglocks = l->l_biglocks; int biglocks = l->l_biglocks;
while (l->l_wchan) { while (l->l_wchan) {
l->l_mutex = mp;
if ((error=cv_timedwait(&sq_cv, mp, timo)) == EWOULDBLOCK) { if ((error=cv_timedwait(&sq_cv, mp, timo)) == EWOULDBLOCK) {
TAILQ_REMOVE(l->l_sleepq, l, l_sleepchain); TAILQ_REMOVE(l->l_sleepq, l, l_sleepchain);
l->l_wchan = NULL; l->l_wchan = NULL;