X-Git-Url: https://git.lttng.org/?p=lttng-tools.git;a=blobdiff_plain;f=src%2Fbin%2Flttng-sessiond%2Fchannel.c;h=d07e1d901fa3ba10170cb259411c105689a6a4a4;hp=58a36b97c8b2f54085ed0b2ad463ef005c68d53a;hb=f3f0db50cf3fbc658bb4ffaf79a7fb694ab05e1d;hpb=b024d072031acc6eec7e083de606fbce11aab882

diff --git a/src/bin/lttng-sessiond/channel.c b/src/bin/lttng-sessiond/channel.c
index 58a36b97c..d07e1d901 100644
--- a/src/bin/lttng-sessiond/channel.c
+++ b/src/bin/lttng-sessiond/channel.c
@@ -1,18 +1,18 @@
 /*
  * Copyright (C) 2011 - David Goulet <david.goulet@polymtl.ca>
  *
- * This program is free software; you can redistribute it and/or modify it
- * under the terms of the GNU General Public License as published by the Free
- * Software Foundation; only version 2 of the License.
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License, version 2 only,
+ * as published by the Free Software Foundation.
  *
- * This program is distributed in the hope that it will be useful, but WITHOUT
- * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
- * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for
- * more details.
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
  *
- * You should have received a copy of the GNU General Public License along with
- * this program; if not, write to the Free Software Foundation, Inc., 59 Temple
- * Place - Suite 330, Boston, MA  02111-1307, USA.
+ * You should have received a copy of the GNU General Public License along
+ * with this program; if not, write to the Free Software Foundation, Inc.,
+ * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
  */
 
 #define _GNU_SOURCE
