X-Git-Url: https://git.lttng.org/?p=lttng-tools.git;a=blobdiff_plain;f=src%2Fbin%2Flttng-sessiond%2Fagent-thread.c;h=f8456b428d49882945c2089bf12e3c4121c7420b;hp=39d2ec4a05bd1297c31e2ac7036600772ea38ffa;hb=2288467f63826a06d25ac361fa04ea92ec7ddfa3;hpb=890d8fe47755c3bad936389cf48ffa141cff41c9

diff --git a/src/bin/lttng-sessiond/agent-thread.c b/src/bin/lttng-sessiond/agent-thread.c
index 39d2ec4a0..f8456b428 100644
--- a/src/bin/lttng-sessiond/agent-thread.c
+++ b/src/bin/lttng-sessiond/agent-thread.c
@@ -32,6 +32,8 @@
 #include "session.h"
 #include "utils.h"
 
+static int agent_tracing_enabled = -1;
+
 /*
  * Note that there is not port here. It's set after this URI is parsed so we
  * can let the user define a custom one. However, localhost is ALWAYS the
@@ -81,6 +83,8 @@ static struct lttcomm_sock *init_tcp_socket(void)
 	int ret;
 	struct lttng_uri *uri = NULL;
 	struct lttcomm_sock *sock = NULL;
+	unsigned int port;
+	bool bind_succeeded = false;
 
 	/*
 	 * This should never fail since the URI is hardcoded and the port is set
@@ -88,8 +92,8 @@ static struct lttcomm_sock *init_tcp_socket(void)
 	 */
 	ret = uri_parse(default_reg_uri, &uri);
 	assert(ret);
-	assert(agent_tcp_port);
-	uri->port = agent_tcp_port;
+	assert(config.agent_tcp_port.begin > 0);
+	uri->port = config.agent_tcp_port.begin;
 
 	sock = lttcomm_alloc_sock_from_uri(uri);
 	uri_free(uri);
@@ -103,11 +107,43 @@ static struct lttcomm_sock *init_tcp_socket(void)
 		goto error;
 	}
 
-	ret = sock->ops->bind(sock);
-	if (ret < 0) {
-		WARN("Another session daemon is using this agent port. Agent support "
-				"will be deactivated to prevent interfering with the tracing.");
-		goto error;
+	for (port = config.agent_tcp_port.begin;
+			port <= config.agent_tcp_port.end; port++) {
+		ret = lttcomm_sock_set_port(sock, (uint16_t) port);
+		if (ret) {
+			ERR("[agent-thread] Failed to set port %u on socket",
+					port);
+			goto error;
+		}
+		DBG3("[agent-thread] Trying to bind on port %u", port);
+		ret = sock->ops->bind(sock);
+		if (!ret) {
+			bind_succeeded = true;
+			break;
+		}
+
+		if (errno == EADDRINUSE) {
+			DBG("Failed to bind to port %u since it is already in use",
+					port);
+		} else {
+			PERROR("Failed to bind to port %u", port);
+			goto error;
+		}
+	}
+
+	if (!bind_succeeded) {
+		if (config.agent_tcp_port.begin == config.agent_tcp_port.end) {
+			WARN("Another process is already using the agent port %i. "
+					"Agent support will be deactivated.",
+					config.agent_tcp_port.begin);
+			goto error;
+		} else {
+			WARN("All ports in the range [%i, %i] are already in use. "
+					"Agent support will be deactivated.",
+					config.agent_tcp_port.begin,
+					config.agent_tcp_port.end);
+			goto error;
+		}
 	}
 
 	ret = sock->ops->listen(sock, -1);
@@ -116,7 +152,7 @@ static struct lttcomm_sock *init_tcp_socket(void)
 	}
 
 	DBG("[agent-thread] Listening on TCP port %u and socket %d",
-			agent_tcp_port, sock->fd);
+			port, sock->fd);
 
 	return sock;
 
@@ -132,9 +168,19 @@ error:
  */
 static void destroy_tcp_socket(struct lttcomm_sock *sock)
 {
+	int ret;
+	uint16_t port;
+
 	assert(sock);
 
-	DBG3("[agent-thread] Destroy TCP socket on port %u", agent_tcp_port);
+	ret = lttcomm_sock_get_port(sock, &port);
+	if (ret) {
+		ERR("[agent-thread] Failed to get port of agent TCP socket");
+		port = 0;
+	}
+
+	DBG3("[agent-thread] Destroy TCP socket on port %" PRIu16,
+			port);
 
 	/* This will return gracefully if fd is invalid. */
 	sock->ops->close(sock);
@@ -223,6 +269,24 @@ error:
 	return ret;
 }
 
+bool agent_tracing_is_enabled(void)
+{
+	int enabled;
+
+	enabled = uatomic_read(&agent_tracing_enabled);
+	assert(enabled != -1);
+	return enabled == 1;
+}
+
+/*
+ * Write agent TCP port using the rundir.
+ */
+static int write_agent_port(uint16_t port)
+{
+	return utils_create_pid_file((pid_t) port,
+			config.agent_port_file_path.value);
+}
+
 /*
  * This thread manage application notify communication.
  */
@@ -248,11 +312,32 @@ void *agent_thread_manage_registration(void *data)
 	}
 
 	reg_sock = init_tcp_socket();
-	if (!reg_sock) {
+	if (reg_sock) {
+		uint16_t port;
+
+		assert(lttcomm_sock_get_port(reg_sock, &port) == 0);
+
+		ret = write_agent_port(port);
+		if (ret) {
+			ERR("[agent-thread] Failed to create agent port file: agent tracing will be unavailable");
+			/* Don't prevent the launch of the sessiond on error. */
+			sessiond_notify_ready();
+			goto error;
+		}
+	} else {
+		/* Don't prevent the launch of the sessiond on error. */
+		sessiond_notify_ready();
 		goto error_tcp_socket;
 	}
 
-	/* Add create valid TCP socket to poll set. */
+	/*
+	 * Signal that the agent thread is ready. The command thread
+	 * may start to query whether or not agent tracing is enabled.
+	 */
+	uatomic_set(&agent_tracing_enabled, 1);
+	sessiond_notify_ready();
+
+	/* Add TCP socket to poll set. */
 	ret = lttng_poll_add(&events, reg_sock->fd,
 			LPOLLIN | LPOLLERR | LPOLLHUP | LPOLLRDHUP);
 	if (ret < 0) {
@@ -354,6 +439,7 @@ error:
 error_tcp_socket:
 	lttng_poll_clean(&events);
 error_poll_create:
+	uatomic_set(&agent_tracing_enabled, 0);
 	DBG("[agent-thread] is cleaning up and stopping.");
 
 	rcu_thread_offline();