Actually, to be a little clearer, it seems like if there is diagnostic messaging that should happen here, perhaps the individual crypto module is *not* the right place. A better place to message things would be in the crypto core itself, when the failure is first detected.
- Garrett Garrett D'Amore wrote: > Anthony Scarpino wrote: >> Got a few bugs up for review.. should be pretty straight forward and >> repetitious.. >> >> 6849769 des _init: crypto_register_provider() failed >> 6885135 assertion failed: kcf_dh != NULL >> 6907099 module load/unload errors could be a little less techie and >> ominous >> >> http://cr.opensolaris.org/~izick/bugs/ >> >> thanks >> >> Tony >> _______________________________________________ >> crypto-discuss mailing list >> crypto-discuss at opensolaris.org >> http://mail.opensolaris.org/mailman/listinfo/crypto-discuss > > You're using a private variable, sys_shutdown, for the sole purpose of > detecting that the system is shutting down to suppress message. This > seems, to me at least, to be rather strange. Is there any real harm in > emitting the message... does it show up in real life? > > - Garrett > >
