Re: [PATCH 1/2] mm/compaction: fix invalid free_pfn and compact_cached_free_pfn

2015-12-22 Thread David Rientjes
On Mon, 21 Dec 2015, Joonsoo Kim wrote: > free_pfn and compact_cached_free_pfn are the pointer that remember > restart position of freepage scanner. When they are reset or invalid, > we set them to zone_end_pfn because freepage scanner works in reverse > direction. But, because zone range is defin

[PATCH 1/2] mm/compaction: fix invalid free_pfn and compact_cached_free_pfn

2015-12-20 Thread Joonsoo Kim
free_pfn and compact_cached_free_pfn are the pointer that remember restart position of freepage scanner. When they are reset or invalid, we set them to zone_end_pfn because freepage scanner works in reverse direction. But, because zone range is defined as [zone_start_pfn, zone_end_pfn), zone_end_pf

Re: [PATCH 1/2] mm/compaction: fix invalid free_pfn and compact_cached_free_pfn

2015-12-15 Thread Joonsoo Kim
On Tue, Dec 15, 2015 at 09:31:39AM +0100, Vlastimil Babka wrote: > On 12/14/2015 04:26 PM, Joonsoo Kim wrote: > >2015-12-14 19:07 GMT+09:00 Vlastimil Babka : > >>On 12/14/2015 06:02 AM, Joonsoo Kim wrote: > >>> > >> > >>Acked-by: Vlastimil Babka > >> > >>Note that until now in compaction we've use

Re: [PATCH 1/2] mm/compaction: fix invalid free_pfn and compact_cached_free_pfn

2015-12-15 Thread Vlastimil Babka
On 12/14/2015 04:26 PM, Joonsoo Kim wrote: 2015-12-14 19:07 GMT+09:00 Vlastimil Babka : On 12/14/2015 06:02 AM, Joonsoo Kim wrote: Acked-by: Vlastimil Babka Note that until now in compaction we've used basically an open-coded round_down(), and ALIGN() for rounding up. You introduce a first

Re: [PATCH 1/2] mm/compaction: fix invalid free_pfn and compact_cached_free_pfn

2015-12-14 Thread Joonsoo Kim
2015-12-14 19:07 GMT+09:00 Vlastimil Babka : > On 12/14/2015 06:02 AM, Joonsoo Kim wrote: >> >> free_pfn and compact_cached_free_pfn are the pointer that remember >> restart position of freepage scanner. When they are reset or invalid, >> we set them to zone_end_pfn because freepage scanner works i

Re: [PATCH 1/2] mm/compaction: fix invalid free_pfn and compact_cached_free_pfn

2015-12-14 Thread Vlastimil Babka
On 12/14/2015 06:02 AM, Joonsoo Kim wrote: free_pfn and compact_cached_free_pfn are the pointer that remember restart position of freepage scanner. When they are reset or invalid, we set them to zone_end_pfn because freepage scanner works in reverse direction. But, because zone range is defined a

[PATCH 1/2] mm/compaction: fix invalid free_pfn and compact_cached_free_pfn

2015-12-13 Thread Joonsoo Kim
free_pfn and compact_cached_free_pfn are the pointer that remember restart position of freepage scanner. When they are reset or invalid, we set them to zone_end_pfn because freepage scanner works in reverse direction. But, because zone range is defined as [zone_start_pfn, zone_end_pfn), zone_end_pf