[PATCH] Tracepoints synchronize unregister static inline

[PATCH] Tracepoints synchronize unregister static inline

[Mathieu Desnoyers]

Ingo, could you pull the following patch into -tip ?

Thanks,

Mathieu

Turn tracepoint synchronize unregister into a static inline. There is no reason
to keep it as a macro over a static inline.

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@polymtl.ca>
CC: Ingo Molnar <mingo@elte.hu>
CC: Andrew Morton <akpm@linux-foundation.org>
---
 include/linux/tracepoint.h |    5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

Index: linux-2.6-lttng/include/linux/tracepoint.h
===================================================================
--- linux-2.6-lttng.orig/include/linux/tracepoint.h	2008-10-03 14:50:53.000000000 -0400
+++ linux-2.6-lttng/include/linux/tracepoint.h	2008-10-03 14:51:44.000000000 -0400
@@ -153,6 +153,9 @@ extern int tracepoint_get_iter_range(str
  * probe unregistration and the end of module exit to make sure there is no
  * caller executing a probe when it is freed.
  */
-#define tracepoint_synchronize_unregister() synchronize_sched()
+static inline void tracepoint_synchronize_unregister(void)
+{
+	synchronize_sched();
+}
 
 #endif
-- 
Mathieu Desnoyers
OpenPGP key fingerprint: 8CD5 52C3 8E3C 4140 715F  BA06 3F25 A8FE 3BAE 9A68

Re: [PATCH] Tracepoints synchronize unregister static inline

[Ingo Molnar]

* Mathieu Desnoyers <mathieu.desnoyers@polymtl.ca> wrote:

> Ingo, could you pull the following patch into -tip ?

sure - but it does not apply to tip/master. Some missing dependencies?

	Ingo

Re: [PATCH] Tracepoints synchronize unregister static inline

[Mathieu Desnoyers]

* Ingo Molnar (mingo@elte.hu) wrote:
> 
> * Mathieu Desnoyers <mathieu.desnoyers@polymtl.ca> wrote:
> 
> > Ingo, could you pull the following patch into -tip ?
> 
> sure - but it does not apply to tip/master. Some missing dependencies?
> 
> 	Ingo

Hi Ingo,

"Tracepoints synchronize unregister static inline"
applies on top of
"Tracepoints : tracepoint_synchronize_unregister()"

I posted earlier but does not seem to be in tip/master. Here is a repost
:

Tracepoints : tracepoint_synchronize_unregister()

Create tracepoint_synchronize_unregister() which must be called before the end
of exit() to make sure every probe callers have exited the non preemptible
section and thus are not executing the probe code anymore.

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@polymtl.ca>
CC: Rusty Russell <rusty@rustcorp.com.au>
CC: "Frank Ch. Eigler" <fche@redhat.com>
CC: Ingo Molnar <mingo@elte.hu>
CC: Peter Zijlstra <peterz@infradead.org>
CC: Steven Rostedt <rostedt@goodmis.org>
---
 include/linux/tracepoint.h |    7 +++++++
 1 file changed, 7 insertions(+)

Index: linux-2.6-lttng/include/linux/tracepoint.h
===================================================================
--- linux-2.6-lttng.orig/include/linux/tracepoint.h	2008-07-31 09:21:40.000000000 -0400
+++ linux-2.6-lttng/include/linux/tracepoint.h	2008-07-31 09:22:27.000000000 -0400
@@ -124,4 +124,11 @@ extern void tracepoint_iter_reset(struct
 extern int tracepoint_get_iter_range(struct tracepoint **tracepoint,
 	struct tracepoint *begin, struct tracepoint *end);
 
+/*
+ * tracepoint_synchronize_unregister must be called between the last tracepoint
+ * probe unregistration and the end of module exit to make sure there is no
+ * caller executing a probe when it is freed.
+ */
+#define tracepoint_synchronize_unregister() synchronize_sched()
+
 #endif

-- 
Mathieu Desnoyers
OpenPGP key fingerprint: 8CD5 52C3 8E3C 4140 715F  BA06 3F25 A8FE 3BAE 9A68

Re: [PATCH] Tracepoints synchronize unregister static inline

[Ingo Molnar]

* Mathieu Desnoyers <mathieu.desnoyers@polymtl.ca> wrote:

> * Ingo Molnar (mingo@elte.hu) wrote:
> > 
> > * Mathieu Desnoyers <mathieu.desnoyers@polymtl.ca> wrote:
> > 
> > > Ingo, could you pull the following patch into -tip ?
> > 
> > sure - but it does not apply to tip/master. Some missing dependencies?
> > 
> > 	Ingo
> 
> Hi Ingo,
> 
> "Tracepoints synchronize unregister static inline"
> applies on top of
> "Tracepoints : tracepoint_synchronize_unregister()"
> 
> I posted earlier but does not seem to be in tip/master. Here is a repost
> :
> 
> Tracepoints : tracepoint_synchronize_unregister()

picked them up into tip/tracing/tracepoints:

1c38db9: tracepoints: synchronize unregister static inline
99067e3: tracepoints: tracepoint_synchronize_unregister()

thanks Mathieu!

	Ingo