Re: CVS commit: [trunk, wrstuden-revivesa] src/sys

[Andrew Doran <ad%netbsd.org@localhost>]

Hi,

On Sat, May 10, 2008 at 11:49:09PM +0000, Bill Stouder-Studenmund wrote:

> Log Message:
> Initial checkin of re-adding SA. Everything except kern_sa.c
> compiles in GENERIC for i386. This is still a work-in-progress, but
> this checkin covers most of the mechanical work
...
> (changing signalling to be able to accomidate SA's process-wide
> signalling

I discussed this with you in private some time ago: the SA code's approach
to signal handling is completely ham-fisted and reviving it would involve
making invasive code changes in the kernel.

An easier and cleaner way to handle it would be to maintain the signal mask
in each VP's lwpctl block, so that the VP's signal mask can be changed on
user context switch without a syscall. If a pending signal become unmasked
then a dummy call to sigprocmask() would be needed to clear it and trigger
delivery.

> re-adding includes of sys/sa.h and savar.h).

The majority that were removed were there only because of unusual types in
syscallargs.h, that should be fixed properly.

Andrew