Re: [PATCH] CMA: use MIGRATE_SYNC in alloc_contig_range()

From: David Rientjes
Date: Thu May 29 2014 - 15:32:27 EST


On Thu, 29 May 2014, Joonsoo Kim wrote:

> Before commit 'mm, compaction: embed migration mode in compact_control'
> from David is merged, alloc_contig_range() used sync migration,
> instead of sync_light migration. This doesn't break anything currently
> because page isolation doesn't have any difference with sync and
> sync_light, but it could in the future, so change back as it was.
>
> And pass cc->mode to migrate_pages(), instead of passing MIGRATE_SYNC
> to migrate_pages().
>
> Signed-off-by: Joonsoo Kim <iamjoonsoo.kim@xxxxxxx>

Acked-by: David Rientjes <rientjes@xxxxxxxxxx>

Should probably be renamed
mm-compaction-embed-migration-mode-in-compact_control-fix-fix though since
it's based on another patch in -mm that properly does the s/sync/mode/
conversion for CMA.
--
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/