Re: [PATCH v3 04/13] mm, compaction: move pageblock checks up from isolate_migratepages_range()

2014-06-26 Thread Joonsoo Kim
On Wed, Jun 25, 2014 at 10:59:19AM +0200, Vlastimil Babka wrote: > On 06/25/2014 02:53 AM, Joonsoo Kim wrote: > >On Tue, Jun 24, 2014 at 05:42:50PM +0200, Vlastimil Babka wrote: > >>On 06/24/2014 10:33 AM, Joonsoo Kim wrote: > >>>On Fri, Jun 20, 2014 at 05:49:34PM +0200, Vlastimil Babka wrote: >

Re: [PATCH v3 04/13] mm, compaction: move pageblock checks up from isolate_migratepages_range()

2014-06-26 Thread Joonsoo Kim
On Wed, Jun 25, 2014 at 10:59:19AM +0200, Vlastimil Babka wrote: On 06/25/2014 02:53 AM, Joonsoo Kim wrote: On Tue, Jun 24, 2014 at 05:42:50PM +0200, Vlastimil Babka wrote: On 06/24/2014 10:33 AM, Joonsoo Kim wrote: On Fri, Jun 20, 2014 at 05:49:34PM +0200, Vlastimil Babka wrote:

Re: [PATCH v3 04/13] mm, compaction: move pageblock checks up from isolate_migratepages_range()

