Buildresult: linus/um-defconfig/um-x86_64 built on Mar 19 2020, 05:10

kisskb

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

Status:	OK
Date/Time:	Mar 19 2020, 05:10
Duration:	0:00:52.599839
Builder:	blade4b
Revision:	mm: slub: be more careful about the double cmpxchg of freelist (`5076190daded2197f62fe92cf69674488be44175)`
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 5076190daded2197f62fe92cf69674488be44175^{commit} 5076190daded2197f62fe92cf69674488be44175 already have revision, skipping fetch # git checkout -q -f -B kisskb 5076190daded2197f62fe92cf69674488be44175 # git clean -qxdf # < git log -1 # commit 5076190daded2197f62fe92cf69674488be44175 # Author: Linus Torvalds <torvalds@linux-foundation.org> # Date: Tue Mar 17 11:04:09 2020 -0700 # # mm: slub: be more careful about the double cmpxchg of freelist # # This is just a cleanup addition to Jann's fix to properly update the # transaction ID for the slub slowpath in commit fd4d9c7d0c71 ("mm: slub: # add missing TID bump.."). # # The transaction ID is what protects us against any concurrent accesses, # but we should really also make sure to make the 'freelist' comparison # itself always use the same freelist value that we then used as the new # next free pointer. # # Jann points out that if we do all of this carefully, we could skip the # transaction ID update for all the paths that only remove entries from # the lists, and only update the TID when adding entries (to avoid the ABA # issue with cmpxchg and list handling re-adding a previously seen value). # # But this patch just does the "make sure to cmpxchg the same value we # used" rather than then try to be clever. # # Acked-by: Jann Horn <jannh@google.com> # 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 5076190daded2197f62fe92cf69674488be44175 # < make -s -j 24 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 24 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 24 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 24 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:52.599839

© Michael Ellerman 2006-2018.