cvs commit: src/sys/dev/streams streams.c src/sys/kernkern_descrip.c kern_event.c sys_pipe.c uipc_syscalls.cvfs_syscalls.c src/sys/opencrypto cryptodev.c

[David Malone]

> > This reminds me that we still hold Giant around pipe(2) because it isn't
> > declared MPSAFE in the syscall table.  Is this still necessary?

> I've been suspicious of this too, and I was hoping that you would have
> an answer.  Can we go ahead and correct this?

I think we may need to check the calling of pipeclose() in pipe(),
but as this is only done in an error case, it is probably safe enough
to just grab Giant for that. (The mac_* calls may need to be checked
too).

> > Additionally, we declare dup2(2) to be MPSAFE, but not dup(2).  Given
> > their implementations that seems odd.

> We likely need to do a review through the syscalls and weed out problems
> like this.  Any volunteers?

I'm pretty sure dup not being marked MPSAFE is just a typo, based on the
CVS logs. I'll change it later today.

	David.