Re: [GIT *] Allow request_firmware() to be satisfied from in-kernel, use it in more drivers.

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: <david@...>
Cc: Willy Tarreau <w@...>, David Miller <davem@...>, <jeff@...>, <arjan@...>, <akpm@...>, <dwmw2@...>, <alan@...>, <linux-kernel@...>
Date: Tuesday, July 15, 2008 - 12:01 pm

On Tue, 15 Jul 2008, david@lang.hm wrote:

No. We haven't had that possibility, probably largely because we haven't 
even really had the ability (Davem and Jeff talk about "choice" and how 
request_firmware() supposedly removes it, but they ignore the fact that it 
very much cuts the other way too - request_firmware() _adds_ choice and 
makes it easier for people to try to upgrade firmware without upgrading 
the kernel).


I don't think we can - or want to - define things until we've seen the 
problems and advantages in practice over a longer time.

Do we want a versioning scheme? Probably not - exactly because people may 
well want to update (or downgrade) firmware from vendors independently of 
the kernel.

But maybe people will eventually want some kind of blacklisting scheme 
where the driver can say things like "I know I won't work with versions 
that match this checksum" and cause a nice big warning at load-time.

And I think we can all _imagine_ scenarios in all kinds of possible ways 
for things to work better or fail spectaclarly, and even come up with 
anecdotal evidence where scheme X (out of any number of random variations) 
is the "only possible one to work". I'd argue that we simply don't have 
any realistic way to decide which ones make sense.

For example, we can be in the situation where we have some case where
"only this particular firmware is known to work, and others are known to 
break", and people can still take two totally different mental approaches 
to the problem.

One approach is the "strict control one", where people argue that the 
above problem means that we absolutely *must* have very strict versioning, 
and we just always pair things up exactly.

But the other argument is that firmware is part of hardware, and the 
driver should strive to work with different versions of vendor firmware, 
the same way it should strive to work with different versions of chip 
hardware.

Which one is right?  Probably neither. Sometimes one, sometimes the other, 
and in both cases it should probably not be black-and-white.

IOW, I suspect that in the long run we want to say

  "Ok, there's no way we can support _all_ firmware versions, but on the 
   other hand, we also don't want to force particular versions and not let 
   people experiment.

   So drivers should strive to be flexible, and driver writers should not 
   always just say 'there is only one "blessed" firmware', but on the 
   other hand we cannot require drivers to work around horrible known bugs 
   in firmware when a fixed firmware is available"

but the exact details will have to depend on particular drivers.

And we should obviously have _a_ version of the firmware available with 
the kernel when that is possible. But I'd hate for it to be 1:1 with a 
particular driver version - because at that point it smells of being a 
single work, and if it is more than mere aggregation it's no longer viable 
with most of our firmware (I don't think we have source for more than one 
or two cases).

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

Messages in current thread:
Prosaic firmware issues, Alexey Dobriyan, (Tue Jul 15, 9:11 pm)
Re: Prosaic firmware issues, Linus Torvalds, (Tue Jul 15, 9:45 pm)
Re: Prosaic firmware issues, David Woodhouse, (Wed Jul 16, 1:54 am)
Re: Prosaic firmware issues, Alexey Dobriyan, (Wed Jul 16, 10:19 am)
Re: [GIT *] Allow request_firmware() to be satisfied from in..., Henrique de Moraes Holschuh..., (Tue Jul 15, 2:05 pm)
Re: [GIT *] Allow request_firmware() to be satisfied from in..., Arjan van de Ven, (Tue Jul 15, 12:04 pm)
Re: [GIT *] Allow request_firmware() to be satisfied from in..., Linus Torvalds, (Tue Jul 15, 12:01 pm)
Re: [GIT *] Allow request_firmware() to be satisfied from in..., Benjamin Herrenschmidt, (Tue Jul 15, 1:05 am)
Re: [GIT *] Allow request_firmware() to be satisfied from in..., Benjamin Herrenschmidt, (Tue Jul 15, 1:15 am)
Re: [GIT *] Allow request_firmware() to be satisfied from in..., Arjan van de Ven, (Tue Jul 15, 12:56 am)
Re: [GIT *] Allow request_firmware() to be satisfied , Oliver Neukum, (Tue Jul 15, 2:23 am)
Re: [GIT *] Allow request_firmware() to be satisfied , Oliver Neukum, (Tue Jul 15, 12:52 pm)
Re: [GIT *] Allow request_firmware() to be satisfied from in..., Rafael J. Wysocki, (Tue Jul 15, 4:45 pm)
Re: [GIT *] Allow request_firmware() to be satisfied from in..., Rafael J. Wysocki, (Wed Jul 16, 5:28 pm)
Re: [GIT *] Allow request_firmware() to be satisfied from in..., Rafael J. Wysocki, (Thu Jul 17, 4:42 pm)
Re: [GIT *] Allow request_firmware() to be satisfied from in..., Rafael J. Wysocki, (Thu Jul 17, 6:25 pm)
Re: [GIT *] Allow request_firmware() to be satisfied from in..., Rafael J. Wysocki, (Tue Jul 15, 7:45 pm)