Messages in this thread | | | Date | Wed, 22 Jun 2016 14:50:13 +0200 | From | Thierry Reding <> | Subject | Re: [PATCH] ARM: tegra: remove board_init_funcs array |
| |
On Wed, Jun 22, 2016 at 02:39:41PM +0200, Arnd Bergmann wrote: > In a configuration that enables CONFIG_UBSAN_SANITIZE_ALL, I am getting > a section mismatch warning for tegra20: > > WARNING: arch/arm/mach-tegra/built-in.o(.data+0x6e0): Section mismatch in reference from the variable board_init_funcs to the function .init.text:paz00_init() > > The array is no longer useful here since there is only one entry, > so we can simply call the function directly after checking > of_machine_is_compatible(). This fixes the section mismatch > and is easier to read. > > Signed-off-by: Arnd Bergmann <arnd@arndb.de> > --- > arch/arm/mach-tegra/tegra.c | 24 +++--------------------- > 1 file changed, 3 insertions(+), 21 deletions(-)
Shouldn't these section mismatches show up with default builds? I haven't seen any when building Tegra configurations.
I'm going to apply this patch because I think it's useful, but it sure would be nice to know why I need to enable this new UBSAN stuff to get these warnings now.
Thierry [unhandled content-type:application/pgp-signature] | |