> > I have one concern for now. Reading the patch, the flag have influence on > > only anonymous pages, so we'd better to note it and I feel it strange to > > set(and clear) the flag of "old page" always(iow, even when !PageAnon) > > in prepare_migration. > > > > Hmm...Checking "Only Anon" is simpler ? I just thought it was inconsistent that we always set/clear the bit about "old page", while we set the bit about "new page" only in PageAnon case. > It will have no meanings for migrating > file caches, but it may have some meanings for easy debugging. > I think "mark it always but it's used only for anonymous page" is reasonable > (if it causes no bug.) > Anyway, I don't have any strong objection. It's all right for me as long as it is well documented or commented. Thanks, Daisuke Nishimura. -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxxx For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>