Messages in this thread | | | Date | Thu, 21 Jul 2016 09:08:24 +0100 | From | Mel Gorman <> | Subject | Re: [PATCH 5/5] mm: consider per-zone inactive ratio to deactivate |
| |
On Thu, Jul 21, 2016 at 02:30:17PM +0900, Minchan Kim wrote: > > The problem is due to the active deactivation logic in inactive_list_is_low. > > > > Node 0 active_anon:404412kB inactive_anon:409040kB > > > > IOW, (inactive_anon of node * inactive_ratio > active_anon of node) due to > > highmem anonymous stat so VM never deactivates normal zone's anonymous pages. > > > > This patch is a modified version of Minchan's original solution but based > > upon it. The problem with Minchan's patch is that it didn't take memcg > > into account and any low zone with an imbalanced list could force a rotation. > > Could you explan why we should consider memcg here? >
It already was and there is no good reason to ignore it if it's memcg reclaim.
> > In this page, a zone-constrained global reclaim will rotate the list if > > patch, >
I'll fix it.
-- Mel Gorman SUSE Labs
| |