[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 10:54 AM, Trond Myklebust
    <> wrote:
    > Hmm... Looking again at the problem, it appears that the same callback
    > needs to be added to truncate_complete_page() and
    > invalidate_complete_page2(). Otherwise we end up in a situation where
    > the page can sometimes be removed from the page cache without calling
    > freepage().

    Yes, I think any caller of __remove_from_page_cache() should do it
    once it has dropped all locks.

    And to be consistent with that rule, even in the __remove_mapping()
    function I suspect the code to call ->freepage() might as well be done
    only in the __remove_from_page_cache() case (ie not in the
    PageSwapCache() case).

    Then, add the case to the end of "remove_page_cache()" itself, and now
    it's really easy to just grep for __remove_from_page_cache() and make
    sure they all do it.

    That sounds sane, no?


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