lkml.org 
[lkml]   [2015]   [May]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [RESEND][PATCH] Bluetooth: Make request workqueue freezable
    On 05/20/2015 05:44 AM, Takashi Iwai wrote:
    > At Wed, 20 May 2015 11:46:31 +0200,
    > Marcel Holtmann wrote:
    >>
    >> Hi Oliver,
    >>
    >>>> The data is cached in RAM. More specifically, the former loaded
    >>>> firmware files are reloaded and saved at suspend for each device
    >>>> object. See fw_pm_notify() in firmware_class.c.
    >>>
    >>> OK, this may be a stupid idea, but do we know the firmware
    >>> was successfully loaded in the first place?
    >>> Also btusb is in the habit of falling back to a generic
    >>> firmware in some places. It seems to me that caching
    >>> firmware is conceptually not enough, but we'd also need
    >>> to record the absence of firmware images.
    >>
    >> in a lot of cases the firmware is optional. The device will operate fine without the firmware. There are a few devices where the firmware is required, but for many it just contains patches.
    >>
    >> It would be nice if we could tell request_firmware() if it is optional or mandatory firmware. Or if it should just cache the status of a missing firmware as well.
    >
    > OK, below is a quick hack to record the failed f/w files, too.
    > Not sure whether this helps, though. Proper tests are appreciated.
    >
    >

    This doesn't quite work. We end up with the name on fw_names but
    the firmware isn't actually on the firmware cache list.

    If request_firmware fails to get the firmware from the filesystem,
    release firmware will be called which is going to free the
    firmware_buf which has been marked as failed anyway. The only
    way to make this work would be to always piggy back and increase
    the ref so it always stays around. But this also marks the firmware
    as a permanent failure. There would need to be a hook somewhere
    to force a cache drop, else there would be no way to add new
    firmware to a running system without a reboot.

    Perhaps we split the difference: keep a list of firmware images
    that failed to load in the past and if one is requested during
    a time when usermodehelper isn't available, silently return an
    error? This way, if correct firmware is loaded at a regular time
    the item can be removed from the list.

    Thanks,
    Laura


    \
     
     \ /
      Last update: 2015-05-21 02:01    [W:2.414 / U:0.052 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site