[lkml]   [2012]   [Sep]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH] userns: Add basic quota support v4

    Dave thank you earlier taking some time to do review. It made me
    realize that my code was not as mature as it needed to be.

    That said you missed a lot of important details, and I will aim at
    to address some of the highlights.

    Precedent in naming. <uidgid.h> that has been merged since 3.5, earlier
    there is make_pte. Much of your review sounds like you are unhappy
    with the general infrastructure that was built for user namespaces,
    and already merged. Short of an overwhelmingly better suggestion
    at this point I am not going to revisit naming.

    Passing structures by value is a core part of the design and a necessity
    for strong type safety in C, and again already in common use in the

    Using BUG on code paths that can not be triggered, simply means that
    in that rare and unlikely event there is a clear trace to follow,
    and xfs already has plenty of invocations of BUG, some of which
    I managed to trigger with the xfs test suite. So I can not find
    much sense in your arguments about being clear that something is a BUG.

    Your comments on where xfs uses projid are just laughable.

    >> So I am inclined to rename this field projid, but I don't know if it is
    >> worth respinning the patch for something so trivial.
    > I'd like to think you are joking, given that the patch series is all
    > about using consistent, verifiable identifiers... :/

    The patch series is all about implementing the user namespace.

    To make it hard to miss places where conversions need to between the
    internal kernel form and the form stored in filesystems or the form that
    userspace uses I use types that are not assignment compatible. Pushing
    the types deeper in the code keeps me honest and let's me find cases
    I was not expecting like the ioctl interface of xfs that largely
    duplicates much of the vfs functionality.


     \ /
      Last update: 2012-09-20 04:01    [W:0.028 / U:14.848 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site