lkml.org 
[lkml]   [2002]   [Oct]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH] Module loader preparation
Date
In message <Pine.LNX.4.44.0210171057210.6301-100000@chaos.physics.uiowa.edu> yo
u write:
> 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.

Since there are only a couple, I didn't get exotic.

> 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)

Didn't think of it, to be honest, and I can't find any reference to
link-once glancing through the ld info page.

You'll still have problems with objects linked into two modules
(ip_conntrack_core being the typical one), but we could ban these and
just #include the .c file rather than linking.

Really, the number of modules which do this is so small, the code to
add init function to them is less than the change in the build system
to get tricky.

Rusty.
--
Anyone who quotes me in their sig is an idiot. -- Rusty Russell.
-
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/

\
 
 \ /
  Last update: 2005-03-22 13:30    [W:0.062 / U:12.144 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site