Hi, David, David Hildenbrand <david@xxxxxxxxxx> writes: > This is against mm/mm-new. > > This series gets rid of MIGRATEPAGE_UNMAP, to then convert the remaining > MIGRATEPAGE_SUCCESS usage to simply use 0 instead. > > Not sure if it makes sense to split the second patch up, a treewide > cleanup felt more reasonable for this simple change in an area where > I don't expect a lot of churn. > > Briefly tested with virtio-mem in a VM, making sure that basic > page migration keeps working. > > Cc: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> > Cc: Madhavan Srinivasan <maddy@xxxxxxxxxxxxx> > Cc: Michael Ellerman <mpe@xxxxxxxxxxxxxx> > Cc: Nicholas Piggin <npiggin@xxxxxxxxx> > Cc: Christophe Leroy <christophe.leroy@xxxxxxxxxx> > Cc: Jerrin Shaji George <jerrin.shaji-george@xxxxxxxxxxxx> > Cc: Arnd Bergmann <arnd@xxxxxxxx> > Cc: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> > Cc: "Michael S. Tsirkin" <mst@xxxxxxxxxx> > Cc: Jason Wang <jasowang@xxxxxxxxxx> > Cc: Xuan Zhuo <xuanzhuo@xxxxxxxxxxxxxxxxx> > Cc: "Eugenio Pérez" <eperezma@xxxxxxxxxx> > Cc: Alexander Viro <viro@xxxxxxxxxxxxxxxxxx> > Cc: Christian Brauner <brauner@xxxxxxxxxx> > Cc: Jan Kara <jack@xxxxxxx> > Cc: Benjamin LaHaise <bcrl@xxxxxxxxx> > Cc: Chris Mason <clm@xxxxxx> > Cc: Josef Bacik <josef@xxxxxxxxxxxxxx> > Cc: David Sterba <dsterba@xxxxxxxx> > Cc: Muchun Song <muchun.song@xxxxxxxxx> > Cc: Oscar Salvador <osalvador@xxxxxxx> > Cc: Dave Kleikamp <shaggy@xxxxxxxxxx> > Cc: Zi Yan <ziy@xxxxxxxxxx> > Cc: Matthew Brost <matthew.brost@xxxxxxxxx> > Cc: Joshua Hahn <joshua.hahnjy@xxxxxxxxx> > Cc: Rakie Kim <rakie.kim@xxxxxx> > Cc: Byungchul Park <byungchul@xxxxxx> > Cc: Gregory Price <gourry@xxxxxxxxxx> > Cc: Ying Huang <ying.huang@xxxxxxxxxxxxxxxxx> > Cc: Alistair Popple <apopple@xxxxxxxxxx> > Cc: Minchan Kim <minchan@xxxxxxxxxx> > Cc: Sergey Senozhatsky <senozhatsky@xxxxxxxxxxxx> > > David Hildenbrand (2): > mm/migrate: remove MIGRATEPAGE_UNMAP > treewide: remove MIGRATEPAGE_SUCCESS LGTM. Feel free to add my Reviewed-by: Huang Ying <ying.huang@xxxxxxxxxxxxxxxxx> for the whole series in the future versions. > arch/powerpc/platforms/pseries/cmm.c | 2 +- > drivers/misc/vmw_balloon.c | 4 +- > drivers/virtio/virtio_balloon.c | 2 +- > fs/aio.c | 2 +- > fs/btrfs/inode.c | 4 +- > fs/hugetlbfs/inode.c | 4 +- > fs/jfs/jfs_metapage.c | 8 +-- > include/linux/migrate.h | 11 +--- > mm/migrate.c | 80 ++++++++++++++-------------- > mm/migrate_device.c | 2 +- > mm/zsmalloc.c | 4 +- > 11 files changed, 56 insertions(+), 67 deletions(-) > > > base-commit: 53c448023185717d0ed56b5546dc2be405da92ff --- Best Regards, Huang, Ying