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 E9B3BC77B61 for ; Thu, 27 Apr 2023 13:40:29 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 3FBC46B0071; Thu, 27 Apr 2023 09:40:29 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 3D2E46B0072; Thu, 27 Apr 2023 09:40:29 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 29AD8900002; Thu, 27 Apr 2023 09:40:29 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0017.hostedemail.com [216.40.44.17]) by kanga.kvack.org (Postfix) with ESMTP id 192DB6B0071 for ; Thu, 27 Apr 2023 09:40:29 -0400 (EDT) Received: from smtpin21.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay02.hostedemail.com (Postfix) with ESMTP id CAFA712022E for ; Thu, 27 Apr 2023 13:40:28 +0000 (UTC) X-FDA: 80727280536.21.9BEB952 Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by imf08.hostedemail.com (Postfix) with ESMTP id EDDD216001B for ; Thu, 27 Apr 2023 13:40:25 +0000 (UTC) Authentication-Results: imf08.hostedemail.com; dkim=none; dmarc=fail reason="SPF not aligned (relaxed), No valid DKIM" header.from=arm.com (policy=none); spf=pass (imf08.hostedemail.com: domain of cmarinas@kernel.org designates 139.178.84.217 as permitted sender) smtp.mailfrom=cmarinas@kernel.org ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1682602826; 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:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=ri5ExrU+tXvqSYKdGRNAMNXg24iWK05vtQVc5m3M7UI=; b=eN2u3tuR+c7yCWtoqoy1OBk8FnjjAErmvOa8LQve3vrzE26A14pov3FMM1rjPGcxWECaLs pkWDLdHppqTDULClv13/cHuWDqT24L0kHlpl+Db26JkycO4osvHhI4W0teBcKP6RCRP1Jq 1iVYI/dcCz8bhjvtlB2huxvVXg0wuPY= ARC-Authentication-Results: i=1; imf08.hostedemail.com; dkim=none; dmarc=fail reason="SPF not aligned (relaxed), No valid DKIM" header.from=arm.com (policy=none); spf=pass (imf08.hostedemail.com: domain of cmarinas@kernel.org designates 139.178.84.217 as permitted sender) smtp.mailfrom=cmarinas@kernel.org ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1682602826; a=rsa-sha256; cv=none; b=WcYwOvdXezc9gsPwEjwsV8LmxE7Ww26JkI68sjMu89w1JCTPajT/kSo0+A64TSi0h0bqqj GDzC7vnEn1AnU9BVjtAYJl3O+sVj2elVC6wWTYW8//ZkthDq0doaFL4aW/Qf1pAbGsw/Kr c7MJ6dqDPkmRLM/ey3RN33c/Ia/2lmg= Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id D2E7A6195F; Thu, 27 Apr 2023 13:40:24 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id DF907C433EF; Thu, 27 Apr 2023 13:40:19 +0000 (UTC) Date: Thu, 27 Apr 2023 14:40:16 +0100 From: Catalin Marinas To: Justin Forbes Cc: Andrew Morton , Mike Rapoport , Arnd Bergmann , Christophe Leroy , "David S. Miller" , Dinh Nguyen , Geert Uytterhoeven , Guo Ren , John Paul Adrian Glaubitz , "Kirill A. Shutemov" , Max Filippov , Michael Ellerman , Rich Felker , Russell King , Will Deacon , Yoshinori Sato , Zi Yan , linux-arm-kernel@lists.infradead.org, linux-csky@vger.kernel.org, linux-ia64@vger.kernel.org, linux-kernel@vger.kernel.org, linux-m68k@lists.linux-m68k.org, linux-mm@kvack.org, linux-sh@vger.kernel.org, linux-xtensa@linux-xtensa.org, linuxppc-dev@lists.ozlabs.org, sparclinux@vger.kernel.org Subject: Re: [PATCH v3 02/14] arm64: drop ranges in definition of ARCH_FORCE_MAX_ORDER Message-ID: References: <20230325060828.2662773-1-rppt@kernel.org> <20230325060828.2662773-3-rppt@kernel.org> <20230418150557.ea8c87c96ec64c899c88ab08@linux-foundation.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: X-Rspamd-Server: rspam05 X-Rspamd-Queue-Id: EDDD216001B X-Stat-Signature: hhn3gopea364z7ktpbmarif56fpj84j3 X-Rspam-User: X-HE-Tag: 1682602825-622904 X-HE-Meta: U2FsdGVkX19AUEIWYtNghfV58YVXjKo0JrGoW2uqRd8wSlMZJn/yku117dc6BfOQ6c1Ei28XkFTzXgYdrcZhO2AB+joHM1OG3kR6X7AgDgnwRrWJpfPwk+1lcJzbv5lCnz8TusHZ4V9heMs2y00EREWzye+/y4Rh4Oq9VCJjaViKahpsAGUEQHsKuQ3LawFC8wpMmGOjk4Gqiw0j8/UWBuyFw5/o1klBlaXOkhJzpaw65rMp8MRibo9rOL8AcUubUsSLfrSzhDlKpoZxAPBqAtp+6iARim5RdWjUvj0RXP1KgLyTMocBzOulVzc9lpSYK1NWAif4sqJyk/n8IPSRLYQO8qkcDd1B0DNLHGylJCCwk4eSEfZkYaR7waYRmfhZ7jTYG4rWe6P/cKPFizY6xyAccaiPWGQdQwpMn7+UbbT6/ciWDvpTPOoTkpEFVky5OCaPmqsDSOgnIGUX1dEkJ92drfEh4h8m4NrPSUriV4YmgwSobVt1xCQO9K/fTKKXFvsXknvmIjVgtY75AhuB28CCxYLLfG80wwqSYAxiLm/sOhzbjslUSx4okG5Hi+A6oxxvXVibxh1eonGBiEhVU/Rqsc5HXTk1/HXLnibjyuVzr2/WmmxGF7hI8htpSFWWnUhnmfcvzYpUk+mDKBXSmPgr9OjjLGkpIN7k74mAkG+uWHtpEAn9Qu9YB/WsR2sh29kXwr6kZWQzRiBNJeAIMAmsfGp+rWo7ofQtfUmKLY4wb7qYc7Q3v5WtzSRTrOB2n/NIzsUQ0Xpn33vhfBc/ZGx0g3D/N4Rihb1KskZE7jOrWhQjBz39FSaJPYDFycgXJ6x/o9j2OakRLbWERwFNJonxqLo2DofzlngaguAc7gqwrg8+C+knBk4JzsS57UvCx/oOLtXZQng1v1ePx8Z/9m3tPo+dsZ5DglS/+116oGTxq76NV5MbJ1W3PYxZPvCpB2pP+p31D0jqFQN3JYy zOxRczPF JWEGWoiI84PZkA2K0HkCAp+NehmJLcUMff86aE8Jkfv5hHNX1MhjIneq+OQDNHar6a+57XqqQ3KoKeP2oLFJoVlkIAALCnl+OWGVzrH/ege+6nPuayQ7WBIu4PMZdSStdDPM7DuADH0v3accO1L6qgt2a9IX2C9lxBjiYKiWR/fF0UjPm1dXW1CWSjCtOXA2ZXI7lhAXvmn9qTJUNc/SbE1CsIeEYYF7VTIir2qpOgHwzR0iOPM1tjXPsCaqUOAHuz79lKZDuVixnwI3In4IhmeuR6oMUG/lh7P4ybNs4XUq9W1S79fYWHl9fkn9OUx/g0ztEMAcZkYizTz3Vb0/3EUATmc2Pyx9kSg8crL9UOh2VxnfpJHhuBuU7vTEAT/1FgGNFgP5DnnqkgbNpD1OxlbRKvtXJJ8u8vLHX+b+xTfXcSV7QhEGrDAfotmHPMCl9OAYA1wDGt55KG8l4BCQKKrey6njIIEuMkCjd+Is0/Bt00N8hi1ZksDMhq+vPyH2TpQO4ZmK5CrACUMaj8Ccg21yxiK0oymoTQ2ySCoVapoUn6aNanq1UrUblqFOxenZ8kFjyqWQyGGOqXRYFddr3+HQ1AoyrZDLN4qZQ7KJyT6fWdfrcLcyYq7ap3w== 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: On Tue, Apr 25, 2023 at 11:09:58AM -0500, Justin Forbes wrote: > On Tue, Apr 18, 2023 at 5:22 PM Andrew Morton wrote: > > On Wed, 12 Apr 2023 18:27:08 +0100 Catalin Marinas wrote: > > > > It sounds nice in theory. In practice. EXPERT hides too much. When you > > > > flip expert, you expose over a 175ish new config options which are > > > > hidden behind EXPERT. You don't have to know what you are doing just > > > > with the MAX_ORDER, but a whole bunch more as well. If everyone were > > > > already running 10, this might be less of a problem. At least Fedora > > > > and RHEL are running 13 for 4K pages on aarch64. This was not some > > > > accidental choice, we had to carry a patch to even allow it for a > > > > while. If this does go in as is, we will likely just carry a patch to > > > > remove the "if EXPERT", but that is a bit of a disservice to users who > > > > might be trying to debug something else upstream, bisecting upstream > > > > kernels or testing a patch. In those cases, people tend to use > > > > pristine upstream sources without distro patches to verify, and they > > > > tend to use their existing configs. With this change, their MAX_ORDER > > > > will drop to 10 from 13 silently. That can look like a different > > > > issue enough to ruin a bisect or have them give bad feedback on a > > > > patch because it introduces a "regression" which is not a regression > > > > at all, but a config change they couldn't see. > > > > > > If we remove EXPERT (as prior to this patch), I'd rather keep the ranges > > > and avoid having to explain to people why some random MAX_ORDER doesn't > > > build (keeping the range would also make sense for randconfig, not sure > > > we got to any conclusion there). > > > > Well this doesn't seem to have got anywhere. I think I'll send the > > patchset into Linus for the next merge window as-is. Please let's take > > a look at this Kconfig presentation issue during the following -rc > > cycle. > > Well, I am very sorry to see this going in as is. It will silently > change people building with oldconfig, and anyone not paying attention > will not notice until an issue is hit where "it worked before, and my > config hasn't changed". If EXPERT is unset, there is no notification, > just a changed behavior. While it would be easy for me to carry a > patch dropping the if EXPERT, it will not help any users building on > upstream with our configs, whether for their own regular use, or while > trying to debug other issues, I expect it will result in a reasonable > amount of frustration from users trying to do the right thing and > bisect or test patches upstream. As I said in a previous reply, I'm fine with reverting this commit if it breaks existing configs. It's only that Andrew had already queued it in his tree but we have time until the final 6.4 kernel is released. That said, would you mind sending a patch reverting it (if removing EXPERT, I'd like to keep the ranges)? ;) Thanks. -- Catalin