Re: [PATCH 6/6] mm: have order > 0 compaction start near a pageblock with free pages

2012-08-08 Thread Mel Gorman
On Wed, Aug 08, 2012 at 01:36:00PM +0900, Minchan Kim wrote: > > > > Second, it updates compact_cached_free_pfn in a more limited set of > > circumstances. > > > > If a scanner has wrapped, it updates compact_cached_free_pfn to the end > > of the zone. Each time a wrapped scanner isoaltes a

Re: [PATCH 6/6] mm: have order 0 compaction start near a pageblock with free pages

2012-08-08 Thread Mel Gorman
On Wed, Aug 08, 2012 at 01:36:00PM +0900, Minchan Kim wrote: Second, it updates compact_cached_free_pfn in a more limited set of circumstances. If a scanner has wrapped, it updates compact_cached_free_pfn to the end of the zone. Each time a wrapped scanner isoaltes a page, it

Re: [PATCH 6/6] mm: have order > 0 compaction start near a pageblock with free pages

2012-08-07 Thread Minchan Kim
On Tue, Aug 07, 2012 at 01:31:17PM +0100, Mel Gorman wrote: > commit [7db8889a: mm: have order > 0 compaction start off where it left] > introduced a caching mechanism to reduce the amount work the free page > scanner does in compaction. However, it has a problem. Consider two process >

Re: [PATCH 6/6] mm: have order > 0 compaction start near a pageblock with free pages

2012-08-07 Thread Mel Gorman
On Tue, Aug 07, 2012 at 09:20:08AM -0600, Jim Schutt wrote: > On 08/07/2012 08:52 AM, Mel Gorman wrote: > >On Tue, Aug 07, 2012 at 10:45:25AM -0400, Rik van Riel wrote: > >>On 08/07/2012 08:31 AM, Mel Gorman wrote: > >>>commit [7db8889a: mm: have order> 0 compaction start off where it left] >

Re: [PATCH 6/6] mm: have order > 0 compaction start near a pageblock with free pages

2012-08-07 Thread Jim Schutt
On 08/07/2012 08:52 AM, Mel Gorman wrote: On Tue, Aug 07, 2012 at 10:45:25AM -0400, Rik van Riel wrote: On 08/07/2012 08:31 AM, Mel Gorman wrote: commit [7db8889a: mm: have order> 0 compaction start off where it left] introduced a caching mechanism to reduce the amount work the free page

Re: [PATCH 6/6] mm: have order > 0 compaction start near a pageblock with free pages

2012-08-07 Thread Mel Gorman
On Tue, Aug 07, 2012 at 10:45:25AM -0400, Rik van Riel wrote: > On 08/07/2012 08:31 AM, Mel Gorman wrote: > >commit [7db8889a: mm: have order> 0 compaction start off where it left] > >introduced a caching mechanism to reduce the amount work the free page > >scanner does in compaction. However, it

Re: [PATCH 6/6] mm: have order > 0 compaction start near a pageblock with free pages

2012-08-07 Thread Rik van Riel
On 08/07/2012 08:31 AM, Mel Gorman wrote: commit [7db8889a: mm: have order> 0 compaction start off where it left] introduced a caching mechanism to reduce the amount work the free page scanner does in compaction. However, it has a problem. Consider two process simultaneously scanning free pages

[PATCH 6/6] mm: have order > 0 compaction start near a pageblock with free pages

2012-08-07 Thread Mel Gorman
commit [7db8889a: mm: have order > 0 compaction start off where it left] introduced a caching mechanism to reduce the amount work the free page scanner does in compaction. However, it has a problem. Consider two process simultaneously scanning free pages

[PATCH 6/6] mm: have order 0 compaction start near a pageblock with free pages

2012-08-07 Thread Mel Gorman
commit [7db8889a: mm: have order 0 compaction start off where it left] introduced a caching mechanism to reduce the amount work the free page scanner does in compaction. However, it has a problem. Consider two process simultaneously scanning free pages

Re: [PATCH 6/6] mm: have order 0 compaction start near a pageblock with free pages

2012-08-07 Thread Rik van Riel
On 08/07/2012 08:31 AM, Mel Gorman wrote: commit [7db8889a: mm: have order 0 compaction start off where it left] introduced a caching mechanism to reduce the amount work the free page scanner does in compaction. However, it has a problem. Consider two process simultaneously scanning free pages

Re: [PATCH 6/6] mm: have order 0 compaction start near a pageblock with free pages

2012-08-07 Thread Mel Gorman
On Tue, Aug 07, 2012 at 10:45:25AM -0400, Rik van Riel wrote: On 08/07/2012 08:31 AM, Mel Gorman wrote: commit [7db8889a: mm: have order 0 compaction start off where it left] introduced a caching mechanism to reduce the amount work the free page scanner does in compaction. However, it has a

Re: [PATCH 6/6] mm: have order 0 compaction start near a pageblock with free pages

2012-08-07 Thread Jim Schutt
On 08/07/2012 08:52 AM, Mel Gorman wrote: On Tue, Aug 07, 2012 at 10:45:25AM -0400, Rik van Riel wrote: On 08/07/2012 08:31 AM, Mel Gorman wrote: commit [7db8889a: mm: have order 0 compaction start off where it left] introduced a caching mechanism to reduce the amount work the free page

Re: [PATCH 6/6] mm: have order 0 compaction start near a pageblock with free pages

2012-08-07 Thread Mel Gorman
On Tue, Aug 07, 2012 at 09:20:08AM -0600, Jim Schutt wrote: On 08/07/2012 08:52 AM, Mel Gorman wrote: On Tue, Aug 07, 2012 at 10:45:25AM -0400, Rik van Riel wrote: On 08/07/2012 08:31 AM, Mel Gorman wrote: commit [7db8889a: mm: have order 0 compaction start off where it left] introduced a

Re: [PATCH 6/6] mm: have order 0 compaction start near a pageblock with free pages

2012-08-07 Thread Minchan Kim
On Tue, Aug 07, 2012 at 01:31:17PM +0100, Mel Gorman wrote: commit [7db8889a: mm: have order 0 compaction start off where it left] introduced a caching mechanism to reduce the amount work the free page scanner does in compaction. However, it has a problem. Consider two process simultaneously