lkml.org 
[lkml]   [2003]   [May]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: request_firmware() hotplug interface, third round.
    On Fri, May 16, 2003 at 05:03:38PM -0700, Greg Kroah-Hartman wrote:
    > On Sat, May 17, 2003 at 01:55:15AM +0200, Oliver Neukum wrote:
    > >
    > > > > - echo 1 > /sysfs/class/firmware/dev_name/loading
    > > > > - cat whatever_fw > /sysfs/class/firmware/dev_name/data
    > > > > - echo 0 > /sysfs/class/firmware/dev_name/loading
    > > >
    > > > Nice, but can't you get rid of the loading file by just relying on
    > > > open() and close()? Oh wait, sysfs doesn't pass that down to you, hm,
    > > > looks like you need that info. But does the new binary interface in
    > > > sysfs that just got merged into the tree provide that info for you?
    > >
    > > But what if the close() is due to irregular termination?
    > > If the script is killed, do you download half a firmware?
    >
    > Good point. Actually I don't think that the binary interface for sysfs
    > passes open and close down to the lower levels, so it's a moot point.
    >
    > echo... works for me.

    I think we'd be better off checking for this case by requiring the
    firmware to include a length field and adapting the binary interface
    so that we can see the open/close boundary. The "loading" thing is
    pretty damn ugly.

    Plus with the "loading" file, a interrupted load will just appear to
    be unterminated - and if the scripts run again they'll have to check
    that the loading file was 0 to start with - and if it isn't there's
    nothing they can do except wait for the dead load to time out.

    Better to catch the close, check the length, then return the firmware
    or throw the junk image away as appropriate.

    --
    David Gibson | For every complex problem there is a
    david@gibson.dropbear.id.au | solution which is simple, neat and
    | wrong.
    http://www.ozlabs.org/people/dgibson
    -
    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/

    \
     
     \ /
      Last update: 2005-03-22 13:35    [W:3.244 / U:0.312 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site