On Thu, 2017-11-02 at 10:19 +0800, kbuild test robot wrote: > Hi Cyril, > > Thank you for the patch! Yet something to improve: >
Once again robot, you have done brilliantly! You're 100% correct and the last thing I want to do is break the build with CONFIG_PPC_TRANSACTIONAL_MEM turned off. Life saver, Thanks so much kbuild. Cyril > [auto build test ERROR on powerpc/next] > [also build test ERROR on v4.14-rc7 next-20171018] > [if your patch is applied to the wrong git tree, please drop us a note to > help improve the system] > > url: > https://github.com/0day-ci/linux/commits/Cyril-Bur/powerpc-Don-t-enable-FP-Altivec-if-not-checkpointed/20171102-073816 > base: https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git next > config: powerpc-asp8347_defconfig (attached as .config) > compiler: powerpc-linux-gnu-gcc (Debian 6.1.1-9) 6.1.1 20160705 > reproduce: > wget > https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O > ~/bin/make.cross > chmod +x ~/bin/make.cross > # save the attached .config to linux build tree > make.cross ARCH=powerpc > > All errors (new ones prefixed by >>): > > arch/powerpc/kernel/process.c: In function 'is_transactionally_fp': > > > arch/powerpc/kernel/process.c:243:15: error: 'struct thread_struct' has > > > no member named 'ckpt_regs' > > (tsk->thread.ckpt_regs.msr & MSR_FP); > ^ > arch/powerpc/kernel/process.c:244:1: error: control reaches end of > non-void function [-Werror=return-type] > } > ^ > cc1: all warnings being treated as errors > > vim +243 arch/powerpc/kernel/process.c > > 239 > 240 static int is_transactionally_fp(struct task_struct *tsk) > 241 { > 242 return msr_tm_active(tsk->thread.regs->msr) && > > 243 (tsk->thread.ckpt_regs.msr & MSR_FP); > 244 } > 245 > > --- > 0-DAY kernel test infrastructure Open Source Technology Center > https://lists.01.org/pipermail/kbuild-all Intel Corporation
