X-Git-Url: https://git.liburcu.org/?a=blobdiff_plain;f=src%2Fbin%2Flttng-sessiond%2Fcontext.c;h=1b87df2e90ab03a3d95192a5afc102fc21404397;hb=aa3514e96f12c13f681a81ea275dc51dd63473c8;hp=6aacbad0ea1c96ad3375739ca908f765f1ba4370;hpb=f263b7fd113e51d0737554e8232b8669e142a260;p=lttng-tools.git

diff --git a/src/bin/lttng-sessiond/context.c b/src/bin/lttng-sessiond/context.c
index 6aacbad0e..1b87df2e9 100644
--- a/src/bin/lttng-sessiond/context.c
+++ b/src/bin/lttng-sessiond/context.c
@@ -92,13 +92,19 @@ static int add_uctx_to_channel(struct ltt_ust_session *usess, int domain,
 {
 	int ret;
 	struct ltt_ust_context *uctx;
-	struct lttng_ht_iter iter;
-	struct lttng_ht_node_ulong *uctx_node;
 
 	assert(usess);
 	assert(uchan);
 	assert(ctx);
 
+	/* Check if context is duplicate */
+	cds_list_for_each_entry(uctx, &uchan->ctx_list, list) {
+		if (trace_ust_match_context(uctx, ctx)) {
+			ret = -EEXIST;
+			goto duplicate;
+		}
+	}
+
 	/* Create ltt UST context */
 	uctx = trace_ust_create_context(ctx);
 	if (uctx == NULL) {
@@ -120,17 +126,8 @@ static int add_uctx_to_channel(struct ltt_ust_session *usess, int domain,
 
 	rcu_read_lock();
 
-	/* Lookup context before adding it */
-	lttng_ht_lookup(uchan->ctx, (void *)((unsigned long)uctx->ctx.ctx), &iter);
-	uctx_node = lttng_ht_iter_get_node_ulong(&iter);
-	if (uctx_node != NULL) {
-		ret = -EEXIST;
-		rcu_read_unlock();
-		goto error;
-	}
-
 	/* Add ltt UST context node to ltt UST channel */
-	lttng_ht_add_unique_ulong(uchan->ctx, &uctx->node);
+	lttng_ht_add_ulong(uchan->ctx, &uctx->node);
 	rcu_read_unlock();
 	cds_list_add_tail(&uctx->list, &uchan->ctx_list);
 
@@ -140,6 +137,7 @@ static int add_uctx_to_channel(struct ltt_ust_session *usess, int domain,
 
 error:
 	free(uctx);
+duplicate:
 	return ret;
 }
 
@@ -162,9 +160,6 @@ int context_kernel_add(struct ltt_kernel_session *ksession,
 	case LTTNG_EVENT_CONTEXT_PID:
 		kctx.ctx = LTTNG_KERNEL_CONTEXT_PID;
 		break;
-	case LTTNG_EVENT_CONTEXT_PERF_COUNTER:
-		kctx.ctx = LTTNG_KERNEL_CONTEXT_PERF_COUNTER;
-		break;
 	case LTTNG_EVENT_CONTEXT_PROCNAME:
 		kctx.ctx = LTTNG_KERNEL_CONTEXT_PROCNAME;
 		break;
@@ -192,6 +187,10 @@ int context_kernel_add(struct ltt_kernel_session *ksession,
 	case LTTNG_EVENT_CONTEXT_HOSTNAME:
 		kctx.ctx = LTTNG_KERNEL_CONTEXT_HOSTNAME;
 		break;
+	case LTTNG_EVENT_CONTEXT_PERF_CPU_COUNTER:
+	case LTTNG_EVENT_CONTEXT_PERF_COUNTER:
+		kctx.ctx = LTTNG_KERNEL_CONTEXT_PERF_CPU_COUNTER;
+		break;
 	default:
 		return LTTNG_ERR_KERN_CONTEXT_FAIL;
 	}