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 562FDC6FD18 for ; Tue, 25 Apr 2023 16:10:38 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id E17986B008C; Tue, 25 Apr 2023 12:10:37 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id DC7956B0092; Tue, 25 Apr 2023 12:10:37 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id CB6A16B0093; Tue, 25 Apr 2023 12:10:37 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0016.hostedemail.com [216.40.44.16]) by kanga.kvack.org (Postfix) with ESMTP id BBF076B008C for ; Tue, 25 Apr 2023 12:10:37 -0400 (EDT) Received: from smtpin03.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay02.hostedemail.com (Postfix) with ESMTP id 896721202AD for ; Tue, 25 Apr 2023 16:10:37 +0000 (UTC) X-FDA: 80720401314.03.AA4EFE3 Received: from mail-lf1-f53.google.com (mail-lf1-f53.google.com [209.85.167.53]) by imf24.hostedemail.com (Postfix) with ESMTP id 5FD3618001C for ; Tue, 25 Apr 2023 16:10:35 +0000 (UTC) Authentication-Results: imf24.hostedemail.com; dkim=pass header.d=linuxtx.org header.s=google header.b=QMkIWwlS; spf=pass (imf24.hostedemail.com: domain of jmforbes@linuxtx.org designates 209.85.167.53 as permitted sender) smtp.mailfrom=jmforbes@linuxtx.org; dmarc=fail reason="SPF not aligned (relaxed), DKIM not aligned (relaxed)" header.from=fedoraproject.org (policy=none) ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1682439035; h=from:from:sender: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:dkim-signature; bh=gzBCWhkUry18zVOWXW5QyNYiFJMq+EJv8KjCsvLzxVI=; b=FIqUa/l0EvPcqnoS4QjWGphqdiHG8ka2WmlOgdbvwplL8M4M7mpAvMoS+UeGd0hP3bYrtl 2junTtMMYnXB2tUPARmgnmIiGPxAtf7DQJX3ddwLQL9/gVvdFZ3UWCRsj9pePZwfyMPBLB GWAyRDKfbmGRPHY3cPhmtYNaTiVpLPM= ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1682439035; a=rsa-sha256; cv=none; b=XRzSSECN4FKYSyUROMcoLbkzKgFDvpfJAepilwJEepz1htkLPuSmMProy+3KS9y1ZjVhzE SYGkz6HgjIWyCWHMNtptWccn6RnDXrh4YaxlUwVpHKbUqQZRos3cyG+/4i5hon8/EKGCKh fV5fzNe1MPX2vqeYqG27oMDeePNvPnk= ARC-Authentication-Results: i=1; imf24.hostedemail.com; dkim=pass header.d=linuxtx.org header.s=google header.b=QMkIWwlS; spf=pass (imf24.hostedemail.com: domain of jmforbes@linuxtx.org designates 209.85.167.53 as permitted sender) smtp.mailfrom=jmforbes@linuxtx.org; dmarc=fail reason="SPF not aligned (relaxed), DKIM not aligned (relaxed)" header.from=fedoraproject.org (policy=none) Received: by mail-lf1-f53.google.com with SMTP id 2adb3069b0e04-4efefbd2c5eso3191254e87.0 for ; Tue, 25 Apr 2023 09:10:34 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linuxtx.org; s=google; t=1682439033; x=1685031033; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:sender:from:to:cc:subject:date :message-id:reply-to; bh=gzBCWhkUry18zVOWXW5QyNYiFJMq+EJv8KjCsvLzxVI=; b=QMkIWwlSa1dFqjZNAvY1XABYARSvvJqstr6B3FBYxnYeEZbMDFWIcbcX289rr+ND5z ZQJCn1lqGn87wlPR/gIR1F+JWxMSOcLRF2PDHJ6MJTwRrd7AutqRm+TTOaEeBT3sRMYQ yzPmvWLbEoY0UXf2J2QHnMJHIPXTROKg9jOH8= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1682439033; x=1685031033; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:sender:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to; bh=gzBCWhkUry18zVOWXW5QyNYiFJMq+EJv8KjCsvLzxVI=; b=hTyWpyidVjiISmY/ghkcCXuDwlr0jqCiiS3SZRcCFTut5Q16nqb9stN1z/gvfsDBFN bNkURoFNDt8d5pHIME0fYW4yrbLJEatsrEYEMkLLu854zdloFnt/foADOfSwM4KgZ86L D5uGbljaE0qj1PPfRFGUAhVoWmkHLQgGM3t/pdGdWDnAyzhZI/WObqf/rFbtaVsihigA 6k7+Y+feLrw0eNLCW7shI2GOmPBV/iLKAelygg//rNH33ippNfIfDvfquosq6w6+E6lk cGkiwoO6aORIUtMuFtufG9vNIgj5w1W/cb5yaVVmrEgb6eHS63o45e+5RjqkL1hkmBRV XAAg== X-Gm-Message-State: AAQBX9dpibKTfUyCtHZhJde0dOSP6Q31hmDkBOU2PEVGN1fzLl77TSZG QcuA4R3cjtCfufKTFNypOroInjb0XD0GDT4cnUs73YmV X-Google-Smtp-Source: AKy350YMu2/c6Gefy3CdZM8VMUGgZyhYGYXZY5lNEGqLkXOOrhxhNOTR2WHwXk1vYzUkjPYsjw/i5w== X-Received: by 2002:ac2:53bb:0:b0:4dd:ac8d:671f with SMTP id j27-20020ac253bb000000b004ddac8d671fmr4164904lfh.34.1682439033148; Tue, 25 Apr 2023 09:10:33 -0700 (PDT) Received: from mail-lf1-f54.google.com (mail-lf1-f54.google.com. [209.85.167.54]) by smtp.gmail.com with ESMTPSA id y24-20020a2e9d58000000b002a8b2b60ba3sm2070111ljj.103.2023.04.25.09.10.32 for (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 25 Apr 2023 09:10:32 -0700 (PDT) Received: by mail-lf1-f54.google.com with SMTP id 2adb3069b0e04-4edcdfa8638so6493074e87.2 for ; Tue, 25 Apr 2023 09:10:32 -0700 (PDT) X-Received: by 2002:a05:6512:145:b0:4d7:44c9:9f4c with SMTP id m5-20020a056512014500b004d744c99f4cmr4233943lfo.4.1682439010843; Tue, 25 Apr 2023 09:10:10 -0700 (PDT) MIME-Version: 1.0 References: <20230325060828.2662773-1-rppt@kernel.org> <20230325060828.2662773-3-rppt@kernel.org> <20230418150557.ea8c87c96ec64c899c88ab08@linux-foundation.org> In-Reply-To: <20230418150557.ea8c87c96ec64c899c88ab08@linux-foundation.org> From: Justin Forbes Date: Tue, 25 Apr 2023 11:09:58 -0500 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH v3 02/14] arm64: drop ranges in definition of ARCH_FORCE_MAX_ORDER To: Andrew Morton Cc: Catalin Marinas , 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 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Rspam-User: X-Rspamd-Queue-Id: 5FD3618001C X-Rspamd-Server: rspam09 X-Stat-Signature: imm6jrknp8o7jsspkofzo4j9d3zjgc4d X-HE-Tag: 1682439035-414699 X-HE-Meta: U2FsdGVkX1/ESAyA3BzbfuwlgPA/Z50/MB3P017EHsRL9E0KHTht4zyK1wCjYF5b+/8C8i/h/FKypFS6WkZKxcKvf36BCXfXDZtEsbQDvY1JUE0dmZVuvsRNacSL60PzkrAptngJ4zNyWeQkzWM1cMdX37bvSjssfYSXsWB+qJQ6BunixyBGFpvwrd2fk7qNkYM2KAx1+feg8/jHGzZlotY17RUC/wH+oQyu+mQ/Ln04s1QwJ0pvTNAxnqFHDn6uLO2cRx8EE934OljbW/jVvf7ggRsBvSew68DNEjLXgJXcfz31UcMbcd2/rbAlgfEhrvWZPoR6/pDMQe/EkDve6f8gIPkESzA2lTo1ygUuz9FEGFM6kVrkFfKW+NmmWnRvpNNBRIvTQVNmQ50db/k9Ajmbm0PNwyUI7zG3z1if2Z31KVVmC7e0HXJgKloBG5v1HGJugukZXtTh+a3IQAOJZm0Q24Tqmm6s0gu27c7ku7PbMvrpia2mnwyOhhaFMSz3PNWTmmNTPaQQzNNbDpTUAICkML+g9cMpYQ2mqaueT6P3Ms/KYyW6cRypKNcEPwHPHQR4Qk0REkuZShrY90Heud7PSIDjgsvPomA/NQWt8stb9dxgBg4/HqEALVQEkr9aWhSxmpL5dZmio/Cvo89nyhkEk39j38f5uEhBpepgY8KG1LeLi9HrE8m7ScEp2TLRYqRv9zCG2nSQR2kNsfXe+67sJQqTyz7JuJzwWg78gvPE02lGFy39SeX6U4Y7MAscEU2VRYy0ib6gnhWEnuNJ9bF9AMUiLX+UJC1BCn5HfmlX/Pw4fr/ceYRN2FFyZtVtDudbITZZeZw6DzwBYcjDO+GmoCE26cGI+1mHlbCVz6y+oqzVAus4X33C8V3b+Cdf1axgPk+d8t5tVJQEKDF28ZpMBDO7w/BK7r2m6ZCnwzZf6rZePDl4/ODSGg3nzxJbPpgGKiwx1hybq9LQkf2 BzqhwBAi cm72M6THbMV4UsrGvouHFs17Mc9ExBbxNE4FotrD903eKrlNYfkShM4eJwfBx/jmcCNeSo+mn0me/xVkIYMtKUP2KrluoiJ+g/zXnW45JhS+4KkDyKgVDStQgT1O+dBxpYeGg0t4Ejk3Ftx1DgM5x3kaLosUQFQ5FfRJ+LffQAnol8EI80+ueqqQBVDpnxBWBoXRg96xGa07rDjdzrGpjkor9lKJ4sy9uqlFJLGDN8FzFl8TDDmp9mGGAuuF9sCBeBakyWbLCkn1KyvosGb94T6Bprhowv99Pca24iURypdLbgGyYLtYAI081d6V46jm1UtbX0DFcXyUcV9dwkTEnvQutS2PTah2YlopwvR0VV4W9734bBiUttf1xvh0FUpbpi3RH5jfvkQASh10f2uOeXw9e8RavBQa0xbSoSLh6U50sbxumAUff/JhU0t79KgUXhIh8/+PRbjP4XaWdFMxXurtGJKLd41sOs21TOxgYS/aYnXmanU4GeJuxBOC2B/QKBsrMH9LNxUaqsBGH9sMH9FGZdFXTVBOIWu+nvxGGB5pkrj/sJQlQ4NjNmQG9KAnQE7xSAoxvXZRKx7i9/aQwd4q7/ffG6q96JD2TMfzIGbjDCU/3WuOiHppnplznVDmWBAvOpEqjd7nIpOm+e39rmXmlBPpGMh5XUusArozY99e48cPg2pa/bV4ieCB2jfpjEw9xpV5U1tyn6bj/+8mWQefeJYCCoKoNisuvUwiBafu2TaQ= 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 18, 2023 at 5:22=E2=80=AFPM 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 yo= u > > > 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 t= o > > > remove the "if EXPERT", but that is a bit of a disservice to users wh= o > > > 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 range= s > > 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. Justin