lkml.org 
[lkml]   [2008]   [Jul]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
SubjectRe: [git pull] core/stacktrace changes for v2.6.27
On Mon, Jul 14, 2008 at 05:56:17PM +0200, Ingo Molnar wrote:
>
> * Stephen Rothwell <sfr@canb.auug.org.au> wrote:
>
> > Hi Ingo,
> >
> > On Mon, 14 Jul 2008 16:44:20 +0200 Ingo Molnar <mingo@elte.hu> wrote:
> > >
> > > Please pull the latest core/stacktrace git tree from:
> > >
> > > git://git.kernel.org/pub/scm/linux/kernel/git/tip/linux-2.6-tip.git core/stacktrace
> >
> > This presumably needs the patch I posted today against linux-next that
> > fixes the powerpc allmodconfig build.
>
> hm - but you sent that against the generic-ipi tree :-)
>
> but i agree - the patch is below, Linus please apply.
>
> Ingo
>
> ---------------->
> commit 7798ed0f57b4d137e660fbf5be1e1528e40f89ac
> Author: Stephen Rothwell <sfr@canb.auug.org.au>
> Date: Mon Jul 14 19:55:03 2008 +1000
>
> generic-ipi: powerpc/generic-ipi tree build failure
>
> Today's linux-next build (powerpc allmodconfig) failed like this:
>
> ERROR: ".save_stack_trace" [tests/backtracetest.ko] undefined!
>
> But save_stack_trace is exported in arch/powerpc/kernel/stacktrace.c
>
> I couldn't figure it out until I noticed these earlier warnings:
>
> arch/powerpc/kernel/stacktrace.c:47: warning: data definition has no type or storage class
> arch/powerpc/kernel/stacktrace.c:47: warning: type defaults to 'int' in declaration of 'EXPORT_SYMBOL_GPL'
> arch/powerpc/kernel/stacktrace.c:47: warning: parameter names (without types) in function declaration
>
> I applied the patch below.
>
> Signed-off-by: Stephen Rothwell <sfr@canb.auug.org.au>
> Cc: Paul Mackerras <paulus@samba.org>
> Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>
> Cc: <linuxppc-dev@ozlabs.org>
> Signed-off-by: Ingo Molnar <mingo@elte.hu>
> ---
> arch/powerpc/kernel/stacktrace.c | 1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/arch/powerpc/kernel/stacktrace.c b/arch/powerpc/kernel/stacktrace.c
> index 9861f17..3cf0d94 100644
> --- a/arch/powerpc/kernel/stacktrace.c
> +++ b/arch/powerpc/kernel/stacktrace.c
> @@ -12,6 +12,7 @@
>
> #include <linux/sched.h>
> #include <linux/stacktrace.h>
> +#include <linux/module.h>
> #include <asm/ptrace.h>

s390 and probably other architectures will suffer from the same compile bug.
I sent you the patch below against linux-next to fix this problem a couple of
days ago:
http://marc.info/?l=linux-next&m=121577286607960&w=2

Subject: [PATCH] Add module.h include to stacktrace.c.

From: Heiko Carstens <heiko.carstens@de.ibm.com>

Fixes this:

CC arch/s390/kernel/stacktrace.o
arch/s390/kernel/stacktrace.c:84: warning: data definition has no type or storage class
arch/s390/kernel/stacktrace.c:84: warning: type defaults to 'int' in declaration of 'EXPORT_SYMBOL_GPL'
arch/s390/kernel/stacktrace.c:84: warning: parameter names (without types) in function declaration
arch/s390/kernel/stacktrace.c:97: warning: data definition has no type or storage class
arch/s390/kernel/stacktrace.c:97: warning: type defaults to 'int' in declaration of 'EXPORT_SYMBOL_GPL'
arch/s390/kernel/stacktrace.c:97: warning: parameter names (without types) in function declaration

caused by "stacktrace: export save_stack_trace[_tsk]"

Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>
---
arch/avr32/kernel/stacktrace.c | 1 +
arch/mips/kernel/stacktrace.c | 1 +
arch/powerpc/kernel/stacktrace.c | 1 +
arch/s390/kernel/stacktrace.c | 1 +
arch/sh/kernel/stacktrace.c | 1 +
5 files changed, 5 insertions(+)

Index: linux-next/arch/s390/kernel/stacktrace.c
===================================================================
--- linux-next.orig/arch/s390/kernel/stacktrace.c
+++ linux-next/arch/s390/kernel/stacktrace.c
@@ -10,6 +10,7 @@
#include <linux/sched.h>
#include <linux/stacktrace.h>
#include <linux/kallsyms.h>
+#include <linux/module.h>

static unsigned long save_context_stack(struct stack_trace *trace,
unsigned long sp,
Index: linux-next/arch/avr32/kernel/stacktrace.c
===================================================================
--- linux-next.orig/arch/avr32/kernel/stacktrace.c
+++ linux-next/arch/avr32/kernel/stacktrace.c
@@ -10,6 +10,7 @@
#include <linux/sched.h>
#include <linux/stacktrace.h>
#include <linux/thread_info.h>
+#include <linux/module.h>

register unsigned long current_frame_pointer asm("r7");

Index: linux-next/arch/mips/kernel/stacktrace.c
===================================================================
--- linux-next.orig/arch/mips/kernel/stacktrace.c
+++ linux-next/arch/mips/kernel/stacktrace.c
@@ -7,6 +7,7 @@
*/
#include <linux/sched.h>
#include <linux/stacktrace.h>
+#include <linux/module.h
#include <asm/stacktrace.h>

/*
Index: linux-next/arch/powerpc/kernel/stacktrace.c
===================================================================
--- linux-next.orig/arch/powerpc/kernel/stacktrace.c
+++ linux-next/arch/powerpc/kernel/stacktrace.c
@@ -12,6 +12,7 @@

#include <linux/sched.h>
#include <linux/stacktrace.h>
+#include <linux/module.h>
#include <asm/ptrace.h>

/*
Index: linux-next/arch/sh/kernel/stacktrace.c
===================================================================
--- linux-next.orig/arch/sh/kernel/stacktrace.c
+++ linux-next/arch/sh/kernel/stacktrace.c
@@ -12,6 +12,7 @@
#include <linux/sched.h>
#include <linux/stacktrace.h>
#include <linux/thread_info.h>
+#include <linux/module.h>
#include <asm/ptrace.h>

/*

\
 
 \ /
  Last update: 2008-07-14 23:35    [W:1.189 / U:0.640 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site