2014-06-25 Thread Naoya Horiguchi
On Wed, Jun 25, 2014 at 10:50:51AM +0200, Vlastimil Babka wrote: > On 06/24/2014 06:58 PM, Naoya Horiguchi wrote: > >On Tue, Jun 24, 2014 at 05:34:32PM +0200, Vlastimil Babka wrote: > >>On 06/24/2014 06:52 AM, Naoya Horiguchi wrote: > - low_pfn = isolate_migratepages_range(zone, cc, low_pfn,

Re: [PATCH v3 04/13] mm, compaction: move pageblock checks up from isolate_migratepages_range()

2014-06-25 Thread Vlastimil Babka
On 06/25/2014 02:53 AM, Joonsoo Kim wrote: On Tue, Jun 24, 2014 at 05:42:50PM +0200, Vlastimil Babka wrote: On 06/24/2014 10:33 AM, Joonsoo Kim wrote: On Fri, Jun 20, 2014 at 05:49:34PM +0200, Vlastimil Babka wrote: isolate_migratepages_range() is the main function of the compaction scanner,

Re: [PATCH v3 04/13] mm, compaction: move pageblock checks up from isolate_migratepages_range()

2014-06-25 Thread Vlastimil Babka
On 06/24/2014 06:58 PM, Naoya Horiguchi wrote: On Tue, Jun 24, 2014 at 05:34:32PM +0200, Vlastimil Babka wrote: On 06/24/2014 06:52 AM, Naoya Horiguchi wrote: - low_pfn = isolate_migratepages_range(zone, cc, low_pfn, end_pfn, false); - if (!low_pfn || cc->contended) -

Re: [PATCH v3 04/13] mm, compaction: move pageblock checks up from isolate_migratepages_range()

2014-06-25 Thread Vlastimil Babka
On 06/24/2014 06:58 PM, Naoya Horiguchi wrote: On Tue, Jun 24, 2014 at 05:34:32PM +0200, Vlastimil Babka wrote: On 06/24/2014 06:52 AM, Naoya Horiguchi wrote: - low_pfn = isolate_migratepages_range(zone, cc, low_pfn, end_pfn, false); - if (!low_pfn || cc-contended) -

Re: [PATCH v3 04/13] mm, compaction: move pageblock checks up from isolate_migratepages_range()

2014-06-25 Thread Vlastimil Babka
On 06/25/2014 02:53 AM, Joonsoo Kim wrote: On Tue, Jun 24, 2014 at 05:42:50PM +0200, Vlastimil Babka wrote: On 06/24/2014 10:33 AM, Joonsoo Kim wrote: On Fri, Jun 20, 2014 at 05:49:34PM +0200, Vlastimil Babka wrote: isolate_migratepages_range() is the main function of the compaction scanner,

Re: [PATCH v3 04/13] mm, compaction: move pageblock checks up from isolate_migratepages_range()

2014-06-25 Thread Naoya Horiguchi
On Wed, Jun 25, 2014 at 10:50:51AM +0200, Vlastimil Babka wrote: On 06/24/2014 06:58 PM, Naoya Horiguchi wrote: On Tue, Jun 24, 2014 at 05:34:32PM +0200, Vlastimil Babka wrote: On 06/24/2014 06:52 AM, Naoya Horiguchi wrote: - low_pfn = isolate_migratepages_range(zone, cc, low_pfn, end_pfn,

Re: [PATCH v3 04/13] mm, compaction: move pageblock checks up from isolate_migratepages_range()

2014-06-24 Thread Joonsoo Kim
On Tue, Jun 24, 2014 at 05:42:50PM +0200, Vlastimil Babka wrote: > On 06/24/2014 10:33 AM, Joonsoo Kim wrote: > >On Fri, Jun 20, 2014 at 05:49:34PM +0200, Vlastimil Babka wrote: > >>isolate_migratepages_range() is the main function of the compaction scanner, > >>called either on a single pageblock

Re: [PATCH v3 04/13] mm, compaction: move pageblock checks up from isolate_migratepages_range()

2014-06-24 Thread Naoya Horiguchi
On Tue, Jun 24, 2014 at 05:34:32PM +0200, Vlastimil Babka wrote: > On 06/24/2014 06:52 AM, Naoya Horiguchi wrote: > >>- low_pfn = isolate_migratepages_range(zone, cc, low_pfn, end_pfn, false); > >>- if (!low_pfn || cc->contended) > >>- return ISOLATE_ABORT; > >>+ /* Do not

Re: [PATCH v3 04/13] mm, compaction: move pageblock checks up from isolate_migratepages_range()

2014-06-24 Thread Vlastimil Babka
On 06/24/2014 10:33 AM, Joonsoo Kim wrote: On Fri, Jun 20, 2014 at 05:49:34PM +0200, Vlastimil Babka wrote: isolate_migratepages_range() is the main function of the compaction scanner, called either on a single pageblock by isolate_migratepages() during regular compaction, or on an arbitrary

Re: [PATCH v3 04/13] mm, compaction: move pageblock checks up from isolate_migratepages_range()

2014-06-24 Thread Vlastimil Babka
On 06/24/2014 06:52 AM, Naoya Horiguchi wrote: - low_pfn = isolate_migratepages_range(zone, cc, low_pfn, end_pfn, false); - if (!low_pfn || cc->contended) - return ISOLATE_ABORT; + /* Do not scan within a memory hole */ + if

Re: [PATCH v3 04/13] mm, compaction: move pageblock checks up from isolate_migratepages_range()

2014-06-24 Thread Joonsoo Kim
On Fri, Jun 20, 2014 at 05:49:34PM +0200, Vlastimil Babka wrote: > isolate_migratepages_range() is the main function of the compaction scanner, > called either on a single pageblock by isolate_migratepages() during regular > compaction, or on an arbitrary range by CMA's

Re: [PATCH v3 04/13] mm, compaction: move pageblock checks up from isolate_migratepages_range()

2014-06-24 Thread Joonsoo Kim
On Fri, Jun 20, 2014 at 05:49:34PM +0200, Vlastimil Babka wrote: isolate_migratepages_range() is the main function of the compaction scanner, called either on a single pageblock by isolate_migratepages() during regular compaction, or on an arbitrary range by CMA's

Re: [PATCH v3 04/13] mm, compaction: move pageblock checks up from isolate_migratepages_range()

2014-06-24 Thread Vlastimil Babka
On 06/24/2014 06:52 AM, Naoya Horiguchi wrote: - low_pfn = isolate_migratepages_range(zone, cc, low_pfn, end_pfn, false); - if (!low_pfn || cc-contended) - return ISOLATE_ABORT; + /* Do not scan within a memory hole */ + if

Re: [PATCH v3 04/13] mm, compaction: move pageblock checks up from isolate_migratepages_range()

2014-06-24 Thread Vlastimil Babka
On 06/24/2014 10:33 AM, Joonsoo Kim wrote: On Fri, Jun 20, 2014 at 05:49:34PM +0200, Vlastimil Babka wrote: isolate_migratepages_range() is the main function of the compaction scanner, called either on a single pageblock by isolate_migratepages() during regular compaction, or on an arbitrary

Re: [PATCH v3 04/13] mm, compaction: move pageblock checks up from isolate_migratepages_range()

2014-06-24 Thread Naoya Horiguchi
On Tue, Jun 24, 2014 at 05:34:32PM +0200, Vlastimil Babka wrote: On 06/24/2014 06:52 AM, Naoya Horiguchi wrote: - low_pfn = isolate_migratepages_range(zone, cc, low_pfn, end_pfn, false); - if (!low_pfn || cc-contended) - return ISOLATE_ABORT; + /* Do not scan within a

Re: [PATCH v3 04/13] mm, compaction: move pageblock checks up from isolate_migratepages_range()

2014-06-24 Thread Joonsoo Kim
On Tue, Jun 24, 2014 at 05:42:50PM +0200, Vlastimil Babka wrote: On 06/24/2014 10:33 AM, Joonsoo Kim wrote: On Fri, Jun 20, 2014 at 05:49:34PM +0200, Vlastimil Babka wrote: isolate_migratepages_range() is the main function of the compaction scanner, called either on a single pageblock by

Re: [PATCH v3 04/13] mm, compaction: move pageblock checks up from isolate_migratepages_range()

2014-06-23 Thread Naoya Horiguchi
On Fri, Jun 20, 2014 at 05:49:34PM +0200, Vlastimil Babka wrote: > isolate_migratepages_range() is the main function of the compaction scanner, > called either on a single pageblock by isolate_migratepages() during regular > compaction, or on an arbitrary range by CMA's

Re: [PATCH v3 04/13] mm, compaction: move pageblock checks up from isolate_migratepages_range()

2014-06-23 Thread Zhang Yanfei
On 06/20/2014 11:49 PM, Vlastimil Babka wrote: > isolate_migratepages_range() is the main function of the compaction scanner, > called either on a single pageblock by isolate_migratepages() during regular > compaction, or on an arbitrary range by CMA's __alloc_contig_migrate_range(). > It

Re: [PATCH v3 04/13] mm, compaction: move pageblock checks up from isolate_migratepages_range()

2014-06-23 Thread Naoya Horiguchi
On Fri, Jun 20, 2014 at 05:49:34PM +0200, Vlastimil Babka wrote: isolate_migratepages_range() is the main function of the compaction scanner, called either on a single pageblock by isolate_migratepages() during regular compaction, or on an arbitrary range by CMA's

Re: [PATCH v3 04/13] mm, compaction: move pageblock checks up from isolate_migratepages_range()

2014-06-23 Thread Zhang Yanfei
On 06/20/2014 11:49 PM, Vlastimil Babka wrote: isolate_migratepages_range() is the main function of the compaction scanner, called either on a single pageblock by isolate_migratepages() during regular compaction, or on an arbitrary range by CMA's __alloc_contig_migrate_range(). It currently

[PATCH v3 04/13] mm, compaction: move pageblock checks up from isolate_migratepages_range()

2014-06-20 Thread Vlastimil Babka
isolate_migratepages_range() is the main function of the compaction scanner, called either on a single pageblock by isolate_migratepages() during regular compaction, or on an arbitrary range by CMA's __alloc_contig_migrate_range(). It currently perfoms two pageblock-wide compaction suitability

[PATCH v3 04/13] mm, compaction: move pageblock checks up from isolate_migratepages_range()

2014-06-20 Thread Vlastimil Babka
isolate_migratepages_range() is the main function of the compaction scanner, called either on a single pageblock by isolate_migratepages() during regular compaction, or on an arbitrary range by CMA's __alloc_contig_migrate_range(). It currently perfoms two pageblock-wide compaction suitability