Re: Build regressions/improvements in v6.8-rc7

Re: Build regressions/improvements in v6.8-rc7

[Geert Uytterhoeven]

On Mon, 4 Mar 2024, Geert Uytterhoeven wrote:
> JFYI, when comparing v6.8-rc7[1] to v6.8-rc6[3], the summaries are:
>  - build errors: +6/-0

   + /kisskb/src/drivers/sbus/char/bbc_envctrl.c: error: no previous prototype for 'bbc_envctrl_cleanup' [-Werror=missing-prototypes]:  => 594:6
   + /kisskb/src/drivers/sbus/char/bbc_envctrl.c: error: no previous prototype for 'bbc_envctrl_init' [-Werror=missing-prototypes]:  => 566:5

sparc64-gcc13/sparc64-allmodconfig (seen before)

   + /kisskb/src/include/linux/rcupdate.h: error: dereferencing pointer to incomplete type 'struct dpll_pin':  => 462:36

arm64-gcc5/arm64-allmodconfig
mips-gcc8/mips-allmodconfig
powerpc-gcc5/powerpc-all{mod,yes}config
powerpc-gcc5/{ppc32,ppc64_book3e,ppc64le}_allmodconfig
sparc64-gcc5/sparc{,64}-allmodconfig
x86_64-gcc8/x86-allmodconfig

     src/net/core/dev.c: In function 'netdev_dpll_pin_assign':
     include/linux/rcupdate.h:462:36: error: dereferencing pointer to incomplete type 'struct dpll_pin'
      #define RCU_INITIALIZER(v) (typeof(*(v)) __force __rcu *)(v)
 					^~~~
     include/asm-generic/rwonce.h:55:33: note: in definition of macro '__WRITE_ONCE'
       *(volatile typeof(x) *)&(x) = (val);    \
 				     ^~~
     arch/x86/include/asm/barrier.h:67:2: note: in expansion of macro 'WRITE_ONCE'
       WRITE_ONCE(*p, v);      \
       ^~~~~~~~~~
     include/asm-generic/barrier.h:172:55: note: in expansion of macro '__smp_store_release'
      #define smp_store_release(p, v) do { kcsan_release(); __smp_store_release(p, v); } while (0)
 							   ^~~~~~~~~~~~~~~~~~~
     include/linux/rcupdate.h:503:3: note: in expansion of macro 'smp_store_release'
        smp_store_release(&p, RCU_INITIALIZER((typeof(p))_r_a_p__v)); \
        ^~~~~~~~~~~~~~~~~
     include/linux/rcupdate.h:503:25: note: in expansion of macro 'RCU_INITIALIZER'
        smp_store_release(&p, RCU_INITIALIZER((typeof(p))_r_a_p__v)); \
 			     ^~~~~~~~~~~~~~~
     net/core/dev.c:9081:2: note: in expansion of macro 'rcu_assign_pointer'
       rcu_assign_pointer(dev->dpll_pin, dpll_pin);
       ^~~~~~~~~~~~~~~~~~

   + {standard input}: Error: invalid operands for opcode:  => 606
   + {standard input}: Error: missing operand:  => 606
   + {standard input}: Error: unknown pseudo-op: `.cfi_def_cfa_offse':  => 605

SH ICE crickets

*** WARNINGS ***

> [1] http://kisskb.ellerman.id.au/kisskb/branch/linus/head/90d35da658da8cff0d4ecbb5113f5fac9d00eb72/ (138 out of 239 configs)

> [3] http://kisskb.ellerman.id.au/kisskb/branch/linus/head/d206a76d7d2726f3b096037f2079ce0bd3ba329b/ (138 out of 239 configs)

Gr{oetje,eeting}s,

 						Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
 							    -- Linus Torvalds

Re: Build regressions/improvements in v6.8-rc7

[Jakub Kicinski]

On Mon, 4 Mar 2024 10:00:07 +0100 (CET) Geert Uytterhoeven wrote:
> arm64-gcc5/arm64-allmodconfig
> mips-gcc8/mips-allmodconfig
> powerpc-gcc5/powerpc-all{mod,yes}config
> powerpc-gcc5/{ppc32,ppc64_book3e,ppc64le}_allmodconfig
> sparc64-gcc5/sparc{,64}-allmodconfig
> x86_64-gcc8/x86-allmodconfig

Ugh, we had an earlier report but it didn't cover rcu_assign_pointer()
or at least we missed if it did. Fix on the way, thanks for the report!