Re: [PATCH] tracing: add trace event for memory-failure

2015-03-20 Thread Xie XiuQi
On 2015/3/21 1:24, Luck, Tony wrote: >> RAS user space tools like rasdaemon which base on trace event, could >> receive mce error event, but no memory recovery result event. So, I >> want to add this event to make this scenario complete. > > Excellent answer. Are you going to write that patch for

RE: [PATCH] tracing: add trace event for memory-failure

2015-03-20 Thread Luck, Tony
> RAS user space tools like rasdaemon which base on trace event, could > receive mce error event, but no memory recovery result event. So, I > want to add this event to make this scenario complete. Excellent answer. Are you going to write that patch for rasdaemon? -Tony

Re: [PATCH] tracing: add trace event for memory-failure

2015-03-19 Thread Xie XiuQi
On 2015/3/19 18:39, Borislav Petkov wrote: > On Thu, Mar 19, 2015 at 11:04:30AM +0800, Xie XiuQi wrote: >> Memory-failure as the high level machine check handler, it's necessary >> to report memory page recovery action result to user space by ftrace. >> >> This patch add a event at ras group for me

Re: [PATCH] tracing: add trace event for memory-failure

2015-03-19 Thread Borislav Petkov
On Thu, Mar 19, 2015 at 11:04:30AM +0800, Xie XiuQi wrote: > Memory-failure as the high level machine check handler, it's necessary > to report memory page recovery action result to user space by ftrace. > > This patch add a event at ras group for memory-failure. > > The output like below: > # t

Re: [PATCH] tracing: add trace event for memory-failure

2015-03-18 Thread Steven Rostedt
On Thu, 19 Mar 2015 11:04:30 +0800 Xie XiuQi wrote: > Memory-failure as the high level machine check handler, it's necessary > to report memory page recovery action result to user space by ftrace. > > This patch add a event at ras group for memory-failure. > > The output like below: > # tracer

Re: [PATCH] tracing: add trace event for memory-failure

2015-03-17 Thread Steven Rostedt
On Tue, 17 Mar 2015 18:47:40 +0800 Xie XiuQi wrote: > I'm not clearly why we need a hard coded here. As the strings or "result" have > defined in mm/memory-failure.c, so passing "action_name[result]" would be more > clean and more flexible here? The TP_printk() is what will be shown in the print

Re: [PATCH] tracing: add trace event for memory-failure

2015-03-17 Thread Xie XiuQi
On 2015/3/14 3:32, Steven Rostedt wrote: > On Fri, 13 Mar 2015 09:37:34 -0700 > Tony Luck wrote: > > >>> int sysctl_memory_failure_early_kill __read_mostly = 0; >>> >>> @@ -837,6 +838,8 @@ static struct page_state { >>> */ >>> static void action_result(unsigned long pfn, char *msg, int resul

Re: [PATCH] tracing: add trace event for memory-failure

2015-03-16 Thread Xie XiuQi
On 2015/3/16 17:27, Naoya Horiguchi wrote: > On Fri, Mar 13, 2015 at 06:10:51PM +0800, Xie XiuQi wrote: >> Memory-failure as the high level machine check handler, it's necessary >> to report memory page recovery action result to user space by ftrace. >> >> This patch add a event at ras group for me

Re: [PATCH] tracing: add trace event for memory-failure

2015-03-16 Thread Naoya Horiguchi
On Fri, Mar 13, 2015 at 06:10:51PM +0800, Xie XiuQi wrote: > Memory-failure as the high level machine check handler, it's necessary > to report memory page recovery action result to user space by ftrace. > > This patch add a event at ras group for memory-failure. > > The output like below: > # tr

Re: [PATCH] tracing: add trace event for memory-failure

2015-03-13 Thread Steven Rostedt
On Fri, 13 Mar 2015 09:37:34 -0700 Tony Luck wrote: > > int sysctl_memory_failure_early_kill __read_mostly = 0; > > > > @@ -837,6 +838,8 @@ static struct page_state { > > */ > > static void action_result(unsigned long pfn, char *msg, int result) > > { > > + trace_memory_failure_event(

Re: [PATCH] tracing: add trace event for memory-failure

2015-03-13 Thread Tony Luck
On Fri, Mar 13, 2015 at 3:10 AM, Xie XiuQi wrote: > Memory-failure as the high level machine check handler, it's necessary > to report memory page recovery action result to user space by ftrace. > > This patch add a event at ras group for memory-failure. > > The output like below: > # tracer: nop