Messages in this thread | | | Date | Thu, 17 Oct 2002 11:02:14 -0500 (CDT) | From | Kai Germaschewski <> | Subject | Re: [PATCH] Module loader preparation |
| |
On Thu, 17 Oct 2002, Rusty Russell wrote:
> This patch fixes a couple of places using the old "I can just call my > function init_module() and it will be called at module init" and a > couple of modules without module_init() declarations. > > These uses are obsolete with the in-kernel module loader, because the > module_init() is where we put the module name in the ".modulename" > section (we could have a "no_init_func()" thing, but it's fairly rare > and hardly intuitive).
Converting things to module_init() is certainly a good thing, but having to provide fake init functions for modules which don't need them doesn't look so nice.
Did you consider just generating the info you need unconditionally in include/linux/module.h and then removing duplicates for multi-part modules using ld's link-once (I didn't try that yet, but it seems doable and would also remove duplicated .modinfo.kernel_version strings and the like)
--Kai
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |