On Tue, May 29, 2012 at 03:00:58PM -0700, Paul Goyette wrote: > Well, at least for sysctl's SYSCTL_SETUP() stuff, you probably don't > want to use the same initialization call for modules as is used for > built-ins. The built-ins are initialized with an explicit NULL argument > passed for the sysctl_clog argument, which makes it difficult for a > module to do its clean-up. Modular code needs to (or at least, should?) > pass a non-null module-specific clog so it can be used during an "undo" > at MODULE_CMD_UNLOAD time.
Indeed I consider it a bug if a module uses SYSCTL_SETUP() (and does not tear down the nodes after unload). This applies more generally to drivers too. - Jukka.
