Re: [PATCH 14/15] stack_trace: Add weak save_stack_trace_regs()

From: Masami Hiramatsu
Date: Mon Jun 13 2011 - 06:52:18 EST


(2011/06/13 19:19), Ingo Molnar wrote:
>
> * Steven Rostedt <rostedt@xxxxxxxxxxx> wrote:
>
>> +save_stack_trace_regs(struct pt_regs *regs, struct stack_trace *trace)
>> +{
>> + WARN_ONCE(1,
>> + KERN_INFO "save_stack_trace_regs() not implemented yet.\n");
>
> - pointless linebreak
> - we don't add KERN_ tags to WARN*() invocations

Hm, I just copied save_stack_trace_tsk(). OK, I'll remove it.

Thanks!

>
> Thanks,
>
> Ingo


--
Masami HIRAMATSU
Software Platform Research Dept. Linux Technology Center
Hitachi, Ltd., Yokohama Research Laboratory
E-mail: masami.hiramatsu.pt@xxxxxxxxxxx
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/