Buildresult: crypto/x86_64-allmodconfig/x86_64-gcc8 built on Nov 15 2019, 17:41

kisskb

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

Status:	OK
Date/Time:	Nov 15 2019, 17:41
Duration:	0:30:20.876889
Builder:	ka4
Revision:	random: Don't freeze in add_hwgenerator_randomness() if stopping kthread (`b5baf00a90701b96e99ca3ca04d9f746bfe8ed89)`
Target:	crypto/x86_64-allmodconfig/x86_64-gcc8
Branch:	crypto
Compiler:	x86_64-gcc8 (x86_64-linux-gcc (GCC) 8.1.0 / GNU ld (GNU Binutils) 2.30)
Config:	allmodconfig (download)
Log:	Download original Possible warnings (4) include/asm-generic/qspinlock.h:65:55: warning: pointer targets in passing argument 2 of 'atomic_try_cmpxchg' differ in signedness [-Wpointer-sign] include/asm-generic/qspinlock.h:78:52: warning: pointer targets in passing argument 2 of 'atomic_try_cmpxchg' differ in signedness [-Wpointer-sign] include/asm-generic/qrwlock.h:65:56: warning: pointer targets in passing argument 2 of 'atomic_try_cmpxchg' differ in signedness [-Wpointer-sign] include/asm-generic/qrwlock.h:92:53: warning: pointer targets in passing argument 2 of 'atomic_try_cmpxchg' differ in signedness [-Wpointer-sign] Full Log # git rev-parse -q --verify b5baf00a90701b96e99ca3ca04d9f746bfe8ed89^{commit} b5baf00a90701b96e99ca3ca04d9f746bfe8ed89 already have revision, skipping fetch # git checkout -q -f -B kisskb b5baf00a90701b96e99ca3ca04d9f746bfe8ed89 # git clean -qxdf # < git log -1 # commit b5baf00a90701b96e99ca3ca04d9f746bfe8ed89 # Author: Maciej S. Szmigiero <mail@maciej.szmigiero.name> # Date: Sun Nov 10 14:55:42 2019 +0100 # # random: Don't freeze in add_hwgenerator_randomness() if stopping kthread # # Since commit 59b569480dc8 # ("random: Use wait_event_freezable() in add_hwgenerator_randomness()") # there is a race in add_hwgenerator_randomness() between freezing and # stopping the calling kthread. # # This commit changed wait_event_interruptible() call with # kthread_freezable_should_stop() as a condition into wait_event_freezable() # with just kthread_should_stop() as a condition to fix a warning that # kthread_freezable_should_stop() might sleep inside the wait. # # wait_event_freezable() ultimately calls __refrigerator() with its # check_kthr_stop argument set to false, which causes it to keep the kthread # frozen even if somebody calls kthread_stop() on it. # # Calling wait_event_freezable() with kthread_should_stop() as a condition # is racy because it doesn't take into account the situation where this # condition becomes true on a kthread marked for freezing only after this # condition has already been checked. # # Calling freezing() should avoid the issue that the commit 59b569480dc8 has # fixed, as it is only a checking function, it doesn't actually do the # freezing. # # add_hwgenerator_randomness() has two post-boot users: in khwrng the # kthread will be frozen anyway by call to kthread_freezable_should_stop() # in its main loop, while its second user (ath9k-hwrng) is not freezable at # all. # # This change allows a VM with virtio-rng loaded to write s2disk image # successfully. # # Fixes: 59b569480dc8 ("random: Use wait_event_freezable() in add_hwgenerator_randomness()") # Signed-off-by: Maciej S. Szmigiero <mail@maciej.szmigiero.name> # Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au> # < /opt/cross/kisskb/korg/gcc-8.1.0-nolibc/x86_64-linux/bin/x86_64-linux-gcc --version # < /opt/cross/kisskb/korg/gcc-8.1.0-nolibc/x86_64-linux/bin/x86_64-linux-ld --version # < git log --format=%s --max-count=1 b5baf00a90701b96e99ca3ca04d9f746bfe8ed89 # < make -s -j 10 ARCH=x86 O=/kisskb/build/crypto_x86-allmodconfig_x86_64-gcc8 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-8.1.0-nolibc/x86_64-linux/bin/x86_64-linux- allmodconfig # Added to kconfig CONFIG_BUILD_DOCSRC=n # Added to kconfig CONFIG_MODULE_SIG=n # Added to kconfig CONFIG_SAMPLES=n # yes \n \| make -s -j 10 ARCH=x86 O=/kisskb/build/crypto_x86-allmodconfig_x86_64-gcc8 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-8.1.0-nolibc/x86_64-linux/bin/x86_64-linux- oldconfig yes: standard output: Broken pipe # make -s -j 10 ARCH=x86 O=/kisskb/build/crypto_x86-allmodconfig_x86_64-gcc8 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-8.1.0-nolibc/x86_64-linux/bin/x86_64-linux- In file included from /kisskb/src/arch/x86/include/asm/bitops.h:16, from /kisskb/src/include/linux/bitops.h:19, from /kisskb/src/lib/crypto/sha256.c:14: /kisskb/src/include/asm-generic/qspinlock.h: In function 'queued_spin_trylock': /kisskb/src/include/asm-generic/qspinlock.h:65:55: warning: pointer targets in passing argument 2 of 'atomic_try_cmpxchg' differ in signedness [-Wpointer-sign] return likely(atomic_try_cmpxchg_acquire(&lock->val, &val, _Q_LOCKED_VAL)); ^~~~ /kisskb/src/include/linux/compiler.h:77:40: note: in definition of macro 'likely' # define likely(x) __builtin_expect(!!(x), 1) ^ In file included from /kisskb/src/arch/x86/include/asm/atomic.h:265, from /kisskb/src/arch/x86/include/asm/msr.h:67, from /kisskb/src/arch/x86/include/asm/processor.h:21, from /kisskb/src/arch/x86/include/asm/cpufeature.h:5, from /kisskb/src/arch/x86/include/asm/thread_info.h:53, from /kisskb/src/include/linux/thread_info.h:38, from /kisskb/src/arch/x86/include/asm/preempt.h:7, from /kisskb/src/include/linux/preempt.h:78, from /kisskb/src/include/linux/spinlock.h:51, from /kisskb/src/include/linux/seqlock.h:36, from /kisskb/src/include/linux/time.h:6, from /kisskb/src/include/linux/stat.h:19, from /kisskb/src/include/linux/module.h:10, from /kisskb/src/lib/crypto/sha256.c:16: /kisskb/src/include/asm-generic/atomic-instrumented.h:691:1: note: expected 'int ' but argument is of type 'u32 ' {aka 'unsigned int '} atomic_try_cmpxchg(atomic_t v, int old, int new) ^~~~~~~~~~~~~~~~~~ In file included from /kisskb/src/arch/x86/include/asm/bitops.h:16, from /kisskb/src/include/linux/bitops.h:19, from /kisskb/src/lib/crypto/sha256.c:14: /kisskb/src/include/asm-generic/qspinlock.h: In function 'queued_spin_lock': /kisskb/src/include/asm-generic/qspinlock.h:78:52: warning: pointer targets in passing argument 2 of 'atomic_try_cmpxchg' differ in signedness [-Wpointer-sign] if (likely(atomic_try_cmpxchg_acquire(&lock->val, &val, _Q_LOCKED_VAL))) ^~~~ /kisskb/src/include/linux/compiler.h:77:40: note: in definition of macro 'likely' # define likely(x) __builtin_expect(!!(x), 1) ^ In file included from /kisskb/src/arch/x86/include/asm/atomic.h:265, from /kisskb/src/arch/x86/include/asm/msr.h:67, from /kisskb/src/arch/x86/include/asm/processor.h:21, from /kisskb/src/arch/x86/include/asm/cpufeature.h:5, from /kisskb/src/arch/x86/include/asm/thread_info.h:53, from /kisskb/src/include/linux/thread_info.h:38, from /kisskb/src/arch/x86/include/asm/preempt.h:7, from /kisskb/src/include/linux/preempt.h:78, from /kisskb/src/include/linux/spinlock.h:51, from /kisskb/src/include/linux/seqlock.h:36, from /kisskb/src/include/linux/time.h:6, from /kisskb/src/include/linux/stat.h:19, from /kisskb/src/include/linux/module.h:10, from /kisskb/src/lib/crypto/sha256.c:16: /kisskb/src/include/asm-generic/atomic-instrumented.h:691:1: note: expected 'int ' but argument is of type 'u32 ' {aka 'unsigned int '} atomic_try_cmpxchg(atomic_t v, int old, int new) ^~~~~~~~~~~~~~~~~~ In file included from /kisskb/src/arch/x86/include/asm/bitops.h:16, from /kisskb/src/include/linux/bitops.h:19, from /kisskb/src/lib/crypto/sha256.c:14: /kisskb/src/include/asm-generic/qrwlock.h: In function 'queued_write_trylock': /kisskb/src/include/asm-generic/qrwlock.h:65:56: warning: pointer targets in passing argument 2 of 'atomic_try_cmpxchg' differ in signedness [-Wpointer-sign] return likely(atomic_try_cmpxchg_acquire(&lock->cnts, &cnts, ^~~~~ /kisskb/src/include/linux/compiler.h:77:40: note: in definition of macro 'likely' # define likely(x) __builtin_expect(!!(x), 1) ^ In file included from /kisskb/src/arch/x86/include/asm/atomic.h:265, from /kisskb/src/arch/x86/include/asm/msr.h:67, from /kisskb/src/arch/x86/include/asm/processor.h:21, from /kisskb/src/arch/x86/include/asm/cpufeature.h:5, from /kisskb/src/arch/x86/include/asm/thread_info.h:53, from /kisskb/src/include/linux/thread_info.h:38, from /kisskb/src/arch/x86/include/asm/preempt.h:7, from /kisskb/src/include/linux/preempt.h:78, from /kisskb/src/include/linux/spinlock.h:51, from /kisskb/src/include/linux/seqlock.h:36, from /kisskb/src/include/linux/time.h:6, from /kisskb/src/include/linux/stat.h:19, from /kisskb/src/include/linux/module.h:10, from /kisskb/src/lib/crypto/sha256.c:16: /kisskb/src/include/asm-generic/atomic-instrumented.h:691:1: note: expected 'int ' but argument is of type 'u32 ' {aka 'unsigned int '} atomic_try_cmpxchg(atomic_t v, int old, int new) ^~~~~~~~~~~~~~~~~~ In file included from /kisskb/src/arch/x86/include/asm/bitops.h:16, from /kisskb/src/include/linux/bitops.h:19, from /kisskb/src/lib/crypto/sha256.c:14: /kisskb/src/include/asm-generic/qrwlock.h: In function 'queued_write_lock': /kisskb/src/include/asm-generic/qrwlock.h:92:53: warning: pointer targets in passing argument 2 of 'atomic_try_cmpxchg' differ in signedness [-Wpointer-sign] if (likely(atomic_try_cmpxchg_acquire(&lock->cnts, &cnts, _QW_LOCKED))) ^~~~~ /kisskb/src/include/linux/compiler.h:77:40: note: in definition of macro 'likely' # define likely(x) __builtin_expect(!!(x), 1) ^ In file included from /kisskb/src/arch/x86/include/asm/atomic.h:265, from /kisskb/src/arch/x86/include/asm/msr.h:67, from /kisskb/src/arch/x86/include/asm/processor.h:21, from /kisskb/src/arch/x86/include/asm/cpufeature.h:5, from /kisskb/src/arch/x86/include/asm/thread_info.h:53, from /kisskb/src/include/linux/thread_info.h:38, from /kisskb/src/arch/x86/include/asm/preempt.h:7, from /kisskb/src/include/linux/preempt.h:78, from /kisskb/src/include/linux/spinlock.h:51, from /kisskb/src/include/linux/seqlock.h:36, from /kisskb/src/include/linux/time.h:6, from /kisskb/src/include/linux/stat.h:19, from /kisskb/src/include/linux/module.h:10, from /kisskb/src/lib/crypto/sha256.c:16: /kisskb/src/include/asm-generic/atomic-instrumented.h:691:1: note: expected 'int ' but argument is of type 'u32 ' {aka 'unsigned int '} atomic_try_cmpxchg(atomic_t v, int old, int new) ^~~~~~~~~~~~~~~~~~ Completed OK # rm -rf /kisskb/build/crypto_x86-allmodconfig_x86_64-gcc8 # Build took: 0:30:20.876889

© Michael Ellerman 2006-2018.