Re: [4.4 STABLE BACKPORT] x86: paravirt: make native_save_fl extern inline
From: Greg KH
Date: Sat Jul 21 2018 - 01:08:18 EST
On Fri, Jul 20, 2018 at 05:06:07PM -0700, Nick Desaulniers wrote:
> On Fri, Jul 20, 2018 at 5:04 PM kbuild test robot <lkp@xxxxxxxxx> wrote:
> >
> > Hi Nick,
> >
> > Thank you for the patch! Yet something to improve:
> >
> > [auto build test ERROR on tip/x86/core]
> > [cannot apply to v4.18-rc5 next-20180720]
> > [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/Nick-Desaulniers/x86-paravirt-make-native_save_fl-extern-inline/20180721-073112
> > config: i386-tinyconfig (attached as .config)
> > compiler: gcc-7 (Debian 7.3.0-16) 7.3.0
> > reproduce:
> > # save the attached .config to linux build tree
> > make ARCH=i386
> >
> > All errors (new ones prefixed by >>):
> >
> > arch/x86/kernel/irqflags.S: Assembler messages:
> > >> arch/x86/kernel/irqflags.S:22: Error: bad register name `%_ASM_ARG1'
> >
> > vim +22 arch/x86/kernel/irqflags.S
> >
> > 16
> > 17 /*
> > 18 * void native_restore_fl(unsigned long flags)
> > 19 * %eax/%rdi: flags
> > 20 */
> > 21 ENTRY(native_restore_fl)
> > > 22 push %_ASM_ARG1
> >
> > ---
> > 0-DAY kernel test infrastructure Open Source Technology Center
> > https://lists.01.org/pipermail/kbuild-all Intel Corporation
>
> oops, probably should not have cc'ed vger.kernel.org for a patch meant
> for stable.
stable@xxxxxxxxxxxxxxx is the right address, you did it correct. I
think 0-day just got a bit trigger-happy on this one for some reason,
don't worry about it.
thanks,
greg k-h