Re: [PATCH] Module loader preparation

From: Kai Germaschewski (kai@tp1.ruhr-uni-bochum.de)
Date: Thu Oct 17 2002 - 11:02:14 EST


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/



This archive was generated by hypermail 2b29 : Wed Oct 23 2002 - 22:00:35 EST