X-Git-Url: http://git.lttng.org./?a=blobdiff_plain;f=ltt-events.c;h=3532dee5a1c8d46e246fa14c8e6c07a1257f26c7;hb=3568c28ca2b801c6fd572aae078bfb474f720e7c;hp=42d486c195d73c8cdfd265b6a54a5d2bd20950ec;hpb=c6485006a92b1b86fbb48c2b277732d3405295c9;p=lttng-modules.git

diff --git a/ltt-events.c b/ltt-events.c
index 42d486c1..3532dee5 100644
--- a/ltt-events.c
+++ b/ltt-events.c
@@ -14,7 +14,7 @@
 #include <linux/sched.h>
 #include <linux/slab.h>
 #include <linux/jiffies.h>
-#include <linux/uuid.h>
+#include "wrapper/uuid.h"
 #include "wrapper/vmalloc.h"	/* for wrapper_vmalloc_sync_all() */
 #include "ltt-events.h"
 #include "ltt-tracer.h"
@@ -34,8 +34,6 @@ int _ltt_event_metadata_statedump(struct ltt_session *session,
 static
 int _ltt_session_metadata_statedump(struct ltt_session *session);
 
-
-static
 void synchronize_trace(void)
 {
 	synchronize_sched();
@@ -108,12 +106,9 @@ int ltt_session_enable(struct ltt_session *session)
 
 	ACCESS_ONCE(session->active) = 1;
 	ACCESS_ONCE(session->been_active) = 1;
-	synchronize_trace();	/* Wait for in-flight events to complete */
 	ret = _ltt_session_metadata_statedump(session);
-	if (ret) {
+	if (ret)
 		ACCESS_ONCE(session->active) = 0;
-		synchronize_trace();	/* Wait for in-flight events to complete */
-	}
 end:
 	mutex_unlock(&sessions_mutex);
 	return ret;
@@ -129,7 +124,6 @@ int ltt_session_disable(struct ltt_session *session)
 		goto end;
 	}
 	ACCESS_ONCE(session->active) = 0;
-	synchronize_trace();	/* Wait for in-flight events to complete */
 end:
 	mutex_unlock(&sessions_mutex);
 	return ret;
@@ -202,7 +196,7 @@ struct ltt_channel *ltt_channel_create(struct ltt_session *session,
 				       unsigned int read_timer_interval)
 {
 	struct ltt_channel *chan;
-	struct ltt_transport *transport;
+	struct ltt_transport *transport = NULL;
 
 	mutex_lock(&sessions_mutex);
 	if (session->been_active)
@@ -213,6 +207,10 @@ struct ltt_channel *ltt_channel_create(struct ltt_session *session,
 		       transport_name);
 		goto notransport;
 	}
+	if (!try_module_get(transport->owner)) {
+		printk(KERN_WARNING "LTT : Can't lock transport module.\n");
+		goto notransport;
+	}
 	chan = kzalloc(sizeof(struct ltt_channel), GFP_KERNEL);
 	if (!chan)
 		goto nomem;
@@ -230,6 +228,7 @@ struct ltt_channel *ltt_channel_create(struct ltt_session *session,
 		goto create_error;
 	chan->enabled = 1;
 	chan->ops = &transport->ops;
+	chan->transport = transport;
 	list_add(&chan->list, &session->chan);
 	mutex_unlock(&sessions_mutex);
 	return chan;
@@ -237,6 +236,8 @@ struct ltt_channel *ltt_channel_create(struct ltt_session *session,
 create_error:
 	kfree(chan);
 nomem:
+	if (transport)
+		module_put(transport->owner);
 notransport:
 active:
 	mutex_unlock(&sessions_mutex);
@@ -250,6 +251,7 @@ static
 void _ltt_channel_destroy(struct ltt_channel *chan)
 {
 	chan->ops->channel_destroy(chan->chan);
+	module_put(chan->transport->owner);
 	list_del(&chan->list);
 	lttng_destroy_context(chan->ctx);
 	kfree(chan);