lkml.org 
[lkml]   [2012]   [Dec]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH, 3.7-rc7, RESEND] fs: revert commit bbdd6808 to fallocate UAPI
    On Wed, Dec 05, 2012 at 05:18:43PM +0100, Martin Steigerwald wrote:
    > Dave provided technical reasons.
    >
    > First in the patch description and then in:
    >
    > https://lkml.org/lkml/2012/11/26/700

    There were no technical reasons. We are only reserving a bit. And
    different file systems don't support all of the various different
    fallocate flags already --- for example, not all file systems support
    the punch system call.

    Yes, I could create an entrely new ioctl() that looks just like
    fallocate, but supports the extra bit, just so that Dave and others
    don't have to be offended about the existence of that extra bit ---
    but Linus (and others) have considered ioctl()'s evil, since there is
    no type checking, and it's just silly to create a separate interface
    just because somebody doesn't think some other file system shouldn't
    implement a particular feature --- especially since it's not like
    we're have any kind of shortage of bits in the fallocate field.

    Heck, I probably have more to complain about with the inode flags
    field, which were originally created specifically for ext2/3/4, and
    which has since been grabbed for use by other file systems, including
    btrfs. You haven't heard me kvetching because btrfs has grabbed
    btrfs-specific inode flags for nocow and notail... no one even bother
    to try to get it past the fs-devel shed painting crew before *those*
    bits were allocated --- and I am absolutely fine with that.

    - Ted


    \
     
     \ /
      Last update: 2012-12-05 18:22    [W:2.899 / U:0.032 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site