From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-alma10-1.taild15c8.ts.net [100.103.45.18]) (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 B19D246EF9B; Fri, 14 Aug 2026 18:50:27 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=100.103.45.18 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786733428; cv=none; b=fSTZ44+1Cri4XQJGSJpB6ZBE6Oo1ERd4JONiAFxYqSgsG/lMzxB8iWaCbPsryc9s3dpA/Rwb3fJTYCuRFunOUOsd+IaX0RBgXt5ziB/IYt0soq0tkV7UyGa40LXt7V8S9YtC+FNnYOstuty4kEMcKOXmUsqBAbRh/8q3o4gSQOw= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1786733428; c=relaxed/simple; bh=6ZGyDkAvmPp5+Boo2phBmC3awhbL+d+EV4cWc+532P8=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=i1o6kkyYBP5f45tu7NubXrkZh5ZVzhMtddPU2bZ1v9mpNClZ2zFeIFCqPN4EBNm+ujj5NSlMFugg9ipfXgmPN7wvKk+VGzC9BgMc8iBugda2tGuf5/8SoVbes4Q+s0Cizytm75ixfW3j19MwYQ+JwUCbXFMVlvruIaaqXK/T4nQ= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=Oz1IlyhO; arc=none smtp.client-ip=100.103.45.18 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="Oz1IlyhO" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 9FED91F000E9; Fri, 14 Aug 2026 18:50:22 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1786733427; bh=9lCmvz6eJXEEERTfL+imfRIsmzyRwZKtZsiC+HSbrbg=; h=Date:From:To:Cc:Subject:References:In-Reply-To; b=Oz1IlyhOm2snclEDHvF0SYNRPqIrosypwB2/OI0vOQ4Q64WFI9EXzRNYJ2H9WX0YP E66lghnB356hVef7ElHyhQwl8OC9Qzef2nLt9A3+ogldevJthPsg+1tafWJZmssFbF UJLpFh/bbznrO4a0ROtvj4htPutE/V1tiCGFUUQCwC+RDtkpyYBrn5uF8UflRxqsxz ozFTiP3CF+VgLlNnXQeJIot56gDUn83CiELVEuFI5e9xdFIjZYRkL/5C8trEaLBwuQ 4/wj64z1Hg0vpPJvLUVGdcFG0ifZJTcp7DqE3a6/WB6KE4JHZ3XyaeGAqNCFtUcfY+ fEnDEAOZgXMAA== Date: Sat, 15 Aug 2026 02:50:19 +0800 From: Gao Xiang To: Guenter Roeck Cc: Geert Uytterhoeven , Gao Xiang , Michael Bommarito , Uwe =?utf-8?Q?Kleine-K=C3=B6nig?= , SJ Park , Gao Xiang , Yue Hu , Jeffle Xu , Sandeep Dhavale , Chunhai Guo , linux-erofs@lists.ozlabs.org, linux-kernel@vger.kernel.org, Chao Yu , David Hildenbrand , linux-kbuild Subject: Re: [PATCH v3] erofs: cap LZMA stream pool size Message-ID: Mail-Followup-To: Guenter Roeck , Geert Uytterhoeven , Gao Xiang , Michael Bommarito , Uwe =?utf-8?Q?Kleine-K=C3=B6nig?= , SJ Park , Gao Xiang , Yue Hu , Jeffle Xu , Sandeep Dhavale , Chunhai Guo , linux-erofs@lists.ozlabs.org, linux-kernel@vger.kernel.org, Chao Yu , David Hildenbrand , linux-kbuild References: <87853c96-cc8f-49e6-81b1-02bfe409e372@roeck-us.net> <5825b5c2-06b3-4358-91b0-d7bf437528a3@roeck-us.net> <1638f2a8-d9cd-423d-83f6-2b436e3c9de0@roeck-us.net> 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=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <1638f2a8-d9cd-423d-83f6-2b436e3c9de0@roeck-us.net> On Fri, Aug 14, 2026 at 11:44:31AM -0700, Guenter Roeck wrote: > On 8/14/26 11:26, Gao Xiang wrote: > > Hi Guenter, > > > > On Wed, Aug 12, 2026 at 07:25:50AM -0700, Guenter Roeck wrote: > > > > ... > > > > > > > > > > > > Thank you, that seems to work (despite still seeing the warning on m68k, > > > > > as NR_CPUS does not exist). > > > > > > > > > > > > > ... > > > > > > > > > > > > > > $ make ARCH=m68k allmodconfig > > > > > fs/erofs/Kconfig:137:warning: range is invalid > > > > > > > > > > Warning... > > > > > > > > > > > > > Yes, other arches shouldn't have the warning because I think other > > > > arches (including microblaze) defines NR_CPUS in Kconfig, except m68k. > > > > > > > > > > That is not entirely true. openrisc only has it if SMP=y. But it still > > > does not generate the warning for some reason. > > > > > > > I looked into the documentation a bit > > (https://docs.kernel.org/kbuild/kconfig-language.html), especially the > > following parts: > > > > - type definition: “bool”/”tristate”/”string”/”hex”/”int” ... > > - input prompt: “prompt” [“if” ] ... > > and "If a prompt is not present, the config option is > > a non-visible symbol, ..." > > > > - dependencies: “depends on” [“if” ] ... > > - Menu dependencies .. > > > > and I think that is true, taking openrisc for example: > > > > config NR_CPUS > > int "Maximum number of CPUs (2-32)" > > range 2 32 > > depends on SMP > > default "2" > > > > which can be transformed into > > > > config NR_CPUS > > int > > prompt "Maximum number of CPUs (2-32)" > > range 2 32 > > depends on SMP > > default "2" > > > > which can be then transformed into > > > > config NR_CPUS > > int > > prompt "Maximum number of CPUs (2-32)" if SMP > > range 2 32 if SMP > > default "2" if SMP > > > > so if `SMP=n`, it's equivalent to > > > > config NR_CPUS > > int > > > > so NR_CPUS symbol is still there I think (and its value is 0.) > > > $ grep NR_CPUS .config > CONFIG_FORCE_NR_CPUS=y > $ grep EROFS .config > CONFIG_EROFS_FS=y > # CONFIG_EROFS_FS_DEBUG is not set > CONFIG_EROFS_FS_XATTR=y > CONFIG_EROFS_FS_POSIX_ACL=y > CONFIG_EROFS_FS_SECURITY=y > CONFIG_EROFS_FS_BACKED_BY_FILE=y > CONFIG_EROFS_FS_ZIP=y > CONFIG_EROFS_FS_ZIP_LZMA=y > CONFIG_EROFS_FS_ZIP_LZMA_DEFAULT_MAX_STREAMS=0 > # CONFIG_EROFS_FS_ZIP_DEFLATE is not set > # CONFIG_EROFS_FS_ZIP_ZSTD is not set > # CONFIG_EROFS_FS_ZIP_ACCEL is not set > # CONFIG_EROFS_FS_PCPU_KTHREAD is not set > # CONFIG_EROFS_FS_PAGE_CACHE_SHARE is not set > > The symbol may be there internally, but it does not show up in .config. As I said, the documentation: If a prompt is not present, the config option is a non-visible symbol, meaning its value cannot be directly changed by the user (such as altering the value in .config) and the option will not appear in any config menus. So It think it's expected, and any config X like this won't be shown in .config: config X int Thanks, Gao Xiang > > Guenter > >