Re: [PATCH 06/10] mm: vmscan: Have kswapd writeback pages based on dirty pages encountered, not priority

2013-04-18 Thread Johannes Weiner
On Thu, Apr 11, 2013 at 08:57:54PM +0100, Mel Gorman wrote: > Currently kswapd queues dirty pages for writeback if scanning at an elevated > priority but the priority kswapd scans at is not related to the number > of unqueued dirty encountered. Since commit "mm: vmscan: Flatten kswapd > priority

Re: [PATCH 06/10] mm: vmscan: Have kswapd writeback pages based on dirty pages encountered, not priority

2013-04-18 Thread Johannes Weiner
On Thu, Apr 11, 2013 at 08:57:54PM +0100, Mel Gorman wrote: Currently kswapd queues dirty pages for writeback if scanning at an elevated priority but the priority kswapd scans at is not related to the number of unqueued dirty encountered. Since commit mm: vmscan: Flatten kswapd priority loop,

Re: [PATCH 06/10] mm: vmscan: Have kswapd writeback pages based on dirty pages encountered, not priority

2013-04-11 Thread Rik van Riel
On 04/09/2013 07:07 AM, Mel Gorman wrote: Currently kswapd queues dirty pages for writeback if scanning at an elevated priority but the priority kswapd scans at is not related to the number of unqueued dirty encountered. Since commit "mm: vmscan: Flatten kswapd priority loop", the priority is

[PATCH 06/10] mm: vmscan: Have kswapd writeback pages based on dirty pages encountered, not priority

2013-04-11 Thread Mel Gorman
Currently kswapd queues dirty pages for writeback if scanning at an elevated priority but the priority kswapd scans at is not related to the number of unqueued dirty encountered. Since commit "mm: vmscan: Flatten kswapd priority loop", the priority is related to the size of the LRU and the zone

[PATCH 06/10] mm: vmscan: Have kswapd writeback pages based on dirty pages encountered, not priority

2013-04-11 Thread Mel Gorman
Currently kswapd queues dirty pages for writeback if scanning at an elevated priority but the priority kswapd scans at is not related to the number of unqueued dirty encountered. Since commit mm: vmscan: Flatten kswapd priority loop, the priority is related to the size of the LRU and the zone

Re: [PATCH 06/10] mm: vmscan: Have kswapd writeback pages based on dirty pages encountered, not priority

2013-04-11 Thread Rik van Riel
On 04/09/2013 07:07 AM, Mel Gorman wrote: Currently kswapd queues dirty pages for writeback if scanning at an elevated priority but the priority kswapd scans at is not related to the number of unqueued dirty encountered. Since commit mm: vmscan: Flatten kswapd priority loop, the priority is

[PATCH 06/10] mm: vmscan: Have kswapd writeback pages based on dirty pages encountered, not priority

2013-04-09 Thread Mel Gorman
Currently kswapd queues dirty pages for writeback if scanning at an elevated priority but the priority kswapd scans at is not related to the number of unqueued dirty encountered. Since commit "mm: vmscan: Flatten kswapd priority loop", the priority is related to the size of the LRU and the zone

[PATCH 06/10] mm: vmscan: Have kswapd writeback pages based on dirty pages encountered, not priority

2013-04-09 Thread Mel Gorman
Currently kswapd queues dirty pages for writeback if scanning at an elevated priority but the priority kswapd scans at is not related to the number of unqueued dirty encountered. Since commit mm: vmscan: Flatten kswapd priority loop, the priority is related to the size of the LRU and the zone

Re: [PATCH 06/10] mm: vmscan: Have kswapd writeback pages based on dirty pages encountered, not priority