@@ -54,7 +54,8 @@ struct lttng_channel *channel_new_default_attr(int dom)
 
 	switch (dom) {
 	case LTTNG_DOMAIN_KERNEL:
-		chan->attr.subbuf_size = DEFAULT_KERNEL_CHANNEL_SUBBUF_SIZE;
+		chan->attr.subbuf_size =
+			default_get_kernel_channel_subbuf_size();
 		chan->attr.num_subbuf = DEFAULT_KERNEL_CHANNEL_SUBBUF_NUM;
 		chan->attr.output = DEFAULT_KERNEL_CHANNEL_OUTPUT;
 		break;
@@ -64,7 +65,7 @@ struct lttng_channel *channel_new_default_attr(int dom)
 	case LTTNG_DOMAIN_UST_PID_FOLLOW_CHILDREN:
 	case LTTNG_DOMAIN_UST_EXEC_NAME:
 #endif
-		chan->attr.subbuf_size = DEFAULT_UST_CHANNEL_SUBBUF_SIZE;
+		chan->attr.subbuf_size = default_get_ust_channel_subbuf_size();
 		chan->attr.num_subbuf = DEFAULT_UST_CHANNEL_SUBBUF_NUM;
 		chan->attr.output = DEFAULT_UST_CHANNEL_OUTPUT;
 		break;
@@ -91,17 +92,17 @@ int channel_kernel_disable(struct ltt_kernel_session *ksession,
 
 	kchan = trace_kernel_get_channel_by_name(channel_name, ksession);
 	if (kchan == NULL) {
-		ret = LTTCOMM_KERN_CHAN_NOT_FOUND;
+		ret = LTTNG_ERR_KERN_CHAN_NOT_FOUND;
 		goto error;
 	} else if (kchan->enabled == 1) {
 		ret = kernel_disable_channel(kchan);
 		if (ret < 0 && ret != -EEXIST) {
-			ret = LTTCOMM_KERN_CHAN_DISABLE_FAIL;
+			ret = LTTNG_ERR_KERN_CHAN_DISABLE_FAIL;
 			goto error;
 		}
 	}
 
-	ret = LTTCOMM_OK;
+	ret = LTTNG_OK;
 
 error:
 	return ret;
@@ -118,12 +119,15 @@ int channel_kernel_enable(struct ltt_kernel_session *ksession,
 	if (kchan->enabled == 0) {
 		ret = kernel_enable_channel(kchan);
 		if (ret < 0) {
-			ret = LTTCOMM_KERN_CHAN_ENABLE_FAIL;
+			ret = LTTNG_ERR_KERN_CHAN_ENABLE_FAIL;
 			goto error;
 		}
+	} else {
+		ret = LTTNG_ERR_KERN_CHAN_EXIST;
+		goto error;
 	}
 
-	ret = LTTCOMM_OK;
+	ret = LTTNG_OK;
 
 error:
 	return ret;
@@ -142,7 +146,7 @@ int channel_kernel_create(struct ltt_kernel_session *ksession,
 	if (attr == NULL) {
 		defattr = channel_new_default_attr(LTTNG_DOMAIN_KERNEL);
 		if (defattr == NULL) {
-			ret = LTTCOMM_FATAL;
+			ret = LTTNG_ERR_FATAL;
 			goto error;
 		}
 		attr = defattr;
@@ -151,18 +155,18 @@ int channel_kernel_create(struct ltt_kernel_session *ksession,
 	/* Channel not found, creating it */
 	ret = kernel_create_channel(ksession, attr, ksession->trace_path);
 	if (ret < 0) {
-		ret = LTTCOMM_KERN_CHAN_FAIL;
+		ret = LTTNG_ERR_KERN_CHAN_FAIL;
 		goto error;
 	}
 
 	/* Notify kernel thread that there is a new channel */
 	ret = notify_thread_pipe(kernel_pipe);
 	if (ret < 0) {
-		ret = LTTCOMM_FATAL;
+		ret = LTTNG_ERR_FATAL;
 		goto error;
 	}
 
-	ret = LTTCOMM_OK;
+	ret = LTTNG_OK;
 error:
 	free(defattr);
 	return ret;
@@ -174,11 +178,12 @@ error:
 int channel_ust_enable(struct ltt_ust_session *usess, int domain,
 		struct ltt_ust_channel *uchan)
 {
-	int ret = LTTCOMM_OK;
+	int ret = LTTNG_OK;
 
 	/* If already enabled, everything is OK */
 	if (uchan->enabled) {
 		DBG3("Channel %s already enabled. Skipping", uchan->name);
+		ret = LTTNG_ERR_UST_CHAN_EXIST;
 		goto end;
 	}
 
@@ -194,16 +199,16 @@ int channel_ust_enable(struct ltt_ust_session *usess, int domain,
 	case LTTNG_DOMAIN_UST_EXEC_NAME:
 #endif
 	default:
-		ret = LTTCOMM_UND;
+		ret = LTTNG_ERR_UND;
 		goto error;
 	}
 
 	if (ret < 0) {
-		if (ret != -EEXIST) {
-			ret = LTTCOMM_UST_CHAN_ENABLE_FAIL;
+		if (ret != -LTTNG_UST_ERR_EXIST) {
+			ret = LTTNG_ERR_UST_CHAN_ENABLE_FAIL;
 			goto error;
 		} else {
-			ret = LTTCOMM_OK;
+			ret = LTTNG_OK;
 		}
 	}
 
@@ -221,7 +226,7 @@ error:
 int channel_ust_create(struct ltt_ust_session *usess, int domain,
 		struct lttng_channel *attr)
 {
-	int ret = LTTCOMM_OK;
+	int ret = LTTNG_OK;
 	struct ltt_ust_channel *uchan = NULL;
 	struct lttng_channel *defattr = NULL;
 
@@ -229,7 +234,7 @@ int channel_ust_create(struct ltt_ust_session *usess, int domain,
 	if (attr == NULL) {
 		defattr = channel_new_default_attr(domain);
 		if (defattr == NULL) {
-			ret = LTTCOMM_FATAL;
+			ret = LTTNG_ERR_FATAL;
 			goto error;
 		}
 		attr = defattr;
@@ -242,18 +247,23 @@ int channel_ust_create(struct ltt_ust_session *usess, int domain,
 	 * (unlike kernel tracing).
 	 */
 	if (!attr->attr.subbuf_size || (attr->attr.subbuf_size & (attr->attr.subbuf_size - 1))) {
-		ret = LTTCOMM_INVALID;
+		ret = LTTNG_ERR_INVALID;
 		goto error;
 	}
 	if (!attr->attr.num_subbuf || (attr->attr.num_subbuf & (attr->attr.num_subbuf - 1))) {
-		ret = LTTCOMM_INVALID;
+		ret = LTTNG_ERR_INVALID;
+		goto error;
+	}
+
+	if (attr->attr.output != LTTNG_EVENT_MMAP) {
+		ret = LTTNG_ERR_NOT_SUPPORTED;
 		goto error;
 	}
 
 	/* Create UST channel */
 	uchan = trace_ust_create_channel(attr, usess->pathname);
 	if (uchan == NULL) {
-		ret = LTTCOMM_FATAL;
+		ret = LTTNG_ERR_FATAL;
 		goto error;
 	}
 	uchan->enabled = 1;
@@ -271,12 +281,12 @@ int channel_ust_create(struct ltt_ust_session *usess, int domain,
 	case LTTNG_DOMAIN_UST_EXEC_NAME:
 #endif
 	default:
-		ret = LTTCOMM_UND;
+		ret = LTTNG_ERR_UND;
 		goto error_free_chan;
 	}
 
-	if (ret < 0 && ret != -EEXIST) {
-		ret = LTTCOMM_UST_CHAN_ENABLE_FAIL;
+	if (ret < 0 && ret != -LTTNG_UST_ERR_EXIST) {
+		ret = LTTNG_ERR_UST_CHAN_FAIL;
 		goto error_free_chan;
 	}
 
@@ -288,7 +298,7 @@ int channel_ust_create(struct ltt_ust_session *usess, int domain,
 	DBG2("Channel %s created successfully", uchan->name);
 
 	free(defattr);
-	return LTTCOMM_OK;
+	return LTTNG_OK;
 
 error_free_chan:
 	/*
@@ -307,7 +317,7 @@ error:
 int channel_ust_disable(struct ltt_ust_session *usess, int domain,
 		struct ltt_ust_channel *uchan)
 {
-	int ret = LTTCOMM_OK;
+	int ret = LTTNG_OK;
 
 	/* Already disabled */
 	if (uchan->enabled == 0) {
@@ -328,12 +338,12 @@ int channel_ust_disable(struct ltt_ust_session *usess, int domain,
 	case LTTNG_DOMAIN_UST_PID:
 #endif
 	default:
-		ret = LTTCOMM_UND;
+		ret = LTTNG_ERR_UND;
 		goto error;
 	}
 
-	if (ret < 0 && ret != -EEXIST) {
-		ret = LTTCOMM_UST_DISABLE_FAIL;
+	if (ret < 0 && ret != -LTTNG_UST_ERR_EXIST) {
+		ret = LTTNG_ERR_UST_CHAN_DISABLE_FAIL;
 		goto error;
 	}
 
@@ -341,7 +351,7 @@ int channel_ust_disable(struct ltt_ust_session *usess, int domain,
 
 	DBG2("Channel %s disabled successfully", uchan->name);
 
-	return LTTCOMM_OK;
+	return LTTNG_OK;
 
 end:
 error: