Re: [PATCH] splitlru: memcg swapbacked pages active

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Hugh Dickins
Date: Sunday, June 29, 2008 - 5:20 am

On Sun, 29 Jun 2008, KOSAKI Motohiro wrote:

Hmm.  Thanks for looking into it (and the others).  This is something
I noticed as a discrepancy in the source, rather than something from
which I observed any OOMs.  So if you're already working on bringing
the global and memcg approaches into line, I defer to your judgement.

But it seems to me worrying and fragile while they diverge in this way.
If the global has anon, active anon, file, active file and unevictable
lrus for the page; and the memcg has anon, active anon, file, active file
and unevictable lrus for the page cgroup; but different choices are made
which to put on where, then we're heading for confusion and trouble.

And when called, __mem_cgroup_move_lists tries to keep the activ-ity of
the memcg lrus in step with the activ-ity of the global lrus, doesn't
it?  So it's beyond my comprehension to start them off out of step.

Hugh
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH] splitlru: memcg swapbacked pages active, Hugh Dickins, (Sat Jun 28, 5:22 pm)
[PATCH] splitlru: BDI_CAP_SWAP_BACKED, Hugh Dickins, (Sat Jun 28, 5:24 pm)
Re: [PATCH] splitlru: shmem_getpage SetPageSwapBacked sooner, KOSAKI Motohiro, (Sun Jun 29, 1:09 am)
Re: [PATCH] splitlru: memcg swapbacked pages active, KOSAKI Motohiro, (Sun Jun 29, 1:19 am)
Re: [PATCH] splitlru: BDI_CAP_SWAP_BACKED, KOSAKI Motohiro, (Sun Jun 29, 1:48 am)
Re: [PATCH] splitlru: memcg swapbacked pages active, Hugh Dickins, (Sun Jun 29, 5:20 am)
Re: [PATCH] splitlru: memcg swapbacked pages active, KOSAKI Motohiro, (Sun Jun 29, 5:58 am)
Re: [PATCH] splitlru: memcg swapbacked pages active, Hugh Dickins, (Sun Jun 29, 6:51 am)
Re: [PATCH] splitlru: BDI_CAP_SWAP_BACKED, Rik van Riel, (Sun Jun 29, 8:16 am)
Re: [PATCH] splitlru: memcg swapbacked pages active, KAMEZAWA Hiroyuki, (Sun Jun 29, 7:44 pm)
Re: [PATCH] splitlru: BDI_CAP_SWAP_BACKED, KAMEZAWA Hiroyuki, (Sun Jun 29, 7:53 pm)
[RFC][PATCH] memcg: change shmem handler., KAMEZAWA Hiroyuki, (Mon Jun 30, 1:36 am)
Re: [PATCH] splitlru: memcg swapbacked pages active, Hugh Dickins, (Mon Jun 30, 12:19 pm)
Re: [PATCH] splitlru: BDI_CAP_SWAP_BACKED, Hugh Dickins, (Mon Jun 30, 12:23 pm)
Re: [PATCH] splitlru: BDI_CAP_SWAP_BACKED, Rik van Riel, (Mon Jun 30, 12:49 pm)
Re: [RFC][PATCH] memcg: change shmem handler., Hugh Dickins, (Mon Jun 30, 12:52 pm)
Re: [PATCH] splitlru: BDI_CAP_SWAP_BACKED, Hugh Dickins, (Mon Jun 30, 1:55 pm)
Re: [PATCH] splitlru: BDI_CAP_SWAP_BACKED, Rik van Riel, (Mon Jun 30, 2:17 pm)
Re: Re: [RFC][PATCH] memcg: change shmem handler., kamezawa.hiroyu, (Mon Jun 30, 5:56 pm)
Re: Re: Re: [RFC][PATCH] memcg: change shmem handler., kamezawa.hiroyu, (Mon Jun 30, 6:06 pm)
[RFC][PATCH] memcg: shmem swap cache, KAMEZAWA Hiroyuki, (Tue Jul 1, 7:16 pm)
Re: [RFC][PATCH] memcg: change shmem handler., Balbir Singh, (Thu Jul 3, 12:15 am)
Re: [RFC][PATCH] memcg: change shmem handler., KAMEZAWA Hiroyuki, (Thu Jul 3, 12:56 am)