linux-kernel - Re: [PATCH v3 2/2] proc: Add getting pages info of ZONE

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 for Android: free password hash cracker in your pocket

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

Message-ID: <YeFOBWz1OZTpEuwt@dhcp22.suse.cz>
Date:   Fri, 14 Jan 2022 11:18:45 +0100
From:   Michal Hocko <mhocko@...e.com>
To:     Xiongwei Song <sxwjean@...il.com>
Cc:     Xiongwei Song <sxwjean@...com>, akpm@...ux-foundation.org,
        David Hildenbrand <david@...hat.com>, dan.j.williams@...el.com,
        osalvador@...e.de, naoya.horiguchi@....com,
        thunder.leizhen@...wei.com, linux-mm@...ck.org,
        linux-fsdevel@...r.kernel.org,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v3 2/2] proc: Add getting pages info of ZONE_DEVICE
 support

On Fri 14-01-22 18:03:04, Xiongwei Song wrote:
> HI Michal,
> 
> On Thu, Jan 13, 2022 at 11:31 PM Michal Hocko <mhocko@...e.com> wrote:
> >
> > On Wed 12-01-22 22:35:17, sxwjean@...com wrote:
> > > From: Xiongwei Song <sxwjean@...il.com>
> > >
> > > When requesting pages info by /proc/kpage*, the pages in ZONE_DEVICE were
> > > ignored.
> > >
> > > The pfn_to_devmap_page() function can help to get page that belongs to
> > > ZONE_DEVICE.
> >
> > Why is this needed? Who would consume that information and what for?
> 
> It's for debug purpose, which checks page flags in system wide. No any other
> special thought. But it looks like it's not appropriate to expose now from my
> understand, which is from David's comment.
> https://lore.kernel.org/linux-mm/20220110141957.259022-1-sxwjean@me.com/T/#m4eccbb2698dbebc80ec00be47382b34b0f64b4fc

yes, I do agree with David. This is the reason I am asking because I do
remember we have deliberately excluded those pages. If there is no real
user to use that information then I do not think we want to make the
code more complex and check for memmap and other peculiarities.

Thanks!
-- 
Michal Hocko
SUSE Labs