Re: [PATCH v2 2/2] modpost: skip ELF local symbols by default during section mismatch check

2018-11-21 Thread Paul Walmsley
Hello Masahiro, On Wed, 21 Nov 2018, Masahiro Yamada wrote: I think this is almost good. Just a nit. Maybe, putting this check in secref_whitelist() (with a comment "Pattern 6:") could look more consistency. Also, if you use strstart() helper, you can remove is_local_symbol() function.

Re: [PATCH v2 2/2] modpost: skip ELF local symbols by default during section mismatch check

2018-11-20 Thread Masahiro Yamada
Hi Paul, On Thu, Nov 15, 2018 at 9:57 AM Paul Walmsley wrote: > > During development of a serial console driver with a gcc 8.2.0 > toolchain for RISC-V, the following modpost warning appeared: > > > WARNING: vmlinux.o(.data+0x19b10): Section mismatch in reference from the > variable .LANCH

Re: [PATCH v2 2/2] modpost: skip ELF local symbols by default during section mismatch check

2018-11-16 Thread Sam Ravnborg
On Wed, Nov 14, 2018 at 04:56:02PM -0800, Paul Walmsley wrote: > During development of a serial console driver with a gcc 8.2.0 > toolchain for RISC-V, the following modpost warning appeared: > > > WARNING: vmlinux.o(.data+0x19b10): Section mismatch in reference from the > variable .LANCHOR1

[PATCH v2 2/2] modpost: skip ELF local symbols by default during section mismatch check

2018-11-14 Thread Paul Walmsley
During development of a serial console driver with a gcc 8.2.0 toolchain for RISC-V, the following modpost warning appeared: WARNING: vmlinux.o(.data+0x19b10): Section mismatch in reference from the variable .LANCHOR1 to the function .init.text:sifive_serial_console_setup() The variable .LAN