linux-kernel - Re: [PATCH v6 0/1] mm: report per-page metadata information

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <2023120645-survey-puppet-4ae0@gregkh>
Date:   Wed, 6 Dec 2023 12:12:10 +0900
From:   Greg KH <gregkh@...uxfoundation.org>
To:     Pasha Tatashin <pasha.tatashin@...een.com>
Cc:     Sourav Panda <souravpanda@...gle.com>, corbet@....net,
        rafael@...nel.org, akpm@...ux-foundation.org,
        mike.kravetz@...cle.com, muchun.song@...ux.dev, rppt@...nel.org,
        david@...hat.com, rdunlap@...radead.org, chenlinxuan@...ontech.com,
        yang.yang29@....com.cn, tomas.mudrunka@...il.com,
        bhelgaas@...gle.com, ivan@...udflare.com, yosryahmed@...gle.com,
        hannes@...xchg.org, shakeelb@...gle.com,
        kirill.shutemov@...ux.intel.com, wangkefeng.wang@...wei.com,
        adobriyan@...il.com, vbabka@...e.cz, Liam.Howlett@...cle.com,
        surenb@...gle.com, linux-kernel@...r.kernel.org,
        linux-fsdevel@...r.kernel.org, linux-doc@...r.kernel.org,
        linux-mm@...ck.org, willy@...radead.org, weixugc@...gle.com
Subject: Re: [PATCH v6 0/1] mm: report per-page metadata information

On Tue, Dec 05, 2023 at 09:57:36PM -0500, Pasha Tatashin wrote:
> Hi Greg,
> 
> Sourav removed the new field from sys/device../nodeN/meminfo as you
> requested; however, in nodeN/vmstat fields still get appended, as
> there is code that displays every item in zone_stat_item,
> node_stat_item without option to opt-out. I mentioned it to you at
> LPC.

Sorry, I thought that was a proc file, not a sysfs file.  Don't grow
that file please, it should not be that way and adding to it will just
cause others to also want to add to it and we end up with the whole proc
file mess...

> In my IOMMU [1] series, there are also fields that are added to
> node_stat_item that as result are appended to nodeN/vmstat.

I missed that, that too shouldn't be done please.

Again, sysfs is "one value per file" for a reason, don't abuse the fact
that we missed this abuse of the rules for that file by adding more
things to it.

thanks,

greg k-h