[lkml]   [2010]   [Dec]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH v2 3/3] NFS: Fix a memory leak in nfs_readdir
    On Wed, Dec 1, 2010 at 11:23 AM, Hugh Dickins <> wrote:
    > But I'd prefer us not to throw in another callback if it's well
    > workable with the ->releasepage we already have.

    The thing is, NFS already really uses releasepage for its "real"
    designed usage, in the form of the fscache case (which the directory
    inodes don't use).

    I really hate mixing things up. The NFS directory case really hasn't
    got anything to do with releasepage, and taking the page lock on the
    read side is just really sad. As is marking the page not up-to-date,
    just so that it will get filled again.

    In fact, I don't even know if it's kosher by VFS standards to clear
    the up-to-date bit in the first place after it has already gotten set.
    It absolutely isn't for anything that can be mmap'ed. Obviously, mmap
    doesn't work on the directory cache anyway, but the point is that the
    work-arounds for the semantic gap are really quite ugly.

    Certainly much uglier than just adding some much simpler semantics to
    the "now I'm releasing the page" case in the VM.


     \ /
      Last update: 2010-12-01 20:55    [W:0.021 / U:13.960 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site