[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <ac3eb2510903101122h2ff348e8waa3aee20867892fb@mail.gmail.com>
Date: Tue, 10 Mar 2009 19:22:16 +0100
From: Kay Sievers <kay.sievers@...y.org>
To: Scott James Remnant <scott@...onical.com>
Cc: Jean Delvare <khali@...ux-fr.org>, linux-kernel@...r.kernel.org,
akpm@...ux-foundation.org
Subject: Re: [PATCH 00/31] Add a lot of module alias statements
On Tue, Mar 10, 2009 at 19:13, Scott James Remnant <scott@...onical.com> wrote:
> On Tue, 2009-03-10 at 18:55 +0100, Jean Delvare wrote:
>> On Tue, 10 Mar 2009 17:49:51 +0000, Scott James Remnant wrote:
>> > On Tue, 2009-03-10 at 18:46 +0100, Jean Delvare wrote:
>> >
>> > > I don't get how it could make any difference in terms of performance.
>> > > As far as I know, all the module aliases that come from the kernel are
>> > > assembled into /lib/modules/$version/modules.alias when the kernel is
>> > > installed, and that file must be processed by modprobe the exact same
>> > > way another configuration file would. Or am I missing something?
>> >
>> > With current modprobe those files are turned into a binary index that
>> > can be read and processed *much* faster.
>>
>> What would prevent the same binary index from being generated from
>> user-provided module aliases?
>>
> Why go to all that effort when adding the alias to the kernel is just a
> one-line change, and then it shows up along with all of the other
> aliases that depmod generates the existing binary index from?
The problem, with a new kernel or module, we know for forever, that we
have to run depmod, but this is not the case for depmod config files,
and not really to manage, to require a binary index update here.
But the main point is that we want to put information where it
belongs: in the module itself. Just look at the crap we ship in
/etc/modprobe* and you know that externally maintained configs for
kernel modules just don't work. :)
Thanks,
Kay
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists