Fix: add session_info object to sessions_ht
authorJérémie Galarneau <jeremie.galarneau@efficios.com>
Fri, 17 Aug 2018 17:25:49 +0000 (13:25 -0400)
committerJérémie Galarneau <jeremie.galarneau@efficios.com>
Fri, 24 Aug 2018 01:15:03 +0000 (21:15 -0400)
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 <jeremie.galarneau@efficios.com>
src/bin/lttng-sessiond/notification-thread-events.c

index e915b33b55631e4035052d4e66a46056dca464c7..0f43e380b20a0799e0e3c3a37a83d1fe4518e7d4 100644 (file)
@@ -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;
This page took 0.02747 seconds and 5 git commands to generate.