Re: [PATCH 11/13] mm: meminit: Free pages in large chunks where possible

From: Mel Gorman
Date: Tue Apr 28 2015 - 07:38:29 EST

On Mon, Apr 27, 2015 at 03:43:56PM -0700, Andrew Morton wrote:
> On Thu, 23 Apr 2015 11:33:14 +0100 Mel Gorman <mgorman@xxxxxxx> wrote:
> > Parallel struct page frees pages one at a time. Try free pages as single
> > large pages where possible.
> >
> > ...
> >
> > void __defermem_init deferred_init_memmap(int nid)
> This function is gruesome in an 80-col display. Even the code comments
> wrap, which is nuts. Maybe hoist the contents of the outermost loop
> into a separate function, called for each zone?

I can do better than that because only the highest zone is deferred
in this version and the loop is no longer necessary. I should post a V4
before the end of my day that addresses your feedback. It caused a lot of
conflicts and it'll be easier to replace the full series than try managing
incremental fixes.

Thanks Andrew.

Mel Gorman
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at
Please read the FAQ at