X-Git-Url: https://git.lttng.org/?a=blobdiff_plain;f=src%2Fbin%2Flttng-sessiond%2Fconsumer.h;h=27da4e38c2bfa7e70a460e4c61abeec40cd82e0b;hb=efa116c6f19ecdc344a82709e7d919703ec96c45;hp=eaa115333584d88e6edcfa3785b46ec3addf78d3;hpb=52898cb1ecfe2cf37bda9b95039826da517b9a5b;p=lttng-tools.git

diff --git a/src/bin/lttng-sessiond/consumer.h b/src/bin/lttng-sessiond/consumer.h
index eaa115333..27da4e38c 100644
--- a/src/bin/lttng-sessiond/consumer.h
+++ b/src/bin/lttng-sessiond/consumer.h
@@ -35,9 +35,10 @@ enum consumer_dst_type {
 struct consumer_socket {
 	/*
 	 * File descriptor. This is just a reference to the consumer data meaning
-	 * that every access must be locked and check for a possible invalid value.
+	 * that every access must be locked and checked for a possible invalid
+	 * value.
 	 */
-	int *fd;
+	int *fd_ptr;
 
 	/*
 	 * To use this socket (send/recv), this lock MUST be acquired.
@@ -94,8 +95,8 @@ struct consumer_data {
 	/*
 	 * The metadata socket object is handled differently and only created
 	 * locally in this object thus it's the only reference available in the
-	 * session daemon. For that reason, a static variable for the fd is
-	 * required and the metadata socket fd points to it.
+	 * session daemon. For that reason, a variable for the fd is required and
+	 * the metadata socket fd points to it.
 	 */
 	int metadata_fd;
 	struct consumer_socket metadata_sock;
@@ -104,7 +105,11 @@ struct consumer_data {
 	char err_unix_sock_path[PATH_MAX];
 	char cmd_unix_sock_path[PATH_MAX];
 
-	/* communication lock */
+	/*
+	 * This lock has two purposes. It protects any change to the consumer
+	 * socket and make sure only one thread uses this object for read/write
+	 * operations.
+	 */
 	pthread_mutex_t lock;
 };