Buildresult: linus/malta_defconfig/mips-gcc4.6 built on Dec 30 2016, 11:26

kisskb

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

Status:	Failed
Date/Time:	Dec 30 2016, 11:26
Duration:	0:00:20.131101
Builder:	ka3
Revision:	mm: optimize PageWaiters bit use for unlock_page() (`b91e1302ad9b80c174a4855533f7e3aa2873355e)`
Target:	linus/malta_defconfig/mips-gcc4.6
Branch:	linus
Compiler:	mips-gcc4.6 (mips-linux-gcc (GCC) 4.6.3)
Config:	malta_defconfig (download)
Log:	Download original Possible errors /home/kisskb/slave/src/mm/filemap.c:933:2: error: too few arguments to function 'test_bit' make[2]: * [mm/filemap.o] Error 1 make[1]: * [mm] Error 2 make: * [sub-make] Error 2 Possible warnings (4) arch/mips/configs/malta_defconfig:62:warning: symbol value 'm' invalid for NF_CT_PROTO_DCCP arch/mips/configs/malta_defconfig:63:warning: symbol value 'm' invalid for NF_CT_PROTO_UDPLITE /home/kisskb/slave/src/mm/filemap.c:934:1: warning: control reaches end of non-void function [-Wreturn-type] /home/kisskb/slave/src/kernel/printk/printk.c:183:16: warning: 'old' may be used uninitialized in this function [-Wuninitialized] Full Log # git rev-parse -q --verify b91e1302ad9b80c174a4855533f7e3aa2873355e^{commit} b91e1302ad9b80c174a4855533f7e3aa2873355e already have revision, skipping fetch # git checkout -q -f -B kisskb b91e1302ad9b80c174a4855533f7e3aa2873355e # git clean -qxdf # git log -1 commit b91e1302ad9b80c174a4855533f7e3aa2873355e Author: Linus Torvalds <torvalds@linux-foundation.org> Date: Tue Dec 27 11:40:38 2016 -0800 mm: optimize PageWaiters bit use for unlock_page() In commit 62906027091f ("mm: add PageWaiters indicating tasks are waiting for a page bit") Nick Piggin made our page locking no longer unconditionally touch the hashed page waitqueue, which not only helps performance in general, but is particularly helpful on NUMA machines where the hashed wait queues can bounce around a lot. However, the "clear lock bit atomically and then test the waiters bit" sequence turns out to be much more expensive than it needs to be, because you get a nasty stall when trying to access the same word that just got updated atomically. On architectures where locking is done with LL/SC, this would be trivial to fix with a new primitive that clears one bit and tests another atomically, but that ends up not working on x86, where the only atomic operations that return the result end up being cmpxchg and xadd. The atomic bit operations return the old value of the same bit we changed, not the value of an unrelated bit. On x86, we could put the lock bit in the high bit of the byte, and use "xadd" with that bit (where the overflow ends up not touching other bits), and look at the other bits of the result. However, an even simpler model is to just use a regular atomic "and" to clear the lock bit, and then the sign bit in eflags will indicate the resulting state of the unrelated bit #7. So by moving the PageWaiters bit up to bit #7, we can atomically clear the lock bit and test the waiters bit on x86 too. And architectures with LL/SC (which is all the usual RISC suspects), the particular bit doesn't matter, so they are fine with this approach too. This avoids the extra access to the same atomic word, and thus avoids the costly stall at page unlock time. The only downside is that the interface ends up being a bit odd and specialized: clear a bit in a byte, and test the sign bit. Nick doesn't love the resulting name of the new primitive, but I'd rather make the name be descriptive and very clear about the limitation imposed by trying to work across all relevant architectures than make it be some generic thing that doesn't make the odd semantics explicit. So this introduces the new architecture primitive clear_bit_unlock_is_negative_byte(); and adds the trivial implementation for x86. We have a generic non-optimized fallback (that just does a "clear_bit()"+"test_bit(7)" combination) which can be overridden by any architecture that can do better. According to Nick, Power has the same hickup x86 has, for example, but some other architectures may not even care. All these optimizations mean that my page locking stress-test (which is just executing a lot of small short-lived shell scripts: "make test" in the git source tree) no longer makes our page locking look horribly bad. Before all these optimizations, just the unlock_page() costs were just over 3% of all CPU overhead on "make test". After this, it's down to 0.66%, so just a quarter of the cost it used to be. (The difference on NUMA is bigger, but there this micro-optimization is likely less noticeable, since the big issue on NUMA was not the accesses to 'struct page', but the waitqueue accesses that were already removed by Nick's earlier commit). Acked-by: Nick Piggin <npiggin@gmail.com> Cc: Dave Hansen <dave.hansen@linux.intel.com> Cc: Bob Peterson <rpeterso@redhat.com> Cc: Steven Whitehouse <swhiteho@redhat.com> Cc: Andrew Lutomirski <luto@kernel.org> Cc: Andreas Gruenbacher <agruenba@redhat.com> Cc: Peter Zijlstra <peterz@infradead.org> Cc: Mel Gorman <mgorman@techsingularity.net> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org> # < /opt/cross/kisskb/gcc-4.6.3-nolibc/mips-linux/bin/mips-linux-gcc --version # < git log --format=%s --max-count=1 b91e1302ad9b80c174a4855533f7e3aa2873355e # < make -j 10 ARCH=mips O=/home/kisskb/slave/build/linus_malta_defconfig_mips CROSS_COMPILE=/opt/cross/kisskb/gcc-4.6.3-nolibc/mips-linux/bin/mips-linux- malta_defconfig arch/mips/configs/malta_defconfig:62:warning: symbol value 'm' invalid for NF_CT_PROTO_DCCP arch/mips/configs/malta_defconfig:63:warning: symbol value 'm' invalid for NF_CT_PROTO_UDPLITE # make -j 10 ARCH=mips O=/home/kisskb/slave/build/linus_malta_defconfig_mips CROSS_COMPILE=/opt/cross/kisskb/gcc-4.6.3-nolibc/mips-linux/bin/mips-linux- make[1]: Entering directory '/home/kisskb/slave/build/linus_malta_defconfig_mips' GEN ./Makefile scripts/kconfig/conf --silentoldconfig Kconfig CHK include/config/kernel.release GEN ./Makefile WRAP arch/mips/include/generated/asm/clkdev.h WRAP arch/mips/include/generated/asm/cputime.h WRAP arch/mips/include/generated/asm/current.h WRAP arch/mips/include/generated/asm/dma-contiguous.h WRAP arch/mips/include/generated/asm/emergency-restart.h WRAP arch/mips/include/generated/asm/irq_work.h WRAP arch/mips/include/generated/asm/local64.h WRAP arch/mips/include/generated/asm/mcs_spinlock.h WRAP arch/mips/include/generated/asm/mm-arch-hooks.h WRAP arch/mips/include/generated/asm/parport.h WRAP arch/mips/include/generated/asm/percpu.h WRAP arch/mips/include/generated/asm/preempt.h WRAP arch/mips/include/generated/asm/sections.h WRAP arch/mips/include/generated/asm/segment.h WRAP arch/mips/include/generated/asm/serial.h WRAP arch/mips/include/generated/asm/trace_clock.h WRAP arch/mips/include/generated/asm/user.h WRAP arch/mips/include/generated/asm/xor.h WRAP arch/mips/include/generated/asm/word-at-a-time.h CHK include/generated/uapi/linux/version.h WRAP arch/mips/include/generated/uapi/asm/ipcbuf.h UPD include/generated/uapi/linux/version.h /home/kisskb/slave/src/arch/mips/vdso/Makefile:30: MIPS VDSO requires binutils >= 2.25 UPD include/config/kernel.release HOSTCC arch/mips/boot/tools/relocs_32.o HOSTCC arch/mips/boot/tools/relocs_64.o HOSTCC arch/mips/boot/tools/relocs_main.o Using /home/kisskb/slave/src as source for kernel CHK include/generated/utsrelease.h UPD include/generated/utsrelease.h HOSTCC scripts/kallsyms HOSTCC scripts/sortextable HOSTCC scripts/conmakehash HOSTCC scripts/dtc/dtc.o HOSTCC scripts/dtc/flattree.o HOSTCC scripts/genksyms/genksyms.o CC scripts/mod/empty.o SHIPPED scripts/genksyms/parse.tab.c HOSTCC scripts/mod/mk_elfconfig HOSTCC scripts/dtc/fstree.o CC scripts/mod/devicetable-offsets.s SHIPPED scripts/genksyms/lex.lex.c HOSTCC scripts/dtc/data.o HOSTLD arch/mips/boot/tools/relocs SHIPPED scripts/genksyms/keywords.hash.c MKELF scripts/mod/elfconfig.h GEN scripts/mod/devicetable-offsets.h HOSTCC scripts/mod/sumversion.o HOSTCC scripts/dtc/livetree.o SHIPPED scripts/genksyms/parse.tab.h HOSTCC scripts/dtc/treesource.o HOSTCC scripts/genksyms/parse.tab.o HOSTCC scripts/genksyms/lex.lex.o HOSTCC scripts/mod/modpost.o HOSTCC scripts/mod/file2alias.o HOSTCC scripts/dtc/srcpos.o CC kernel/bounds.s HOSTCC scripts/dtc/checks.o CHK include/generated/timeconst.h UPD include/generated/timeconst.h CHK include/generated/bounds.h HOSTCC scripts/dtc/util.o SHIPPED scripts/dtc/dtc-lexer.lex.c UPD include/generated/bounds.h SHIPPED scripts/dtc/dtc-parser.tab.h SHIPPED scripts/dtc/dtc-parser.tab.c HOSTCC scripts/dtc/dtc-lexer.lex.o HOSTCC scripts/dtc/dtc-parser.tab.o CC arch/mips/kernel/asm-offsets.s CHK include/generated/asm-offsets.h UPD include/generated/asm-offsets.h CALL /home/kisskb/slave/src/scripts/checksyscalls.sh HOSTLD scripts/genksyms/genksyms HOSTLD scripts/dtc/dtc HOSTLD scripts/mod/modpost CC init/main.o CHK include/generated/compile.h CC init/do_mounts.o CC init/do_mounts_rd.o CC init/noinitramfs.o CC init/calibrate.o CC init/init_task.o LD usr/built-in.o UPD include/generated/compile.h CC init/version.o CC arch/mips/mm/cache.o LD arch/mips/boot/dts/brcm/built-in.o LD arch/mips/boot/dts/cavium-octeon/built-in.o LD arch/mips/boot/dts/ingenic/built-in.o CC arch/mips/kernel/cpu-probe.o LD arch/mips/boot/dts/lantiq/built-in.o DTC arch/mips/boot/dts/mti/malta.dtb DTB arch/mips/boot/dts/mti/malta.dtb.S AS arch/mips/boot/dts/mti/malta.dtb.o LD arch/mips/boot/dts/mti/built-in.o rm arch/mips/boot/dts/mti/malta.dtb.S LD arch/mips/boot/dts/netlogic/built-in.o LD arch/mips/boot/dts/pic32/built-in.o LD arch/mips/boot/dts/qca/built-in.o LD arch/mips/boot/dts/ralink/built-in.o CC arch/mips/mti-malta/malta-display.o LD arch/mips/boot/dts/xilfpga/built-in.o LD arch/mips/boot/dts/built-in.o CC arch/mips/kernel/branch.o CC arch/mips/kernel/elf.o CC arch/mips/mti-malta/malta-dt.o CC arch/mips/mti-malta/malta-dtshim.o CC arch/mips/mti-malta/malta-init.o LD arch/mips/net/built-in.o CC arch/mips/mti-malta/malta-int.o CC arch/mips/mm/dma-default.o /home/kisskb/slave/src/arch/mips/vdso/Makefile:30: MIPS VDSO requires binutils >= 2.25 LDS arch/mips/vdso/vdso.lds AS arch/mips/vdso/elf.o AS arch/mips/vdso/sigreturn.o HOSTCC arch/mips/vdso/genvdso AS arch/mips/kernel/entry.o CC arch/mips/mti-malta/malta-memory.o CC arch/mips/mti-malta/malta-platform.o VDSO arch/mips/vdso/vdso.so.dbg.raw CC arch/mips/mti-malta/malta-reset.o CC arch/mips/mm/extable.o CC arch/mips/mti-malta/malta-setup.o CC arch/mips/mm/fault.o OBJCOPY arch/mips/vdso/vdso.so.raw GENVDSO arch/mips/vdso/vdso-image.c CC arch/mips/vdso/vdso-image.o CC arch/mips/mm/gup.o CC arch/mips/mm/init.o LD init/mounts.o LD init/built-in.o AS arch/mips/kernel/genex.o CC arch/mips/kernel/idle.o CC arch/mips/kernel/irq.o CC arch/mips/kernel/process.o CC kernel/fork.o CC arch/mips/mti-malta/malta-time.o LD arch/mips/vdso/built-in.o CC kernel/exec_domain.o CC kernel/panic.o CC kernel/cpu.o CC kernel/exit.o CC arch/mips/kernel/prom.o CC arch/mips/kernel/ptrace.o CC kernel/softirq.o CC arch/mips/mm/mmap.o LD certs/built-in.o CC arch/mips/kernel/reset.o CC arch/mips/mti-malta/malta-pm.o CC mm/filemap.o CC mm/mempool.o CC mm/oom_kill.o CC mm/maccess.o CC arch/mips/mm/page.o CC arch/mips/kernel/setup.o /home/kisskb/slave/src/mm/filemap.c: In function 'clear_bit_unlock_is_negative_byte': /home/kisskb/slave/src/mm/filemap.c:933:2: error: too few arguments to function 'test_bit' /home/kisskb/slave/src/include/asm-generic/bitops/non-atomic.h:103:19: note: declared here LD arch/mips/mti-malta/built-in.o CC arch/mips/kernel/signal.o /home/kisskb/slave/src/mm/filemap.c:934:1: warning: control reaches end of non-void function [-Wreturn-type] /home/kisskb/slave/src/scripts/Makefile.build:293: recipe for target 'mm/filemap.o' failed make[2]: * [mm/filemap.o] Error 1 make[2]: * Waiting for unfinished jobs.... CC kernel/resource.o CC kernel/sysctl.o CC arch/mips/kernel/syscall.o CC arch/mips/kernel/time.o AS arch/mips/mm/page-funcs.o CC kernel/sysctl_binary.o CC arch/mips/mm/tlbex.o CC arch/mips/kernel/topology.o CC arch/mips/kernel/traps.o AS arch/mips/mm/tlbex-fault.o AS arch/mips/mm/tlb-funcs.o CC arch/mips/mm/uasm-mips.o /home/kisskb/slave/src/Makefile:988: recipe for target 'mm' failed make[1]: * [mm] Error 2 make[1]: * Waiting for unfinished jobs.... CC arch/mips/mm/ioremap.o CC arch/mips/mm/pgtable-32.o CC arch/mips/kernel/unaligned.o CC arch/mips/kernel/watch.o CC arch/mips/kernel/vdso.o CC arch/mips/kernel/cevt-r4k.o CC kernel/capability.o CC arch/mips/kernel/csrc-r4k.o CC arch/mips/kernel/sync-r4k.o CC arch/mips/mm/c-r4k.o AS arch/mips/mm/cex-gen.o CC arch/mips/mm/tlb-r4k.o CC kernel/ptrace.o CC kernel/user.o CC kernel/signal.o CC arch/mips/mm/sc-mips.o CC kernel/sys.o CC arch/mips/kernel/mips_ksyms.o CC kernel/kmod.o CC arch/mips/kernel/module.o CC kernel/workqueue.o CC kernel/pid.o CC kernel/task_work.o CC kernel/extable.o AS arch/mips/kernel/r4k_fpu.o AS arch/mips/kernel/r4k_switch.o CC kernel/params.o CC arch/mips/kernel/smp.o CC arch/mips/kernel/smp-up.o CC arch/mips/kernel/mips-mt.o CC arch/mips/kernel/mips-mt-fpaff.o CC kernel/kthread.o CC arch/mips/kernel/smp-mt.o CC arch/mips/kernel/spram.o LD arch/mips/mm/built-in.o CC arch/mips/kernel/irq-msc01.o AS arch/mips/kernel/scall32-o32.o CC arch/mips/kernel/proc.o CC arch/mips/kernel/i8253.o CC arch/mips/kernel/mips-cm.o AS arch/mips/kernel/head.o LDS arch/mips/kernel/vmlinux.lds CC kernel/sys_ni.o CC kernel/nsproxy.o CC kernel/notifier.o CC kernel/ksysfs.o CC kernel/cred.o CC kernel/reboot.o CC kernel/async.o CC kernel/range.o LD arch/mips/kernel/built-in.o LD arch/mips/built-in.o CC kernel/smpboot.o CC kernel/ucount.o CC kernel/groups.o CC kernel/bpf/core.o CC kernel/irq/irqdesc.o LD kernel/livepatch/built-in.o CC kernel/locking/mutex.o CC kernel/irq/handle.o CC kernel/locking/semaphore.o CC kernel/locking/rwsem.o CC kernel/power/qos.o CC kernel/printk/printk.o CC kernel/locking/percpu-rwsem.o CC kernel/locking/spinlock.o CC kernel/printk/nmi.o CC kernel/rcu/update.o CC kernel/locking/rtmutex.o CC kernel/irq/manage.o CC kernel/irq/spurious.o CC kernel/irq/resend.o CC kernel/locking/rwsem-spinlock.o CC kernel/rcu/sync.o LD kernel/power/built-in.o CC kernel/rcu/srcu.o CC kernel/sched/core.o CC kernel/irq/chip.o LD kernel/bpf/built-in.o CC kernel/time/time.o CC kernel/time/timer.o CC kernel/irq/dummychip.o /home/kisskb/slave/src/kernel/printk/printk.c: In function 'devkmsg_sysctl_set_loglvl': /home/kisskb/slave/src/kernel/printk/printk.c:183:16: warning: 'old' may be used uninitialized in this function [-Wuninitialized] CC kernel/irq/devres.o LD kernel/locking/built-in.o CC kernel/irq/generic-chip.o CC kernel/rcu/tree.o CC kernel/irq/autoprobe.o CC kernel/irq/irqdomain.o LD kernel/printk/built-in.o CC kernel/time/hrtimer.o CC kernel/irq/proc.o CC kernel/futex.o CC kernel/dma.o CC kernel/smp.o CC kernel/module.o CC kernel/kallsyms.o CC kernel/time/timekeeping.o CC kernel/time/ntp.o CC kernel/irq/ipi.o CC kernel/irq/affinity.o CC kernel/utsname.o CC kernel/sched/loadavg.o CC kernel/sched/clock.o CC kernel/sched/cputime.o CC kernel/sched/idle_task.o LD kernel/irq/built-in.o CC kernel/time/clocksource.o CC kernel/sched/fair.o CC kernel/pid_namespace.o CC kernel/sched/rt.o CC kernel/stop_machine.o CC kernel/seccomp.o LD kernel/rcu/built-in.o CC kernel/relay.o CC kernel/utsname_sysctl.o CC kernel/sched/deadline.o CC kernel/time/jiffies.o CC kernel/time/timer_list.o CC kernel/time/timeconv.o CC kernel/time/timecounter.o CC kernel/sched/stop_task.o CC kernel/sched/wait.o CC kernel/elfcore.o CC kernel/time/alarmtimer.o CC kernel/sched/swait.o CC kernel/time/posix-timers.o CC kernel/sched/completion.o CC kernel/sched/idle.o CC kernel/time/posix-cpu-timers.o CC kernel/sched/cpupri.o CC kernel/time/posix-clock.o CC kernel/time/itimer.o CC kernel/time/clockevents.o CC kernel/sched/cpudeadline.o CC kernel/sched/debug.o CC kernel/irq_work.o CC kernel/time/tick-common.o CC kernel/membarrier.o CC kernel/memremap.o CC kernel/time/sched_clock.o CC kernel/time/tick-oneshot.o CC kernel/time/tick-sched.o LD kernel/sched/built-in.o LD kernel/time/built-in.o LD kernel/built-in.o make[1]: Leaving directory '/home/kisskb/slave/build/linus_malta_defconfig_mips' Makefile:150: recipe for target 'sub-make' failed make: * [sub-make] Error 2 Command 'make -j 10 ARCH=mips O=/home/kisskb/slave/build/linus_malta_defconfig_mips CROSS_COMPILE=/opt/cross/kisskb/gcc-4.6.3-nolibc/mips-linux/bin/mips-linux- ' returned non-zero exit status 2 # rm -rf /home/kisskb/slave/build/linus_malta_defconfig_mips # Build took: 0:00:20.131101

© Michael Ellerman 2006-2018.