[PATCH v1 0/2] mm: remove MIGRATEPAGE_*

[Huang, Ying]

Hi, David,

David Hildenbrand <david at redhat.com> 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 at linux-foundation.org>
> Cc: Madhavan Srinivasan <maddy at linux.ibm.com>
> Cc: Michael Ellerman <mpe at ellerman.id.au>
> Cc: Nicholas Piggin <npiggin at gmail.com>
> Cc: Christophe Leroy <christophe.leroy at csgroup.eu>
> Cc: Jerrin Shaji George <jerrin.shaji-george at broadcom.com>
> Cc: Arnd Bergmann <arnd at arndb.de>
> Cc: Greg Kroah-Hartman <gregkh at linuxfoundation.org>
> Cc: "Michael S. Tsirkin" <mst at redhat.com>
> Cc: Jason Wang <jasowang at redhat.com>
> Cc: Xuan Zhuo <xuanzhuo at linux.alibaba.com>
> Cc: "Eugenio Pérez" <eperezma at redhat.com>
> Cc: Alexander Viro <viro at zeniv.linux.org.uk>
> Cc: Christian Brauner <brauner at kernel.org>
> Cc: Jan Kara <jack at suse.cz>
> Cc: Benjamin LaHaise <bcrl at kvack.org>
> Cc: Chris Mason <clm at fb.com>
> Cc: Josef Bacik <josef at toxicpanda.com>
> Cc: David Sterba <dsterba at suse.com>
> Cc: Muchun Song <muchun.song at linux.dev>
> Cc: Oscar Salvador <osalvador at suse.de>
> Cc: Dave Kleikamp <shaggy at kernel.org>
> Cc: Zi Yan <ziy at nvidia.com>
> Cc: Matthew Brost <matthew.brost at intel.com>
> Cc: Joshua Hahn <joshua.hahnjy at gmail.com>
> Cc: Rakie Kim <rakie.kim at sk.com>
> Cc: Byungchul Park <byungchul at sk.com>
> Cc: Gregory Price <gourry at gourry.net>
> Cc: Ying Huang <ying.huang at linux.alibaba.com>
> Cc: Alistair Popple <apopple at nvidia.com>
> Cc: Minchan Kim <minchan at kernel.org>
> Cc: Sergey Senozhatsky <senozhatsky at chromium.org>
>
> David Hildenbrand (2):
>   mm/migrate: remove MIGRATEPAGE_UNMAP
>   treewide: remove MIGRATEPAGE_SUCCESS

LGTM.  Feel free to add my

Reviewed-by: Huang Ying <ying.huang at linux.alibaba.com>

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