Re: CVS commit: src/sys/opencrypto

[christos%zoulas.com@localhost (Christos Zoulas)]

On Jan 19, 11:21am, paul%whooppee.com@localhost (Paul Goyette) wrote:
-- Subject: Re: CVS commit: src/sys/opencrypto

| > | Handled indirectly.  The MODULE_CMD_FINI calls config_cfdata_detach()
| > | which attempts to detach each device instance.  If a detach fails, then
| > | config_cfdata_detach fails, and the unload will fail.
| >
| > Ok then! Did you test it? If it works, I guess commit it.
| 
| It does address the simple case I described.  It has a minor/cosmetic 
| issue of printing an error message
| 
|       crypto0: unable to detach instance
| 
| from config_cfdata_detach().

yes, it looks like it is not designed to be called if it is not going
to work. This is why most other things do the test in the MODULE_FINI
part, see bpf for example.

| But David Laight's post has me more concerned, that perhaps we really 
| need to solve this sort of issue more generically.

Yes, this is why I took the EOPNOTSUPP shortcut.

| So, not sure if this should be committed, or if we should leave your 
| code in to prevent unload in all cases.

Well, you can move your test from detach to MODULE_FINI and it should
work just fine. But yes, we should solve it more generically, but I still
think the reference counting code is needed.

christos