linux-kernel - Re: [PATCH] mm: mempolicy: don't have to split pmd for huge zero page

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <CAHbLzkrQdt+gvq-SdQoH8sVVNiSxdMqreTGsrZC4AsVpYzysWQ@mail.gmail.com>
Date:   Tue, 8 Jun 2021 12:36:05 -0700
From:   Yang Shi <shy828301@...il.com>
To:     Michal Hocko <mhocko@...e.com>
Cc:     Zi Yan <ziy@...dia.com>, nao.horiguchi@...il.com,
        "Kirill A. Shutemov" <kirill.shutemov@...ux.intel.com>,
        Hugh Dickins <hughd@...gle.com>,
        Andrew Morton <akpm@...ux-foundation.org>,
        Linux MM <linux-mm@...ck.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] mm: mempolicy: don't have to split pmd for huge zero page

On Tue, Jun 8, 2021 at 10:49 AM Michal Hocko <mhocko@...e.com> wrote:
>
> On Tue 08-06-21 10:15:36, Yang Shi wrote:
> [...]
> > I did some archeology, the findings are:
> >
> > The zero page has PageReserved flag set, it was skipped by the
> > explicit PageReserved check in mempolicy.c since commit f4598c8b3678
> > ("[PATCH] migration: make sure there is no attempt to migrate reserved
> > pages."). The zero page was not used anymore by do_anonymous_page()
> > since 2.6.24 by commit 557ed1fa2620 ("remove ZERO_PAGE"), then
> > reinstated by commit a13ea5b759645 ("mm: reinstate ZERO_PAGE") and
> > this commit added zero page check in vm_normal_page(), so mempolicy
> > doesn't depend on PageReserved check to skip zero page anymore since
> > then.
> >
> > So the zero page is skipped by mempolicy.c since 2.6.16.
>
> Thanks a lot! This is really useful. Can you just add it to the
> changelog so others do not have to go through the painful archeology.
>
> With that, feel free to add
> Acked-by: Michal Hocko <mhocko@...e.com>

Thanks. Will add that into v2.

>
> Thanls!
> --
> Michal Hocko
> SUSE Labs