lkml.org 
[lkml]   [2010]   [Jun]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    SubjectRe: [PATCH 3/3] Add a pair of system calls to make extended file stats available [ver #2]
    Date
    Arnd Bergmann <arnd@arndb.de> wrote:

    > It also makes things like strace more complicated.

    That's the most compelling argument.

    > No, I think that would be worse than the current version. But if you remove
    > the structure version in favor of the flags, you only need six arguments
    > anyway.

    I want to keep the structure version, just in case we need to expand fields in
    the stat struct in future. Otherwise we may need to create yet another stat
    syscall.

    > You can also go further and fold the structure length into flags, because
    > the length is just a function of the data you are passing.

    The potential problem with passing the flags as a syscall argument is that
    we're then limited to a single 32-bit integer. It might be enough, but if I
    do as at least one person has suggested and assign each field in the struct
    its own bit, that uses up half right there, plus I'd like to add at least one
    operational flag (to force synchronisation with the server).

    > Having a system call with flags, size and version is like wearing a belt,
    > braces and suspenders. An unsigned long flags argument should be enough to
    > hold up your pants[1].

    I would like the size argument for two reasons: firstly, to prevent buffer
    overruns and, secondly, because I can see some scope for variable-size fields
    (such as for volume IDs or security labels), though the latter might be better
    handled through getxattr() (which would mean extra overhead).

    David


    \
     
     \ /
      Last update: 2010-06-30 12:03    [W:5.653 / U:2.140 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site