Re: [PATCH] firmware: Allow release-specific firmware dir

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: David Woodhouse
Date: Wednesday, September 10, 2008 - 5:39 pm

On Thu, 2008-09-11 at 10:23 +1000, Herbert Xu wrote:

Absolutely -- nobody should ever find themselves in that situation.
If a firmware changes in an incompatible way, its filename needs to
change too. The old one should still be available.


Even safer would be to have content-addressed firmware. Instead of
requesting it by filename, you request it by its md5sum. That way,
you're _guaranteed_ to have precisely what you expected.

But I think that's a bad idea too. As a general rule firmware isn't, and
shouldn't be, tied intimately to one particular version of the kernel.
It isn't even tied intimately to one particular version of the _driver_.


That's what we are doing already, as you say -- and what we should
always be doing.

And remember, ipw2200 isn't a particularly good example, because it's a
relatively recent driver and thus has always had its firmware shipped
_separately_ from the kernel.

All we're talking about here is the behaviour for the handful of older
drivers which I've recently dragged into this century by converting them
to use request_firmware(). Those drivers, by virtue of their age, are
mostly quite unlikely to receive _any_ kind of firmware update --
especially an update which changes their ABI.

-- 
David Woodhouse                            Open Source Technology Centre
David.Woodhouse@intel.com                              Intel Corporation

--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH] firmware: Allow release-specific firmware dir, Jeff Mahoney, (Tue Sep 9, 7:15 am)
Re: [PATCH] firmware: Allow release-specific firmware dir, Andrew Morton, (Wed Sep 10, 3:35 pm)
Re: [PATCH] firmware: Allow release-specific firmware dir, David Woodhouse, (Wed Sep 10, 4:01 pm)
Re: [PATCH] firmware: Allow release-specific firmware dir, David Woodhouse, (Wed Sep 10, 4:15 pm)
Re: [PATCH] firmware: Allow release-specific firmware dir, Andrew Morton, (Wed Sep 10, 4:24 pm)
Re: [PATCH] firmware: Allow release-specific firmware dir, David Woodhouse, (Wed Sep 10, 4:36 pm)
Re: [PATCH] firmware: Allow release-specific firmware dir, David Woodhouse, (Wed Sep 10, 5:39 pm)
Re: [PATCH] firmware: Allow release-specific firmware dir, Faidon Liambotis, (Wed Sep 10, 11:35 pm)
Re: [PATCH] firmware: Allow release-specific firmware dir, Jeff Mahoney, (Thu Sep 11, 12:15 am)
Re: [PATCH] firmware: Allow release-specific firmware dir, Marcel Holtmann, (Thu Sep 11, 12:44 am)
Re: [PATCH] firmware: Allow release-specific firmware dir, Faidon Liambotis, (Thu Sep 11, 1:29 am)
Re: [PATCH] firmware: Allow release-specific firmware dir, Thierry Vignaud, (Thu Sep 11, 4:29 am)
Re: [PATCH] firmware: Allow release-specific firmware dir, David Woodhouse, (Thu Sep 11, 7:50 am)
Re: [PATCH] firmware: Allow release-specific firmware dir, David Woodhouse, (Thu Sep 11, 8:31 am)
Re: [PATCH] firmware: Allow release-specific firmware dir, Linus Torvalds, (Thu Sep 11, 8:57 am)
Re: [PATCH] firmware: Allow release-specific firmware dir, David Woodhouse, (Thu Sep 11, 9:01 am)
Re: [PATCH] firmware: Allow release-specific firmware dir, Marcel Holtmann, (Thu Sep 11, 9:09 am)
Re: [PATCH] firmware: Allow release-specific firmware dir, Marcel Holtmann, (Thu Sep 11, 9:12 am)
Re: [PATCH] firmware: Allow release-specific firmware dir, Marcel Holtmann, (Thu Sep 11, 9:16 am)
Re: [PATCH] firmware: Allow release-specific firmware dir, Marcel Holtmann, (Thu Sep 11, 9:20 am)
Re: [PATCH] firmware: Allow release-specific firmware dir, Marcel Holtmann, (Thu Sep 11, 9:29 am)
Re: [PATCH] firmware: Allow release-specific firmware dir, Marcel Holtmann, (Thu Sep 11, 9:39 am)
Re: [PATCH] firmware: Allow release-specific firmware dir, David Woodhouse, (Thu Sep 11, 9:45 am)
Re: [PATCH] firmware: Allow release-specific firmware dir, Linus Torvalds, (Thu Sep 11, 10:49 am)
Re: [PATCH] firmware: Allow release-specific firmware dir, David Woodhouse, (Thu Sep 11, 1:38 pm)
Re: [PATCH] firmware: Allow release-specific firmware dir, David Woodhouse, (Thu Sep 11, 2:15 pm)
Re: [PATCH] firmware: Allow release-specific firmware dir, David Woodhouse, (Thu Sep 11, 3:25 pm)
Re: [PATCH] firmware: Allow release-specific firmware dir, Joseph Fannin, (Fri Sep 12, 1:39 am)
Re: [PATCH] firmware: Allow release-specific firmware dir, David Woodhouse, (Fri Sep 12, 7:32 am)
Re: [PATCH] firmware: Allow release-specific firmware dir, Kai Henningsen, (Fri Sep 12, 1:24 pm)