>> + vfrom = kmap_local_page(from); >> + vto = kmap_local_page(to); >> + ret = copy_mc_to_kernel(vto, vfrom, PAGE_SIZE); > > In copy_user_highpage(), kmsan_unpoison_memory(page_address(to), PAGE_SIZE) is done after the copy when > __HAVE_ARCH_COPY_USER_HIGHPAGE isn't defined. Do we need to do something similar here? But I'm not familiar > with kmsan, so I can easy be wrong. It looks like that kmsan_unpoison_memory() call was added recently, after I copied copy_user_highpage() to create copy_mc_user_highpage(). I'm not familiar with kmsan either. Adding Alexander to this thread since they added that code. > Anyway, this patch looks good to me. Thanks. > > Reviewed-by: Miaohe Lin <linmiaohe@xxxxxxxxxx> Thanks for the review. -Tony
