X-Git-Url: https://git.lttng.org/?p=lttng-tools.git;a=blobdiff_plain;f=src%2Fbin%2Flttng-sessiond%2Fagent-thread.c;h=9c98d30ab4e71accd303a442d41b45feeee212b6;hp=b238656812c69390b2d7e16ee841d7830cd3ba03;hb=5b0936814a6004448eeec430b2dfea534f35a2fc;hpb=04fd2d2e705f7d9956f4cb8b18f6c43452913a16

diff --git a/src/bin/lttng-sessiond/agent-thread.c b/src/bin/lttng-sessiond/agent-thread.c
index b23865681..9c98d30ab 100644
--- a/src/bin/lttng-sessiond/agent-thread.c
+++ b/src/bin/lttng-sessiond/agent-thread.c
@@ -31,6 +31,7 @@
 #include "lttng-sessiond.h"
 #include "session.h"
 #include "utils.h"
+#include "thread.h"
 
 static int agent_tracing_enabled = -1;
 
@@ -59,6 +60,10 @@ static void update_agent_app(struct agent_app *app)
 
 	session_lock_list();
 	cds_list_for_each_entry_safe(session, stmp, &list->head, list) {
+		if (!session_get(session)) {
+			continue;
+		}
+
 		session_lock(session);
 		if (session->ust_session) {
 			struct agent *agt;
@@ -71,6 +76,7 @@ static void update_agent_app(struct agent_app *app)
 			rcu_read_unlock();
 		}
 		session_unlock(session);
+		session_put(session);
 	}
 	session_unlock_list();
 }
@@ -290,12 +296,14 @@ static int write_agent_port(uint16_t port)
 /*
  * This thread manage application notify communication.
  */
-void *agent_thread_manage_registration(void *data)
+static void *thread_agent_management(void *data)
 {
 	int i, ret, pollfd;
 	uint32_t revents, nb_fd;
 	struct lttng_poll_event events;
 	struct lttcomm_sock *reg_sock;
+	struct lttng_pipe *quit_pipe = data;
+	const int quit_pipe_read_fd = lttng_pipe_get_readfd(quit_pipe);
 
 	DBG("[agent-thread] Manage agent application registration.");
 
@@ -305,13 +313,18 @@ void *agent_thread_manage_registration(void *data)
 	/* Agent initialization call MUST be called before starting the thread. */
 	assert(agent_apps_ht_by_sock);
 
-	/* Create pollset with size 2, quit pipe and socket. */
-	ret = sessiond_set_thread_pollset(&events, 2);
+	/* Create pollset with size 2, quit pipe and registration socket. */
+	ret = lttng_poll_create(&events, 2, LTTNG_CLOEXEC);
 	if (ret < 0) {
-		sessiond_notify_ready();
 		goto error_poll_create;
 	}
 
+	ret = lttng_poll_add(&events, quit_pipe_read_fd,
+			LPOLLIN | LPOLLERR);
+	if (ret < 0) {
+		goto error_tcp_socket;
+	}
+
 	reg_sock = init_tcp_socket();
 	if (reg_sock) {
 		uint16_t port;
@@ -376,8 +389,7 @@ restart:
 			}
 
 			/* Thread quit pipe has been closed. Killing thread. */
-			ret = sessiond_check_thread_quit_pipe(pollfd, revents);
-			if (ret) {
+			if (pollfd == quit_pipe_read_fd) {
 				goto exit;
 			}
 
@@ -441,9 +453,48 @@ error_tcp_socket:
 	lttng_poll_clean(&events);
 error_poll_create:
 	uatomic_set(&agent_tracing_enabled, 0);
-	DBG("[agent-thread] is cleaning up and stopping.");
-
+	DBG("[agent-thread] Cleaning up and stopping.");
 	rcu_thread_offline();
 	rcu_unregister_thread();
 	return NULL;
 }
+
+static bool shutdown_agent_management_thread(void *data)
+{
+	struct lttng_pipe *quit_pipe = data;
+	const int write_fd = lttng_pipe_get_writefd(quit_pipe);
+
+	return notify_thread_pipe(write_fd) == 1;
+}
+
+static void cleanup_agent_management_thread(void *data)
+{
+	struct lttng_pipe *quit_pipe = data;
+
+	lttng_pipe_destroy(quit_pipe);
+}
+
+bool launch_agent_management_thread(void)
+{
+	struct lttng_pipe *quit_pipe;
+	struct lttng_thread *thread;
+
+	quit_pipe = lttng_pipe_open(FD_CLOEXEC);
+	if (!quit_pipe) {
+		goto error;
+	}
+	thread = lttng_thread_create("Agent management",
+			thread_agent_management,
+			shutdown_agent_management_thread,
+			cleanup_agent_management_thread,
+			quit_pipe);
+	if (!thread) {
+		goto error;
+	}
+
+	lttng_thread_put(thread);
+	return true;
+error:
+	cleanup_agent_management_thread(quit_pipe);
+	return false;
+}