Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-04-12 Thread Rik van Riel
On 04/12/2013 05:34 AM, Mel Gorman wrote: I do not speak for Rik but I at least am ignoring most of these questions because there is not enough time in the day already. Pings are not likely to change my opinion. Same here. Some questions are just so lacking in context that trying to answer

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-04-12 Thread Mel Gorman
On Fri, Apr 12, 2013 at 01:46:22PM +0800, Ric Mason wrote: > Ping Rik, I also want to know the answer. ;-) This question, like a *lot* of list traffic recently, is a "how long is a piece of string" with hints that it is an important question but really is just going to waste a developers time

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-04-12 Thread Mel Gorman
On Fri, Apr 12, 2013 at 01:46:22PM +0800, Ric Mason wrote: Ping Rik, I also want to know the answer. ;-) This question, like a *lot* of list traffic recently, is a how long is a piece of string with hints that it is an important question but really is just going to waste a developers time

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-04-12 Thread Rik van Riel
On 04/12/2013 05:34 AM, Mel Gorman wrote: I do not speak for Rik but I at least am ignoring most of these questions because there is not enough time in the day already. Pings are not likely to change my opinion. Same here. Some questions are just so lacking in context that trying to answer

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-04-11 Thread Ric Mason
Ping Rik, I also want to know the answer. ;-) On 04/11/2013 01:58 PM, Will Huck wrote: Hi Rik, On 03/22/2013 11:52 AM, Rik van Riel wrote: On 03/21/2013 08:05 PM, Will Huck wrote: One offline question, how to understand this in function balance_pgdat: /* * Do some background aging of the

[PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-04-11 Thread Mel Gorman
The number of pages kswapd can reclaim is bound by the number of pages it scans which is related to the size of the zone and the scanning priority. In many cases the priority remains low because it's reset every SWAP_CLUSTER_MAX reclaimed pages but in the event kswapd scans a large number of pages

[PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-04-11 Thread Mel Gorman
The number of pages kswapd can reclaim is bound by the number of pages it scans which is related to the size of the zone and the scanning priority. In many cases the priority remains low because it's reset every SWAP_CLUSTER_MAX reclaimed pages but in the event kswapd scans a large number of pages

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-04-11 Thread Ric Mason
Ping Rik, I also want to know the answer. ;-) On 04/11/2013 01:58 PM, Will Huck wrote: Hi Rik, On 03/22/2013 11:52 AM, Rik van Riel wrote: On 03/21/2013 08:05 PM, Will Huck wrote: One offline question, how to understand this in function balance_pgdat: /* * Do some background aging of the

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-04-10 Thread Will Huck
Hi Rik, On 03/22/2013 11:52 AM, Rik van Riel wrote: On 03/21/2013 08:05 PM, Will Huck wrote: One offline question, how to understand this in function balance_pgdat: /* * Do some background aging of the anon list, to give * pages a chance to be referenced before reclaiming. */

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-04-10 Thread Will Huck
Hi Rik, On 03/22/2013 11:52 AM, Rik van Riel wrote: On 03/21/2013 08:05 PM, Will Huck wrote: One offline question, how to understand this in function balance_pgdat: /* * Do some background aging of the anon list, to give * pages a chance to be referenced before reclaiming. */

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-04-10 Thread Kamezawa Hiroyuki
(2013/04/09 20:06), Mel Gorman wrote: > The number of pages kswapd can reclaim is bound by the number of pages it > scans which is related to the size of the zone and the scanning priority. In > many cases the priority remains low because it's reset every SWAP_CLUSTER_MAX > reclaimed pages but in

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-04-10 Thread Kamezawa Hiroyuki
(2013/04/09 20:06), Mel Gorman wrote: The number of pages kswapd can reclaim is bound by the number of pages it scans which is related to the size of the zone and the scanning priority. In many cases the priority remains low because it's reset every SWAP_CLUSTER_MAX reclaimed pages but in the

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-04-10 Thread Will Huck
Hi Rik, On 03/22/2013 11:52 AM, Rik van Riel wrote: On 03/21/2013 08:05 PM, Will Huck wrote: One offline question, how to understand this in function balance_pgdat: /* * Do some background aging of the anon list, to give * pages a chance to be referenced before reclaiming. */

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-04-10 Thread Will Huck
Hi Rik, On 03/22/2013 11:52 AM, Rik van Riel wrote: On 03/21/2013 08:05 PM, Will Huck wrote: One offline question, how to understand this in function balance_pgdat: /* * Do some background aging of the anon list, to give * pages a chance to be referenced before reclaiming. */

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-04-09 Thread Michal Hocko
On Tue 09-04-13 12:06:56, Mel Gorman wrote: > The number of pages kswapd can reclaim is bound by the number of pages it > scans which is related to the size of the zone and the scanning priority. In > many cases the priority remains low because it's reset every SWAP_CLUSTER_MAX > reclaimed pages

[PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-04-09 Thread Mel Gorman
The number of pages kswapd can reclaim is bound by the number of pages it scans which is related to the size of the zone and the scanning priority. In many cases the priority remains low because it's reset every SWAP_CLUSTER_MAX reclaimed pages but in the event kswapd scans a large number of pages

[PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-04-09 Thread Mel Gorman
The number of pages kswapd can reclaim is bound by the number of pages it scans which is related to the size of the zone and the scanning priority. In many cases the priority remains low because it's reset every SWAP_CLUSTER_MAX reclaimed pages but in the event kswapd scans a large number of pages

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-04-09 Thread Michal Hocko
On Tue 09-04-13 12:06:56, Mel Gorman wrote: The number of pages kswapd can reclaim is bound by the number of pages it scans which is related to the size of the zone and the scanning priority. In many cases the priority remains low because it's reset every SWAP_CLUSTER_MAX reclaimed pages but

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-04-07 Thread Will Huck
cc Fengguang, On 04/05/2013 08:05 AM, Will Huck wrote: Hi Rik, On 03/22/2013 09:01 PM, Rik van Riel wrote: On 03/22/2013 12:59 AM, Will Huck wrote: Hi Rik, On 03/22/2013 11:56 AM, Will Huck wrote: Hi Rik, On 03/22/2013 11:52 AM, Rik van Riel wrote: On 03/21/2013 08:05 PM, Will Huck wrote:

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-04-07 Thread Will Huck
Ping Rik. On 04/05/2013 08:05 AM, Will Huck wrote: Hi Rik, On 03/22/2013 09:01 PM, Rik van Riel wrote: On 03/22/2013 12:59 AM, Will Huck wrote: Hi Rik, On 03/22/2013 11:56 AM, Will Huck wrote: Hi Rik, On 03/22/2013 11:52 AM, Rik van Riel wrote: On 03/21/2013 08:05 PM, Will Huck wrote: One

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-04-07 Thread Will Huck
Ping Rik. On 04/05/2013 08:05 AM, Will Huck wrote: Hi Rik, On 03/22/2013 09:01 PM, Rik van Riel wrote: On 03/22/2013 12:59 AM, Will Huck wrote: Hi Rik, On 03/22/2013 11:56 AM, Will Huck wrote: Hi Rik, On 03/22/2013 11:52 AM, Rik van Riel wrote: On 03/21/2013 08:05 PM, Will Huck wrote: One

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-04-07 Thread Will Huck
cc Fengguang, On 04/05/2013 08:05 AM, Will Huck wrote: Hi Rik, On 03/22/2013 09:01 PM, Rik van Riel wrote: On 03/22/2013 12:59 AM, Will Huck wrote: Hi Rik, On 03/22/2013 11:56 AM, Will Huck wrote: Hi Rik, On 03/22/2013 11:52 AM, Rik van Riel wrote: On 03/21/2013 08:05 PM, Will Huck wrote:

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-04-04 Thread Will Huck
Hi Rik, On 03/22/2013 09:01 PM, Rik van Riel wrote: On 03/22/2013 12:59 AM, Will Huck wrote: Hi Rik, On 03/22/2013 11:56 AM, Will Huck wrote: Hi Rik, On 03/22/2013 11:52 AM, Rik van Riel wrote: On 03/21/2013 08:05 PM, Will Huck wrote: One offline question, how to understand this in function

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-04-04 Thread Will Huck
Hi Rik, On 03/22/2013 09:01 PM, Rik van Riel wrote: On 03/22/2013 12:59 AM, Will Huck wrote: Hi Rik, On 03/22/2013 11:56 AM, Will Huck wrote: Hi Rik, On 03/22/2013 11:52 AM, Rik van Riel wrote: On 03/21/2013 08:05 PM, Will Huck wrote: One offline question, how to understand this in function

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-04-02 Thread Mel Gorman
On Sat, Mar 30, 2013 at 11:07:03PM +0100, Jiri Slaby wrote: > On 03/29/2013 09:22 AM, Michal Hocko wrote: > > On Thu 28-03-13 23:31:18, Jiri Slaby wrote: > >> On 03/25/2013 10:13 AM, Jiri Slaby wrote: > >>> BTW I very pray this will fix also the issue I have when I run ltp tests > >>> (highly I/O

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-04-02 Thread Mel Gorman
On Sat, Mar 30, 2013 at 11:07:03PM +0100, Jiri Slaby wrote: On 03/29/2013 09:22 AM, Michal Hocko wrote: On Thu 28-03-13 23:31:18, Jiri Slaby wrote: On 03/25/2013 10:13 AM, Jiri Slaby wrote: BTW I very pray this will fix also the issue I have when I run ltp tests (highly I/O intensive,

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-30 Thread Jiri Slaby
On 03/29/2013 09:22 AM, Michal Hocko wrote: > On Thu 28-03-13 23:31:18, Jiri Slaby wrote: >> On 03/25/2013 10:13 AM, Jiri Slaby wrote: >>> BTW I very pray this will fix also the issue I have when I run ltp tests >>> (highly I/O intensive, esp. `growfiles') in a VM while playing a movie >>> on the

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-30 Thread Jiri Slaby
On 03/29/2013 09:22 AM, Michal Hocko wrote: On Thu 28-03-13 23:31:18, Jiri Slaby wrote: On 03/25/2013 10:13 AM, Jiri Slaby wrote: BTW I very pray this will fix also the issue I have when I run ltp tests (highly I/O intensive, esp. `growfiles') in a VM while playing a movie on the host

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-29 Thread Michal Hocko
On Thu 28-03-13 23:31:18, Jiri Slaby wrote: > On 03/25/2013 10:13 AM, Jiri Slaby wrote: > > BTW I very pray this will fix also the issue I have when I run ltp tests > > (highly I/O intensive, esp. `growfiles') in a VM while playing a movie > > on the host resulting in a stuttered playback ;). > >

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-29 Thread Michal Hocko
On Thu 28-03-13 23:31:18, Jiri Slaby wrote: On 03/25/2013 10:13 AM, Jiri Slaby wrote: BTW I very pray this will fix also the issue I have when I run ltp tests (highly I/O intensive, esp. `growfiles') in a VM while playing a movie on the host resulting in a stuttered playback ;). No, this

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-28 Thread Jiri Slaby
On 03/25/2013 10:13 AM, Jiri Slaby wrote: > BTW I very pray this will fix also the issue I have when I run ltp tests > (highly I/O intensive, esp. `growfiles') in a VM while playing a movie > on the host resulting in a stuttered playback ;). No, this is still terrible. I was now updating a kernel

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-28 Thread Jiri Slaby
On 03/25/2013 10:13 AM, Jiri Slaby wrote: BTW I very pray this will fix also the issue I have when I run ltp tests (highly I/O intensive, esp. `growfiles') in a VM while playing a movie on the host resulting in a stuttered playback ;). No, this is still terrible. I was now updating a kernel in

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-25 Thread Jiri Slaby
On 03/25/2013 10:07 AM, Michal Hocko wrote: > On Sun 17-03-13 13:04:07, Mel Gorman wrote: >> The number of pages kswapd can reclaim is bound by the number of pages it >> scans which is related to the size of the zone and the scanning priority. In >> many cases the priority remains low because it's

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-25 Thread Michal Hocko
On Sun 17-03-13 13:04:07, Mel Gorman wrote: > The number of pages kswapd can reclaim is bound by the number of pages it > scans which is related to the size of the zone and the scanning priority. In > many cases the priority remains low because it's reset every SWAP_CLUSTER_MAX > reclaimed pages

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-25 Thread Michal Hocko
On Sun 17-03-13 13:04:07, Mel Gorman wrote: The number of pages kswapd can reclaim is bound by the number of pages it scans which is related to the size of the zone and the scanning priority. In many cases the priority remains low because it's reset every SWAP_CLUSTER_MAX reclaimed pages but

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-25 Thread Jiri Slaby
On 03/25/2013 10:07 AM, Michal Hocko wrote: On Sun 17-03-13 13:04:07, Mel Gorman wrote: The number of pages kswapd can reclaim is bound by the number of pages it scans which is related to the size of the zone and the scanning priority. In many cases the priority remains low because it's reset

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-22 Thread Rik van Riel
On 03/22/2013 12:59 AM, Will Huck wrote: Hi Rik, On 03/22/2013 11:56 AM, Will Huck wrote: Hi Rik, On 03/22/2013 11:52 AM, Rik van Riel wrote: On 03/21/2013 08:05 PM, Will Huck wrote: One offline question, how to understand this in function balance_pgdat: /* * Do some background aging of

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-22 Thread Rik van Riel
On 03/22/2013 12:59 AM, Will Huck wrote: Hi Rik, On 03/22/2013 11:56 AM, Will Huck wrote: Hi Rik, On 03/22/2013 11:52 AM, Rik van Riel wrote: On 03/21/2013 08:05 PM, Will Huck wrote: One offline question, how to understand this in function balance_pgdat: /* * Do some background aging of

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-21 Thread Will Huck
Hi Rik, On 03/22/2013 11:52 AM, Rik van Riel wrote: On 03/21/2013 08:05 PM, Will Huck wrote: One offline question, how to understand this in function balance_pgdat: /* * Do some background aging of the anon list, to give * pages a chance to be referenced before reclaiming. */

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-21 Thread Will Huck
Hi Rik, On 03/22/2013 11:56 AM, Will Huck wrote: Hi Rik, On 03/22/2013 11:52 AM, Rik van Riel wrote: On 03/21/2013 08:05 PM, Will Huck wrote: One offline question, how to understand this in function balance_pgdat: /* * Do some background aging of the anon list, to give * pages a chance to

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-21 Thread Rik van Riel
On 03/21/2013 08:05 PM, Will Huck wrote: One offline question, how to understand this in function balance_pgdat: /* * Do some background aging of the anon list, to give * pages a chance to be referenced before reclaiming. */ age_acitve_anon(zone, ); The anon lrus use a two-handed clock

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-21 Thread Will Huck
Hi Rik, On 03/21/2013 08:52 AM, Rik van Riel wrote: On 03/20/2013 12:18 PM, Michal Hocko wrote: On Sun 17-03-13 13:04:07, Mel Gorman wrote: [...] diff --git a/mm/vmscan.c b/mm/vmscan.c index 88c5fed..4835a7a 100644 --- a/mm/vmscan.c +++ b/mm/vmscan.c @@ -2593,6 +2593,32 @@ static bool

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-21 Thread Will Huck
Hi Johannes, On 03/21/2013 11:57 PM, Johannes Weiner wrote: On Sun, Mar 17, 2013 at 01:04:07PM +, Mel Gorman wrote: The number of pages kswapd can reclaim is bound by the number of pages it scans which is related to the size of the zone and the scanning priority. In many cases the priority

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-21 Thread Mel Gorman
On Thu, Mar 21, 2013 at 11:57:05AM -0400, Johannes Weiner wrote: > On Sun, Mar 17, 2013 at 01:04:07PM +, Mel Gorman wrote: > > The number of pages kswapd can reclaim is bound by the number of pages it > > scans which is related to the size of the zone and the scanning priority. In > > many

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-21 Thread Johannes Weiner
On Sun, Mar 17, 2013 at 01:04:07PM +, Mel Gorman wrote: > The number of pages kswapd can reclaim is bound by the number of pages it > scans which is related to the size of the zone and the scanning priority. In > many cases the priority remains low because it's reset every SWAP_CLUSTER_MAX >

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-21 Thread Michal Hocko
On Thu 21-03-13 09:47:13, Mel Gorman wrote: > On Wed, Mar 20, 2013 at 05:18:47PM +0100, Michal Hocko wrote: > > On Sun 17-03-13 13:04:07, Mel Gorman wrote: > > [...] > > > diff --git a/mm/vmscan.c b/mm/vmscan.c > > > index 88c5fed..4835a7a 100644 > > > --- a/mm/vmscan.c > > > +++ b/mm/vmscan.c > >

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-21 Thread Mel Gorman
On Wed, Mar 20, 2013 at 05:18:47PM +0100, Michal Hocko wrote: > On Sun 17-03-13 13:04:07, Mel Gorman wrote: > [...] > > diff --git a/mm/vmscan.c b/mm/vmscan.c > > index 88c5fed..4835a7a 100644 > > --- a/mm/vmscan.c > > +++ b/mm/vmscan.c > > @@ -2593,6 +2593,32 @@ static bool

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-21 Thread Mel Gorman
On Wed, Mar 20, 2013 at 05:18:47PM +0100, Michal Hocko wrote: On Sun 17-03-13 13:04:07, Mel Gorman wrote: [...] diff --git a/mm/vmscan.c b/mm/vmscan.c index 88c5fed..4835a7a 100644 --- a/mm/vmscan.c +++ b/mm/vmscan.c @@ -2593,6 +2593,32 @@ static bool prepare_kswapd_sleep(pg_data_t

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-21 Thread Michal Hocko
On Thu 21-03-13 09:47:13, Mel Gorman wrote: On Wed, Mar 20, 2013 at 05:18:47PM +0100, Michal Hocko wrote: On Sun 17-03-13 13:04:07, Mel Gorman wrote: [...] diff --git a/mm/vmscan.c b/mm/vmscan.c index 88c5fed..4835a7a 100644 --- a/mm/vmscan.c +++ b/mm/vmscan.c @@ -2593,6

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-21 Thread Johannes Weiner
On Sun, Mar 17, 2013 at 01:04:07PM +, Mel Gorman wrote: The number of pages kswapd can reclaim is bound by the number of pages it scans which is related to the size of the zone and the scanning priority. In many cases the priority remains low because it's reset every SWAP_CLUSTER_MAX

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-21 Thread Mel Gorman
On Thu, Mar 21, 2013 at 11:57:05AM -0400, Johannes Weiner wrote: On Sun, Mar 17, 2013 at 01:04:07PM +, Mel Gorman wrote: The number of pages kswapd can reclaim is bound by the number of pages it scans which is related to the size of the zone and the scanning priority. In many cases the

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-21 Thread Will Huck
Hi Johannes, On 03/21/2013 11:57 PM, Johannes Weiner wrote: On Sun, Mar 17, 2013 at 01:04:07PM +, Mel Gorman wrote: The number of pages kswapd can reclaim is bound by the number of pages it scans which is related to the size of the zone and the scanning priority. In many cases the priority

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-21 Thread Will Huck
Hi Rik, On 03/21/2013 08:52 AM, Rik van Riel wrote: On 03/20/2013 12:18 PM, Michal Hocko wrote: On Sun 17-03-13 13:04:07, Mel Gorman wrote: [...] diff --git a/mm/vmscan.c b/mm/vmscan.c index 88c5fed..4835a7a 100644 --- a/mm/vmscan.c +++ b/mm/vmscan.c @@ -2593,6 +2593,32 @@ static bool

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-21 Thread Rik van Riel
On 03/21/2013 08:05 PM, Will Huck wrote: One offline question, how to understand this in function balance_pgdat: /* * Do some background aging of the anon list, to give * pages a chance to be referenced before reclaiming. */ age_acitve_anon(zone, sc); The anon lrus use a two-handed

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-21 Thread Will Huck
Hi Rik, On 03/22/2013 11:56 AM, Will Huck wrote: Hi Rik, On 03/22/2013 11:52 AM, Rik van Riel wrote: On 03/21/2013 08:05 PM, Will Huck wrote: One offline question, how to understand this in function balance_pgdat: /* * Do some background aging of the anon list, to give * pages a chance to

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-21 Thread Will Huck
Hi Rik, On 03/22/2013 11:52 AM, Rik van Riel wrote: On 03/21/2013 08:05 PM, Will Huck wrote: One offline question, how to understand this in function balance_pgdat: /* * Do some background aging of the anon list, to give * pages a chance to be referenced before reclaiming. */

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-20 Thread Rik van Riel
On 03/20/2013 12:18 PM, Michal Hocko wrote: On Sun 17-03-13 13:04:07, Mel Gorman wrote: [...] diff --git a/mm/vmscan.c b/mm/vmscan.c index 88c5fed..4835a7a 100644 --- a/mm/vmscan.c +++ b/mm/vmscan.c @@ -2593,6 +2593,32 @@ static bool prepare_kswapd_sleep(pg_data_t *pgdat, int order, long

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-20 Thread Rik van Riel
On 03/17/2013 09:04 AM, Mel Gorman wrote: The number of pages kswapd can reclaim is bound by the number of pages it scans which is related to the size of the zone and the scanning priority. In many cases the priority remains low because it's reset every SWAP_CLUSTER_MAX reclaimed pages but in

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-20 Thread Michal Hocko
On Sun 17-03-13 13:04:07, Mel Gorman wrote: [...] > diff --git a/mm/vmscan.c b/mm/vmscan.c > index 88c5fed..4835a7a 100644 > --- a/mm/vmscan.c > +++ b/mm/vmscan.c > @@ -2593,6 +2593,32 @@ static bool prepare_kswapd_sleep(pg_data_t *pgdat, int > order, long remaining, > } > > /* > + * kswapd

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-20 Thread Michal Hocko
On Sun 17-03-13 13:04:07, Mel Gorman wrote: [...] diff --git a/mm/vmscan.c b/mm/vmscan.c index 88c5fed..4835a7a 100644 --- a/mm/vmscan.c +++ b/mm/vmscan.c @@ -2593,6 +2593,32 @@ static bool prepare_kswapd_sleep(pg_data_t *pgdat, int order, long remaining, } /* + * kswapd shrinks the

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-20 Thread Rik van Riel
On 03/17/2013 09:04 AM, Mel Gorman wrote: The number of pages kswapd can reclaim is bound by the number of pages it scans which is related to the size of the zone and the scanning priority. In many cases the priority remains low because it's reset every SWAP_CLUSTER_MAX reclaimed pages but in

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-20 Thread Rik van Riel
On 03/20/2013 12:18 PM, Michal Hocko wrote: On Sun 17-03-13 13:04:07, Mel Gorman wrote: [...] diff --git a/mm/vmscan.c b/mm/vmscan.c index 88c5fed..4835a7a 100644 --- a/mm/vmscan.c +++ b/mm/vmscan.c @@ -2593,6 +2593,32 @@ static bool prepare_kswapd_sleep(pg_data_t *pgdat, int order, long

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-19 Thread Mel Gorman
On Tue, Mar 19, 2013 at 06:16:50PM +0800, Simon Jeons wrote: > Hi Mel, > On 03/19/2013 05:55 PM, Mel Gorman wrote: > >On Tue, Mar 19, 2013 at 07:53:16AM +0800, Simon Jeons wrote: > >>Hi Mel, > >>On 03/17/2013 09:04 PM, Mel Gorman wrote: > >>>The number of pages kswapd can reclaim is bound by the

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-19 Thread Simon Jeons
Hi Mel, On 03/19/2013 05:55 PM, Mel Gorman wrote: On Tue, Mar 19, 2013 at 07:53:16AM +0800, Simon Jeons wrote: Hi Mel, On 03/17/2013 09:04 PM, Mel Gorman wrote: The number of pages kswapd can reclaim is bound by the number of pages it scans which is related to the size of the zone and the

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-19 Thread Mel Gorman
On Tue, Mar 19, 2013 at 07:53:16AM +0800, Simon Jeons wrote: > Hi Mel, > On 03/17/2013 09:04 PM, Mel Gorman wrote: > >The number of pages kswapd can reclaim is bound by the number of pages it > >scans which is related to the size of the zone and the scanning priority. In > >many cases the priority

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-19 Thread Mel Gorman
On Tue, Mar 19, 2013 at 07:53:16AM +0800, Simon Jeons wrote: Hi Mel, On 03/17/2013 09:04 PM, Mel Gorman wrote: The number of pages kswapd can reclaim is bound by the number of pages it scans which is related to the size of the zone and the scanning priority. In many cases the priority remains

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-19 Thread Simon Jeons
Hi Mel, On 03/19/2013 05:55 PM, Mel Gorman wrote: On Tue, Mar 19, 2013 at 07:53:16AM +0800, Simon Jeons wrote: Hi Mel, On 03/17/2013 09:04 PM, Mel Gorman wrote: The number of pages kswapd can reclaim is bound by the number of pages it scans which is related to the size of the zone and the

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-19 Thread Mel Gorman
On Tue, Mar 19, 2013 at 06:16:50PM +0800, Simon Jeons wrote: Hi Mel, On 03/19/2013 05:55 PM, Mel Gorman wrote: On Tue, Mar 19, 2013 at 07:53:16AM +0800, Simon Jeons wrote: Hi Mel, On 03/17/2013 09:04 PM, Mel Gorman wrote: The number of pages kswapd can reclaim is bound by the number of pages

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-18 Thread Simon Jeons
Hi Mel, On 03/17/2013 09:04 PM, Mel Gorman wrote: The number of pages kswapd can reclaim is bound by the number of pages it scans which is related to the size of the zone and the scanning priority. In many cases the priority remains low because it's reset every SWAP_CLUSTER_MAX reclaimed pages

Re: [PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-18 Thread Simon Jeons
Hi Mel, On 03/17/2013 09:04 PM, Mel Gorman wrote: The number of pages kswapd can reclaim is bound by the number of pages it scans which is related to the size of the zone and the scanning priority. In many cases the priority remains low because it's reset every SWAP_CLUSTER_MAX reclaimed pages

[PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-17 Thread Mel Gorman
The number of pages kswapd can reclaim is bound by the number of pages it scans which is related to the size of the zone and the scanning priority. In many cases the priority remains low because it's reset every SWAP_CLUSTER_MAX reclaimed pages but in the event kswapd scans a large number of pages

[PATCH 01/10] mm: vmscan: Limit the number of pages kswapd reclaims at each priority

2013-03-17 Thread Mel Gorman
The number of pages kswapd can reclaim is bound by the number of pages it scans which is related to the size of the zone and the scanning priority. In many cases the priority remains low because it's reset every SWAP_CLUSTER_MAX reclaimed pages but in the event kswapd scans a large number of pages