lkml.org 
[lkml]   [2012]   [Dec]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    Date
    SubjectRe: [PATCH, 3.7-rc7, RESEND] fs: revert commit bbdd6808 to fallocate UAPI
    On Wed, Dec 5, 2012 at 5:14 PM, Dave Chinner <david@fromorbit.com> wrote:
    >
    > And for changes to syscalls? That's something that must be peer
    > reviewed because we are going to be stuck with those changes forever
    > as we can't undo them at a later date. It doesn't matter who made the
    > change in question, I would have done exactly the same thing....

    The thing that people are complaining about is exactly the reverse of
    this. It's *protecting* us from making mistakes, and doesn't actually
    add any new interfaces in itself.

    This is why I'm so annoyed with this stupid thread. It's been going on
    forever, and reverting that change WOULD BE OBJECTIVELY A BAD IDEA.

    The change is a one-liner, doesn't add any code at all, seeks to
    protect us from potential future mistakes, and had a good technical
    rationale for it. Seriously. Why are we even discussing this two weeks
    later?

    Don't even bother to answer that question. This thread isn't worth it.

    Linus


    \
     
     \ /
      Last update: 2012-12-06 04:41    [W:6.087 / U:0.016 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site