Buildresult: linus/sun3_defconfig/m68k-gcc8 built on Aug 26, 11:32

kisskb

Revisions | Branches | Compilers | Configs | Build Results | Build Failures |

Status:	OK
Date/Time:	Aug 26, 11:32
Duration:	0:01:22.176955
Builder:	alpine3
Revision:	lib/clz_ctz.c: Fix __clzdi2() and __ctzdi2() for 32-bit kernels (`382d4cd1847517ffcb1800fd462b625db7b2ebea)`
Target:	linus/sun3_defconfig/m68k-gcc8
Branch:	linus
Compiler:	m68k-gcc8 (m68k-linux-gcc (GCC) 8.5.0 / GNU ld (GNU Binutils) 2.36.1)
Config:	sun3_defconfig (download)
Log:	Download original Possible warnings (1) drivers/net/ethernet/i825xx/sun3_82586.c:990:108: warning: array subscript 1 is above array bounds of 'volatile struct transmit_cmd_struct [1]' [-Warray-bounds] Full Log # git rev-parse -q --verify 382d4cd1847517ffcb1800fd462b625db7b2ebea^{commit} 382d4cd1847517ffcb1800fd462b625db7b2ebea already have revision, skipping fetch # git checkout -q -f -B kisskb 382d4cd1847517ffcb1800fd462b625db7b2ebea # git clean -qxdf # < git log -1 # commit 382d4cd1847517ffcb1800fd462b625db7b2ebea # Author: Helge Deller <deller@gmx.de> # Date: Fri Aug 25 21:50:33 2023 +0200 # # lib/clz_ctz.c: Fix __clzdi2() and __ctzdi2() for 32-bit kernels # # The gcc compiler translates on some architectures the 64-bit # __builtin_clzll() function to a call to the libgcc function __clzdi2(), # which should take a 64-bit parameter on 32- and 64-bit platforms. # # But in the current kernel code, the built-in __clzdi2() function is # defined to operate (wrongly) on 32-bit parameters if BITS_PER_LONG == # 32, thus the return values on 32-bit kernels are in the range from # [0..31] instead of the expected [0..63] range. # # This patch fixes the in-kernel functions __clzdi2() and __ctzdi2() to # take a 64-bit parameter on 32-bit kernels as well, thus it makes the # functions identical for 32- and 64-bit kernels. # # This bug went unnoticed since kernel 3.11 for over 10 years, and here # are some possible reasons for that: # # a) Some architectures have assembly instructions to count the bits and # which are used instead of calling __clzdi2(), e.g. on x86 the bsr # instruction and on ppc cntlz is used. On such architectures the # wrong __clzdi2() implementation isn't used and as such the bug has # no effect and won't be noticed. # # b) Some architectures link to libgcc.a, and the in-kernel weak # functions get replaced by the correct 64-bit variants from libgcc.a. # # c) __builtin_clzll() and __clzdi2() doesn't seem to be used in many # places in the kernel, and most likely only in uncritical functions, # e.g. when printing hex values via seq_put_hex_ll(). The wrong return # value will still print the correct number, but just in a wrong # formatting (e.g. with too many leading zeroes). # # d) 32-bit kernels aren't used that much any longer, so they are less # tested. # # A trivial testcase to verify if the currently running 32-bit kernel is # affected by the bug is to look at the output of /proc/self/maps: # # Here the kernel uses a correct implementation of __clzdi2(): # # root@debian:~# cat /proc/self/maps # 00010000-00019000 r-xp 00000000 08:05 787324 /usr/bin/cat # 00019000-0001a000 rwxp 00009000 08:05 787324 /usr/bin/cat # 0001a000-0003b000 rwxp 00000000 00:00 0 [heap] # f7551000-f770d000 r-xp 00000000 08:05 794765 /usr/lib/hppa-linux-gnu/libc.so.6 # ... # # and this kernel uses the broken implementation of __clzdi2(): # # root@debian:~# cat /proc/self/maps # 0000000010000-0000000019000 r-xp 00000000 000000008:000000005 787324 /usr/bin/cat # 0000000019000-000000001a000 rwxp 000000009000 000000008:000000005 787324 /usr/bin/cat # 000000001a000-000000003b000 rwxp 00000000 00:00 0 [heap] # 00000000f73d1000-00000000f758d000 r-xp 00000000 000000008:000000005 794765 /usr/lib/hppa-linux-gnu/libc.so.6 # ... # # Signed-off-by: Helge Deller <deller@gmx.de> # Fixes: 4df87bb7b6a22 ("lib: add weak clz/ctz functions") # Cc: Chanho Min <chanho.min@lge.com> # Cc: Geert Uytterhoeven <geert@linux-m68k.org> # Cc: stable@vger.kernel.org # v3.11+ # Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org> # < /opt/cross/kisskb/korg/gcc-8.5.0-nolibc/m68k-linux/bin/m68k-linux-gcc --version # < /opt/cross/kisskb/korg/gcc-8.5.0-nolibc/m68k-linux/bin/m68k-linux-ld --version # < git log --format=%s --max-count=1 382d4cd1847517ffcb1800fd462b625db7b2ebea # make -s -j 160 ARCH=m68k O=/kisskb/build/linus_sun3_defconfig_m68k-gcc8 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-8.5.0-nolibc/m68k-linux/bin/m68k-linux- sun3_defconfig # < make -s -j 160 ARCH=m68k O=/kisskb/build/linus_sun3_defconfig_m68k-gcc8 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-8.5.0-nolibc/m68k-linux/bin/m68k-linux- help # make -s -j 160 ARCH=m68k O=/kisskb/build/linus_sun3_defconfig_m68k-gcc8 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-8.5.0-nolibc/m68k-linux/bin/m68k-linux- olddefconfig # make -s -j 160 ARCH=m68k O=/kisskb/build/linus_sun3_defconfig_m68k-gcc8 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-8.5.0-nolibc/m68k-linux/bin/m68k-linux- In file included from /kisskb/src/include/linux/swab.h:5, from /kisskb/src/include/uapi/linux/byteorder/big_endian.h:14, from /kisskb/src/include/linux/byteorder/big_endian.h:5, from /kisskb/src/arch/m68k/include/uapi/asm/byteorder.h:5, from /kisskb/src/include/asm-generic/bitops/le.h:6, from /kisskb/src/arch/m68k/include/asm/bitops.h:545, from /kisskb/src/include/linux/bitops.h:68, from /kisskb/src/include/linux/kernel.h:22, from /kisskb/src/drivers/net/ethernet/i825xx/sun3_82586.c:32: /kisskb/src/drivers/net/ethernet/i825xx/sun3_82586.c: In function 'sun3_82586_timeout': /kisskb/src/drivers/net/ethernet/i825xx/sun3_82586.c:990:108: warning: array subscript 1 is above array bounds of 'volatile struct transmit_cmd_struct [1]' [-Warray-bounds] printk("%s: command-stats: %04x %04x\n",dev->name,swab16(p->xmit_cmds[0]->cmd_status),swab16(p->xmit_cmds[1]->cmd_status)); ~~~~~~~~~~~~^~~ /kisskb/src/include/uapi/linux/swab.h:107:12: note: in definition of macro '__swab16' __fswab16(x)) ^ /kisskb/src/include/linux/printk.h:455:26: note: in expansion of macro 'printk_index_wrap' #define printk(fmt, ...) printk_index_wrap(_printk, fmt, ##__VA_ARGS__) ^~~~~~~~~~~~~~~~~ /kisskb/src/drivers/net/ethernet/i825xx/sun3_82586.c:990:3: note: in expansion of macro 'printk' printk("%s: command-stats: %04x %04x\n",dev->name,swab16(p->xmit_cmds[0]->cmd_status),swab16(p->xmit_cmds[1]->cmd_status)); ^~~~~~ Completed OK # rm -rf /kisskb/build/linus_sun3_defconfig_m68k-gcc8 # Build took: 0:01:22.176955

© Michael Ellerman 2006-2018.