Subject: Re: AD's CVS commit: src/sys/arch (spl.S)
To: Geoff Wing <mason@primenet.com.au>
From: Andrew Doran <ad@netbsd.org>
List: current-users
Date: 12/04/2007 08:35:38
On Tue, Dec 04, 2007 at 08:14:18AM +0000, Geoff Wing wrote:

> : Modified Files:
> : 	src/sys/arch/amd64/amd64: spl.S
> : 	src/sys/arch/i386/i386: spl.S
> : Log Message:
> : softintr_ret: fix it to work properly after yesterday's change to mi_switch.
> : cvs rdiff -r1.13 -r1.14 src/sys/arch/amd64/amd64/spl.S
> : cvs rdiff -r1.21 -r1.22 src/sys/arch/i386/i386/spl.S
> 
> Does this fix the problem I had, with these warnings in my dmesg (with
> overlaps) and a crash into ddb in syslogd (even with ddb.onpanic=0)?
> System is uniprocessor.
..
> WARNING: SPL NOT LOWERED ON SYSCALL EXIT
> WARNING:PL NOT LOWERED ON SYSCALL EXIT
> WARNING: SPL NOT LOWERED ON SYSCALL EXIT
> WARNING: SPL NOT LOWERED ON SYSCALL EXIT
> WARNING: SPL NOSPL NOT LOWERED ON SYSCALL EXIT
> WARNING: SPL NOT LOWERED ON SYSCALL EXIT
> WARNING: SPL NOT LOWERED ON SYSCALL EXIT
> WARNING: SPL N31:17771

Yes, it does fix that! So if you were running i386 or amd64 and got the soft
interrupt changes, please update again. I have been running the code for 6
months now I don't believe that there are any other serious problems.

Thanks,
Andrew