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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <fc45a12e-71e8-7a54-3644-456a2b3fccbb@suse.com>
Date:   Tue, 7 Feb 2017 11:05:43 +0100
From:   Juergen Gross <jgross@...e.com>
To:     Boris Ostrovsky <boris.ostrovsky@...cle.com>,
        kbuild test robot <lkp@...el.com>
Cc:     kbuild-all@...org, xen-devel@...ts.xenproject.org,
        linux-kernel@...r.kernel.org, roger.pau@...rix.com
Subject: Re: [Xen-devel] [PATCH v3 4/9] xen/pvh: Bootstrap PVH guest

On 06/02/17 21:37, Boris Ostrovsky wrote:
> 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).

With this change folded in:

Reviewed-by: Juergen Gross <jgross@...e.com>


Juergen