X-Git-Url: https://git.lttng.org/?p=lttng-tools.git;a=blobdiff_plain;f=src%2Fcommon%2Fconsumer-timer.c;h=a32dbf64d2d229e730f6e87a4add490c75567a39;hp=f2dff4cd980aae5258cfc19a3c268bd800c955c8;hb=4fa3dc0ebca381a211ace01c3320591529b5b833;hpb=75d83e506938a3e673ad19bb11838ee88e977e72

diff --git a/src/common/consumer-timer.c b/src/common/consumer-timer.c
index f2dff4cd9..a32dbf64d 100644
--- a/src/common/consumer-timer.c
+++ b/src/common/consumer-timer.c
@@ -78,6 +78,22 @@ static void metadata_switch_timer(struct lttng_consumer_local_data *ctx,
 	switch (ctx->type) {
 	case LTTNG_CONSUMER32_UST:
 	case LTTNG_CONSUMER64_UST:
+		/*
+		 * Locks taken by lttng_ustconsumer_request_metadata():
+		 * - metadata_socket_lock
+		 *   - Calling lttng_ustconsumer_recv_metadata():
+		 *     - consumer_data.lock
+		 *       - channel->lock
+		 *         - channel->metadata_cache->lock
+		 *     - Calling consumer_metadata_cache_flushed():
+		 *       - consumer_data.lock
+		 *         - channel->lock
+		 *           - channel->metadata_cache->lock
+		 *
+		 * Both consumer_data.lock and channel->lock currently
+		 * cause a deadlock, since they are held while
+		 * consumer_timer_switch_stop() is called.
+		 */
 		ret = lttng_ustconsumer_request_metadata(ctx, channel);
 		if (ret < 0) {
 			channel->switch_timer_error = 1;