Re: [patch 1/8] mm: memcg: only evict file pages when we have plenty

2012-12-19 Thread Mel Gorman
On Wed, Dec 19, 2012 at 12:21:55AM -0500, Simon Jeons wrote: > On Mon, 2012-12-17 at 16:54 +0100, Michal Hocko wrote: > > On Sun 16-12-12 09:21:54, Simon Jeons wrote: > > > On 12/13/2012 10:55 PM, Michal Hocko wrote: > > > >On Wed 12-12-12 17:28:44, Johannes Weiner wrote: > > > >>On Wed, Dec 12,

Re: [patch 1/8] mm: memcg: only evict file pages when we have plenty

2012-12-19 Thread Mel Gorman
On Wed, Dec 19, 2012 at 12:21:55AM -0500, Simon Jeons wrote: On Mon, 2012-12-17 at 16:54 +0100, Michal Hocko wrote: On Sun 16-12-12 09:21:54, Simon Jeons wrote: On 12/13/2012 10:55 PM, Michal Hocko wrote: On Wed 12-12-12 17:28:44, Johannes Weiner wrote: On Wed, Dec 12, 2012 at

Re: [patch 1/8] mm: memcg: only evict file pages when we have plenty

2012-12-18 Thread Simon Jeons
On Mon, 2012-12-17 at 16:54 +0100, Michal Hocko wrote: > On Sun 16-12-12 09:21:54, Simon Jeons wrote: > > On 12/13/2012 10:55 PM, Michal Hocko wrote: > > >On Wed 12-12-12 17:28:44, Johannes Weiner wrote: > > >>On Wed, Dec 12, 2012 at 04:53:36PM -0500, Rik van Riel wrote: > > >>>On 12/12/2012 04:43

Re: [patch 1/8] mm: memcg: only evict file pages when we have plenty

2012-12-18 Thread Simon Jeons
On Mon, 2012-12-17 at 16:54 +0100, Michal Hocko wrote: On Sun 16-12-12 09:21:54, Simon Jeons wrote: On 12/13/2012 10:55 PM, Michal Hocko wrote: On Wed 12-12-12 17:28:44, Johannes Weiner wrote: On Wed, Dec 12, 2012 at 04:53:36PM -0500, Rik van Riel wrote: On 12/12/2012 04:43 PM, Johannes

Re: [patch 1/8] mm: memcg: only evict file pages when we have plenty

2012-12-17 Thread Michal Hocko
On Sun 16-12-12 09:21:54, Simon Jeons wrote: > On 12/13/2012 10:55 PM, Michal Hocko wrote: > >On Wed 12-12-12 17:28:44, Johannes Weiner wrote: > >>On Wed, Dec 12, 2012 at 04:53:36PM -0500, Rik van Riel wrote: > >>>On 12/12/2012 04:43 PM, Johannes Weiner wrote: > dc0422c "mm: vmscan: only evict

Re: [patch 1/8] mm: memcg: only evict file pages when we have plenty

2012-12-17 Thread Michal Hocko
On Sun 16-12-12 09:21:54, Simon Jeons wrote: On 12/13/2012 10:55 PM, Michal Hocko wrote: On Wed 12-12-12 17:28:44, Johannes Weiner wrote: On Wed, Dec 12, 2012 at 04:53:36PM -0500, Rik van Riel wrote: On 12/12/2012 04:43 PM, Johannes Weiner wrote: dc0422c mm: vmscan: only evict file pages when

Re: [patch 1/8] mm: memcg: only evict file pages when we have plenty

2012-12-15 Thread Simon Jeons
On 12/13/2012 10:55 PM, Michal Hocko wrote: On Wed 12-12-12 17:28:44, Johannes Weiner wrote: On Wed, Dec 12, 2012 at 04:53:36PM -0500, Rik van Riel wrote: On 12/12/2012 04:43 PM, Johannes Weiner wrote: dc0422c "mm: vmscan: only evict file pages when we have plenty" makes a point of not going

