lkml.org 
[lkml]   [2016]   [Sep]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    Date
    SubjectRe: [RFC] fs: add userspace critical mounts event support
    On Tue, Sep 6, 2016 at 2:11 PM, Bjorn Andersson
    <bjorn.andersson@linaro.org> wrote:
    > On Tue 06 Sep 11:32 PDT 2016, Linus Torvalds wrote:
    >
    >> On Tue, Sep 6, 2016 at 10:46 AM, Bjorn Andersson
    >> Nobody has actually answered the "why don't we just tie the firmware
    >> and module together" question.
    >
    > The answer to this depends on the details of the suggestion; but
    > generally there's a much stronger bond between the kernel and the driver
    > than between the driver and the firmware in my cases.

    I call BS.

    Let me be very clear. I'm not applying that shit-for-brains stupid
    patch, and will not be pulling it unless somebody tricks me into it.

    Because all these arguments make no sense at all.

    If the driver doesn't work without the firmware, then anybody who
    distributes the driver binary without a firmware is just
    *fundamentally* doing something insane. You may do it for
    *development* purposes, but doing so for actual *use* would be
    entirely pointless.

    See my point? If a distribution is distributing the driver without the
    firmware, then what the hell is the point of such a thing?

    But even if you decide to do that for some odd reason, the patch is
    still just stupid. Instead of adding some crazy infrastructure for
    "now I've mounted everything", you could equally well just

    (a) make the driver fail the module load if it cannot find a firmware binary

    (b) after user space has mounted everything, just do "insmod -a"
    again (or insmod just that driver).

    See? The point is, this "generic" hacky interface is just stupid. It's
    not adding any value. If you add user space "I'm ready now" points
    anyway, you might as well make those points do the right thing and
    just load the module that is now loadable.

    We could mark such "late loading" modules explicitly if people want
    to, so that you can automate the whole thing about delaying the
    loading in user space.

    At no point does it make sense to say "I have now mounted all the
    important filesystems". Maybe the firmware is extracted later by user
    space downloading it from the internet, and the module will then work
    only after that point"./

    This whole "I have mounted important filesystems" is just pure and
    utter garbage. Stop pushing this shit.

    Linus

    \
     
     \ /
      Last update: 2016-09-17 09:58    [W:4.264 / U:0.012 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site