Re: [PATCH] sh: Fix build with CONFIG_UBSAN=y

From: "Liu, Yujie" <yujie.liu@intel.com>
To: "glaubitz@physik.fu-berlin.de" <glaubitz@physik.fu-berlin.de>
Cc: lkp <lkp@intel.com>,
	"masahiroy@kernel.org" <masahiroy@kernel.org>,
	"dalias@libc.org" <dalias@libc.org>,
	"linux-sh@vger.kernel.org" <linux-sh@vger.kernel.org>,
	"keescook@chromium.org" <keescook@chromium.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"ysato@users.osdn.me" <ysato@users.osdn.me>,
	"linux-hardening@vger.kernel.org"
	<linux-hardening@vger.kernel.org>,
	"Schier, Nicolas" <n.schier@avm.de>
Subject: Re: [PATCH] sh: Fix build with CONFIG_UBSAN=y
Date: Fri, 1 Mar 2024 05:46:45 +0000	[thread overview]
Message-ID: <e7a80eeafa9b916108ee53f2fe60524554ab0fa6.camel@intel.com> (raw)
In-Reply-To: <93572a771cb3a0b36a9213f18dd43c034c84cf6d.camel@physik.fu-berlin.de>

Hi Adrian,

On Wed, 2024-02-14 at 13:52 +0100, John Paul Adrian Glaubitz wrote:
> Hi Kees,
> 
> On Mon, 2024-02-12 at 19:45 +0100, John Paul Adrian Glaubitz wrote:
> > On Mon, 2024-02-12 at 10:26 -0800, Kees Cook wrote:
> > > > I just wanted to try reproduce the problem again with the reproducer in [1] as well
> > > > as with gcc-13.2.0, but your branch devel/overflow/ubsan-only no longer exists.
> > > > 
> > > > Can you tell me where to find the patches now?
> > > 
> > > Sure, they're in -next, but for an -rc2 based tree, see:
> > > https://git.kernel.org/pub/scm/linux/kernel/git/kees/linux.git/log/?h=for-next/kspp
> > 
> > OK, thanks. I will give it a try with gcc-13.
> 
> I'm still unable to reproduce the error that the kernel test robot reported.
> 
> I'm using gcc-13:
> 
> glaubitz@node54:/data/home/glaubitz/linux-kees> sh4-linux-gcc
> sh4-linux-gcc: fatal error: no input files
> compilation terminated.
> glaubitz@node54:/data/home/glaubitz/linux-kees> sh4-linux-gcc --version
> sh4-linux-gcc (GCC) 13.2.0
> Copyright (C) 2023 Free Software Foundation, Inc.
> This is free software; see the source for copying conditions.  There is NO
> warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
> 
> glaubitz@node54:/data/home/glaubitz/linux-kees>
> 
> I checked out your tree and the for-next/kspp branch.
> 
> Then fetched the config that triggered the bug like this:
> 
> $ wget https://download.01.org/0day-ci/archive/20240131/202401310416.s8HLiLnC-lkp@intel.com/config -O .config
> 
> Building the kernel with "make -j32 uImage" works fine. No errors except for
> some unreleated warnings that still need to be fixed.

Sorry for late reply. Seems like the warnings can be reproduced by
"make zImage" which is the default make target, but cannot be
reproduced by "make uImage".

HEAD is now at 918327e9b7ffb ubsan: Remove CONFIG_UBSAN_SANITIZE_ALL

$ wget https://download.01.org/0day-ci/archive/20240131/202401310416.s8HLiLnC-lkp@intel.com/config -O .config
$ sh4-linux-gcc --version
sh4-linux-gcc (GCC) 13.2.0

$ make -j72 CROSS_COMPILE=sh4-linux- ARCH=sh olddefconfig
$ make -j72 CROSS_COMPILE=sh4-linux- ARCH=sh
...
  LD      arch/sh/boot/compressed/vmlinux
sh4-linux-ld: arch/sh/boot/compressed/misc.o: in function `zlib_inflate_table':
misc.c:(.text+0x670): undefined reference to `__ubsan_handle_shift_out_of_bounds'
sh4-linux-ld: arch/sh/boot/compressed/misc.o: in function `inflate_fast':
misc.c:(.text+0xc5c): undefined reference to `__ubsan_handle_shift_out_of_bounds'
sh4-linux-ld: arch/sh/boot/compressed/misc.o: in function `zlib_inflateReset':
misc.c:(.text+0xd00): undefined reference to `__ubsan_handle_shift_out_of_bounds'
sh4-linux-ld: arch/sh/boot/compressed/misc.o: in function `zlib_inflate':
misc.c:(.text+0x23fc): undefined reference to `__ubsan_handle_shift_out_of_bounds'
sh4-linux-ld: arch/sh/boot/compressed/ashldi3.o: in function `__ashldi3':
ashldi3.c:(.text+0xc8): undefined reference to `__ubsan_handle_shift_out_of_bounds'
make[3]: *** [arch/sh/boot/compressed/Makefile:38: arch/sh/boot/compressed/vmlinux] Error 1
make[2]: *** [arch/sh/boot/Makefile:40: arch/sh/boot/compressed/vmlinux] Error 2
make[1]: *** [arch/sh/Makefile:170: zImage] Error 2
make: *** [Makefile:240: __sub-make] Error 2

$ make -j72 CROSS_COMPILE=sh4-linux- ARCH=sh uImage
...
  OBJCOPY arch/sh/boot/vmlinux.bin
  GZIP    arch/sh/boot/vmlinux.bin.gz
  UIMAGE  arch/sh/boot/uImage.gz
Image Name:   Linux-6.8.0-rc2+
Created:      Fri Mar  1 13:31:36 2024
Image Type:   SuperH Linux Kernel Image (gzip compressed)
Data Size:    9297141 Bytes = 9079.24 KiB = 8.87 MiB
Load Address: 08001000
Entry Point:  08002000
  Image arch/sh/boot/uImage is ready

--
Best Regards,
Yujie