From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 422BB1E52F for ; Wed, 27 Dec 2023 18:42:45 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linux-foundation.org header.i=@linux-foundation.org header.b="Odl1enjz" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 4D7FAC433C7; Wed, 27 Dec 2023 18:42:45 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linux-foundation.org; s=korg; t=1703702565; bh=Itb1htvmWYpO4t/GAfB1X1BEGP2JCKRav8Pymo8NuEI=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=Odl1enjzu74MzygvCIiuGdQhGuwBJGYIrfEtU38Js+pJaoZgqZtkgAzxnt8KMGu1Z HnIBfcbwruRGCu4fnHHGpQUY7F610YMk2gcSRgtmGCxsz2CP9vBZFtS/g54K0EOiqb vXLQMw500IYoKRuQK2K0rtEuQiKxOdMtZzsO8p2I= Date: Wed, 27 Dec 2023 10:42:44 -0800 From: Andrew Morton To: David Rientjes Cc: Pasha Tatashin , Linus Torvalds , Greg Kroah-Hartman , rafael@kernel.org, surenb@google.com, linux-kernel@vger.kernel.org, linux-mm@kvack.org, souravpanda@google.com Subject: Re: Sysfs one-value-per-file (was Re: [PATCH] vmstat: don't auto expand the sysfs files) Message-Id: <20231227104244.824b0977ae6d4bb6b37f6f79@linux-foundation.org> In-Reply-To: <13e5fbd4-d84d-faba-47f1-d0024d2c572d@google.com> References: <20231211154644.4103495-1-pasha.tatashin@soleen.com> <3d415ab4-e8c7-7e72-0379-952370612bdd@google.com> <13e5fbd4-d84d-faba-47f1-d0024d2c572d@google.com> X-Mailer: Sylpheed 3.8.0beta1 (GTK+ 2.24.33; x86_64-pc-linux-gnu) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit On Tue, 26 Dec 2023 16:53:31 -0800 (PST) David Rientjes wrote: > But for existing files and conventions, I think we should settle it as > "keep doing what you've been doing for 13+ years" and don't force this > argument every time a kernel developer wants to just add one more stat. Absolutely. Let's do what makes most sense. For new things, one value per file. For stats which logically group with other existing stats, do whatever the existing other stats are currently doing.