Re: [PATCH 06/17] tools lib traceevent: Add pevent_unregister_event_handler()

2014-01-15 Thread Steven Rostedt
On Thu, 16 Jan 2014 10:58:05 +0900 Namhyung Kim wrote: > Thanks, can I add your ack with this change? Yeah, go ahead. -- Steve -- 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.ke

Re: [PATCH 06/17] tools lib traceevent: Add pevent_unregister_event_handler()

2014-01-15 Thread Namhyung Kim
On Wed, 15 Jan 2014 20:52:59 -0500, Steven Rostedt wrote: > On Thu, 16 Jan 2014 09:00:28 +0900 > Namhyung Kim wrote: > >> Hi Steve, >> >> 2014-01-16 AM 1:18, Steven Rostedt wrote: >> > On Wed, 15 Jan 2014 10:45:29 +0900 >> > Namhyung Kim wrote: >> >> + if (event->handler == func && event->contex

Re: [PATCH 06/17] tools lib traceevent: Add pevent_unregister_event_handler()

2014-01-15 Thread Steven Rostedt
On Thu, 16 Jan 2014 09:00:28 +0900 Namhyung Kim wrote: > Hi Steve, > > 2014-01-16 AM 1:18, Steven Rostedt wrote: > > On Wed, 15 Jan 2014 10:45:29 +0900 > > Namhyung Kim wrote: > >> + if (event->handler == func && event->context == context) { > >> + pr_stat("overriding event (%d) %s:%s

Re: [PATCH 06/17] tools lib traceevent: Add pevent_unregister_event_handler()

2014-01-15 Thread Namhyung Kim
Hi Steve, 2014-01-16 AM 1:18, Steven Rostedt wrote: On Wed, 15 Jan 2014 10:45:29 +0900 Namhyung Kim wrote: + if (event->handler == func && event->context == context) { + pr_stat("overriding event (%d) %s:%s with default print handler", Should we use the word "overriding"

Re: [PATCH 06/17] tools lib traceevent: Add pevent_unregister_event_handler()

2014-01-15 Thread Arnaldo Carvalho de Melo
Em Wed, Jan 15, 2014 at 11:18:39AM -0500, Steven Rostedt escreveu: > On Wed, 15 Jan 2014 10:45:29 +0900 > Namhyung Kim wrote: > > > pr_stat("overriding event (%d) %s:%s with new print handler", > > event->id, event->system, event->name); > > @@ -5637,6 +5649,79 @@ int pevent_reg

Re: [PATCH 06/17] tools lib traceevent: Add pevent_unregister_event_handler()

2014-01-15 Thread Steven Rostedt
On Wed, 15 Jan 2014 10:45:29 +0900 Namhyung Kim wrote: > pr_stat("overriding event (%d) %s:%s with new print handler", > event->id, event->system, event->name); > @@ -5637,6 +5649,79 @@ int pevent_register_event_handler(struct pevent > *pevent, int id, > return -1; >

[PATCH 06/17] tools lib traceevent: Add pevent_unregister_event_handler()

2014-01-14 Thread Namhyung Kim
When a plugin unloaded it needs to unregister its handler from pevent. So add unregister function to do it. Cc: Jiri Olsa Signed-off-by: Namhyung Kim --- tools/lib/traceevent/event-parse.c | 113 - tools/lib/traceevent/event-parse.h | 3 + 2 files changed,