Re: [patch 1/8] mm: memcg: only evict file pages when we have plenty

2012-12-15 Thread Simon Jeons
On 12/13/2012 10:55 PM, Michal Hocko wrote: On Wed 12-12-12 17:28:44, Johannes Weiner wrote: On Wed, Dec 12, 2012 at 04:53:36PM -0500, Rik van Riel wrote: On 12/12/2012 04:43 PM, Johannes Weiner wrote: dc0422c mm: vmscan: only evict file pages when we have plenty makes a point of not going

Re: [patch 1/8] mm: memcg: only evict file pages when we have plenty

2012-12-13 Thread Michal Hocko
On Wed 12-12-12 17:28:44, Johannes Weiner wrote: > On Wed, Dec 12, 2012 at 04:53:36PM -0500, Rik van Riel wrote: > > On 12/12/2012 04:43 PM, Johannes Weiner wrote: > > >dc0422c "mm: vmscan: only evict file pages when we have plenty" makes > > >a point of not going for anonymous memory while there

Re: [patch 1/8] mm: memcg: only evict file pages when we have plenty

2012-12-13 Thread Mel Gorman
On Thu, Dec 13, 2012 at 10:07:04AM +, Mel Gorman wrote: > On Wed, Dec 12, 2012 at 05:28:44PM -0500, Johannes Weiner wrote: > > On Wed, Dec 12, 2012 at 04:53:36PM -0500, Rik van Riel wrote: > > > On 12/12/2012 04:43 PM, Johannes Weiner wrote: > > > >dc0422c "mm: vmscan: only evict file pages

Re: [patch 1/8] mm: memcg: only evict file pages when we have plenty

2012-12-13 Thread Mel Gorman
On Wed, Dec 12, 2012 at 05:28:44PM -0500, Johannes Weiner wrote: > On Wed, Dec 12, 2012 at 04:53:36PM -0500, Rik van Riel wrote: > > On 12/12/2012 04:43 PM, Johannes Weiner wrote: > > >dc0422c "mm: vmscan: only evict file pages when we have plenty" makes You are using some internal tree for that

Re: [patch 1/8] mm: memcg: only evict file pages when we have plenty

2012-12-13 Thread Mel Gorman
On Wed, Dec 12, 2012 at 05:28:44PM -0500, Johannes Weiner wrote: On Wed, Dec 12, 2012 at 04:53:36PM -0500, Rik van Riel wrote: On 12/12/2012 04:43 PM, Johannes Weiner wrote: dc0422c mm: vmscan: only evict file pages when we have plenty makes You are using some internal tree for that commit.

Re: [patch 1/8] mm: memcg: only evict file pages when we have plenty

2012-12-13 Thread Mel Gorman
On Thu, Dec 13, 2012 at 10:07:04AM +, Mel Gorman wrote: On Wed, Dec 12, 2012 at 05:28:44PM -0500, Johannes Weiner wrote: On Wed, Dec 12, 2012 at 04:53:36PM -0500, Rik van Riel wrote: On 12/12/2012 04:43 PM, Johannes Weiner wrote: dc0422c mm: vmscan: only evict file pages when we have

Re: [patch 1/8] mm: memcg: only evict file pages when we have plenty

2012-12-13 Thread Michal Hocko
On Wed 12-12-12 17:28:44, Johannes Weiner wrote: On Wed, Dec 12, 2012 at 04:53:36PM -0500, Rik van Riel wrote: On 12/12/2012 04:43 PM, Johannes Weiner wrote: dc0422c mm: vmscan: only evict file pages when we have plenty makes a point of not going for anonymous memory while there is still

Re: [patch 1/8] mm: memcg: only evict file pages when we have plenty

