Re: [GIT *] Allow request_firmware() to be satisfied from in-kernel, use it in more drivers.
From: Willy Tarreau
Date: Tue Jul 15 2008 - 00:59:43 EST
On Mon, Jul 14, 2008 at 07:51:55PM -0700, Linus Torvalds wrote:
>
>
> On Mon, 14 Jul 2008, David Miller wrote:
> >
> > Jeff's not against request_firmware() in an of itself.
> >
> > He's against the fact that it's not possible to build a
> > self-contained foo.ko module file any longer, and that
> > would definitely be possible with using request_firmware().
>
> .. but is it really that big of a deal?
>
> If it were about not being able to build a self-contained non-modular
> "vmlinux", I'd be upset too.
>
> But once you can load a module, you can load the firmware. You just have
> to _remember_ to move it along with the module.
In order to transfer it, right now you feed it through a working device.
When that device itself requires firmware to work, you will suddenly
discover that it becomes harder and harder to get any communication device
to work on your target system.
Then, in order to load it, you need to have a properly set up system
(hotplug scripts, firmware at the proper location, etc...). This is
really not convenient on embedded systems or live CD systems. In fact,
among all my machines, only my notebook is currently able to load a
firmware.
Also, there's a versionning problem. It's easy to rename a driver
"foo.ko" => "foo-1.ko" and just load the exact file or hold several
versions there by simply moving a symlink. Having to fiddle around
with firmware files depending on the driver version will get much
harder.
Also, when you're cleaning up a system after installation, how do
you know which firmware files you need to keep and which ones you
can safely remove ?
Just like Jeff and Davem, I really want to be able to continue to
use my drivers like I have done since modules were introduced.
Switching a single-file to a multi-file/multi-dir installation would
be a huge maintenance regression IMHO.
Willy
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/