Re: [PATCH v9 1/2] mm/khugepaged: recover from poisoned anonymous memory

[Jiaqi Yan <jiaqiyan@google.com>]

Pinging KMSAN experts, for the general guidance of
kmsan_copy_page_meta vs kmsan_unpoison_memory

On Wed, Feb 1, 2023 at 4:30 PM <kirill@shutemov.name> wrote:
>
> On Thu, Feb 02, 2023 at 03:01:02AM +0300, kirill.shutemov@linux.intel.com wrote:
> > On Tue, Jan 31, 2023 at 09:16:45PM -0800, Jiaqi Yan wrote:
> > > > > > > +/*
> > > > > > > + * Copies memory with #MC in source page (@from) handled. Returns number
> > > > > > > + * of bytes not copied if there was an exception; otherwise 0 for success.
> > > > > > > + * Note handling #MC requires arch opt-in.
> > > > > > > + */
> > > > > > > +static int copy_mc_page(struct page *to, struct page *from)
> > > > > > > +{
> > > > > > > +     char *vfrom, *vto;
> > > > > > > +     unsigned long ret;
> > > > > > > +
> > > > > > > +     vfrom = kmap_local_page(from);
> > > > > > > +     vto = kmap_local_page(to);
> > > > > > > +     ret = copy_mc_to_kernel(vto, vfrom, PAGE_SIZE);
> > > > > > > +     if (ret == 0)
> > > > > > > +             kmsan_copy_page_meta(to, from);
> > > > > > > +     kunmap_local(vto);
> > > > > > > +     kunmap_local(vfrom);
> > > > > > > +
> > > > > > > +     return ret;
> > > > > > > +}
> > > > > >
> > > > > >
> > > > > > It is very similar to copy_mc_user_highpage(), but uses
> > > > > > kmsan_copy_page_meta() instead of kmsan_unpoison_memory().
> > > > > >
> > > > > > Could you explain the difference? I don't quite get it.
> > > > >
> > > > > copy_mc_page is actually the MC version of copy_highpage, which uses
> > > > > kmsan_copy_page_meta instead of kmsan_unpoison_memory.
> > > > >
> > > > > My understanding is kmsan_copy_page_meta covers kmsan_unpoison_memory.
> > > > > When there is no metadata (kmsan_shadow or kmsan_origin), both
> > > > > kmsan_copy_page_meta and kmsan_unpoison_memory just do
> > > > > kmsan_internal_unpoison_memory to mark the memory range as
> > > > > initialized; when there is metadata in src page, kmsan_copy_page_meta
> > > > > will copy whatever metadata in src to dst. So I think
> > > > > kmsan_copy_page_meta is the right thing to do.
> > > >
> > > > Should we fix copy_mc_user_highpage() then?
> > >
> > > I think it depends on what copy_user_highpage() (the original of
> > > copy_mc_user_highpage) is used for. copy_mc_user_highpage is currently
> > > only used by __wp_page_copy_user, is it possible that here we don't
> > > want to (or don't need to) copy page metadata for userspace pages?
> >
> > Tony, could chime in on this? Can we modify copy_mc_user_highpage() to
> > also use kmsan_copy_page_meta()? I don't really understand KMSAN here.
>
> +KMSAN folks.
>
> --
>   Kiryl Shutsemau / Kirill A. Shutemov