Cleanup: sessiond jul.c comments and error path
[lttng-tools.git] / src / bin / lttng-sessiond / jul.c
index 87ab654a34f3c109ce7c477fe6b51eb1892ade0d..c4d81217a4ded46ebe712620782dd616fdc63d67 100644 (file)
@@ -27,7 +27,7 @@
 #include "utils.h"
 
 /*
- * URCU intermediate call to complete destroy a JUL event.
+ * URCU delayed JUL event reclaim.
  */
 static void destroy_event_jul_rcu(struct rcu_head *head)
 {
@@ -40,7 +40,7 @@ static void destroy_event_jul_rcu(struct rcu_head *head)
 }
 
 /*
- * URCU intermediate call to complete destroy a JUL event.
+ * URCU delayed JUL app reclaim.
  */
 static void destroy_app_jul_rcu(struct rcu_head *head)
 {
@@ -53,8 +53,8 @@ static void destroy_app_jul_rcu(struct rcu_head *head)
 }
 
 /*
- * Communication with Java agent call. Send the message header to the given
- * socket all in big endian.
+ * Communication with Java agent. Send the message header to the given
+ * socket in big endian.
  *
  * Return 0 on success or else a negative errno message of sendmsg() op.
  */
@@ -133,8 +133,96 @@ error:
        return ret;
 }
 