2013-03-21 Thread Rik van Riel
On 03/21/2013 02:15 PM, Mel Gorman wrote: On Thu, Mar 21, 2013 at 01:53:41PM -0400, Rik van Riel wrote: On 03/17/2013 11:11 AM, Mel Gorman wrote: On Sun, Mar 17, 2013 at 07:42:39AM -0700, Andi Kleen wrote: Mel Gorman writes: @@ -495,6 +495,9 @@ typedef enum { ZONE_CONGESTED,

Re: [PATCH 06/10] mm: vmscan: Have kswapd writeback pages based on dirty pages encountered, not priority

2013-03-21 Thread Mel Gorman
On Thu, Mar 21, 2013 at 01:53:41PM -0400, Rik van Riel wrote: > On 03/17/2013 11:11 AM, Mel Gorman wrote: > >On Sun, Mar 17, 2013 at 07:42:39AM -0700, Andi Kleen wrote: > >>Mel Gorman writes: > >> > >>>@@ -495,6 +495,9 @@ typedef enum { > >>> ZONE_CONGESTED, /* zone has many

Re: [PATCH 06/10] mm: vmscan: Have kswapd writeback pages based on dirty pages encountered, not priority

2013-03-21 Thread Rik van Riel
On 03/17/2013 11:11 AM, Mel Gorman wrote: On Sun, Mar 17, 2013 at 07:42:39AM -0700, Andi Kleen wrote: Mel Gorman writes: @@ -495,6 +495,9 @@ typedef enum { ZONE_CONGESTED, /* zone has many dirty pages backed by * a congested

Re: [PATCH 06/10] mm: vmscan: Have kswapd writeback pages based on dirty pages encountered, not priority

2013-03-21 Thread Rik van Riel
On 03/17/2013 11:11 AM, Mel Gorman wrote: On Sun, Mar 17, 2013 at 07:42:39AM -0700, Andi Kleen wrote: Mel Gorman mgor...@suse.de writes: @@ -495,6 +495,9 @@ typedef enum { ZONE_CONGESTED, /* zone has many dirty pages backed by *

Re: [PATCH 06/10] mm: vmscan: Have kswapd writeback pages based on dirty pages encountered, not priority

2013-03-21 Thread Mel Gorman
On Thu, Mar 21, 2013 at 01:53:41PM -0400, Rik van Riel wrote: On 03/17/2013 11:11 AM, Mel Gorman wrote: On Sun, Mar 17, 2013 at 07:42:39AM -0700, Andi Kleen wrote: Mel Gorman mgor...@suse.de writes: @@ -495,6 +495,9 @@ typedef enum { ZONE_CONGESTED, /* zone has many dirty

Re: [PATCH 06/10] mm: vmscan: Have kswapd writeback pages based on dirty pages encountered, not priority

2013-03-21 Thread Rik van Riel
On 03/21/2013 02:15 PM, Mel Gorman wrote: On Thu, Mar 21, 2013 at 01:53:41PM -0400, Rik van Riel wrote: On 03/17/2013 11:11 AM, Mel Gorman wrote: On Sun, Mar 17, 2013 at 07:42:39AM -0700, Andi Kleen wrote: Mel Gorman mgor...@suse.de writes: @@ -495,6 +495,9 @@ typedef enum {

Re: [PATCH 06/10] mm: vmscan: Have kswapd writeback pages based on dirty pages encountered, not priority

2013-03-19 Thread Mel Gorman
On Mon, Mar 18, 2013 at 07:08:50PM +0800, Wanpeng Li wrote: > >@@ -2735,8 +2748,12 @@ static unsigned long balance_pgdat(pg_data_t *pgdat, > >int order, > > end_zone = i; > > break; > > } else { > >-

Re: [PATCH 06/10] mm: vmscan: Have kswapd writeback pages based on dirty pages encountered, not priority

2013-03-19 Thread Mel Gorman
On Mon, Mar 18, 2013 at 07:08:50PM +0800, Wanpeng Li wrote: @@ -2735,8 +2748,12 @@ static unsigned long balance_pgdat(pg_data_t *pgdat, int order, end_zone = i; break; } else { -/* If

Re: [PATCH 06/10] mm: vmscan: Have kswapd writeback pages based on dirty pages encountered, not priority

2013-03-17 Thread Mel Gorman
On Sun, Mar 17, 2013 at 07:42:39AM -0700, Andi Kleen wrote: > Mel Gorman writes: > > > @@ -495,6 +495,9 @@ typedef enum { > > ZONE_CONGESTED, /* zone has many dirty pages backed by > > * a congested BDI > >

Re: [PATCH 06/10] mm: vmscan: Have kswapd writeback pages based on dirty pages encountered, not priority

2013-03-17 Thread Andi Kleen
Mel Gorman writes: > @@ -495,6 +495,9 @@ typedef enum { > ZONE_CONGESTED, /* zone has many dirty pages backed by >* a congested BDI >*/ > + ZONE_DIRTY, /* reclaim

[PATCH 06/10] mm: vmscan: Have kswapd writeback pages based on dirty pages encountered, not priority

2013-03-17 Thread Mel Gorman
Currently kswapd queues dirty pages for writeback if scanning at an elevated priority but the priority kswapd scans at is not related to the number of unqueued dirty encountered. Since commit "mm: vmscan: Flatten kswapd priority loop", the priority is related to the size of the LRU and the zone

[PATCH 06/10] mm: vmscan: Have kswapd writeback pages based on dirty pages encountered, not priority

2013-03-17 Thread Mel Gorman
Currently kswapd queues dirty pages for writeback if scanning at an elevated priority but the priority kswapd scans at is not related to the number of unqueued dirty encountered. Since commit mm: vmscan: Flatten kswapd priority loop, the priority is related to the size of the LRU and the zone

Re: [PATCH 06/10] mm: vmscan: Have kswapd writeback pages based on dirty pages encountered, not priority

2013-03-17 Thread Andi Kleen
Mel Gorman mgor...@suse.de writes: @@ -495,6 +495,9 @@ typedef enum { ZONE_CONGESTED, /* zone has many dirty pages backed by * a congested BDI */ + ZONE_DIRTY, /*

Re: [PATCH 06/10] mm: vmscan: Have kswapd writeback pages based on dirty pages encountered, not priority

2013-03-17 Thread Mel Gorman
On Sun, Mar 17, 2013 at 07:42:39AM -0700, Andi Kleen wrote: Mel Gorman mgor...@suse.de writes: @@ -495,6 +495,9 @@ typedef enum { ZONE_CONGESTED, /* zone has many dirty pages backed by * a congested BDI