All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 00/14] arch,mm: cleanup Kconfig entries for ARCH_FORCE_MAX_ORDER
@ 2023-03-23  9:21 ` Mike Rapoport
  0 siblings, 0 replies; 95+ messages in thread
From: Mike Rapoport @ 2023-03-23  9:21 UTC (permalink / raw)
  To: Andrew Morton
  Cc: Arnd Bergmann, Catalin Marinas, Christophe Leroy, David S. Miller,
	Dinh Nguyen, Geert Uytterhoeven, Guo Ren,
	John Paul Adrian Glaubitz, Kirill A. Shutemov, Max Filippov,
	Michael Ellerman, Mike Rapoport, Rich Felker, Russell King,
	Will Deacon, Yoshinori Sato, Zi Yan, linux-arm-kernel, linux-csky,
	linux-ia64, linux-kernel, linux-m68k, linux-mm, linux-sh,
	linux-xtensa, linuxppc-dev, sparclinux

From: "Mike Rapoport (IBM)" <rppt@kernel.org>

Hi,

Several architectures have ARCH_FORCE_MAX_ORDER in their Kconfig and
they all have wrong and misleading prompt and help text for this option.

Besides, some define insane limits for possible values of
ARCH_FORCE_MAX_ORDER, some carefully define ranges only for a subset of
possible configurations, some make this option configurable by users for no
good reason.

This set updates the prompt and help text everywhere and does its best to
update actual definitions of ranges where applicable.

Mike Rapoport (IBM) (14):
  arm: reword ARCH_FORCE_MAX_ORDER prompt and help text
  arm64: drop ranges in definition of ARCH_FORCE_MAX_ORDER
  arm64: reword ARCH_FORCE_MAX_ORDER prompt and help text
  csky: drop ARCH_FORCE_MAX_ORDER
  ia64: don't allow users to override ARCH_FORCE_MAX_ORDER
  m68k: reword ARCH_FORCE_MAX_ORDER prompt and help text
  nios2: reword ARCH_FORCE_MAX_ORDER prompt and help text
  nios2: drop ranges for definition of ARCH_FORCE_MAX_ORDER
  powerpc: reword ARCH_FORCE_MAX_ORDER prompt and help text
  powerpc: drop ranges for definition of ARCH_FORCE_MAX_ORDER
  sh: reword ARCH_FORCE_MAX_ORDER prompt and help text
  sh: drop ranges for definition of ARCH_FORCE_MAX_ORDER
  sparc: reword ARCH_FORCE_MAX_ORDER prompt and help text
  xtensa: reword ARCH_FORCE_MAX_ORDER prompt and help text

 arch/arm/Kconfig      | 16 +++++++++-------
 arch/arm64/Kconfig    | 27 ++++++++++++---------------
 arch/csky/Kconfig     |  4 ----
 arch/ia64/Kconfig     |  3 +--
 arch/m68k/Kconfig.cpu | 16 +++++++++-------
 arch/nios2/Kconfig    | 17 +++++++++--------
 arch/powerpc/Kconfig  | 22 +++++++++-------------
 arch/sh/mm/Kconfig    | 19 +++++++++----------
 arch/sparc/Kconfig    | 16 +++++++++-------
 arch/xtensa/Kconfig   | 16 +++++++++-------
 10 files changed, 76 insertions(+), 80 deletions(-)


base-commit: 51551d71edbc998fd8c8afa7312db3d270f5998e
-- 
2.35.1


^ permalink raw reply	[flat|nested] 95+ messages in thread
* Re: [PATCH 02/14] arm64: drop ranges in definition of ARCH_FORCE_MAX_ORDER
@ 2023-03-23 20:47 kernel test robot
  0 siblings, 0 replies; 95+ messages in thread
From: kernel test robot @ 2023-03-23 20:47 UTC (permalink / raw)
  To: oe-kbuild; +Cc: lkp

:::::: 
:::::: Manual check reason: "low confidence static check first_new_problem: include/linux/compiler_types.h:397:45: error: call to '__compiletime_assert_491' declared with attribute error: BUILD_BUG_ON failed: (((12 - 3) * (4 - (2)) + 3)-12) > -1"
:::::: 

BCC: lkp@intel.com
CC: oe-kbuild-all@lists.linux.dev
In-Reply-To: <20230323092156.2545741-3-rppt@kernel.org>
References: <20230323092156.2545741-3-rppt@kernel.org>
TO: Mike Rapoport <rppt@kernel.org>
TO: Andrew Morton <akpm@linux-foundation.org>
CC: Linux Memory Management List <linux-mm@kvack.org>
CC: Arnd Bergmann <arnd@arndb.de>
CC: Catalin Marinas <catalin.marinas@arm.com>
CC: Christophe Leroy <christophe.leroy@csgroup.eu>
CC: Dinh Nguyen <dinguyen@kernel.org>
CC: Geert Uytterhoeven <geert@linux-m68k.org>
CC: Guo Ren <guoren@kernel.org>
CC: John Paul Adrian Glaubitz <glaubitz@physik.fu-berlin.de>
CC: "Kirill A. Shutemov" <kirill.shutemov@linux.intel.com>
CC: Max Filippov <jcmvbkbc@gmail.com>
CC: Michael Ellerman <mpe@ellerman.id.au>
CC: Mike Rapoport <rppt@kernel.org>
CC: Rich Felker <dalias@libc.org>
CC: Russell King <linux@armlinux.org.uk>
CC: Will Deacon <will@kernel.org>
CC: Yoshinori Sato <ysato@users.sourceforge.jp>
CC: Zi Yan <ziy@nvidia.com>
CC: linux-arm-kernel@lists.infradead.org
CC: linux-csky@vger.kernel.org
CC: linux-ia64@vger.kernel.org
CC: linux-kernel@vger.kernel.org
CC: linux-m68k@lists.linux-m68k.org
CC: linux-sh@vger.kernel.org
CC: linux-xtensa@linux-xtensa.org
CC: linuxppc-dev@lists.ozlabs.org
CC: sparclinux@vger.kernel.org

Hi Mike,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on 51551d71edbc998fd8c8afa7312db3d270f5998e]

url:    https://github.com/intel-lab-lkp/linux/commits/Mike-Rapoport/arm-reword-ARCH_FORCE_MAX_ORDER-prompt-and-help-text/20230323-172512
base:   51551d71edbc998fd8c8afa7312db3d270f5998e
patch link:    https://lore.kernel.org/r/20230323092156.2545741-3-rppt%40kernel.org
patch subject: [PATCH 02/14] arm64: drop ranges in definition of ARCH_FORCE_MAX_ORDER
:::::: branch date: 11 hours ago
:::::: commit date: 11 hours ago
config: arm64-randconfig-c034-20230322 (https://download.01.org/0day-ci/archive/20230324/202303240430.oprzwsoZ-lkp@intel.com/config)
compiler: aarch64-linux-gcc (GCC) 12.1.0
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # https://github.com/intel-lab-lkp/linux/commit/0522f943c071abf1610651ea40405b7489c50987
        git remote add linux-review https://github.com/intel-lab-lkp/linux
        git fetch --no-tags linux-review Mike-Rapoport/arm-reword-ARCH_FORCE_MAX_ORDER-prompt-and-help-text/20230323-172512
        git checkout 0522f943c071abf1610651ea40405b7489c50987
        # save the config file
        mkdir build_dir && cp config build_dir/.config
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=arm64 olddefconfig
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=arm64 SHELL=/bin/bash

If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot <lkp@intel.com>
| Link: https://lore.kernel.org/r/202303240430.oprzwsoZ-lkp@intel.com/

All errors (new ones prefixed by >>):

   In file included from <command-line>:
   mm/huge_memory.c: In function 'hugepage_init':
>> include/linux/compiler_types.h:397:45: error: call to '__compiletime_assert_491' declared with attribute error: BUILD_BUG_ON failed: (((12 - 3) * (4 - (2)) + 3)-12) > -1
     397 |         _compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__)
         |                                             ^
   include/linux/compiler_types.h:378:25: note: in definition of macro '__compiletime_assert'
     378 |                         prefix ## suffix();                             \
         |                         ^~~~~~
   include/linux/compiler_types.h:397:9: note: in expansion of macro '_compiletime_assert'
     397 |         _compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__)
         |         ^~~~~~~~~~~~~~~~~~~
   include/linux/build_bug.h:39:37: note: in expansion of macro 'compiletime_assert'
      39 | #define BUILD_BUG_ON_MSG(cond, msg) compiletime_assert(!(cond), msg)
         |                                     ^~~~~~~~~~~~~~~~~~
   include/linux/build_bug.h:50:9: note: in expansion of macro 'BUILD_BUG_ON_MSG'
      50 |         BUILD_BUG_ON_MSG(condition, "BUILD_BUG_ON failed: " #condition)
         |         ^~~~~~~~~~~~~~~~
   include/linux/bug.h:24:25: note: in expansion of macro 'BUILD_BUG_ON'
      24 |                         BUILD_BUG_ON(cond);             \
         |                         ^~~~~~~~~~~~
   mm/huge_memory.c:470:9: note: in expansion of macro 'MAYBE_BUILD_BUG_ON'
     470 |         MAYBE_BUILD_BUG_ON(HPAGE_PMD_ORDER > MAX_ORDER);
         |         ^~~~~~~~~~~~~~~~~~


vim +/__compiletime_assert_491 +397 include/linux/compiler_types.h

eb5c2d4b45e3d2 Will Deacon 2020-07-21  383  
eb5c2d4b45e3d2 Will Deacon 2020-07-21  384  #define _compiletime_assert(condition, msg, prefix, suffix) \
eb5c2d4b45e3d2 Will Deacon 2020-07-21  385  	__compiletime_assert(condition, msg, prefix, suffix)
eb5c2d4b45e3d2 Will Deacon 2020-07-21  386  
eb5c2d4b45e3d2 Will Deacon 2020-07-21  387  /**
eb5c2d4b45e3d2 Will Deacon 2020-07-21  388   * compiletime_assert - break build and emit msg if condition is false
eb5c2d4b45e3d2 Will Deacon 2020-07-21  389   * @condition: a compile-time constant condition to check
eb5c2d4b45e3d2 Will Deacon 2020-07-21  390   * @msg:       a message to emit if condition is false
eb5c2d4b45e3d2 Will Deacon 2020-07-21  391   *
eb5c2d4b45e3d2 Will Deacon 2020-07-21  392   * In tradition of POSIX assert, this macro will break the build if the
eb5c2d4b45e3d2 Will Deacon 2020-07-21  393   * supplied condition is *false*, emitting the supplied error message if the
eb5c2d4b45e3d2 Will Deacon 2020-07-21  394   * compiler has support to do so.
eb5c2d4b45e3d2 Will Deacon 2020-07-21  395   */
eb5c2d4b45e3d2 Will Deacon 2020-07-21  396  #define compiletime_assert(condition, msg) \
eb5c2d4b45e3d2 Will Deacon 2020-07-21 @397  	_compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__)
eb5c2d4b45e3d2 Will Deacon 2020-07-21  398  

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests

^ permalink raw reply	[flat|nested] 95+ messages in thread

end of thread, other threads:[~2023-03-23 20:47 UTC | newest]

Thread overview: 95+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-03-23  9:21 [PATCH 00/14] arch,mm: cleanup Kconfig entries for ARCH_FORCE_MAX_ORDER Mike Rapoport
2023-03-23  9:21 ` Mike Rapoport
2023-03-23  9:21 ` Mike Rapoport
2023-03-23  9:21 ` Mike Rapoport
2023-03-23  9:21 ` [PATCH 01/14] arm: reword ARCH_FORCE_MAX_ORDER prompt and help text Mike Rapoport
2023-03-23  9:21   ` Mike Rapoport
2023-03-23  9:21   ` Mike Rapoport
2023-03-23  9:21   ` Mike Rapoport
2023-03-23  9:21 ` [PATCH 02/14] arm64: drop ranges in definition of ARCH_FORCE_MAX_ORDER Mike Rapoport
2023-03-23  9:21   ` Mike Rapoport
2023-03-23  9:21   ` Mike Rapoport
2023-03-23  9:21   ` Mike Rapoport
2023-03-23 10:15   ` Catalin Marinas
2023-03-23 10:15     ` Catalin Marinas
2023-03-23 10:15     ` Catalin Marinas
2023-03-23 10:37     ` Mike Rapoport
2023-03-23 10:37       ` Mike Rapoport
2023-03-23 10:37       ` Mike Rapoport
2023-03-23 10:37       ` Mike Rapoport
2023-03-23 11:57       ` Zi Yan
2023-03-23 11:57         ` Zi Yan
2023-03-23 11:57         ` Zi Yan
2023-03-23 11:57         ` Zi Yan
2023-03-23 13:46   ` kernel test robot
2023-03-23 13:46     ` kernel test robot
2023-03-23 13:46     ` kernel test robot
2023-03-23 13:46     ` kernel test robot
2023-03-23 17:11   ` kernel test robot
2023-03-23 17:11     ` kernel test robot
2023-03-23 17:11     ` kernel test robot
2023-03-23 17:11     ` kernel test robot
2023-03-23  9:21 ` [PATCH 03/14] arm64: reword ARCH_FORCE_MAX_ORDER prompt and help text Mike Rapoport
2023-03-23  9:21   ` Mike Rapoport
2023-03-23  9:21   ` Mike Rapoport
2023-03-23  9:21   ` Mike Rapoport
2023-03-23 10:30   ` Catalin Marinas
2023-03-23 10:30     ` Catalin Marinas
2023-03-23 10:30     ` Catalin Marinas
2023-03-23  9:21 ` [PATCH 04/14] csky: drop ARCH_FORCE_MAX_ORDER Mike Rapoport
2023-03-23  9:21   ` Mike Rapoport
2023-03-23  9:21   ` Mike Rapoport
2023-03-23  9:21   ` Mike Rapoport
2023-03-23  9:21 ` [PATCH 05/14] ia64: don't allow users to override ARCH_FORCE_MAX_ORDER Mike Rapoport
2023-03-23  9:21   ` Mike Rapoport
2023-03-23  9:21   ` Mike Rapoport
2023-03-23  9:21   ` Mike Rapoport
2023-03-23  9:21 ` [PATCH 06/14] m68k: reword ARCH_FORCE_MAX_ORDER prompt and help text Mike Rapoport
2023-03-23  9:21   ` Mike Rapoport
2023-03-23  9:21   ` Mike Rapoport
2023-03-23  9:21   ` Mike Rapoport
2023-03-23 16:24   ` Geert Uytterhoeven
2023-03-23 16:24     ` Geert Uytterhoeven
2023-03-23 16:24     ` Geert Uytterhoeven
2023-03-23 16:24     ` Geert Uytterhoeven
2023-03-23  9:21 ` [PATCH 07/14] nios2: " Mike Rapoport
2023-03-23  9:21   ` Mike Rapoport
2023-03-23  9:21   ` Mike Rapoport
2023-03-23  9:21   ` Mike Rapoport
2023-03-23  9:21 ` [PATCH 08/14] nios2: drop ranges for definition of ARCH_FORCE_MAX_ORDER Mike Rapoport
2023-03-23  9:21   ` Mike Rapoport
2023-03-23  9:21   ` Mike Rapoport
2023-03-23  9:21   ` Mike Rapoport
2023-03-23  9:21 ` [PATCH 09/14] powerpc: reword ARCH_FORCE_MAX_ORDER prompt and help text Mike Rapoport
2023-03-23  9:21   ` Mike Rapoport
2023-03-23  9:21   ` Mike Rapoport
2023-03-23  9:21   ` Mike Rapoport
2023-03-23  9:21 ` [PATCH 10/14] powerpc: drop ranges for definition of ARCH_FORCE_MAX_ORDER Mike Rapoport
2023-03-23  9:21   ` Mike Rapoport
2023-03-23  9:21   ` Mike Rapoport
2023-03-23  9:21   ` Mike Rapoport
2023-03-23  9:21 ` [PATCH 11/14] sh: reword ARCH_FORCE_MAX_ORDER prompt and help text Mike Rapoport
2023-03-23  9:21   ` Mike Rapoport
2023-03-23  9:21   ` Mike Rapoport
2023-03-23  9:21   ` Mike Rapoport
2023-03-23  9:21 ` [PATCH 12/14] sh: drop ranges for definition of ARCH_FORCE_MAX_ORDER Mike Rapoport
2023-03-23  9:21   ` Mike Rapoport
2023-03-23  9:21   ` Mike Rapoport
2023-03-23  9:21   ` Mike Rapoport
2023-03-23  9:21 ` [PATCH 13/14] sparc: reword ARCH_FORCE_MAX_ORDER prompt and help text Mike Rapoport
2023-03-23  9:21   ` Mike Rapoport
2023-03-23  9:21   ` Mike Rapoport
2023-03-23  9:21   ` Mike Rapoport
2023-03-23  9:21 ` [PATCH 14/14] xtensa: " Mike Rapoport
2023-03-23  9:21   ` Mike Rapoport
2023-03-23  9:21   ` Mike Rapoport
2023-03-23  9:21   ` Mike Rapoport
2023-03-23  9:39   ` Max Filippov
2023-03-23  9:39     ` Max Filippov
2023-03-23  9:39     ` Max Filippov
2023-03-23  9:39     ` Max Filippov
2023-03-23 10:17 ` [PATCH 00/14] arch,mm: cleanup Kconfig entries for ARCH_FORCE_MAX_ORDER Kirill A. Shutemov
2023-03-23 10:17   ` Kirill A. Shutemov
2023-03-23 10:17   ` Kirill A. Shutemov
2023-03-23 10:17   ` Kirill A. Shutemov
  -- strict thread matches above, loose matches on Subject: below --
2023-03-23 20:47 [PATCH 02/14] arm64: drop ranges in definition of ARCH_FORCE_MAX_ORDER kernel test robot

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.