On Tue, Apr 15, 2014 at 11:18:27AM +0200, Vlastimil Babka wrote: > isolate_freepages() is currently somewhat hard to follow thanks to many > different pfn variables. Especially misleading is the name 'high_pfn' which > looks like it is related to the 'low_pfn' variable, but in fact it is not. > > This patch renames the 'high_pfn' variable to a hopefully less confusing name, > and slightly changes its handling without a functional change. A comment made > obsolete by recent changes is also updated. > > Signed-off-by: Vlastimil Babka <vbabka@xxxxxxx> > Cc: Minchan Kim <minchan@xxxxxxxxxx> > Cc: Mel Gorman <mgorman@xxxxxxx> > Cc: Joonsoo Kim <iamjoonsoo.kim@xxxxxxx> > Cc: Bartlomiej Zolnierkiewicz <b.zolnierkie@xxxxxxxxxxx> > Cc: Michal Nazarewicz <mina86@xxxxxxxxxx> > Cc: Naoya Horiguchi <n-horiguchi@xxxxxxxxxxxxx> > Cc: Christoph Lameter <cl@xxxxxxxxx> > Cc: Rik van Riel <riel@xxxxxxxxxx> > --- > mm/compaction.c | 17 ++++++++--------- > 1 file changed, 8 insertions(+), 9 deletions(-) Acked-by: Joonsoo Kim <iamjoonsoo.kim@xxxxxxx> -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>