projects
/
lttng-tools.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Merge branch 'master' of git://git.lttng.org/lttng-tools
[lttng-tools.git]
/
lttng-sessiond
/
channel.c
diff --git
a/lttng-sessiond/channel.c
b/lttng-sessiond/channel.c
index 7ff974da05930c772b5734e1e7de3d4098766df8..3dc04ccb83829ad2077dc2ced938d3ee6c40671b 100644
(file)
--- a/
lttng-sessiond/channel.c
+++ b/
lttng-sessiond/channel.c
@@
-236,11
+236,16
@@
int channel_ust_create(struct ltt_ust_session *usess, int domain,
ret = LTTCOMM_FATAL;
goto error;
}
ret = LTTCOMM_FATAL;
goto error;
}
+ uchan->enabled = 1;
switch (domain) {
case LTTNG_DOMAIN_UST:
DBG2("Channel %s being created in UST global domain", uchan->name);
switch (domain) {
case LTTNG_DOMAIN_UST:
DBG2("Channel %s being created in UST global domain", uchan->name);
- chan_ht = usess->domain_global.channels;
+
+ /* Adding the channel to the channel hash table. */
+ rcu_read_lock();
+ lttng_ht_add_unique_str(usess->domain_global.channels, &uchan->node);
+ rcu_read_unlock();
/* Enable channel for global domain */
ret = ust_app_create_channel_glb(usess, uchan);
/* Enable channel for global domain */
ret = ust_app_create_channel_glb(usess, uchan);
@@
-258,14
+263,16
@@
int channel_ust_create(struct ltt_ust_session *usess, int domain,
goto error_free_chan;
}
goto error_free_chan;
}
- uchan->enabled = 1;
- lttng_ht_add_unique_str(chan_ht, &uchan->node);
DBG2("Channel %s created successfully", uchan->name);
free(defattr);
return LTTCOMM_OK;
error_free_chan:
DBG2("Channel %s created successfully", uchan->name);
free(defattr);
return LTTCOMM_OK;
error_free_chan:
+ /*
+ * No need to remove the channel from the hash table because at this point
+ * it was not added hence the direct call and no call_rcu().
+ */
trace_ust_destroy_channel(uchan);
error:
free(defattr);
trace_ust_destroy_channel(uchan);
error:
free(defattr);
This page took
0.024697 seconds
and
4
git commands to generate.