Re: [PATCH 05/12] mm: alloc_contig_range() added

From: Dave Hansen
Date: Thu Mar 31 2011 - 12:27:22 EST


On Thu, 2011-03-31 at 15:16 +0200, Marek Szyprowski wrote:
> + _start = start & (~0UL << ret);
> + _end = alloc_contig_freed_pages(_start, end, flag);

These names are a wee bit lacking. Care to give them proper names that
might let a reader figure out how the "_" makes the variable different
from its nearly-identical twin?

-- Dave

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/