projects
/
lttng-tools.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
rculfhash: check for callers from RCU read-side C.S.
[lttng-tools.git]
/
src
/
common
/
consumer.c
diff --git
a/src/common/consumer.c
b/src/common/consumer.c
index 52a4ed171d6b0dde5d1759c781f54ef45994cab0..34385579741f6f0273aee213576bd0a188a95b0e 100644
(file)
--- a/
src/common/consumer.c
+++ b/
src/common/consumer.c
@@
-516,8
+516,7
@@
void consumer_del_stream(struct lttng_consumer_stream *stream,
}
rcu_read_unlock();
}
rcu_read_unlock();
- uatomic_dec(&stream->chan->refcount);
- if (!uatomic_read(&stream->chan->refcount)
+ if (!uatomic_sub_return(&stream->chan->refcount, 1)
&& !uatomic_read(&stream->chan->nb_init_stream_left)) {
free_chan = stream->chan;
}
&& !uatomic_read(&stream->chan->nb_init_stream_left)) {
free_chan = stream->chan;
}
@@
-659,6
+658,8
@@
static int add_stream(struct lttng_consumer_stream *stream,
* stream.
*/
if (uatomic_read(&stream->chan->nb_init_stream_left) > 0) {
* stream.
*/
if (uatomic_read(&stream->chan->nb_init_stream_left) > 0) {
+ /* Increment refcount before decrementing nb_init_stream_left */
+ cmm_smp_wmb();
uatomic_dec(&stream->chan->nb_init_stream_left);
}
uatomic_dec(&stream->chan->nb_init_stream_left);
}
@@
-1937,8
+1938,7
@@
void consumer_del_metadata_stream(struct lttng_consumer_stream *stream,
rcu_read_unlock();
/* Atomically decrement channel refcount since other threads can use it. */
rcu_read_unlock();
/* Atomically decrement channel refcount since other threads can use it. */
- uatomic_dec(&stream->chan->refcount);
- if (!uatomic_read(&stream->chan->refcount)
+ if (!uatomic_sub_return(&stream->chan->refcount, 1)
&& !uatomic_read(&stream->chan->nb_init_stream_left)) {
/* Go for channel deletion! */
free_chan = stream->chan;
&& !uatomic_read(&stream->chan->nb_init_stream_left)) {
/* Go for channel deletion! */
free_chan = stream->chan;
@@
-2008,6
+2008,8
@@
static int add_metadata_stream(struct lttng_consumer_stream *stream,
* stream.
*/
if (uatomic_read(&stream->chan->nb_init_stream_left) > 0) {
* stream.
*/
if (uatomic_read(&stream->chan->nb_init_stream_left) > 0) {
+ /* Increment refcount before decrementing nb_init_stream_left */
+ cmm_smp_wmb();
uatomic_dec(&stream->chan->nb_init_stream_left);
}
uatomic_dec(&stream->chan->nb_init_stream_left);
}
@@
-2557,6
+2559,13
@@
void consumer_close_channel_streams(struct lttng_consumer_channel *channel)
ht->hash_fct(&channel->key, lttng_ht_seed),
ht->match_fct, &channel->key,
&iter.iter, stream, node_channel_id.node) {
ht->hash_fct(&channel->key, lttng_ht_seed),
ht->match_fct, &channel->key,
&iter.iter, stream, node_channel_id.node) {
+ /*
+ * Protect against teardown with mutex.
+ */
+ pthread_mutex_lock(&stream->lock);
+ if (cds_lfht_is_node_deleted(&stream->node.node)) {
+ goto next;
+ }
switch (consumer_data.type) {
case LTTNG_CONSUMER_KERNEL:
break;
switch (consumer_data.type) {
case LTTNG_CONSUMER_KERNEL:
break;
@@
-2573,6
+2582,8
@@
void consumer_close_channel_streams(struct lttng_consumer_channel *channel)
ERR("Unknown consumer_data type");
assert(0);
}
ERR("Unknown consumer_data type");
assert(0);
}
+ next:
+ pthread_mutex_unlock(&stream->lock);
}
rcu_read_unlock();
}
}
rcu_read_unlock();
}
@@
-2737,6
+2748,12
@@
restart:
ret = lttng_ht_del(channel_ht, &iter);
assert(ret == 0);
consumer_close_channel_streams(chan);
ret = lttng_ht_del(channel_ht, &iter);
assert(ret == 0);
consumer_close_channel_streams(chan);
+
+ /* Release our own refcount */
+ if (!uatomic_sub_return(&chan->refcount, 1)
+ && !uatomic_read(&chan->nb_init_stream_left)) {
+ consumer_del_channel(chan);
+ }
}
/* Release RCU lock for the channel looked up */
}
/* Release RCU lock for the channel looked up */
@@
-2999,7
+3016,7
@@
int consumer_add_relayd_socket(int net_seq_idx, int sock_type,
{
int fd = -1, ret = -1, relayd_created = 0;
enum lttng_error_code ret_code = LTTNG_OK;
{
int fd = -1, ret = -1, relayd_created = 0;
enum lttng_error_code ret_code = LTTNG_OK;
- struct consumer_relayd_sock_pair *relayd;
+ struct consumer_relayd_sock_pair *relayd
= NULL
;
assert(ctx);
assert(relayd_sock);
assert(ctx);
assert(relayd_sock);
This page took
0.028738 seconds
and
4
git commands to generate.