On Sun, Oct 23, 2011 at 09:05:05PM -0700, Michal Nazarewicz wrote:
On Thu, Oct 06, 2011 at 03:54:42PM +0200, Marek Szyprowski wrote:
This commit introduces alloc_contig_freed_pages() function
which allocates (ie. removes from buddy system) free pages
in range. Caller has to guarantee that
On Tue, 01 Nov 2011 16:04:48 +0100, Mel Gorman m...@csn.ul.ie wrote:
For the purposes of review, have a separate patch for moving
isolate_freepages_block to another file that does not alter the
function in any way. When the function is updated in a follow-on patch,
it'll be far easier to see
On Tue, Nov 01, 2011 at 07:06:56PM +0100, Michal Nazarewicz wrote:
page_isolation.c may also be a better fit than page_alloc.c
Since isolate_freepages_block() is the only user of split_free_page(),
would it make sense to move split_free_page() to page_isolation.c as
well? I sort of like the
On Fri, 21 Oct 2011 03:06:24 -0700, Mel Gorman m...@csn.ul.ie wrote:
On Tue, Oct 18, 2011 at 10:26:37AM -0700, Michal Nazarewicz wrote:
On Tue, 18 Oct 2011 05:21:09 -0700, Mel Gorman m...@csn.ul.ie wrote:
At this point, I'm going to apologise for not reviewing this a long long
time ago.
On
On Tue, Oct 18, 2011 at 10:26:37AM -0700, Michal Nazarewicz wrote:
On Tue, 18 Oct 2011 05:21:09 -0700, Mel Gorman m...@csn.ul.ie wrote:
At this point, I'm going to apologise for not reviewing this a long long
time ago.
On Thu, Oct 06, 2011 at 03:54:42PM +0200, Marek Szyprowski wrote:
At this point, I'm going to apologise for not reviewing this a long long
time ago.
On Thu, Oct 06, 2011 at 03:54:42PM +0200, Marek Szyprowski wrote:
From: KAMEZAWA Hiroyuki kamezawa.hir...@jp.fujitsu.com
This commit introduces alloc_contig_freed_pages() function
which allocates (ie. removes
On Tue, 18 Oct 2011 05:21:09 -0700, Mel Gorman m...@csn.ul.ie wrote:
At this point, I'm going to apologise for not reviewing this a long long
time ago.
On Thu, Oct 06, 2011 at 03:54:42PM +0200, Marek Szyprowski wrote:
From: KAMEZAWA Hiroyuki kamezawa.hir...@jp.fujitsu.com
This commit
On Tue, 2011-10-18 at 10:26 -0700, Michal Nazarewicz wrote:
You can do this in a more general fashion by checking the
zone boundaries and resolving the pfn-page every MAX_ORDER_NR_PAGES.
That will not be SPARSEMEM specific.
I've tried doing stuff that way but it ended up with much more
On Tue, 18 Oct 2011 10:48:46 -0700, Dave Hansen d...@linux.vnet.ibm.com wrote:
On Tue, 2011-10-18 at 10:26 -0700, Michal Nazarewicz wrote:
You can do this in a more general fashion by checking the
zone boundaries and resolving the pfn-page every MAX_ORDER_NR_PAGES.
That will not be
Hello Andrew,
Thanks for your comments. I will try to address them in the next round of
CMA patches.
On Saturday, October 15, 2011 1:30 AM Andrew Morton wrote:
(snipped)
+
+void free_contig_pages(unsigned long pfn, unsigned nr_pages)
+{
+ struct page *page = pfn_to_page(pfn);
+
+
On Mon, 17 Oct 2011 14:21:07 +0200
Marek Szyprowski m.szyprow...@samsung.com wrote:
+
+void free_contig_pages(unsigned long pfn, unsigned nr_pages)
+{
+ struct page *page = pfn_to_page(pfn);
+
+ while (nr_pages--) {
+ __free_page(page);
+ ++pfn;
+
On Sat, 15 Oct 2011 01:29:33 +0200, Andrew Morton a...@linux-foundation.org
wrote:
On Thu, 06 Oct 2011 15:54:42 +0200
Marek Szyprowski m.szyprow...@samsung.com wrote:
From: KAMEZAWA Hiroyuki kamezawa.hir...@jp.fujitsu.com
This commit introduces alloc_contig_freed_pages() function
The
On Sun, 16 Oct 2011 10:01:36 +0200 Michal Nazarewicz min...@mina86.com
wrote:
Still, as I think of it now, maybe alloc_contig_free_range() would be
better?
Nope. Of *course* the pages were free. Otherwise we couldn't
(re)allocate them. I still think the free part is redundant.
What could
On Sun, 16 Oct 2011 10:01:36 +0200 Michal Nazarewicz wrote:
Still, as I think of it now, maybe alloc_contig_free_range() would be
better?
On Sun, 16 Oct 2011 10:31:16 +0200, Andrew Morton wrote:
Nope. Of *course* the pages were free. Otherwise we couldn't
(re)allocate them. I still think
On Thu, 06 Oct 2011 15:54:42 +0200
Marek Szyprowski m.szyprow...@samsung.com wrote:
From: KAMEZAWA Hiroyuki kamezawa.hir...@jp.fujitsu.com
This commit introduces alloc_contig_freed_pages() function
The freed seems redundant to me. Wouldn't alloc_contig_pages be a
better name?
which
From: KAMEZAWA Hiroyuki kamezawa.hir...@jp.fujitsu.com
This commit introduces alloc_contig_freed_pages() function
which allocates (ie. removes from buddy system) free pages
in range. Caller has to guarantee that all pages in range
are in buddy system.
Along with this function, a
From: KAMEZAWA Hiroyuki kamezawa.hir...@jp.fujitsu.com
This commit introduces alloc_contig_freed_pages() function
which allocates (ie. removes from buddy system) free pages
in range. Caller has to guarantee that all pages in range
are in buddy system.
Along with this function, a
17 matches
Mail list logo