Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752328AbaG1XrN (ORCPT ); Mon, 28 Jul 2014 19:47:13 -0400 Received: from mail.linuxfoundation.org ([140.211.169.12]:49932 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751233AbaG1XrL (ORCPT ); Mon, 28 Jul 2014 19:47:11 -0400 Date: Mon, 28 Jul 2014 16:46:44 -0700 From: Greg KH To: "Luis R. Rodriguez" Cc: Santosh Rastapur , Hariprasad S , Takashi Iwai , "linux-kernel@vger.kernel.org" , Tetsuo Handa , Joseph Salisbury , Kay Sievers , One Thousand Gnomes , Tim Gardner , Pierre Fersing , Andrew Morton , Oleg Nesterov , Benjamin Poirier , Nagalakshmi Nandigama , Praveen Krishnamoorthy , Sreekanth Reddy , Abhijit Mahajan , MPT-FusionLinux.pdl@avagotech.com, Linux SCSI List , "netdev@vger.kernel.org" Subject: Re: [PATCH v2 2/4] driver core: enable drivers to use deferred probe from init Message-ID: <20140728234644.GA18729@kroah.com> References: <1406572110-26823-1-git-send-email-mcgrof@do-not-panic.com> <1406572110-26823-3-git-send-email-mcgrof@do-not-panic.com> <20140728185538.GB17609@kroah.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Jul 28, 2014 at 12:48:32PM -0700, Luis R. Rodriguez wrote: > On Mon, Jul 28, 2014 at 12:04 PM, Luis R. Rodriguez > wrote: > > On Mon, Jul 28, 2014 at 11:55 AM, Greg KH wrote: > >> So, what drivers are having problems in their init sequence, and why > >> aren't they using async firmware loading? > > > > Fixing drivers is one thing, fixing drivers *now* because *now* > > drivers are failing due to a regression is another thing and that's > > what we have now so lets just be clear on that. The 30 second rule is > > a major driver requirement change and it should not be taken slightly, > > all of a sudden this is a new requirement but you won't know that > > unless you're reading these threads or hit an issue. That's an issue > > in itself. That "regression" is something that userspace has decided to do, not anything the kernel changed, so perhaps you should just patch your modprobe and be done with it until you can fix up those drivers? And putting a horrid hack in the driver core, just because of some really bad drivers, is not ok, that's an interface _I_ will have to support for the next few decades. And it's going to take you a while to get something like this ever merged in anyway, odds are you can fix up the driver faster... > > The cxgb4: driver is an example where although I did propose patches > > to use asynch firmware loading the entire registration of the > > netdevice would need to be changed as well in order to get this right. > > In short we have to scramble now to first identify drivers that have > > long init sequences and then fix. There's an assumption that we can > > easily fix drivers, this can take time. This series, although does > > take advantage of a kernel interface for other uses, lets us identify > > these drivers on the kernel ring buffer, so we can go and address > > fixing them with time. > > Another thing that came up during asynch firmware review / integration > on cxgb4 was that it would not be the only thing that would need to be > fixed, the driver also has a ton of ports and at least as per my > review the proper fix would be to use platform regiister stuff. It is > a major rewrite of the driver but an example of a driver that needs > quite a bit of work to meet this new 30 second driver requirement. It shouldn't be using any platform driver stuff, it's a pci device, not a platform device. Why not just put the initial "register the device" in a single-shot workqueue or thread or something like that so that modprobe returns instantly back with a success and all is fine? Again, trying to hack the "deferred init" logic for PCI drivers isn't ok, I'm not going to take that into the driver core if at all possible, sorry. greg k-h -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/