Re: [PATCH] Add module load profile hook

From: Rusty Russell (rusty@rustcorp.com.au)
Date: Tue Feb 25 2003 - 03:07:41 EST


In message <20030225025852.GB49589@compsoc.man.ac.uk> you write:
> On Tue, Feb 25, 2003 at 12:25:23PM +1100, Rusty Russell wrote:
> > That would be because that was a HACK, and it's my job to say "no",
> > even when that means we're not "feature complete" by someone's
> > definition.
>
> You've yet to explain why it's a hack as opposed to a reasonable level
> of discoverability. This includes your comments on IRC where you agreed
> I had a point.

You're still mistaking politeness for agreement. Of course you have a
point: there *is* benefit in being able to tell where modules are
without changing any code, otherwise you wouldn't be asking for it.

But it's not going to happen.

It's the bit where you add it a "store this filename" and "get the
filename" kernel which makes no sense whatsoever: the kernel has no
need for the information, why should it hold it?

Making modprobe store this somewhere kind of makes sense, but since
the algorithm that modprobe uses to map names to filenames is trivial,
I'm not convinced that the complexity is sensible (unless you want to
handle special cases like module renaming with -o).

Making insmod store this information, since insmod is supposed to be
the dumb workhorse util (ie. "use modprobe") doesn't make as much
sense. But this is exactly the tool that kernel hackers are likely to
use when they want fine control over their own modules (ie. likely to
be used with oprofile).

The way that gdb solves this is to have a path directive, where you
can say "look here for source". Or you could send me a patch for
modprobe to put the information somewhere sensible if you prefer that.
What makes most sense to you?

> > You seem to have taken the politeness of my previous response as an
> > indication of uncertainty.
>
> I took your point and agreed to translate into whatever you like.

And you took a pot-shot at me for being inconsistent: did you expect
me not to clarify?

> I'm not a kernel hacker and I don't particularly give a shit ...

Huh? You sent a patch. If you don't care, noone will.

Confused,
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/



This archive was generated by hypermail 2b29 : Fri Feb 28 2003 - 22:00:27 EST