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

2012-08-09 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 5/5] mm: have order > 0 compaction start near a pageblock with free pages

2012-08-09 Thread Minchan Kim
On Wed, Aug 08, 2012 at 08:08:44PM +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 5/5] mm: have order > 0 compaction start near a pageblock with free pages

2012-08-09 Thread Minchan Kim
On Thu, Aug 09, 2012 at 09:23:28AM +0100, Mel Gorman wrote: > On Thu, Aug 09, 2012 at 09:12:12AM +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

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

2012-08-09 Thread Mel Gorman
On Thu, Aug 09, 2012 at 09:12:12AM +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. When a wrapped scanner isolates a

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

2012-08-09 Thread Mel Gorman
On Thu, Aug 09, 2012 at 09:12:12AM +0900, Minchan Kim wrote: SNIP 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. When a wrapped scanner isolates a page, it

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

2012-08-09 Thread Minchan Kim
On Thu, Aug 09, 2012 at 09:23:28AM +0100, Mel Gorman wrote: On Thu, Aug 09, 2012 at 09:12:12AM +0900, Minchan Kim wrote: SNIP 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

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

2012-08-09 Thread Minchan Kim
On Wed, Aug 08, 2012 at 08:08:44PM +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

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

2012-08-09 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 5/5] mm: have order > 0 compaction start near a pageblock with free pages

2012-08-08 Thread Minchan Kim
Hi Mel, On Wed, Aug 08, 2012 at 08:08:44PM +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 >

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

2012-08-08 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 5/5] mm: have order 0 compaction start near a pageblock with free pages

2012-08-08 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 5/5] mm: have order 0 compaction start near a pageblock with free pages

2012-08-08 Thread Minchan Kim
Hi Mel, On Wed, Aug 08, 2012 at 08:08:44PM +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