From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932310AbbIBUnp (ORCPT ); Wed, 2 Sep 2015 16:43:45 -0400 Received: from mail-ob0-f196.google.com ([209.85.214.196]:34801 "EHLO mail-ob0-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932283AbbIBUnn (ORCPT ); Wed, 2 Sep 2015 16:43:43 -0400 MIME-Version: 1.0 In-Reply-To: <55E6E7FD.8030401@broadcom.com> References: <1440576394.2443.17.camel@loki> <20150829011130.GK8051@wotan.suse.de> <55E1724E.6040303@broadcom.com> <20150829183802.480b1425@tom-T450> <55E2BE0E.1070907@broadcom.com> <20150902011912.GY8051@wotan.suse.de> <55E6E70F.80907@broadcom.com> <55E6E7FD.8030401@broadcom.com> Date: Wed, 2 Sep 2015 13:43:42 -0700 Message-ID: Subject: Re: Problems loading firmware using built-in drivers with kernels that use initramfs. From: Dmitry Torokhov To: Arend van Spriel Cc: "Luis R. Rodriguez" , Ming Lei , Takashi Iwai , Linus Torvalds , Liam Girdwood , "Jie, Yang" , "joonas.lahtinen@linux.intel.com" , Tom Gundersen , Al Viro , Greg Kroah-Hartman , Kay Sievers , David Woodhouse , Luis Rodriguez , lkml , yalin wang Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Sep 2, 2015 at 5:13 AM, Arend van Spriel wrote: > On 09/02/2015 02:09 PM, Arend van Spriel wrote: >> >> On 09/02/2015 03:19 AM, Luis R. Rodriguez wrote: >>> >>> On Mon, Aug 31, 2015 at 10:21:34PM +0800, Ming Lei wrote: >>>> >>>> On Sun, Aug 30, 2015 at 4:25 PM, Arend van Spriel >>>> wrote: >>>>> >>>>> Does this mean a built-in driver can not get firmware from initramfs or >>>>> built in the kernel early. Seems a bit too aggressive. The problem >>>>> stated in >>>>> this thread is when the firmware is not on initramfs but only on the >>>>> rootfs. >>>> >>>> >>>> Yes, strictly speaking, user mode request can't be handled with defer >>>> probe >>>> during booting because we don't know how the user helper handles the >>>> request, >>> >>> >>> FWIW I have a strategy in mind to help us compartamentalize the user mode >>> helper only to the dell-rbu driver, and as such phase out that code >>> eventually >>> completely. Its part of the goals I have with the extensible firmware >>> API I've >>> been proposing. >>> >>>> that said even checking if the firmware exists in current path doesn't >>>> make sense for user mode request. >>>> >>>> So the patch should have used defer proble for direct load only >>>> during booting. >>> >>> >>> What exact guarantees would we be giving to callers if they follow up >>> on probe >>> with -EDEFER_PROBE ? I'd much prefer to try to avoid such uses in init >>> / probe >>> (note that unless you're using async probe since we batch both so it >>> doesn't really >>> matter where you place your code) all together and then for the few >>> remaining >>> stragglers understand the requirements and provide an interface that >>> lets them >>> claim their requirements and try to meets them. >>> >>> A grammatical hunt for drivers who call fw API on init / probe can be >>> completed, although I know the hunt needs a bit more fine tuning it >>> surely can >>> be completed. If we don't have many callers the compexity added for >>> only a >>> few callers with rather loose criteria seems rather unnecessary, >>> specially if >>> we can change the drivers and make these driver sthe exception rather >>> than >>> a norm. >>> >>> Then as for drivers *needing* the fw at probe why not have a proper >>> interface >>> that does guarantee they get the requirements they ask for first ? For >>> instance >>> a new probe type specified by the driver could enable the core to wait >>> for say >>> an event and then tirgger a probe, kind of how we ended up defining >>> the async >>> probe type preference: >>> >>> static struct some_bus_driver some_driver = { >>> .probe = some_probe, >>> .id_table = some_id, >>> .driver = { >>> .name = DEVICE_NAME, >>> .pm = &some_pm_ops, >>> .probe_type = PROBE_PREFER_POST_FOO, >>> }, >>> }; >>> >>> Then we just don't try just hoping for completion but rather can do >>> something >>> about the criteria passed. > > > So should the probe type indicate some event or should it just indicate what > the driver needs, ie. .probe_type = PROBE_TYPE_NEED_FW. What will kernel do when it sees such an option? It does not know when firmware will become available (if ever). We can only try and fail and then userspace may try re-probing devices once firmware is available. I guess the question is how to let userspace know what devices failed to probe because of the lack of firmware. Thanks. -- Dmitry