Buildresult: linus/pseries_le_defconfig/powerpc-gcc5 built on Dec 8 2019, 06:47

kisskb

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

Status:	OK
Date/Time:	Dec 8 2019, 06:47
Duration:	0:03:03.726862
Builder:	blade46
Revision:	pipe: fix poll/select race introduced by the pipe rework (`ad910e36da4ca3a1bd436989f632d062dda0c921)`
Target:	linus/pseries_le_defconfig/powerpc-gcc5
Branch:	linus
Compiler:	powerpc-gcc5 (powerpc64-linux-gcc (GCC) 5.5.0 / GNU ld (GNU Binutils) 2.29.1.20170915)
Config:	pseries_le_defconfig (download)
Log:	Download original Possible warnings (2) WARNING: vmlinux.o(.text+0x31dc): Section mismatch in reference from the variable __boot_from_prom to the function .init.text:prom_init() WARNING: vmlinux.o(.text+0x33c8): Section mismatch in reference from the variable start_here_common to the function .init.text:start_kernel() Full Log # git rev-parse -q --verify ad910e36da4ca3a1bd436989f632d062dda0c921^{commit} ad910e36da4ca3a1bd436989f632d062dda0c921 already have revision, skipping fetch # git checkout -q -f -B kisskb ad910e36da4ca3a1bd436989f632d062dda0c921 # git clean -qxdf # < git log -1 # commit ad910e36da4ca3a1bd436989f632d062dda0c921 # Author: Linus Torvalds <torvalds@linux-foundation.org> # Date: Sat Dec 7 10:41:17 2019 -0800 # # pipe: fix poll/select race introduced by the pipe rework # # The kernel wait queues have a basic rule to them: you add yourself to # the wait-queue first, and then you check the things that you're going to # wait on. That avoids the races with the event you're waiting for. # # The same goes for poll/select logic: the "poll_wait()" goes first, and # then you check the things you're polling for. # # Of course, if you use locking, the ordering doesn't matter since the # lock will serialize with anything that changes the state you're looking # at. That's not the case here, though. # # So move the poll_wait() first in pipe_poll(), before you start looking # at the pipe state. # # Fixes: 8cefc107ca54 ("pipe: Use head and tail pointers for the ring, not cursor and length") # Cc: David Howells <dhowells@redhat.com> # Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org> # < /opt/cross/kisskb/korg/gcc-5.5.0-nolibc/powerpc64-linux/bin/powerpc64-linux-gcc --version # < /opt/cross/kisskb/korg/gcc-5.5.0-nolibc/powerpc64-linux/bin/powerpc64-linux-ld --version # < git log --format=%s --max-count=1 ad910e36da4ca3a1bd436989f632d062dda0c921 # < make -s -j 32 ARCH=powerpc O=/kisskb/build/linus_pseries_le_defconfig_ppc64le-gcc5 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-5.5.0-nolibc/powerpc64-linux/bin/powerpc64-linux- pseries_le_defconfig # make -s -j 32 ARCH=powerpc O=/kisskb/build/linus_pseries_le_defconfig_ppc64le-gcc5 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-5.5.0-nolibc/powerpc64-linux/bin/powerpc64-linux- WARNING: vmlinux.o(.text+0x31dc): Section mismatch in reference from the variable __boot_from_prom to the function .init.text:prom_init() The function __boot_from_prom() references the function __init prom_init(). This is often because __boot_from_prom lacks a __init annotation or the annotation of prom_init is wrong. WARNING: vmlinux.o(.text+0x33c8): Section mismatch in reference from the variable start_here_common to the function .init.text:start_kernel() The function start_here_common() references the function __init start_kernel(). This is often because start_here_common lacks a __init annotation or the annotation of start_kernel is wrong. Completed OK # rm -rf /kisskb/build/linus_pseries_le_defconfig_ppc64le-gcc5 # Build took: 0:03:03.726862

© Michael Ellerman 2006-2018.