diff options
| author | Artur Grabowski <art@cvs.openbsd.org> | 2008-07-18 23:43:32 +0000 |
|---|---|---|
| committer | Artur Grabowski <art@cvs.openbsd.org> | 2008-07-18 23:43:32 +0000 |
| commit | cbf40e0f6a4ee8fde77e05f8b74ca4b989937d38 (patch) | |
| tree | 761b8494073324eacc2b020b8dcffd3c81afb3ee /sys/kern/sched_bsd.c | |
| parent | 7dde4e5a62193339964b0d2205abffd301cc786b (diff) | |
Add a macro that clears the want_resched flag that need_resched sets.
Right now when mi_switch picks up the same proc, we didn't clear the
flag which would mean that every time we service an AST we would attempt
a context switch. For some architectures, amd64 being probably the
most extreme, that meant attempting to context switch for every
trap and interrupt.
Now we clear_resched explicitly after every context switch, even if it
didn't do anything. Which also allows us to remove some more code
in cpu_switchto (not done yet).
miod@ ok
Diffstat (limited to 'sys/kern/sched_bsd.c')
| -rw-r--r-- | sys/kern/sched_bsd.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/sys/kern/sched_bsd.c b/sys/kern/sched_bsd.c index 2bfbadbec94..e7babbb88ed 100644 --- a/sys/kern/sched_bsd.c +++ b/sys/kern/sched_bsd.c @@ -1,4 +1,4 @@ -/* $OpenBSD: sched_bsd.c,v 1.16 2008/05/22 14:07:14 thib Exp $ */ +/* $OpenBSD: sched_bsd.c,v 1.17 2008/07/18 23:43:31 art Exp $ */ /* $NetBSD: kern_synch.c,v 1.37 1996/04/22 01:38:37 christos Exp $ */ /*- @@ -415,6 +415,8 @@ mi_switch(void) p->p_stat = SONPROC; } + clear_resched(curcpu()); + SCHED_ASSERT_LOCKED(); /* |