Messages in this thread | | | Date | Sun, 29 Jun 2008 14:51:08 +0100 (BST) | From | Hugh Dickins <> | Subject | Re: [PATCH] splitlru: memcg swapbacked pages active |
| |
On Sun, 29 Jun 2008, KOSAKI Motohiro wrote: > > Well... > you are right.
Not proved! This is all quite complex. But it looks that way.
> > Hmm.. OK, I propse alternative way. > > step1: commit this patch > step2: implement active/inactive anon balancing routine
That sounds a good plan to me.
> > Reviewed-by: KOSAKI Motohiro <kosaki.motohiro@jp.fujitsu.com> > > Thank you for good patch.
Thank you; and please accept my apology for not Cc'ing you on the patches - I had intended to, but forgot just when sending.
> > btw, furtunately, memcg reclaim has some retry. > thus, lru imbalancing doesn't cause OOM, it only cause small performance degression. > IMHO your patch doesn't have any risk.
These things can work out so differently in practice than one would expect. Tests seem to be chugging along okay with the change in, but I haven't explicitly tested performance with and without.
It just seems a sensible starting point to have the global and memcg views in synch: if departing from that proves to work better, then we should do so later.
I've no view yet on the performance characteristics of any of this, just focussed on getting tmpfs working correctly again. The notion of keeping unevictable pages away from the regularly scanned lists seems an obvious gain; but the swapbacked/filebacked distinction is not so obvious to me (and particularly problematic for tmpfs).
Hugh
| |