Re: [PATCH 1/3] fixup! mm: alloc_contig_freed_pages() added

2011-09-21 Thread Dave Hansen
On Wed, 2011-09-21 at 18:26 +0200, Michal Nazarewicz wrote: > - page += 1 << order; > + > + if (zone_pfn_same_memmap(pfn - count, pfn)) > + page += count; > + else > + page = pfn_to_page(pfn); > } That al

[PATCH 1/3] fixup! mm: alloc_contig_freed_pages() added

2011-09-21 Thread Michal Nazarewicz
From: Michal Nazarewicz Signed-off-by: Michal Nazarewicz --- include/asm-generic/memory_model.h | 17 ++ include/linux/page-isolation.h |4 ++- mm/page_alloc.c| 43 +++ 3 files changed, 53 insertions(+), 11 deletions(-

Re: [PATCH 1/3] fixup! mm: alloc_contig_freed_pages() added

2011-09-21 Thread Dave Hansen
On Wed, 2011-09-21 at 17:19 +0200, Michal Nazarewicz wrote: > Do the attached changes seem to make sense? The logic looks OK. > I wanted to avoid calling pfn_to_page() each time as it seem fairly > expensive in sparsemem and disctontig modes. At the same time, the > macro trickery is so that use

[PATCH 1/3] fixup! mm: alloc_contig_freed_pages() added

2011-09-21 Thread Michal Nazarewicz
From: Michal Nazarewicz Signed-off-by: Michal Nazarewicz --- include/linux/page-isolation.h |4 ++- mm/page_alloc.c| 66 ++- 2 files changed, 60 insertions(+), 10 deletions(-) > On Fri, 2011-08-19 at 16:27 +0200, Marek Szyprowski wrote