tracing: append a comma to INIT_FTRACE_GRAPH
authorTetsuo Handa <penguin-kernel@i-love.sakura.ne.jp>
Wed, 8 Apr 2009 05:05:43 +0000 (14:05 +0900)
committerIngo Molnar <mingo@elte.hu>
Wed, 8 Apr 2009 08:25:25 +0000 (10:25 +0200)
Impact: dont break future extensions of INIT_TASK

While not a problem right now, due to lack of a comma, build fails if
elements are appended to INIT_TASK() macro in development code:

 arch/x86/kernel/init_task.c:33: error: request for member `XXXXXXXXXX' in something not a structure or union
 arch/x86/kernel/init_task.c:33: error: initializer element is not constant
 arch/x86/kernel/init_task.c:33: error: (near initialization for `init_task.ret_stack')
 make[1]: *** [arch/x86/kernel/init_task.o] Error 1
 make: *** [arch/x86/kernel] Error 2

Signed-off-by: Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp>
Cc: srostedt@redhat.com
LKML-Reference: <200904080505.n3855hcn017109@www262.sakura.ne.jp>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
include/linux/ftrace.h

index da5405dce34746aa4c7ab757af6a46e430f9b538..ff112a872d7520b4ac5426d5b3a38f907d8c3656 100644 (file)
@@ -357,7 +357,7 @@ struct ftrace_graph_ret {
 #ifdef CONFIG_FUNCTION_GRAPH_TRACER
 
 /* for init task */
-#define INIT_FTRACE_GRAPH              .ret_stack = NULL
+#define INIT_FTRACE_GRAPH              .ret_stack = NULL,
 
 /*
  * Stack of return addresses for functions