[lkml]   [2010]   [May]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH] tmpfs: Insert tmpfs cache pages to inactive list at first
    On Fri, 21 May 2010, KOSAKI Motohiro wrote:
    > > Acked-by: Hugh Dickins <>
    > >
    > > Thanks - though I don't quite agree with your description: I can't
    > > see why the lru_cache_add_active_anon() was ever justified - that
    > > "active" came in along with the separate anon and file LRU lists.
    > If you have any worry, can you please share it? I'll test such workload
    > and fix the issue if necessary. You are expert than me in this area.

    ?? I've acked the patch: my worry is only with the detail of your
    comments on the history - in my view it was always wrong to put on
    the active LRU there, and I'm glad that you have now fixed it.

    If you really want to test some workload on 2.6.28 to see if it too
    works better with your fix, I won't stop you - but I'd much prefer
    you to be applying your mind to 2.6.35 and 2.6.36!


     \ /
      Last update: 2010-05-21 04:13    [W:0.021 / U:63.336 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site