From 577983cb5ac581ff691d650bd6c5c68719feb31f Mon Sep 17 00:00:00 2001 From: =?utf8?q?J=C3=A9r=C3=A9mie=20Galarneau?= Date: Fri, 17 Aug 2018 13:25:49 -0400 Subject: [PATCH] Fix: add session_info object to sessions_ht MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit The notification subsystem currently does not add the session_info structures it uses to account for session consumed sizes to its sessions_ht. This means that a new internal session is created for every channel, resulting in an incorrect accounting of session consumed sizes, and thus delayed rotations. Signed-off-by: Jérémie Galarneau --- src/bin/lttng-sessiond/notification-thread-events.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/bin/lttng-sessiond/notification-thread-events.c b/src/bin/lttng-sessiond/notification-thread-events.c index e915b33b5..0f43e380b 100644 --- a/src/bin/lttng-sessiond/notification-thread-events.c +++ b/src/bin/lttng-sessiond/notification-thread-events.c @@ -1025,6 +1025,9 @@ struct session_info *find_or_create_session_info( name, uid, gid); goto end; } + + cds_lfht_add(state->sessions_ht, hash_key_str(name, lttng_ht_seed), + &sessions->sessions_ht_node); end: rcu_read_unlock(); return session; -- 2.34.1