Buildresult: linus/um-defconfig/um-x86_64 built on Jul 6 2020, 08:55

kisskb

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

Status:	OK
Date/Time:	Jul 6 2020, 08:55
Duration:	0:00:41.187568
Builder:	ka2
Revision:	x86/ldt: use "pr_info_once()" instead of open-coding it badly (`bb5a93aaf25261321db0c499cde7da6ee9d8b164)`
Target:	linus/um-defconfig/um-x86_64
Branch:	linus
Compiler:	um-x86_64 (x86_64-linux-gcc.br_real (Buildroot 2017.05) 5.4.0 / GNU ld (GNU Binutils) 2.27)
Config:	defconfig (download)
Log:	Download original Possible warnings (2) arch/um/os-Linux/signal.c:51:1: warning: the frame size of 2960 bytes is larger than 1024 bytes [-Wframe-larger-than=] arch/um/os-Linux/signal.c:95:1: warning: the frame size of 2960 bytes is larger than 1024 bytes [-Wframe-larger-than=] Full Log # git rev-parse -q --verify bb5a93aaf25261321db0c499cde7da6ee9d8b164^{commit} bb5a93aaf25261321db0c499cde7da6ee9d8b164 already have revision, skipping fetch # git checkout -q -f -B kisskb bb5a93aaf25261321db0c499cde7da6ee9d8b164 # git clean -qxdf # < git log -1 # commit bb5a93aaf25261321db0c499cde7da6ee9d8b164 # Author: Linus Torvalds <torvalds@linux-foundation.org> # Date: Sun Jul 5 12:50:20 2020 -0700 # # x86/ldt: use "pr_info_once()" instead of open-coding it badly # # Using a mutex for "print this warning only once" is so overdesigned as # to be actively offensive to my sensitive stomach. # # Just use "pr_info_once()" that already does this, although in a # (harmlessly) racy manner that can in theory cause the message to be # printed twice if more than one CPU races on that "is this the first # time" test. # # [ If somebody really cares about that harmless data race (which sounds # very unlikely indeed), that person can trivially fix printk_once() by # using a simple atomic access, preferably with an optimistic non-atomic # test first before even bothering to treat the pointless "make sure it # is _really_ just once" case. # # A mutex is most definitely never the right primitive to use for # something like this. ] # # Yes, this is a small and meaningless detail in a code path that hardly # matters. But let's keep some code quality standards here, and not # accept outrageously bad code. # # Link: https://lore.kernel.org/lkml/CAHk-=wgV9toS7GU3KmNpj8hCS9SeF+A0voHS8F275_mgLhL4Lw@mail.gmail.com/ # Cc: Andy Lutomirski <luto@kernel.org> # Cc: Thomas Gleixner <tglx@linutronix.de> # Cc: Peter Zijlstra (Intel) <peterz@infradead.org> # Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org> # < /opt/cross/kisskb/fe-x86-64-core-i7-2017.05/bin/x86_64-linux-gcc --version # < /opt/cross/kisskb/fe-x86-64-core-i7-2017.05/bin/x86_64-linux-ld --version # < git log --format=%s --max-count=1 bb5a93aaf25261321db0c499cde7da6ee9d8b164 # < make -s -j 48 ARCH=um O=/kisskb/build/linus_um-defconfig_um-x86_64 CROSS_COMPILE=/opt/cross/kisskb/fe-x86-64-core-i7-2017.05/bin/x86_64-linux- SUBARCH=x86_64 defconfig # < make -s -j 48 ARCH=um O=/kisskb/build/linus_um-defconfig_um-x86_64 CROSS_COMPILE=/opt/cross/kisskb/fe-x86-64-core-i7-2017.05/bin/x86_64-linux- SUBARCH=x86_64 help # make -s -j 48 ARCH=um O=/kisskb/build/linus_um-defconfig_um-x86_64 CROSS_COMPILE=/opt/cross/kisskb/fe-x86-64-core-i7-2017.05/bin/x86_64-linux- SUBARCH=x86_64 olddefconfig # make -s -j 48 ARCH=um O=/kisskb/build/linus_um-defconfig_um-x86_64 CROSS_COMPILE=/opt/cross/kisskb/fe-x86-64-core-i7-2017.05/bin/x86_64-linux- SUBARCH=x86_64 /kisskb/src/arch/um/os-Linux/signal.c: In function 'sig_handler_common': /kisskb/src/arch/um/os-Linux/signal.c:51:1: warning: the frame size of 2960 bytes is larger than 1024 bytes [-Wframe-larger-than=] } ^ /kisskb/src/arch/um/os-Linux/signal.c: In function 'timer_real_alarm_handler': /kisskb/src/arch/um/os-Linux/signal.c:95:1: warning: the frame size of 2960 bytes is larger than 1024 bytes [-Wframe-larger-than=] } ^ LINK linux Completed OK # rm -rf /kisskb/build/linus_um-defconfig_um-x86_64 # Build took: 0:00:41.187568

© Michael Ellerman 2006-2018.