X-Git-Url: http://git.lttng.org/?a=blobdiff_plain;f=lttng-events.c;h=ff28c9af559aecff2d5952a6f5fe9a43e2e343fe;hb=0c95667685cf8612eb0d231883874c173b7bb5a1;hp=97d82a297b911d9ad945cc6b88e484d87d2d8fb3;hpb=abc0446a8da1fe7cc09a546389cbcc932b265520;p=lttng-modules.git

diff --git a/lttng-events.c b/lttng-events.c
index 97d82a29..ff28c9af 100644
--- a/lttng-events.c
+++ b/lttng-events.c
@@ -368,7 +368,7 @@ struct lttng_event *lttng_event_create(struct lttng_channel *chan,
 
 	mutex_lock(&sessions_mutex);
 	if (chan->free_event_id == -1U) {
-		event = ERR_PTR(-EMFILE);
+		ret = -EMFILE;
 		goto full;
 	}
 	/*
@@ -377,13 +377,13 @@ struct lttng_event *lttng_event_create(struct lttng_channel *chan,
 	 */
 	list_for_each_entry(event, &chan->session->events, list) {
 		if (!strcmp(event->desc->name, event_param->name)) {
-			event = ERR_PTR(-EEXIST);
+			ret = -EEXIST;
 			goto exist;
 		}
 	}
 	event = kmem_cache_zalloc(event_cache, GFP_KERNEL);
 	if (!event) {
-		event = ERR_PTR(-ENOMEM);
+		ret = -ENOMEM;
 		goto cache_error;
 	}
 	event->chan = chan;
@@ -397,14 +397,14 @@ struct lttng_event *lttng_event_create(struct lttng_channel *chan,
 	case LTTNG_KERNEL_TRACEPOINT:
 		event->desc = lttng_event_get(event_param->name);
 		if (!event->desc) {
-			event = ERR_PTR(-ENOENT);
+			ret = -ENOENT;
 			goto register_error;
 		}
-		ret = kabi_2635_tracepoint_probe_register(event_param->name,
+		ret = kabi_2635_tracepoint_probe_register(event->desc->kname,
 				event->desc->probe_callback,
 				event);
 		if (ret) {
-			event = ERR_PTR(-EINVAL);
+			ret = -EINVAL;
 			goto register_error;
 		}
 		break;
@@ -415,7 +415,7 @@ struct lttng_event *lttng_event_create(struct lttng_channel *chan,
 				event_param->u.kprobe.addr,
 				event);
 		if (ret) {
-			event = ERR_PTR(-EINVAL);
+			ret = -EINVAL;
 			goto register_error;
 		}
 		ret = try_module_get(event->desc->owner);
@@ -429,7 +429,7 @@ struct lttng_event *lttng_event_create(struct lttng_channel *chan,
 		event_return =
 			kmem_cache_zalloc(event_cache, GFP_KERNEL);
 		if (!event_return) {
-			event = ERR_PTR(-ENOMEM);
+			ret = -ENOMEM;
 			goto register_error;
 		}
 		event_return->chan = chan;
@@ -448,7 +448,7 @@ struct lttng_event *lttng_event_create(struct lttng_channel *chan,
 				event, event_return);
 		if (ret) {
 			kmem_cache_free(event_cache, event_return);
-			event = ERR_PTR(-EINVAL);
+			ret = -EINVAL;
 			goto register_error;
 		}
 		/* Take 2 refs on the module: one per event. */
@@ -463,7 +463,6 @@ struct lttng_event *lttng_event_create(struct lttng_channel *chan,
 			kmem_cache_free(event_cache, event_return);
 			module_put(event->desc->owner);
 			module_put(event->desc->owner);
-			event = ERR_PTR(ret);
 			goto statedump_error;
 		}
 		list_add(&event_return->list, &chan->session->events);
@@ -474,7 +473,6 @@ struct lttng_event *lttng_event_create(struct lttng_channel *chan,
 				event_param->u.ftrace.symbol_name,
 				event);
 		if (ret) {
-			event = ERR_PTR(ret);
 			goto register_error;
 		}
 		ret = try_module_get(event->desc->owner);
@@ -483,19 +481,18 @@ struct lttng_event *lttng_event_create(struct lttng_channel *chan,
 	case LTTNG_KERNEL_NOOP:
 		event->desc = internal_desc;
 		if (!event->desc) {
-			event = ERR_PTR(-EINVAL);
+			ret = -EINVAL;
 			goto register_error;
 		}
 		break;
 	default:
 		WARN_ON_ONCE(1);
-		event = ERR_PTR(-EINVAL);
+		ret = -EINVAL;
 		goto register_error;
 	}
 	ret = _lttng_event_metadata_statedump(chan->session, chan, event);
 	WARN_ON_ONCE(ret > 0);
 	if (ret) {
-		event = ERR_PTR(ret);
 		goto statedump_error;
 	}
 	list_add(&event->list, &chan->session->events);
@@ -510,7 +507,7 @@ cache_error:
 exist:
 full:
 	mutex_unlock(&sessions_mutex);
-	return event;
+	return ERR_PTR(ret);
 }
 
 /*
@@ -522,7 +519,7 @@ int _lttng_event_unregister(struct lttng_event *event)
 
 	switch (event->instrumentation) {
 	case LTTNG_KERNEL_TRACEPOINT:
-		ret = kabi_2635_tracepoint_probe_unregister(event->desc->name,
+		ret = kabi_2635_tracepoint_probe_unregister(event->desc->kname,
 						  event->desc->probe_callback,
 						  event);
 		if (ret)
@@ -1264,7 +1261,13 @@ static int __init lttng_events_init(void)
 	ret = lttng_abi_init();
 	if (ret)
 		goto error_abi;
+	ret = lttng_logger_init();
+	if (ret)
+		goto error_logger;
 	return 0;
+
+error_logger:
+	lttng_abi_exit();
 error_abi:
 	kmem_cache_destroy(event_cache);
 	return ret;
@@ -1276,6 +1279,7 @@ static void __exit lttng_events_exit(void)
 {
 	struct lttng_session *session, *tmpsession;
 
+	lttng_logger_exit();
 	lttng_abi_exit();
 	list_for_each_entry_safe(session, tmpsession, &sessions, list)
 		lttng_session_destroy(session);
@@ -1289,4 +1293,5 @@ MODULE_AUTHOR("Mathieu Desnoyers <mathieu.desnoyers@efficios.com>");
 MODULE_DESCRIPTION("LTTng Events");
 MODULE_VERSION(__stringify(LTTNG_MODULES_MAJOR_VERSION) "."
 	__stringify(LTTNG_MODULES_MINOR_VERSION) "."
-	__stringify(LTTNG_MODULES_PATCHLEVEL_VERSION));
+	__stringify(LTTNG_MODULES_PATCHLEVEL_VERSION)
+	LTTNG_MODULES_EXTRAVERSION);