diff options
| author | Niklas Hallqvist <niklas@cvs.openbsd.org> | 2005-05-25 23:17:48 +0000 |
|---|---|---|
| committer | Niklas Hallqvist <niklas@cvs.openbsd.org> | 2005-05-25 23:17:48 +0000 |
| commit | d6ec0bc1862a4fed11c7f4ac537413b2c7e89de4 (patch) | |
| tree | 6c1ed544e4a8e11ea3e107d10a95bad1273dc5a4 /sys/arch/sparc64/include | |
| parent | fee642f79221488ebcacbd0ca219a563c8607281 (diff) | |
This patch is mortly art's work and was done *a year* ago. Art wants to thank
everyone for the prompt review and ok of this work ;-) Yeah, that includes me
too, or maybe especially me. I am sorry.
Change the sched_lock to a mutex. This fixes, among other things, the infamous
"telnet localhost &" problem. The real bug in that case was that the sched_lock
which is by design a non-recursive lock, was recursively acquired, and not
enough releases made us hold the lock in the idle loop, blocking scheduling
on the other processors. Some of the other processors would hold the biglock though,
which made it impossible for cpu 0 to enter the kernel... A nice deadlock.
Let me just say debugging this for days just to realize that it was all fixed
in an old diff noone ever ok'd was somewhat of an anti-climax.
This diff also changes splsched to be correct for all our architectures.
Diffstat (limited to 'sys/arch/sparc64/include')
| -rw-r--r-- | sys/arch/sparc64/include/psl.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sys/arch/sparc64/include/psl.h b/sys/arch/sparc64/include/psl.h index 32a36b71ec2..633a97467a9 100644 --- a/sys/arch/sparc64/include/psl.h +++ b/sys/arch/sparc64/include/psl.h @@ -1,4 +1,4 @@ -/* $OpenBSD: psl.h,v 1.13 2005/04/19 15:29:48 mickey Exp $ */ +/* $OpenBSD: psl.h,v 1.14 2005/05/25 23:17:47 niklas Exp $ */ /* $NetBSD: psl.h,v 1.20 2001/04/13 23:30:05 thorpej Exp $ */ /* @@ -90,7 +90,7 @@ #define PIL_SER 12 #define PIL_STATCLOCK 14 #define PIL_HIGH 15 -#define PIL_SCHED PIL_CLOCK +#define PIL_SCHED PIL_STATCLOCK #define PIL_LOCK PIL_HIGH /* |