linux-kernel - Re: [s390x] build error: undefined reference to `sie

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] [day] [month] [year] [list]

Date:	Fri, 26 Jul 2013 00:40:18 +0800
From:	Zhouping Liu <zliu@...hat.com>
To:	Heiko Carstens <heiko.carstens@...ibm.com>
CC:	LKML <linux-kernel@...r.kernel.org>
Subject: Re: [s390x] build error: undefined reference to `sie_exit'

On 07/25/2013 05:21 PM, Heiko Carstens wrote:
> On Wed, Jul 24, 2013 at 10:09:13PM -0400, Zhouping Liu wrote:
>> Hello All,
>>
>> I met the below error on b3a3a9c441 with s390x arch:
>> arch/s390/built-in.o: In function `sys_call_table_emu':
>> (.rodata+0x2b98): undefined reference to `sie_exit'
>> arch/s390/built-in.o: In function `sys_call_table_emu':
>> (.rodata+0x2ba0): undefined reference to `sie_exit'
>> make: *** [vmlinux] Error 1
> Thanks for reporting, I just committed the patch below to our
> local branch:
>
>  From c073dc1f474094b5610739e752d83bcb547b1d7d Mon Sep 17 00:00:00 2001
> From: Heiko Carstens <heiko.carstens@...ibm.com>
> Date: Thu, 25 Jul 2013 11:16:48 +0200
> Subject: [PATCH] s390/perf: fix compile error (undefined reference sie_exit)
>
> The perf_event code references sie_exit even if KVM is not available.
> So add proper ifdefs to fix this one:
>
> arch/s390/built-in.o: In function `sys_call_table_emu':
> (.rodata+0x2b98): undefined reference to `sie_exit'
> arch/s390/built-in.o: In function `sys_call_table_emu':
> (.rodata+0x2ba0): undefined reference to `sie_exit'
> make: *** [vmlinux] Error 1

The patch is good for me.

Tested-by: Zhouping Liu <zliu@...hat.com>

Thanks,
Zhouping

> Reported-by: Zhouping Liu <zliu@...hat.com>
> Signed-off-by: Heiko Carstens <heiko.carstens@...ibm.com>
> ---
>   arch/s390/kernel/perf_event.c | 9 +++++----
>   1 file changed, 5 insertions(+), 4 deletions(-)
>
> diff --git a/arch/s390/kernel/perf_event.c b/arch/s390/kernel/perf_event.c
> index a6fc037..500aa10 100644
> --- a/arch/s390/kernel/perf_event.c
> +++ b/arch/s390/kernel/perf_event.c
> @@ -52,12 +52,13 @@ static struct kvm_s390_sie_block *sie_block(struct pt_regs *regs)
>   
>   static bool is_in_guest(struct pt_regs *regs)
>   {
> -	unsigned long ip = instruction_pointer(regs);
> -
>   	if (user_mode(regs))
>   		return false;
> -
> -	return ip == (unsigned long) &sie_exit;
> +#if defined(CONFIG_KVM) || defined(CONFIG_KVM_MODULE)
> +	return instruction_pointer(regs) == (unsigned long) &sie_exit;
> +#else
> +	return false;
> +#endif
>   }
>   
>   static unsigned long guest_is_user_mode(struct pt_regs *regs)

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/