Re: CVS commit: src/sys/opencrypto

[Taylor R Campbell <campbell+netbsd-source-changes-d%mumble.net@localhost>]

   Date: Fri, 24 Jan 2014 17:35:41 +0000 (UTC)
   From: christos%astron.com@localhost (Christos Zoulas)

   In article <7458.1390534472%splode.eterna.com.au@localhost>,
   matthew green  <mrg%eterna.com.au@localhost> wrote:
   >
   >> Log Message:
   >> XXX Also note that there is still a small window where the ref-count can
   >> XXX be decremented, and then the process/thread preempted.  If auto-unload
   >> XXX happens before that thread can return from the module's code, bad
   >> XXX things (tm) could happen.
   >
   >in this case, please simply disallow unload for this module always.
   >if the race is fixed, it can be enabled again.

   I think that most module unloads suffer from this race, any ideas how to
   fix it?

Shouldn't devsw_detach or config_fini_component handle this?  Why does
the crypto device need special reference counting that other devices
don't?