All of lore.kernel.org
 help / color / mirror / Atom feed
From: Shuah Khan <shuahkhan@gmail.com>
To: Jean-Michel Hautbois <jhautbois@gmail.com>
Cc: Takashi Iwai <tiwai@suse.de>,
	B_B_Singh@dell.com, Greg KH <gregkh@linuxfoundation.org>,
	Arnd Bergmann <arnd@arndb.de>, Kay Sievers <kay@vrfy.org>,
	Ming Lei <ming.lei@canonical.com>, Stefan Roese <sr@denx.de>,
	Tom Gundersen <teg@jklm.no>,
	Stuart_Hayes@dell.com, Srinivas_G_Gowda@dell.com,
	linux-kernel <linux-kernel@vger.kernel.org>
Subject: Re: patch "firmware loader: allow disabling of udev as firmware loader" added to driver-core tree
Date: Wed, 6 Aug 2014 13:00:24 -0600	[thread overview]
Message-ID: <CAKocOOOXAPR4-K2pLDABsC=Gpohct+zZ9p+3FNp4_dUfT-2BkQ@mail.gmail.com> (raw)
In-Reply-To: <CAL8zT=hbwRXRP7yNbrh9BEYzqAyipkTmrFCSqM5Z0U8hr0Yz+g@mail.gmail.com>

On Wed, Aug 6, 2014 at 3:44 AM, Jean-Michel Hautbois
<jhautbois@gmail.com> wrote:
> 2014-08-06 11:24 GMT+02:00 Takashi Iwai <tiwai@suse.de>:
>> At Wed, 6 Aug 2014 11:10:27 +0200,
>> Jean-Michel Hautbois wrote:
>>>
>>> 2014-08-05 21:22 GMT+02:00 Shuah Khan <shuahkhan@gmail.com>:
>>> > On Tue, Aug 5, 2014 at 10:01 AM, Takashi Iwai <tiwai@suse.de> wrote:
>>> >>> Yes, it worked, if the firmware was in the /lib/firmware directory.
>>> >>
>>> >
>>> > ok it works when the firmware is in /lib/firmware. It sounds to me the
>>> > reason load fails when the firmware is under /sys/class/firmware is
>>> > fw_load_from_user_helper() returns -ENOENT when
>>> > CONFIG_FW_LOADER_USER_HELPER is disabled.
>>> >
>>> > It would be nice to see the entire dmesg with debug enabled though.
>>> >
>>> > -- Shuah
>>>
>>> It does not work with the 3.16 kernel even when firmware is in
>>> /lib/firmware, it worked before this patch is applied.
>>
>> Hm?  3.16 doesn't contain my patch yet.  It's merged for 3.17-rc1.
>
> Oh, you are right of course, I am on upstream kernel and I have your
> patch. I don't mean your match is causing the issue though ;-).
>

I think this is what is going on and this patch is the cause:
fw_load_from_user_helper() is a stub that returns -ENOENT with this
patch. As a result, in _request_firmware() after fw_get_filesystem_firmware()
fails to find the file, fw_load_from_user_helper() gets called and it
returns right
away with -ENOENT.

In some cases if rootfs mount is in progress, fw_load_from_user_helper()
steps into load the firmware.

A quick way to test is by adding a dev_info() in the fw_load_from_user_helper()
and see if that is path it is taking.

-- Shuah

  parent reply	other threads:[~2014-08-06 19:00 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <14048583172593@kroah.com>
     [not found] ` <8B2F6FFD0BD1E448853114367400A37306FD6FBBE9@BLRX7MCDC203.AMER.DELL.COM>
     [not found]   ` <20140710143144.GA17639@kroah.com>
     [not found]     ` <8B2F6FFD0BD1E448853114367400A37306FD6FBBF0@BLRX7MCDC203.AMER.DELL.COM>
2014-07-10 14:54       ` patch "firmware loader: allow disabling of udev as firmware loader" added to driver-core tree Greg KH
2014-07-10 15:24         ` B_B_Singh
     [not found]   ` <s5h38e9mg5v.wl%tiwai@suse.de>
     [not found]     ` <8B2F6FFD0BD1E448853114367400A37306FD6FBBF8@BLRX7MCDC203.AMER.DELL.COM>
     [not found]       ` <s5hy4w1l0s3.wl%tiwai@suse.de>
     [not found]         ` <8B2F6FFD0BD1E448853114367400A37306FD6FBC06@BLRX7MCDC203.AMER.DELL.COM>
     [not found]           ` <s5hpphdkyi6.wl%tiwai@suse.de>
     [not found]             ` <8B2F6FFD0BD1E448853114367400A37306FD6FBDAB@BLRX7MCDC203.AMER.DELL.COM>
     [not found]               ` <s5hoawvkimg.wl%tiwai@suse.de>
     [not found]                 ` <8B2F6FFD0BD1E448853114367400A37306FE07C773@BLRX7MCDC203.AMER.DELL.COM>
     [not found]                   ` <s5h7g3frmg7.wl%tiwai@suse.de>
     [not found]                     ` <8B2F6FFD0BD1E448853114367400A37306FE07CC64@BLRX7MCDC203.AMER.DELL.COM>
     [not found]                       ` <s5hlhrrizbq.wl%tiwai@suse.de>
2014-08-01 20:01                         ` B_B_Singh
2014-08-05 15:43                           ` Jean-Michel Hautbois
2014-08-05 15:53                             ` Takashi Iwai
2014-08-05 15:55                               ` Jean-Michel Hautbois
2014-08-05 16:01                                 ` Takashi Iwai
2014-08-05 19:22                                   ` Shuah Khan
2014-08-06  9:10                                     ` Jean-Michel Hautbois
2014-08-06  9:24                                       ` Takashi Iwai
2014-08-06  9:44                                         ` Jean-Michel Hautbois
2014-08-06 10:21                                           ` Takashi Iwai
2014-08-06 10:50                                             ` Jean-Michel Hautbois
2014-08-06 10:52                                               ` Takashi Iwai
2014-08-06 11:24                                                 ` Jean-Michel Hautbois
2014-08-06 12:24                                                   ` Takashi Iwai
2014-08-06 19:00                                           ` Shuah Khan [this message]
2014-08-07  7:57                                             ` Takashi Iwai
2014-08-07 13:23                                               ` Shuah Khan

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CAKocOOOXAPR4-K2pLDABsC=Gpohct+zZ9p+3FNp4_dUfT-2BkQ@mail.gmail.com' \
    --to=shuahkhan@gmail.com \
    --cc=B_B_Singh@dell.com \
    --cc=Srinivas_G_Gowda@dell.com \
    --cc=Stuart_Hayes@dell.com \
    --cc=arnd@arndb.de \
    --cc=gregkh@linuxfoundation.org \
    --cc=jhautbois@gmail.com \
    --cc=kay@vrfy.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=ming.lei@canonical.com \
    --cc=sr@denx.de \
    --cc=teg@jklm.no \
    --cc=tiwai@suse.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.