Re: [Xen-devel] [PATCH v3 4/9] xen/pvh: Bootstrap PVH guest

From: Boris Ostrovsky
Date: Mon Feb 06 2017 - 15:37:22 EST


On 02/06/2017 02:29 PM, kbuild test robot wrote:
> Hi Boris,
>
> [auto build test WARNING on xen-tip/linux-next]
> [also build test WARNING on v4.10-rc7]
> [cannot apply to next-20170206]
> [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/Boris-Ostrovsky/x86-boot-32-Convert-the-32-bit-pgtable-setup-code-from-assembly-to-C/20170207-014642
> base: https://git.kernel.org/pub/scm/linux/kernel/git/xen/tip.git linux-next
> config: x86_64-randconfig-it0-02062149 (attached as .config)
> compiler: gcc-4.9 (Debian 4.9.4-2) 4.9.4
> reproduce:
> # save the attached .config to linux build tree
> make ARCH=x86_64
>
> All warnings (new ones prefixed by >>):
>
>>> arch/x86/xen/xen-pvh.o: warning: objtool: pvh_start_xen()+0x57: call without frame pointer save/setup

pvh_start_xen() is never called so this needs

diff --git a/arch/x86/xen/xen-pvh.S b/arch/x86/xen/xen-pvh.S
index 512fda0..5e24671 100644
--- a/arch/x86/xen/xen-pvh.S
+++ b/arch/x86/xen/xen-pvh.S
@@ -133,7 +133,7 @@ ENTRY(pvh_start_xen)

ljmp $__BOOT_CS, $_pa(startup_32)
#endif
-ENDPROC(pvh_start_xen)
+END(pvh_start_xen)

.section ".init.data","aw"
.balign 8

(I don't think it's worth re-sending the series with just this change).

-boris