Re: [PATCH 1/3] bitmap: generalize node_random()

[Yury Norov <yury.norov@gmail.com>]

On Mon, Jun 09, 2025 at 05:44:47AM +0800, kernel test robot wrote:
> Hi Yury,
> 
> kernel test robot noticed the following build errors:
> 
> [auto build test ERROR on akpm-mm/mm-nonmm-unstable]
> [also build test ERROR on tip/timers/core akpm-mm/mm-everything linus/master v6.15 next-20250606]
> [If your patch is applied to the wrong git tree, kindly drop us a note.
> And when submitting patch, we suggest to use '--base' as documented in
> https://git-scm.com/docs/git-format-patch#_base_tree_information]
> 
> url:    https://github.com/intel-lab-lkp/linux/commits/Yury-Norov/bitmap-generalize-node_random/20250609-034657
> base:   https://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm.git mm-nonmm-unstable
> patch link:    https://lore.kernel.org/r/20250608194536.28130-2-yury.norov%40gmail.com
> patch subject: [PATCH 1/3] bitmap: generalize node_random()
> config: arm-randconfig-002-20250609 (https://download.01.org/0day-ci/archive/20250609/202506090550.y3ypu2EO-lkp@intel.com/config)
> compiler: clang version 17.0.6 (https://github.com/llvm/llvm-project 6009708b4367171ccdbf4b5905cb6a803753fe18)
> reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250609/202506090550.y3ypu2EO-lkp@intel.com/reproduce)
> 
> If you fix the issue in a separate patch/commit (i.e. not just a new version of
> the same patch/commit), kindly add following tags
> | Reported-by: kernel test robot <lkp@intel.com>
> | Closes: https://lore.kernel.org/oe-kbuild-all/202506090550.y3ypu2EO-lkp@intel.com/
> 
> All errors (new ones prefixed by >>):
> 
>    In file included from ./arch/arm/include/generated/asm/preempt.h:1:
>    In file included from include/asm-generic/preempt.h:5:
>    In file included from include/linux/thread_info.h:27:
>    In file included from include/linux/bitops.h:29:
>    include/asm-generic/bitops/generic-non-atomic.h:140:16: warning: attribute 'error' is already applied with different arguments [-Wignored-attributes]
>      140 |         return 1UL & (smp_load_acquire(p) >> (nr & (BITS_PER_LONG-1)));
>          |                       ^
>    include/asm-generic/barrier.h:204:37: note: expanded from macro 'smp_load_acquire'
>      204 |         __unqual_scalar_typeof(*p) ___p1 = READ_ONCE(*p);               \
>          |                                            ^
>    include/asm-generic/rwonce.h:49:2: note: expanded from macro 'READ_ONCE'
>       49 |         compiletime_assert_rwonce_type(x);                              \
>          |         ^
>    include/asm-generic/rwonce.h:36:2: note: expanded from macro 'compiletime_assert_rwonce_type'
>       36 |         compiletime_assert(__native_word(t) || sizeof(t) == sizeof(long long),  \
>          |         ^
>    include/linux/compiler_types.h:565:2: note: expanded from macro 'compiletime_assert'
>      565 |         __compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__)
>          |         ^
>    include/linux/compiler_types.h:547:4: note: expanded from macro '__compiletime_assert'
>      547 |                         __compiletime_error(msg);                       \
>          |                         ^
>    include/linux/compiler_attributes.h:138:56: note: expanded from macro '__compiletime_error'
>      138 | # define __compiletime_error(msg)       __attribute__((__error__(msg)))
>          |                                                        ^
>    include/linux/mm.h:2618:9: note: previous attribute is here
>     2618 |         return max(mm->hiwater_rss, get_mm_rss(mm));
>          |                ^
>    include/linux/minmax.h:112:19: note: expanded from macro 'max'
>      112 | #define max(x, y)       __careful_cmp(max, x, y)
>          |                         ^
>    include/linux/minmax.h:98:2: note: expanded from macro '__careful_cmp'
>       98 |         __careful_cmp_once(op, x, y, __UNIQUE_ID(x_), __UNIQUE_ID(y_))
>          |         ^
>    include/linux/minmax.h:93:2: note: expanded from macro '__careful_cmp_once'
>       93 |         BUILD_BUG_ON_MSG(!__types_ok(ux, uy),           \
>          |         ^
>    note: (skipping 1 expansions in backtrace; use -fmacro-backtrace-limit=0 to see all)
>    include/linux/compiler_types.h:565:2: note: expanded from macro 'compiletime_assert'
>      565 |         __compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__)
>          |         ^
>    include/linux/compiler_types.h:547:4: note: expanded from macro '__compiletime_assert'
>      547 |                         __compiletime_error(msg);                       \
>          |                         ^
>    include/linux/compiler_attributes.h:138:56: note: expanded from macro '__compiletime_error'
>      138 | # define __compiletime_error(msg)       __attribute__((__error__(msg)))
>          |                                                        ^
>    In file included from crypto/krb5/rfc8009_aes2.c:10:
>    In file included from include/linux/slab.h:16:
>    In file included from include/linux/gfp.h:7:
>    In file included from include/linux/mmzone.h:8:
>    In file included from include/linux/spinlock.h:56:
>    In file included from include/linux/preempt.h:79:
>    In file included from ./arch/arm/include/generated/asm/preempt.h:1:
>    In file included from include/asm-generic/preempt.h:5:
>    In file included from include/linux/thread_info.h:27:
>    In file included from include/linux/bitops.h:29:
>    include/asm-generic/bitops/generic-non-atomic.h:140:16: warning: attribute 'error' is already applied with different arguments [-Wignored-attributes]
>      140 |         return 1UL & (smp_load_acquire(p) >> (nr & (BITS_PER_LONG-1)));
>          |                       ^
>    include/asm-generic/barrier.h:204:37: note: expanded from macro 'smp_load_acquire'
>      204 |         __unqual_scalar_typeof(*p) ___p1 = READ_ONCE(*p);               \
>          |                                            ^
>    include/asm-generic/rwonce.h:49:2: note: expanded from macro 'READ_ONCE'
>       49 |         compiletime_assert_rwonce_type(x);                              \
>          |         ^
>    include/asm-generic/rwonce.h:36:2: note: expanded from macro 'compiletime_assert_rwonce_type'
>       36 |         compiletime_assert(__native_word(t) || sizeof(t) == sizeof(long long),  \
>          |         ^
>    include/linux/compiler_types.h:565:2: note: expanded from macro 'compiletime_assert'
>      565 |         __compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__)
>          |         ^
>    include/linux/compiler_types.h:547:4: note: expanded from macro '__compiletime_assert'
>      547 |                         __compiletime_error(msg);                       \
>          |                         ^
>    include/linux/compiler_attributes.h:138:56: note: expanded from macro '__compiletime_error'
>      138 | # define __compiletime_error(msg)       __attribute__((__error__(msg)))
>          |                                                        ^
>    include/linux/mm.h:2623:9: note: previous attribute is here
>     2623 |         return max(mm->hiwater_vm, mm->total_vm);
>          |                ^
>    include/linux/minmax.h:112:19: note: expanded from macro 'max'
>      112 | #define max(x, y)       __careful_cmp(max, x, y)
>          |                         ^
>    include/linux/minmax.h:98:2: note: expanded from macro '__careful_cmp'
>       98 |         __careful_cmp_once(op, x, y, __UNIQUE_ID(x_), __UNIQUE_ID(y_))
>          |         ^
>    include/linux/minmax.h:93:2: note: expanded from macro '__careful_cmp_once'
>       93 |         BUILD_BUG_ON_MSG(!__types_ok(ux, uy),           \
>          |         ^
>    note: (skipping 1 expansions in backtrace; use -fmacro-backtrace-limit=0 to see all)
>    include/linux/compiler_types.h:565:2: note: expanded from macro 'compiletime_assert'
>      565 |         __compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__)
>          |         ^
>    include/linux/compiler_types.h:547:4: note: expanded from macro '__compiletime_assert'
>      547 |                         __compiletime_error(msg);                       \
>          |                         ^
>    include/linux/compiler_attributes.h:138:56: note: expanded from macro '__compiletime_error'
>      138 | # define __compiletime_error(msg)       __attribute__((__error__(msg)))
>          |                                                        ^
> >> crypto/krb5/rfc8009_aes2.c:209:3: error: call to undeclared function 'get_random_bytes'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
>      209 |                 get_random_bytes(buffer, krb5->conf_len);
>          |                 ^
>    27 warnings and 1 error generated.

OK, I'll enable those locally, buld-test and send a v3.