Nit. I would like to add following hunk. This is just nit because moving reserve pageblock is extreme rare. if (block_migratetype == MIGRATE_RESERVE) { + found++; set_pageblock_migratetype(page, MIGRATE_MOVABLE); move_freepages_block(zone, page, MIGRATE_MOVABLE); }I don't really see the advantage but if you think it is necessary then I do not object either.
For example, a zone has five pageblock b1,b2,b3,b4,b5 and b1 has MIGRATE_RESERVE. When hotremove b1 and hotadd again, your code need to scan all of blocks. But mine only need to scan b1 and b2. I mean that's a hotplug specific optimization. -- 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>