On 03/14/23 13:13, Peng Zhang wrote: > From: ZhangPeng <zhangpeng362@xxxxxxxxxx> > > Change copy_huge_page_from_user() to copy_folio_from_user(). Any reason why you did not change copy_user_huge_page to folios as well? All callers are passing &folio->page. Just my opinion, but it might be better/easier to review if the copy routines were done in a separate patch. There is a little more than folio conversion happening there. -- Mike Kravetz > Convert hugetlb_mcopy_atomic_pte() and __mcopy_atomic_hugetlb() to use a > folio. > > Signed-off-by: ZhangPeng <zhangpeng362@xxxxxxxxxx> > --- > include/linux/hugetlb.h | 4 ++-- > include/linux/mm.h | 3 +-- > mm/hugetlb.c | 25 ++++++++++++------------- > mm/memory.c | 27 ++++++++++++--------------- > mm/userfaultfd.c | 20 +++++++++----------- > 5 files changed, 36 insertions(+), 43 deletions(-)
