From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from he.sipsolutions.net ([78.46.109.217]:46810 "EHLO sipsolutions.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751276Ab2APMQw (ORCPT ); Mon, 16 Jan 2012 07:16:52 -0500 Subject: Re: calling request_firmware() from module init will not work with recent/future udev versions From: Johannes Berg To: Kay Sievers Cc: netdev@vger.kernel.org, linux-wireless@vger.kernel.org, Tom Gundersen , Andy Whitcroft In-Reply-To: (sfid-20120116_130612_776092_70A618B1) References: <1326621743.3448.1.camel@jlt3.sipsolutions.net> <1326704259.3510.3.camel@jlt3.sipsolutions.net> (sfid-20120116_130612_776092_70A618B1) Content-Type: text/plain; charset="UTF-8" Date: Mon, 16 Jan 2012 13:16:49 +0100 Message-ID: <1326716209.3510.7.camel@jlt3.sipsolutions.net> (sfid-20120116_131700_910031_326BA2C4) Mime-Version: 1.0 Sender: linux-wireless-owner@vger.kernel.org List-ID: On Mon, 2012-01-16 at 13:05 +0100, Kay Sievers wrote: > > What I'm was asking then is this: Can udev know that it is running from > > initramfs (presumably that can't be too hard) and simply not reply to > > async requests it doesn't have firmware for? Then once the real root is > > mounted it could satisfy (or not) firmware requests from the real root. > > We can surely change it to not cancel the firmware request. > > Either by making it aware that we run from initramfs, or by never > cancelling any firmware request and just leave it hanging around for > forever? I think not cancelling it from initramfs and then providing or cancelling it once we have normal root mounted should be sufficient? I don't see how letting it hang around forever (a minute until it times out) would be useful. > We need to decide what's the best model here, if we want a timeout at > all, if yes, how large it should be, and if and when we should cancel > requests. I do think cancelling requests still gives us desirable behaviour in terms of being able to reproduce it etc., it's just that cancelling it when we have only partial data (from initramfs) isn't helpful. johannes