Re: [4.4 STABLE BACKPORT] x86: paravirt: make native_save_fl extern inline

From: Nick Desaulniers
Date: Fri Jul 20 2018 - 20:06:24 EST


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.

--
Thanks,
~Nick Desaulniers