Return-path: Received: from cavan.codon.org.uk ([93.93.128.6]:40525 "EHLO cavan.codon.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752335Ab2ABXAp (ORCPT ); Mon, 2 Jan 2012 18:00:45 -0500 Date: Mon, 2 Jan 2012 23:00:37 +0000 From: Matthew Garrett To: Linus Torvalds Cc: Jack Stone , Alan Stern , Oliver Neukum , Dave Jones , Linux Kernel , Larry Finger , Chaoming Li , "John W. Linville" , Greg Kroah-Hartman , USB list , Linux Wireless List Subject: Re: loading firmware while usermodehelper disabled. Message-ID: <20120102230037.GA16458@srcf.ucam.org> (sfid-20120103_000100_545882_F1F0FCB8) References: <4F02165C.1060400@fastmail.fm> <20120102211904.GA15316@srcf.ucam.org> <20120102215028.GA15701@srcf.ucam.org> <20120102221235.GA16012@srcf.ucam.org> <20120102222920.GA16160@srcf.ucam.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: Sender: linux-wireless-owner@vger.kernel.org List-ID: On Mon, Jan 02, 2012 at 02:46:56PM -0800, Linus Torvalds wrote: > It's *trivial* to attach the firmware driver and load the firmware > even if the firmware isn't needed - because you know it *will* be > needed if somebody suspends. Why not just do that? Why make up these > horrible problems that are totally irrelevant? It means adding complexity to drivers that don't currently care about it, and carrying that cost even for hardware that doesn't need it. It can certainly be made to work, but it's inelegant. We could avoid this specific instance of the problem by just punting responsibility to userland instead. It's clear that we can solve this. All I'm saying is that just making the firmware loader cache things isn't a solution in itself. In this specific case, it means merging the isight_firmware driver into uvcvideo, which is something the uvcvideo maintainer didn't seem keen on a few years ago. -- Matthew Garrett | mjg59@srcf.ucam.org