Search Linux Wireless

Re: loading firmware while usermodehelper disabled.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Jan 02, 2012 at 02:19:37PM -0800, Linus Torvalds wrote:
> On Mon, Jan 2, 2012 at 2:12 PM, Matthew Garrett <mjg@xxxxxxxxxx> wrote:
> >
> > The kernel that loaded the firmware to make the device work is not
> > required to be the same kernel that we're running when we suspend.
> 
> Why do you make up all these idiotic theoretical cases that nobody
> cares about and has no relevance what-so-ever for the 99%?

Rebooting is a theoretical case?

> And if you *can* load the firmware in the new kernel, just do it
> before the suspend event. Problem solved.

How do we know to do that? The device appears to be a UVC device at 
suspend. It's only at resume that we discover that it's actually a 
programmable device.

> And once more: your made-up scenario has *nothing* what-so-ever to do
> with the actual warnings that started this whole thread.

The warnings that started the thread are associated with a driver that I 
wrote. I'm explaining to you that your suggestion doesn't work for 
a real-world scenario involving that driver. If the firmware has been 
loaded and then the user reboots, the new kernel has no indication that 
it needs to load the firmware. There are various ways of handling this, 
but it's simply untrue that we'll always load the firmware at kernel 
init time.

-- 
Matthew Garrett | mjg59@xxxxxxxxxxxxx
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux