From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 1C237C4167B for ; Thu, 7 Dec 2023 07:31:52 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 671A66B0075; Thu, 7 Dec 2023 02:31:51 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 621DB6B0078; Thu, 7 Dec 2023 02:31:51 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 4E97F6B007B; Thu, 7 Dec 2023 02:31:51 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0012.hostedemail.com [216.40.44.12]) by kanga.kvack.org (Postfix) with ESMTP id 3B8FC6B0075 for ; Thu, 7 Dec 2023 02:31:51 -0500 (EST) Received: from smtpin29.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay06.hostedemail.com (Postfix) with ESMTP id 15916A0254 for ; Thu, 7 Dec 2023 07:31:51 +0000 (UTC) X-FDA: 81539202822.29.6F532BD Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by imf10.hostedemail.com (Postfix) with ESMTP id 34CA3C0009 for ; Thu, 7 Dec 2023 07:31:49 +0000 (UTC) Authentication-Results: imf10.hostedemail.com; dkim=pass header.d=linuxfoundation.org header.s=korg header.b=DfhrW129; dmarc=pass (policy=none) header.from=linuxfoundation.org; spf=pass (imf10.hostedemail.com: domain of gregkh@linuxfoundation.org designates 139.178.84.217 as permitted sender) smtp.mailfrom=gregkh@linuxfoundation.org ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1701934309; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=+xHl7ZmW/VH0P2R9EK5nOpVWx3EwTWrA/F3JK29N79c=; b=245rIran+eXTOHx/yMEU8ffhE1N2F359y+a0tKH2A8z1FkIptt4FVCCpw02OMUrk5SP1C/ n4KzpnsqeXv/tN4XS5D7ub0Q9TipOdlpv2eh8IgiB6E1E6to81wBAy6e6y+LSZ/5sLuPKn mj6cs5VPNSh9uYgaE2su6xcnUblOisk= ARC-Authentication-Results: i=1; imf10.hostedemail.com; dkim=pass header.d=linuxfoundation.org header.s=korg header.b=DfhrW129; dmarc=pass (policy=none) header.from=linuxfoundation.org; spf=pass (imf10.hostedemail.com: domain of gregkh@linuxfoundation.org designates 139.178.84.217 as permitted sender) smtp.mailfrom=gregkh@linuxfoundation.org ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1701934309; a=rsa-sha256; cv=none; b=ZrnNLqjDn3fzoKrIVBO7tKyFDE43CbWunzQP+0u09/ACm+y421CdR/VfEXgUh3pQMX/oxk HKZvQ7YdZVzSIDEIn+fWqQbWQX5Ti/uBE/uZtxv97WvwcXU7kO/pTgXZXE/aTJhtzJ9GSo JdTMTZFIYXOywL692MVgO7RIF/N0Coo= Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by dfw.source.kernel.org (Postfix) with ESMTP id DE5F461FE5; Thu, 7 Dec 2023 07:31:47 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 0E745C433CA; Thu, 7 Dec 2023 07:31:46 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1701934307; bh=AQ3zRaWsxl12kZ8IbNOLzKipBmlx9MwTy7N5XYMJexA=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=DfhrW129VGKpUM3TCX2gNqhQjeGfTNPP0SAxtinRm24SLzTOlmXfo800Ka24OTXpr 87dvSP087+Z3Z2U65zg19ilH9Aj/9G9DRZhpw2R/Wz2nN/Rc8t+a0ia1SQfRIhjNS0 yxed0pwSEub43xZZYv2Wy0FozCrcInu2zwI2iOEA= Date: Thu, 7 Dec 2023 08:31:44 +0100 From: Greg KH To: Andrew Morton Cc: Pasha Tatashin , Sourav Panda , corbet@lwn.net, rafael@kernel.org, mike.kravetz@oracle.com, muchun.song@linux.dev, rppt@kernel.org, david@redhat.com, rdunlap@infradead.org, chenlinxuan@uniontech.com, yang.yang29@zte.com.cn, tomas.mudrunka@gmail.com, bhelgaas@google.com, ivan@cloudflare.com, yosryahmed@google.com, hannes@cmpxchg.org, shakeelb@google.com, kirill.shutemov@linux.intel.com, wangkefeng.wang@huawei.com, adobriyan@gmail.com, vbabka@suse.cz, Liam.Howlett@oracle.com, surenb@google.com, linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-doc@vger.kernel.org, linux-mm@kvack.org, willy@infradead.org, weixugc@google.com Subject: Re: [PATCH v6 0/1] mm: report per-page metadata information Message-ID: <2023120731-deception-handmade-8d49@gregkh> References: <20231205223118.3575485-1-souravpanda@google.com> <2023120648-droplet-slit-0e31@gregkh> <2023120645-survey-puppet-4ae0@gregkh> <20231206075913.fa2633991bf257f5ffe5f3f8@linux-foundation.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20231206075913.fa2633991bf257f5ffe5f3f8@linux-foundation.org> X-Rspamd-Queue-Id: 34CA3C0009 X-Rspam-User: X-Rspamd-Server: rspam05 X-Stat-Signature: jtaewk98g7bjxiw7pbyxw5k6jifdtxhb X-HE-Tag: 1701934309-365481 X-HE-Meta: U2FsdGVkX1/Pii98FtaVlLaiGUCfjzaCp6DuSbhREyfaGzk41Tktc2Lll/cT4Gen8cBqmJszI4mmHXZAuDLwIAeQ4cVCFyv5vTt75eTQzlEyDfcmzkbgztl4p058yCf1Jz58Edy91jfc3r4+l23sNlP/VIaluk9zqbseToMYBjY0ZSlnokntHnXK1vPZpn0e+lDu42p8pGupbSzHZW0lA+63cAStVBDvjI5uX2LBot06YtwN7KJdh4TGchnMw3Jz0F3C0JbWF0Llk4H+ATKbnwwQote5AvLpZSHCtg4hTNToblCAq69U6SCODKFajOhJpGSiQ9wHsziehMVYzigqfFhzmEbgEzg0+mjeOL1j+KkZQLydnDYeAn3JKHrickDnfhj5ehdoqB24x+cp9l8gkPqsjJCjf8T0K1oAHzkmTW5z4mf3PF4r+oULY4osNPgyHM1+tof8RFANHHsoyImQ9JTg7n/U4PKjix2REzgZF6fU61tAJwIsWp1rNk0a1Bsci5btYW9HafxAyfRPt8fdR39g5ub2gEaPur7MEru0ojLraVxd0Kve0/YIh0RIbpo4M+jGRHqjlE0U2a0iZyVaiTbMXLNXHW4aBZJ9qL0DelsE25lF3BydrAHd6Jseote1FCuwP8Qk6hy4kTUKN69ThfZapH+Jz1q/0AVW7gK7BQKdjM4JLzD4OePLpGIGgQOZG2R+mjVtsr8MYpqTvz2Jr9/4g/rc5/frmay9eNRmbYvQsBjwgTZCFiE0qIh/y5NA/cEA6DsOXlJfd2W+MuSdA8F//XFCrMVQnSPr5PrG9Gm5Mv+nbtxFFhYVrqJzg+MhMssz2t8AeGOtZmiINbKdJtHE1+nEHwNhp35YT1iCZWWu8eFX22T6PBUPNLn0oGY5FMCNiihHPuTRba8UC7WmOWwJJY2MdGJvxzjjJbQyQjQ/YYaUnhjoBXXBmGKAViBlHDTJ0N6x79WTUK7tMiC 5j2TEKSm 30YYC5ZstncsO/wWzsq33kMWM4krOm8P1raUAoyg9c/UXzxPFfVpKk3fGwwwxnu1QS7pR9e6Z4n1hQ4ou8j9ePTgk6F6JKJxND9AX4KbgYOFMTK/+6JchQWCVbf0awNXfTKzhJQQmLQc7B7n2yLzCMSfVEqdNaTW3UXt2yqgFco58R1ywHASYqRAdQGDSVn/RF3JJ2pDG24C7J36RYthSgGvUp/35WNXcL/RSWekVnqTutHiDEiCUI1ZdnzPHhIc0fQeLpyrH5M1tful0SIGHzDnkHcp2YeKhvsgtJ/BSFGODJ7HK/qM6GJKr9RfiSw4T5013mQApeYdDX2unTWm1xwx9lsM9IOKwywvS X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: On Wed, Dec 06, 2023 at 07:59:13AM -0800, Andrew Morton wrote: > On Wed, 6 Dec 2023 12:12:10 +0900 Greg KH wrote: > > > 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. > > I'm afraid that horse has bolted. > > hp2:/usr/src/25> wc /sys/devices/system/node/node0/vmstat > 61 122 1309 /sys/devices/system/node/node0/vmstat > > We're never going to unpick this into 61 separate files so adding new > files at this stage is pointless. But if it keeps growing, it will eventually overflow and start crashing the kernel and you will then have to do the horrid thing of turning it into a binary sysfs file. So I can please ask that no new entries be added to the file please, let's not keep making things worse. For new items, just add new files, don't add to the existing mess. thanks, greg k-h