From: Roman Zippel <zippel@linux-m68k.org>
To: Dave Jones <davej@codemonkey.org.uk>
Cc: Bill Davidsen <davidsen@tmr.com>,
Rusty Russell <rusty@rustcorp.com.au>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: the modules problems
Date: Tue, 11 Feb 2003 14:59:33 +0100 (CET) [thread overview]
Message-ID: <Pine.LNX.4.44.0302111053440.32518-100000@serv> (raw)
In-Reply-To: <20030211013521.GA23638@codemonkey.org.uk>
Hi,
On Tue, 11 Feb 2003, Dave Jones wrote:
> > Too radical? After the modules rewrite how could anything short of a
> > rewrite in another language be too radical.
>
> The modules rewrite highlighted a *lot* of bugs, which had nothing
> to do with modules whatsoever. It was unfortunate for Rusty that his
> work got merged the same time as a lot of other changes went in
> which broke a lot of stuff (The cli/sti stuff springs to mind).
> It also highlighted another bunch of bugs which were *real problems*
> like using code marked __init after it was freed.
All this could have been done without a complete rewrite.
> The "This doesn't compile, Rusty sucks" brigade then appeared,
> not taking a second to realise that said driver was fscked regardless
> of Rusty's code being merged.
>
> Sure the modules rewrite wasn't painless, and like everything that
> gets merged, there were bugs, but give the guy a break already.[1]
> If there are still problems with modules, let Rusty know about it.
> If not, please STFU already, its getting tiring hearing the same
> old FUD.
Dave, it's really not that simple. Sure, there were bugs misattributed to
the modules rewrite, but these are usually easy to identify and can be
forwarded to the right culprit.
This leaves us with the real module problems and the biggest is an
unresponsive modules maintainer, when it comes to serious criticism. I'm
not the only one, whose questions are ignored. Rusty somehow assumes
nobody would understand his brilliant ideas (1), so we have to live
"executive summaries" (2). Sorry, but this is fucking arrogant and this
slowly really pisses me off.
The modules rewrite has some serious problems and so far I haven't found
an explaination, how they will be addressed. Feel free to flame me if the
solutions is so obvious that my little mind couldn't grasp it.
The biggest problem which doesn't let me look forward to 2.6 regarding
modules are the user visible changes - the new module tools. Why wasn't it
possible to update modutils? The new modprobe.conf is far less powerful
and I don't really want to know what tricks distributions will come up
with to keep modules.conf and modprobe.conf in sync.
request_module() is still broken, because modprobe/insmod does not
synchronise multiple calls. The module tools are still experimental at
best. I posted a patch to keep both tools working (no reaction of course).
I could go on with various other (serious) design problems, but I
mentioned most of them already in other mails, again without any reaction
from Rusty. I'd really like to know, why I'm not worth an answer anymore.
I'd really like to help, but ignoring me doesn't solve anything, I
wouldn't be that insistent, if I wouldn't see some real problems, I don't
do this just to annoy Rusty. Either these problems are real, then they
should be addressed somehow or I'm wrong, but then I'd like to know why
and I had no problem to admit mistakes. If there are multiple possible
solutions, we should at least consider all possible advantages/
disadvantages.
I'm really annoyed with this situation and if Rusty doesn't want to
cooperate, I slowly think it's the best solution to just revert the whole
mess and just port the few good things. The old module support isn't
perfect, but at least we know its problems and can address them one by
one.
bye, Roman
(1) http://marc.theaimsgroup.com/?l=linux-kernel&m=104440182324752&w=2
(2) http://marc.theaimsgroup.com/?l=linux-kernel&m=104261939523782&w=2
next prev parent reply other threads:[~2003-02-11 13:50 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2003-02-06 15:02 [BK PATCH] LSM changes for 2.5.59 Stephen D. Smalley
2003-02-06 15:18 ` Christoph Hellwig
2003-02-06 17:16 ` David Wagner
2003-02-06 17:45 ` Christoph Hellwig
2003-02-06 17:51 ` Alan Cox
2003-02-08 2:20 ` jmjones
2003-02-09 20:06 ` Christoph Hellwig
2003-02-10 1:39 ` Crispin Cowan
2003-02-10 3:02 ` LA Walsh
2003-02-10 3:40 ` Crispin Cowan
2003-02-10 7:34 ` LA Walsh
2003-02-10 8:11 ` Chris Wright
2003-02-10 8:21 ` 'Christoph Hellwig'
2003-02-10 8:33 ` Crispin Cowan
2003-02-10 8:39 ` 'Christoph Hellwig'
2003-02-10 13:31 ` Alan Cox
2003-02-10 17:29 ` Casey Schaufler
2003-02-12 8:12 ` side issues of baloney with that ham...(was LSM changes for 2.5.59) LA Walsh
2003-02-10 20:51 ` [BK PATCH] LSM changes for 2.5.59 LA Walsh
2003-02-10 21:36 ` David Wagner
2003-02-10 22:14 ` Bill Davidsen
2003-02-11 1:35 ` Dave Jones
2003-02-11 13:59 ` Roman Zippel [this message]
2003-02-11 19:44 ` Bill Davidsen
2003-02-10 4:06 ` J Sloan
2003-02-10 5:59 ` David Wagner
2003-02-10 7:31 ` Christoph Hellwig
2003-02-08 4:13 ` Miles Bader
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=Pine.LNX.4.44.0302111053440.32518-100000@serv \
--to=zippel@linux-m68k.org \
--cc=davej@codemonkey.org.uk \
--cc=davidsen@tmr.com \
--cc=linux-kernel@vger.kernel.org \
--cc=rusty@rustcorp.com.au \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®