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.carst...@de.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 Reported-by: Zhouping Liu <z...@redhat.com> Signed-off-by: Heiko Carstens <heiko.carst...@de.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) -- 1.8.2.3 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/