+
 /*
- * Internal enable JUL event call on a JUL application. This function
+ * Internal event listing for a given app. Populate events.
+ *
+ * Return number of element in the list or else a negative LTTNG_ERR* code.
+ * On success, the caller is responsible for freeing the memory
+ * allocated for "events".
+ */
+static ssize_t list_events(struct jul_app *app, struct lttng_event **events)
+{
+       int ret, i, len = 0, offset = 0;
+       uint32_t nb_event;
+       size_t data_size;
+       struct lttng_event *tmp_events = NULL;
+       struct lttcomm_jul_list_reply *reply = NULL;
+       struct lttcomm_jul_list_reply_hdr reply_hdr;
+
+       assert(app);
+       assert(app->sock);
+       assert(events);
+
+       DBG2("JUL listing events for app pid: %d and socket %d", app->pid,
+                       app->sock->fd);
+
+       ret = send_header(app->sock, 0, JUL_CMD_LIST, 0);
+       if (ret < 0) {
+               goto error_io;
+       }
+
+       /* Get list header so we know how much we'll receive. */
+       ret = recv_reply(app->sock, &reply_hdr, sizeof(reply_hdr));
+       if (ret < 0) {
+               goto error_io;
+       }
+
+       switch (be32toh(reply_hdr.ret_code)) {
+       case JUL_RET_CODE_SUCCESS:
+               data_size = be32toh(reply_hdr.data_size) + sizeof(*reply);
+               break;
+       default:
+               ERR("Java agent returned an unknown code: %" PRIu32,
+                               be32toh(reply_hdr.ret_code));
+               ret = LTTNG_ERR_FATAL;
+               goto error;
+       }
+
+       reply = zmalloc(data_size);
+       if (!reply) {
+               ret = LTTNG_ERR_NOMEM;
+               goto error;
+       }
+
+       /* Get the list with the appropriate data size. */
+       ret = recv_reply(app->sock, reply, data_size);
+       if (ret < 0) {
+               goto error_io;
+       }
+
+       nb_event = be32toh(reply->nb_event);
+       tmp_events = zmalloc(sizeof(*tmp_events) * nb_event);
+       if (!tmp_events) {
+               ret = LTTNG_ERR_NOMEM;
+               goto error;
+       }
+
+       for (i = 0; i < nb_event; i++) {
+               offset += len;
+               strncpy(tmp_events[i].name, reply->payload + offset,
+                               sizeof(tmp_events[i].name));
+               tmp_events[i].pid = app->pid;
+               tmp_events[i].enabled = -1;
+               len = strlen(reply->payload + offset) + 1;
+       }
+
+       *events = tmp_events;
+
+       free(reply);
+       return nb_event;
+
+error_io:
+       ret = LTTNG_ERR_UST_LIST_FAIL;
+error:
+       free(reply);
+       free(tmp_events);
+       return -ret;
+
+}
+
+/*
+ * Internal enable JUL event on a JUL application. This function
  * communicates with the Java agent to enable a given event (Logger name).
  *
  * Return LTTNG_OK on success or else a LTTNG_ERR* code.
@@ -274,9 +362,9 @@ int jul_enable_event(struct jul_event *event)
                if (ret != LTTNG_OK) {
                        goto error;
                }
-               event->enabled = 1;
        }
 
+       event->enabled = 1;
        ret = LTTNG_OK;
 
 error:
@@ -307,9 +395,9 @@ int jul_disable_event(struct jul_event *event)
                if (ret != LTTNG_OK) {
                        goto error;
                }
-               event->enabled = 0;
        }
 
+       event->enabled = 0;
        ret = LTTNG_OK;
 
 error:
@@ -328,7 +416,7 @@ int jul_list_events(struct lttng_event **events)
        int ret;
        size_t nbmem, count = 0;
        struct jul_app *app;
-       struct lttng_event *tmp_events;
+       struct lttng_event *tmp_events = NULL;
        struct lttng_ht_iter iter;
 
        assert(events);
@@ -350,8 +438,7 @@ int jul_list_events(struct lttng_event **events)
                nb_ev = list_events(app, &jul_events);
                if (nb_ev < 0) {
                        ret = nb_ev;
-                       rcu_read_unlock();
-                       goto error;
+                       goto error_unlock;
                }
 
                if (count >= nbmem) {
@@ -364,10 +451,9 @@ int jul_list_events(struct lttng_event **events)
                        ptr = realloc(tmp_events, nbmem * sizeof(*tmp_events));
                        if (!ptr) {
                                PERROR("realloc JUL events");
-                               free(tmp_events);
                                ret = -ENOMEM;
-                               rcu_read_unlock();
-                               goto error;
+                               free(jul_events);
+                               goto error_unlock;
                        }
                        tmp_events = ptr;
                }
@@ -380,8 +466,12 @@ int jul_list_events(struct lttng_event **events)
 
        ret = count;
        *events = tmp_events;
+       return ret;
 
+error_unlock:
+       rcu_read_unlock();
 error:
+       free(tmp_events);
        return ret;
 }
 
@@ -530,7 +620,8 @@ void jul_delete_app(struct jul_app *app)
 
 /*
  * Destroy a JUL application object by detaching it from its corresponding UST
- * app if one, closing the socket and freeing the memory.
+ * app if one is connected by closing the socket. Finally, perform a
+ * delayed memory reclaim.
  */
 void jul_destroy_app(struct jul_app *app)
 {
@@ -608,6 +699,7 @@ void jul_add_event(struct jul_event *event, struct jul_domain *dom)
        rcu_read_lock();
        lttng_ht_add_unique_str(dom->events, &event->node);
        rcu_read_unlock();
+       dom->being_used = 1;
 }
 
 /*
@@ -662,7 +754,9 @@ void jul_delete_event(struct jul_event *event, struct jul_domain *dom)
 }
 
 /*
- * Free given JUl event. After this call, the pointer is not usable anymore.
+ * Free given JUL event. This event must not be globally visible at this
+ * point (only expected to be used on failure just after event
+ * creation). After this call, the pointer is not usable anymore.
  */
 void jul_destroy_event(struct jul_event *event)
 {
@@ -673,7 +767,7 @@ void jul_destroy_event(struct jul_event *event)
 
 /*
  * Destroy a JUL domain completely. Note that the given pointer is NOT freed
- * thus a reference can be passed to this function.
+ * thus a reference to static or stack data can be passed to this function.
  */
 void jul_destroy_domain(struct jul_domain *dom)
 {
This page took 0.026878 seconds and 5 git commands to generate.