Buildresult: gustavoars/cavium_octeon_defconfig/mips-gcc8 built on Jun 28 2022, 23:17

kisskb

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

Status:	OK
Date/Time:	Jun 28 2022, 23:17
Duration:	0:00:33.861238
Builder:	ka4
Revision:	treewide: uapi: Replace zero-length arrays with flexible-array members (`7f62ad1aedf6048a3d0354a9838415f97bb5ec14)`
Target:	gustavoars/cavium_octeon_defconfig/mips-gcc8
Branch:	gustavoars
Compiler:	mips-gcc8 (mips-linux-gcc (GCC) 8.5.0 / GNU ld (GNU Binutils) 2.36.1)
Config:	cavium_octeon_defconfig (download)
Log:	Download original No warnings found in log. Full Log # git rev-parse -q --verify 7f62ad1aedf6048a3d0354a9838415f97bb5ec14^{commit} 7f62ad1aedf6048a3d0354a9838415f97bb5ec14 already have revision, skipping fetch # git checkout -q -f -B kisskb 7f62ad1aedf6048a3d0354a9838415f97bb5ec14 # git clean -qxdf # < git log -1 # commit 7f62ad1aedf6048a3d0354a9838415f97bb5ec14 # Author: Gustavo A. R. Silva <gustavoars@kernel.org> # Date: Wed Apr 6 19:36:51 2022 -0500 # # treewide: uapi: Replace zero-length arrays with flexible-array members # # There is a regular need in the kernel to provide a way to declare # having a dynamically sized set of trailing elements in a structure. # Kernel code should always use “flexible array members”[1] for these # cases. The older style of one-element or zero-length arrays should # no longer be used[2]. # # This code was transformed with the help of Coccinelle: # (linux-5.19-rc2$ spatch --jobs $(getconf _NPROCESSORS_ONLN) --sp-file script.cocci --include-headers --dir . > output.patch) # # @@ # identifier S, member, array; # type T1, T2; # @@ # # struct S { # ... # T1 member; # T2 array[ # - 0 # ]; # }; # # -fstrict-flex-arrays=3 is coming and we need to land these changes # to prevent issues like these in the short future: # # ../fs/minix/dir.c:337:3: warning: 'strcpy' will always overflow; destination buffer has size 0, # but the source string has length 2 (including NUL byte) [-Wfortify-source] # strcpy(de3->name, "."); # ^ # # Since these are all [0] to [] changes, the risk to UAPI is nearly zero. If # this breaks anything, we can use a union with a new member name. # # [1] https://en.wikipedia.org/wiki/Flexible_array_member # [2] https://www.kernel.org/doc/html/v5.16/process/deprecated.html#zero-length-and-one-element-arrays # # Link: https://github.com/KSPP/linux/issues/78 # Build-tested-by: kernel test robot <lkp@intel.com> # Link: https://lore.kernel.org/lkml/62b675ec.wKX6AOZ6cbE71vtF%25lkp@intel.com/ # Acked-by: Dan Williams <dan.j.williams@intel.com> # For ndctl.h # Signed-off-by: Gustavo A. R. Silva <gustavoars@kernel.org> # < /opt/cross/kisskb/korg/gcc-8.5.0-nolibc/mips-linux/bin/mips-linux-gcc --version # < /opt/cross/kisskb/korg/gcc-8.5.0-nolibc/mips-linux/bin/mips-linux-ld --version # < git log --format=%s --max-count=1 7f62ad1aedf6048a3d0354a9838415f97bb5ec14 # < make -s -j 120 ARCH=mips O=/kisskb/build/gustavoars_cavium_octeon_defconfig_mips-gcc8 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-8.5.0-nolibc/mips-linux/bin/mips-linux- cavium_octeon_defconfig # < make -s -j 120 ARCH=mips O=/kisskb/build/gustavoars_cavium_octeon_defconfig_mips-gcc8 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-8.5.0-nolibc/mips-linux/bin/mips-linux- help # make -s -j 120 ARCH=mips O=/kisskb/build/gustavoars_cavium_octeon_defconfig_mips-gcc8 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-8.5.0-nolibc/mips-linux/bin/mips-linux- olddefconfig # make -s -j 120 ARCH=mips O=/kisskb/build/gustavoars_cavium_octeon_defconfig_mips-gcc8 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-8.5.0-nolibc/mips-linux/bin/mips-linux- Completed OK # rm -rf /kisskb/build/gustavoars_cavium_octeon_defconfig_mips-gcc8 # Build took: 0:00:33.861238

© Michael Ellerman 2006-2018.