Re: [RFC] NUMA memory policy support for HUGE pages

[Christoph Lameter <clameter@engr.sgi.com>]

On Mon, 14 Nov 2005, Adam Litke wrote:

> On Mon, 2005-11-14 at 13:46 -0800, Christoph Lameter wrote:
> > This is V2 of the patch.
> > 
> > Changes:
> > 
> > - Cleaned up by folding find_or_alloc() into hugetlb_no_page().
> 
> IMHO this is not really a cleanup.  When the demand fault patch stack
> was first accepted, we decided to separate out find_or_alloc_huge_page()
> because it has the page_cache retry loop with several exit conditions.
> no_page() has its own backout logic and mixing the two makes for a
> tangled mess.  Can we leave that hunk out please?

It seemed to me that find_or_alloc_huge_pages has a pretty simple backout 
logic that folds nicely into no_page(). Both functions share a lot of 
variables and putting them together not only increases the readability of 
the code but also makes the function smaller and execution more efficient.

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>