2012-12-12 Thread Simon Jeons
On Wed, 2012-12-12 at 16:53 -0500, Rik van Riel wrote: > On 12/12/2012 04:43 PM, Johannes Weiner wrote: > > dc0422c "mm: vmscan: only evict file pages when we have plenty" makes > > a point of not going for anonymous memory while there is still enough > > inactive cache around. > > > > The check

Re: [patch 1/8] mm: memcg: only evict file pages when we have plenty

2012-12-12 Thread Simon Jeons
On Wed, 2012-12-12 at 16:43 -0500, Johannes Weiner wrote: > dc0422c "mm: vmscan: only evict file pages when we have plenty" makes Can't find dc0422c. > a point of not going for anonymous memory while there is still enough > inactive cache around. > > The check was added only for global reclaim,

Re: [patch 1/8] mm: memcg: only evict file pages when we have plenty

2012-12-12 Thread Johannes Weiner
On Wed, Dec 12, 2012 at 04:53:36PM -0500, Rik van Riel wrote: > On 12/12/2012 04:43 PM, Johannes Weiner wrote: > >dc0422c "mm: vmscan: only evict file pages when we have plenty" makes > >a point of not going for anonymous memory while there is still enough > >inactive cache around. > > > >The

Re: [patch 1/8] mm: memcg: only evict file pages when we have plenty

2012-12-12 Thread Rik van Riel
On 12/12/2012 04:43 PM, Johannes Weiner wrote: dc0422c "mm: vmscan: only evict file pages when we have plenty" makes a point of not going for anonymous memory while there is still enough inactive cache around. The check was added only for global reclaim, but it is just as useful for memory

[patch 1/8] mm: memcg: only evict file pages when we have plenty

2012-12-12 Thread Johannes Weiner
dc0422c "mm: vmscan: only evict file pages when we have plenty" makes a point of not going for anonymous memory while there is still enough inactive cache around. The check was added only for global reclaim, but it is just as useful for memory cgroup reclaim. Signed-off-by: Johannes Weiner ---

[patch 1/8] mm: memcg: only evict file pages when we have plenty

2012-12-12 Thread Johannes Weiner
dc0422c mm: vmscan: only evict file pages when we have plenty makes a point of not going for anonymous memory while there is still enough inactive cache around. The check was added only for global reclaim, but it is just as useful for memory cgroup reclaim. Signed-off-by: Johannes Weiner

Re: [patch 1/8] mm: memcg: only evict file pages when we have plenty

2012-12-12 Thread Rik van Riel
On 12/12/2012 04:43 PM, Johannes Weiner wrote: dc0422c mm: vmscan: only evict file pages when we have plenty makes a point of not going for anonymous memory while there is still enough inactive cache around. The check was added only for global reclaim, but it is just as useful for memory cgroup

Re: [patch 1/8] mm: memcg: only evict file pages when we have plenty

2012-12-12 Thread Johannes Weiner
On Wed, Dec 12, 2012 at 04:53:36PM -0500, Rik van Riel wrote: On 12/12/2012 04:43 PM, Johannes Weiner wrote: dc0422c mm: vmscan: only evict file pages when we have plenty makes a point of not going for anonymous memory while there is still enough inactive cache around. The check was added

Re: [patch 1/8] mm: memcg: only evict file pages when we have plenty

2012-12-12 Thread Simon Jeons
On Wed, 2012-12-12 at 16:43 -0500, Johannes Weiner wrote: dc0422c mm: vmscan: only evict file pages when we have plenty makes Can't find dc0422c. a point of not going for anonymous memory while there is still enough inactive cache around. The check was added only for global reclaim, but it

Re: [patch 1/8] mm: memcg: only evict file pages when we have plenty

2012-12-12 Thread Simon Jeons
On Wed, 2012-12-12 at 16:53 -0500, Rik van Riel wrote: On 12/12/2012 04:43 PM, Johannes Weiner wrote: dc0422c mm: vmscan: only evict file pages when we have plenty makes a point of not going for anonymous memory while there is still enough inactive cache around. The check was added only