Buildresult: powerpc-fixes/mgcoge_defconfig/powerpc-gcc5 built on Jun 19 2019, 00:17

kisskb

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

Status:	OK
Date/Time:	Jun 19 2019, 00:17
Duration:	0:00:55.075604
Builder:	ka1
Revision:	KVM: PPC: Book3S HV: Only write DAWR[X] when handling h_set_dawr in real mode (`84b028243ef07a3f65c1857343ada2b1022f8bed)`
Target:	powerpc-fixes/mgcoge_defconfig/powerpc-gcc5
Branch:	powerpc-fixes
Compiler:	powerpc-gcc5 (powerpc64-linux-gcc (GCC) 5.5.0 / GNU ld (GNU Binutils) 2.29.1.20170915)
Config:	mgcoge_defconfig (download)
Log:	Download original Possible warnings (2) kernel/futex.c:1658:17: warning: 'oldval' may be used uninitialized in this function [-Wmaybe-uninitialized] arch/powerpc/kernel/head_32.S:932: Warning: invalid register expression Full Log # git rev-parse -q --verify 84b028243ef07a3f65c1857343ada2b1022f8bed^{commit} 84b028243ef07a3f65c1857343ada2b1022f8bed already have revision, skipping fetch # git checkout -q -f -B kisskb 84b028243ef07a3f65c1857343ada2b1022f8bed # git clean -qxdf # < git log -1 # commit 84b028243ef07a3f65c1857343ada2b1022f8bed # Author: Suraj Jitindar Singh <sjitindarsingh@gmail.com> # Date: Mon Jun 17 17:16:19 2019 +1000 # # KVM: PPC: Book3S HV: Only write DAWR[X] when handling h_set_dawr in real mode # # The hcall H_SET_DAWR is used by a guest to set the data address # watchpoint register (DAWR). This hcall is handled in the host in # kvmppc_h_set_dawr() which can be called in either real mode on the # guest exit path from hcall_try_real_mode() in book3s_hv_rmhandlers.S, # or in virtual mode when called from kvmppc_pseries_do_hcall() in # book3s_hv.c. # # The function kvmppc_h_set_dawr() updates the dawr and dawrx fields in # the vcpu struct accordingly and then also writes the respective values # into the DAWR and DAWRX registers directly. It is necessary to write # the registers directly here when calling the function in real mode # since the path to re-enter the guest won't do this. However when in # virtual mode the host DAWR and DAWRX values have already been # restored, and so writing the registers would overwrite these. # Additionally there is no reason to write the guest values here as # these will be read from the vcpu struct and written to the registers # appropriately the next time the vcpu is run. # # This also avoids the case when handling h_set_dawr for a nested guest # where the guest hypervisor isn't able to write the DAWR and DAWRX # registers directly and must rely on the real hypervisor to do this for # it when it calls H_ENTER_NESTED. # # Fixes: c1fe190c0672 ("powerpc: Add force enable of DAWR on P9 option") # Signed-off-by: Suraj Jitindar Singh <sjitindarsingh@gmail.com> # Signed-off-by: Michael Ellerman <mpe@ellerman.id.au> # < /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 84b028243ef07a3f65c1857343ada2b1022f8bed # < make -s -j 48 ARCH=powerpc O=/kisskb/build/powerpc-fixes_mgcoge_defconfig_powerpc-gcc5 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-5.5.0-nolibc/powerpc64-linux/bin/powerpc64-linux- mgcoge_defconfig # make -s -j 48 ARCH=powerpc O=/kisskb/build/powerpc-fixes_mgcoge_defconfig_powerpc-gcc5 CROSS_COMPILE=/opt/cross/kisskb/korg/gcc-5.5.0-nolibc/powerpc64-linux/bin/powerpc64-linux- /kisskb/src/kernel/futex.c: In function 'do_futex': /kisskb/src/kernel/futex.c:1658:17: warning: 'oldval' may be used uninitialized in this function [-Wmaybe-uninitialized] return oldval == cmparg; ^ /kisskb/src/kernel/futex.c:1633:6: note: 'oldval' was declared here int oldval, ret; ^ /kisskb/src/arch/powerpc/kernel/head_32.S: Assembler messages: /kisskb/src/arch/powerpc/kernel/head_32.S:932: Warning: invalid register expression Completed OK # rm -rf /kisskb/build/powerpc-fixes_mgcoge_defconfig_powerpc-gcc5 # Build took: 0:00:55.075604

© Michael Ellerman 2006-2018.