cvs commit: src/sys/dev/ubsec ubsec.c ubsecvar.h

[Nate Lawson]

On Mon, 2 Jun 2003, Sam Leffler wrote:
>   Redo locking for proper SMP operation:
>
>   o replace driver-global lock with three locks: one for the handling of mcr1
>     operations, one for handling of mcr2 operations, and one for the mcr1
>     free list
>   o mark the interrupt handler MPSAFE
>   o don't use locking on detach; disabling interrupts is sufficient (I think)

Unfortunately it's not if you're on a shared interrupt.  You need to
call bus_teardown_intr() before you are detached from the ithread.  See
the thread near this message:
  Message-ID: <1742240000.1051807110 at aslan.btc.adaptec.com>

The end result, I thought, was that jhb@ was going to look into
bus_teardown_intr() to change it to install a null handler with proper
locking (intr masking?).  I hate to start this topic again but I've
brought it up a couple times and still haven't heard that this has been
addressed.

-Nate