X-Git-Url: https://git.lttng.org/?a=blobdiff_plain;f=src%2Fbin%2Flttng-sessiond%2Fnotification-thread-events.cpp;h=909ccbff2cc1b6f36eba822957bdfec10a11a957;hb=8a880a84dcb8ee64cbc4a5e04cae6775b4f9babd;hp=279bc7c5a26aa797c08d800b38b16ae98aa04627;hpb=8ebf1688e54cb656c2a9d17d70a829664e1eaf6b;p=lttng-tools.git

diff --git a/src/bin/lttng-sessiond/notification-thread-events.cpp b/src/bin/lttng-sessiond/notification-thread-events.cpp
index 279bc7c5a..909ccbff2 100644
--- a/src/bin/lttng-sessiond/notification-thread-events.cpp
+++ b/src/bin/lttng-sessiond/notification-thread-events.cpp
@@ -3123,28 +3123,43 @@ end:
 	return 0;
 }
 
+static
+int pop_cmd_queue(struct notification_thread_handle *handle,
+		struct notification_thread_command **cmd)
+{
+	int ret;
+	uint64_t counter;
+
+	pthread_mutex_lock(&handle->cmd_queue.lock);
+	ret = lttng_read(handle->cmd_queue.event_fd, &counter, sizeof(counter));
+	if (ret != sizeof(counter)) {
+		ret = -1;
+		goto error_unlock;
+	}
+
+	*cmd = cds_list_first_entry(&handle->cmd_queue.list,
+			struct notification_thread_command, cmd_list_node);
+	cds_list_del(&((*cmd)->cmd_list_node));
+	ret = 0;
+
+error_unlock:
+	pthread_mutex_unlock(&handle->cmd_queue.lock);
+	return ret;
+}
+
 /* Returns 0 on success, 1 on exit requested, negative value on error. */
 int handle_notification_thread_command(
 		struct notification_thread_handle *handle,
 		struct notification_thread_state *state)
 {
 	int ret;
-	uint64_t counter;
 	struct notification_thread_command *cmd;
 
-	/* Read the event pipe to put it back into a quiescent state. */
-	ret = lttng_read(lttng_pipe_get_readfd(handle->cmd_queue.event_pipe), &counter,
-			sizeof(counter));
-	if (ret != sizeof(counter)) {
+	ret = pop_cmd_queue(handle, &cmd);
+	if (ret) {
 		goto error;
 	}
 
-	pthread_mutex_lock(&handle->cmd_queue.lock);
-	cmd = cds_list_first_entry(&handle->cmd_queue.list,
-			struct notification_thread_command, cmd_list_node);
-	cds_list_del(&cmd->cmd_list_node);
-	pthread_mutex_unlock(&handle->cmd_queue.lock);
-
 	DBG("Received `%s` command",
 			notification_command_type_str(cmd->type));
 	switch (cmd->type) {