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 C513AC83F17 for ; Mon, 14 Jul 2025 02:35:41 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 510FA8D0003; Sun, 13 Jul 2025 22:35:41 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 4E8DC8D0001; Sun, 13 Jul 2025 22:35:41 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 3FE868D0003; Sun, 13 Jul 2025 22:35:41 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0015.hostedemail.com [216.40.44.15]) by kanga.kvack.org (Postfix) with ESMTP id 2BFDA8D0001 for ; Sun, 13 Jul 2025 22:35:41 -0400 (EDT) Received: from smtpin10.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay01.hostedemail.com (Postfix) with ESMTP id B70471D9FB0 for ; Mon, 14 Jul 2025 02:35:40 +0000 (UTC) X-FDA: 83661304440.10.2D06331 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by imf27.hostedemail.com (Postfix) with ESMTP id B00EC40009 for ; Mon, 14 Jul 2025 02:35:38 +0000 (UTC) Authentication-Results: imf27.hostedemail.com; dkim=none; dmarc=pass (policy=none) header.from=arm.com; spf=pass (imf27.hostedemail.com: domain of anshuman.khandual@arm.com designates 217.140.110.172 as permitted sender) smtp.mailfrom=anshuman.khandual@arm.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1752460539; a=rsa-sha256; cv=none; b=drxJy9TGjOk5KDP/skAN1K/Qjzo0ee6tJLc2eQOGKPfKmRJL+7jJ7cSk6PwvaYO7suVV02 sDjeHdq40WJ4x8oSaUX6I0KENqNULv8/v5/CqY5EqQ5V6AkfNP7zoDJUVkgY1mB6h8X/8y qXFuMwRAoz417i0R6BxC1nWHD/aGTfM= ARC-Authentication-Results: i=1; imf27.hostedemail.com; dkim=none; dmarc=pass (policy=none) header.from=arm.com; spf=pass (imf27.hostedemail.com: domain of anshuman.khandual@arm.com designates 217.140.110.172 as permitted sender) smtp.mailfrom=anshuman.khandual@arm.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1752460539; 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=7q03RkowSk2+aL0RE0M6yatPy/SNSW8h0KLew0zGbPQ=; b=hQ4gebrrO1D1q6LQO0oOdFf9kHzxPHDU1VBSZKLVdJIF4PMVNSMuQvzXrtscwH1SsMxAXG iQrxL/Ay/OujajVzg+lvPxoKORN2214ZxUzf/PLOjuZAnxdGXSPfVx44ZZ71p8N5ICM/34 XFnFa05/Ok1tg9TQvVaVCES8mHSkzBc= Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 476A912FC; Sun, 13 Jul 2025 19:35:28 -0700 (PDT) Received: from [10.164.146.15] (J09HK2D2RT.blr.arm.com [10.164.146.15]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 9A1303F6A8; Sun, 13 Jul 2025 19:35:34 -0700 (PDT) Message-ID: Date: Mon, 14 Jul 2025 08:05:31 +0530 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH] fs/Kconfig: Enable HUGETLBFS only if ARCH_SUPPORTS_HUGETLBFS To: Andrew Morton Cc: linux-mm@kvack.org, "David S. Miller" , Thomas Gleixner , Ingo Molnar , x86@kernel.org, sparclinux@vger.kernel.org, linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org References: <20250711102934.2399533-1-anshuman.khandual@arm.com> <20250712161549.499ec62de664904bd86ffa90@linux-foundation.org> Content-Language: en-US From: Anshuman Khandual In-Reply-To: <20250712161549.499ec62de664904bd86ffa90@linux-foundation.org> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Rspamd-Queue-Id: B00EC40009 X-Stat-Signature: cchbh9sg8nmiih1b4soni5ppbw7k8j4e X-Rspam-User: X-Rspamd-Server: rspam08 X-HE-Tag: 1752460538-3684 X-HE-Meta: U2FsdGVkX18zyVA1M+650j6j5y04WvYZhxTny8p1bZJ0JRmn5RM8nVckmJU9ju8mMo0cEauVbIOgvclrxXO0CSBcfmHxX49Td35cskeOzMnBjM3k3hk0+sdnhvZYp1z57HPn6eQv/QsIK4kpjCEVTFkSpV5RmeA+H45ABsT+k/bC0lvfZfYmWXJZlfvQyfdidt0lcdA0Pd/B1lG6I4J7I5cBV1Vfbux/xzO+29MAZ16N7jR++53VPFvEhHXIZ5QyOLiH+AE1QytOr/W4fajtozde/PbQb5SyKodPABqOFMGJt0N1+wavktZgr07w6bv98MPRGnjmFxQe3LFGLMQ59NcAufWLaBMj2aiFL2udStQRTFZVGcGrhUXwefGoN8tXxR9jroVyQ76siqmOU7J8lmH+Gq1pjrWpEwy1DrYUVw1K6rBnN/Mmue8Z3XGqWoan8geEy5pMy8naGTl9v+JvCqHoBqhs1cn7QKvLej29hFQZdg3AvaVmwYndByCgsL28LwNjps8Un2fxpmb1oMjxKYVt6uiDuEJVp4h3fp55HUqhZow2ql7HG/TRCdTY65rLd2xmSa5xmpxeo+uDOa54y5u18vSYKGP4Smp0W5ZV3h3HUZTdlb/sAYHEAwNBo/1nbVFgwaloH/KkiyyC64UWnRSTGGDUrtAmmBqMhrfEzabqyGvpFimXaLuyQu13nVL1grCNDqul5ml6R1WPS3uCXS4mzx6y2Wef5mnxgON09DsLrkIBwa98tebp0tc1TZhHrfsrQTt0FlKXXHoH/lWT1U2sA1/UBoV288t8PuYUfipabDlfi+m95Ma2Vr7o/4U8Drnn4NZg/HduL+5++0sIyGWJWxs/0BC6Grk06dYsLnndq431owYNQSScp1SW0keuUI+FQ2dQIF6zi/o39eMVLDOYkidQlTiT3mcQ5t64h/cmkWecESnU8WimreW6N5Jbnyl+y9KgzXY4EH0QrOA DWpNyR4T 5khK17qAvKkcK2ai6O+6ZA7ns0cfxBzlfuXYIi8J/P2oPPaNFZKQN4Id41H7SKISRB1+JNS+X6ww28LRl2Mn8QQaCGwEATXbU0WMOvkLqUj+li+GU5cwqSDResSaISU5rg+K+PZ9K1KjEYCwnvrZR1Q3WDuaOSKPMMVZRhJ+7C8uCIAu1KIJOzlKabbJvLzGXJnHoiT3Y5nu/0U6KT3tZbG9xsUC5qMiV/X6UVMsBXjDns8cQxASNTyVEx6g01yDXV6+0eBxJE2RpJGf5UIhVM32aNy9QmpLXyEtRrRrV/oqckc4Y1GFfeFgu5g== 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 13/07/25 4:45 AM, Andrew Morton wrote: > On Fri, 11 Jul 2025 15:59:34 +0530 Anshuman Khandual wrote: > >> Enable HUGETLBFS only when platform subscrbes via ARCH_SUPPORTS_HUGETLBFS. >> Hence select ARCH_SUPPORTS_HUGETLBFS on existing x86 and sparc for their >> continuing HUGETLBFS support. > > Looks nice. > >> While here also just drop existing 'BROKEN' dependency. > > Why? > > What is BROKEN for, anyway? I don't recall having dealt with it > before. It predates kernel git and we forgot to document it. The original first commit had added 'BROKEN', although currently there are no explanations about it in the tree. But looks like this might be used for feature gating (selectively disabling features) etc. But I am not much aware about it. commit 1da177e4c3f41524e886b7f1b8a0c1fc7321cac2 (tag: v2.6.12-rc2) Author: Linus Torvalds Date: Sat Apr 16 15:20:36 2005 -0700 Linux-2.6.12-rc2 Initial git repository build. I'm not bothering with the full history, even though we have it. We can create a separate "historical" git archive of that later if we want to, and in the meantime it's about 3.2GB when imported into git - space that would just make the early git days unnecessarily complicated, when we don't have a lot of good infrastructure for it. Let it rip! BROKEN still gets used for multiple config options. git grep "depends on BROKEN" arch/m68k/Kconfig.devices: depends on BROKEN && (Q40 || SUN3X) arch/mips/loongson64/Kconfig: depends on BROKEN arch/parisc/Kconfig: depends on BROKEN arch/powerpc/lib/crypto/Kconfig: depends on BROKEN # Needs to be fixed to work in softirq context drivers/edac/Kconfig: depends on BROKEN drivers/edac/Kconfig: depends on BROKEN drivers/gpu/drm/Kconfig.debug: depends on BROKEN drivers/gpu/drm/amd/display/Kconfig: depends on BROKEN || !CC_IS_CLANG || ARM64 || LOONGARCH || RISCV || SPARC64 || X86_64 drivers/gpu/drm/i915/Kconfig.debug: depends on BROKEN drivers/net/wireless/intel/iwlwifi/Kconfig: depends on BROKEN drivers/s390/block/Kconfig: depends on BROKEN drivers/staging/gpib/TODO:- fix device drivers that are broken ("depends on BROKEN" in Kconfig) drivers/tty/Kconfig: depends on BROKEN drivers/virtio/Kconfig: depends on BROKEN init/Kconfig: depends on BROKEN || !SMP init/Kconfig: depends on BROKEN lib/Kconfig.ubsan: depends on BROKEN git grep "&& BROKEN" arch/parisc/Kconfig: depends on PA8X00 && BROKEN && !KFENCE arch/parisc/Kconfig: depends on PA8X00 && BROKEN && !KFENCE arch/powerpc/platforms/amigaone/Kconfig: depends on PPC_BOOK3S_32 && BROKEN_ON_SMP arch/powerpc/platforms/embedded6xx/Kconfig: depends on PPC_BOOK3S_32 && BROKEN_ON_SMP arch/sh/Kconfig.debug: depends on DEBUG_KERNEL && BROKEN drivers/gpu/drm/Kconfig.debug: depends on DRM && EXPERT && BROKEN drivers/i2c/busses/Kconfig: depends on ISA && HAS_IOPORT_MAP && BROKEN_ON_SMP drivers/leds/Kconfig: depends on LEDS_CLASS && BROKEN drivers/net/wireless/broadcom/b43/Kconfig: depends on B43 && BROKEN drivers/net/wireless/broadcom/b43/Kconfig: depends on B43 && B43_BCMA && BROKEN drivers/pps/generators/Kconfig: depends on PARPORT && BROKEN drivers/staging/greybus/Kconfig: depends on MEDIA_SUPPORT && LEDS_CLASS_FLASH && BROKEN drivers/video/fbdev/Kconfig: depends on FB && ((AMIGA && BROKEN) || PCI) fs/quota/Kconfig: depends on QUOTA && BROKEN fs/smb/client/Kconfig: depends on CIFS && BROKEN net/ax25/Kconfig: depends on AX25_DAMA_SLAVE && BROKEN git grep "|| BROKEN" arch/sh/Kconfig.debug: depends on DEBUG_KERNEL && (MMU || BROKEN) && !PAGE_SIZE_64KB drivers/misc/Kconfig: depends on X86_64 || BROKEN drivers/net/ethernet/faraday/Kconfig: depends on !64BIT || BROKEN drivers/net/ethernet/faraday/Kconfig: depends on !64BIT || BROKEN drivers/net/ethernet/intel/Kconfig: depends on PCI && (!SPARC32 || BROKEN) drivers/usb/gadget/udc/Kconfig: depends on !64BIT || BROKEN kernel/power/Kconfig: if ARCH_WANTS_FREEZER_CONTROL || BROKEN