Fix enable syscall and bad value of poll set size
[lttng-tools.git] / ltt-sessiond / kernel-ctl.c
index 27210708e64f1f791b9322d78e01fe7d3258d5e9..e486fa10ada0002e8e2f7437ce677a0f24475ba6 100644 (file)
 #include <string.h>
 #include <unistd.h>
 
-#include "lttngerr.h"
-#include "libkernelctl.h"
+#include <lttng-kernel-ctl.h>
+#include <lttngerr.h>
+
 #include "kernel-ctl.h"
 
 /*
- *  kernel_add_channel_context
- *
- *  Add context on a kernel channel.
+ * Add context on a kernel channel.
  */
 int kernel_add_channel_context(struct ltt_kernel_channel *chan,
                struct lttng_kernel_context *ctx)
@@ -65,9 +64,7 @@ error:
 }
 
 /*
- *  kernel_add_event_context
- *
- *  Add context on a kernel event.
+ * Add context on a kernel event.
  */
 int kernel_add_event_context(struct ltt_kernel_event *event,
                struct lttng_kernel_context *ctx)
@@ -96,10 +93,8 @@ error:
 }
 
 /*
- *  kernel_create_session
- *
- *  Create a new kernel session, register it to the kernel tracer and add it to
- *  the session daemon session.
+ * Create a new kernel session, register it to the kernel tracer and add it to
+ * the session daemon session.
  */
 int kernel_create_session(struct ltt_session *session, int tracer_fd)
 {
@@ -107,7 +102,7 @@ int kernel_create_session(struct ltt_session *session, int tracer_fd)
        struct ltt_kernel_session *lks;
 
        /* Allocate data structure */
-       lks = trace_create_kernel_session();
+       lks = trace_kernel_create_session(session->path);
        if (lks == NULL) {
                ret = -1;
                goto error;
@@ -139,18 +134,17 @@ error:
 }
 
 /*
- *  kernel_create_channel
- *
- *  Create a kernel channel, register it to the kernel tracer and add it to the
- *  kernel session.
+ * Create a kernel channel, register it to the kernel tracer and add it to the
+ * kernel session.
  */
-int kernel_create_channel(struct ltt_kernel_session *session, struct lttng_channel *chan, char *path)
+int kernel_create_channel(struct ltt_kernel_session *session,
+               struct lttng_channel *chan, char *path)
 {
        int ret;
        struct ltt_kernel_channel *lkc;
 
        /* Allocate kernel channel */
-       lkc = trace_create_kernel_channel(chan, path);
+       lkc = trace_kernel_create_channel(chan, path);
        if (lkc == NULL) {
                goto error;
        }
@@ -184,27 +178,35 @@ error:
 }
 
 /*
- *  kernel_create_event
- *
- *  Create a kernel event, enable it to the kernel tracer and add it to the
- *  channel event list of the kernel session.
+ * Create a kernel event, enable it to the kernel tracer and add it to the
+ * channel event list of the kernel session.
  */
-int kernel_create_event(struct lttng_event *ev, struct ltt_kernel_channel *channel)
+int kernel_create_event(struct lttng_event *ev,
+               struct ltt_kernel_channel *channel)
 {
        int ret;
        struct ltt_kernel_event *event;
 
-       event = trace_create_kernel_event(ev);
+       event = trace_kernel_create_event(ev);
        if (event == NULL) {
                goto error;
        }
 
        ret = kernctl_create_event(channel->fd, event->event);
        if (ret < 0) {
-               perror("create event ioctl");
+               PERROR("create event ioctl");
                goto free_event;
        }
 
+       /*
+        * LTTNG_KERNEL_SYSCALL event creation will return 0 on success. However
+        * this FD must not be added to the event list.
+        */
+       if (ret == 0 && event->event->instrumentation == LTTNG_KERNEL_SYSCALL) {
+               DBG2("Kernel event syscall creation success");
+               goto end;
+       }
+
        event->fd = ret;
        /* Prevent fd duplication after execlp() */
        ret = fcntl(event->fd, F_SETFD, FD_CLOEXEC);
@@ -218,6 +220,7 @@ int kernel_create_event(struct lttng_event *ev, struct ltt_kernel_channel *chann
 
        DBG("Event %s created (fd: %d)", ev->name, event->fd);
 
+end:
        return 0;
 
 free_event:
@@ -227,9 +230,7 @@ error:
 }
 
 /*
- *  kernel_disable_channel
- *
- *  Disable a kernel channel.
+ * Disable a kernel channel.
  */
 int kernel_disable_channel(struct ltt_kernel_channel *chan)
 {
@@ -252,18 +253,15 @@ error:
 }
 
 /*
- *  kernel_enable_channel
- *
- *  Enable a kernel channel.
+ * Enable a kernel channel.
  */
 int kernel_enable_channel(struct ltt_kernel_channel *chan)
 {
        int ret;
 
        ret = kernctl_enable(chan->fd);
-       if (ret < 0) {
-               perror("enable chan ioctl");
-               ret = errno;
+       if (ret < 0 && errno != EEXIST) {
+               perror("Enable kernel chan");
                goto error;
        }
 
@@ -277,20 +275,15 @@ error:
 }
 
 /*
- *  kernel_enable_event
- *
- *  Enable a kernel event.
+ * Enable a kernel event.
  */
 int kernel_enable_event(struct ltt_kernel_event *event)
 {
        int ret;
 
        ret = kernctl_enable(event->fd);
-       if (ret < 0) {
-               perror("enable event ioctl");
-               if (errno == EEXIST) {
-                       ret = -EEXIST;
-               }
+       if (ret < 0 && errno != EEXIST) {
+               perror("enable kernel event");
                goto error;
        }
 
@@ -304,17 +297,15 @@ error:
 }
 
 /*
- *  kernel_disable_event
- *
- *  Disable a kernel event.
+ * Disable a kernel event.
  */
 int kernel_disable_event(struct ltt_kernel_event *event)
 {
        int ret;
 
        ret = kernctl_disable(event->fd);
-       if (ret < 0) {
-               perror("disable event ioctl");
+       if (ret < 0 && errno != EEXIST) {
+               perror("disable kernel event");
                goto error;
        }
 
@@ -328,10 +319,8 @@ error:
 }
 
 /*
- *  kernel_open_metadata
- *
- *  Create kernel metadata, open from the kernel tracer and add it to the
- *  kernel session.
+ * Create kernel metadata, open from the kernel tracer and add it to the
+ * kernel session.
  */
 int kernel_open_metadata(struct ltt_kernel_session *session, char *path)
 {
@@ -339,7 +328,7 @@ int kernel_open_metadata(struct ltt_kernel_session *session, char *path)
        struct ltt_kernel_metadata *lkm;
 
        /* Allocate kernel metadata */
-       lkm = trace_create_kernel_metadata(path);
+       lkm = trace_kernel_create_metadata(path);
        if (lkm == NULL) {
                goto error;
        }
@@ -368,9 +357,7 @@ error:
 }
 
 /*
- *  kernel_start_session
- *
- *  Start tracing session.
+ * Start tracing session.
  */
 int kernel_start_session(struct ltt_kernel_session *session)
 {
@@ -391,9 +378,7 @@ error:
 }
 
 /*
- *  kernel_wait_quiescent
- *
- *  Make a kernel wait to make sure in-flight probe have completed.
+ * Make a kernel wait to make sure in-flight probe have completed.
  */
 void kernel_wait_quiescent(int fd)
 {
@@ -409,8 +394,23 @@ void kernel_wait_quiescent(int fd)
 }
 
 /*
- *  kernel_metadata_flush_buffer
- *
+ * Kernel calibrate
+ */
+int kernel_calibrate(int fd, struct lttng_kernel_calibrate *calibrate)
+{
+       int ret;
+
+       ret = kernctl_calibrate(fd, calibrate);
+       if (ret < 0) {
+               perror("calibrate ioctl");
+               return -1;
+       }
+
+       return 0;
+}
+
+
+/*
  *  Force flush buffer of metadata.
  */
 int kernel_metadata_flush_buffer(int fd)
@@ -426,9 +426,7 @@ int kernel_metadata_flush_buffer(int fd)
 }
 
 /*
- *  kernel_flush_buffer
- *
- *  Force flush buffer for channel.
+ * Force flush buffer for channel.
  */
 int kernel_flush_buffer(struct ltt_kernel_channel *channel)
 {
@@ -451,9 +449,7 @@ int kernel_flush_buffer(struct ltt_kernel_channel *channel)
 }
 
 /*
- *  kernel_stop_session
- *
- *  Stop tracing session.
+ * Stop tracing session.
  */
 int kernel_stop_session(struct ltt_kernel_session *session)
 {
@@ -473,12 +469,10 @@ error:
 }
 
 /*
- *  kernel_open_channel_stream
+ * Open stream of channel, register it to the kernel tracer and add it
+ * to the stream list of the channel.
  *
- *  Open stream of channel, register it to the kernel tracer and add it
- *  to the stream list of the channel.
- *
- *  Return the number of created stream. Else, a negative value.
+ * Return the number of created stream. Else, a negative value.
  */
 int kernel_open_channel_stream(struct ltt_kernel_channel *channel)
 {
@@ -486,7 +480,7 @@ int kernel_open_channel_stream(struct ltt_kernel_channel *channel)
        struct ltt_kernel_stream *lks;
 
        while ((ret = kernctl_create_stream(channel->fd)) > 0) {
-               lks = trace_create_kernel_stream();
+               lks = trace_kernel_create_stream();
                if (lks == NULL) {
                        close(ret);
                        goto error;
@@ -521,9 +515,7 @@ error:
 }
 
 /*
- *  kernel_open_metadata_stream
- *
- *  Open the metadata stream and set it to the kernel session.
+ * Open the metadata stream and set it to the kernel session.
  */
 int kernel_open_metadata_stream(struct ltt_kernel_session *session)
 {
@@ -570,7 +562,7 @@ ssize_t kernel_list_events(int tracer_fd, struct lttng_event **events)
        fp = fdopen(fd, "r");
        if (fp == NULL) {
                perror("kernel tracepoint list fdopen");
-               goto error;
+               goto error_fp;
        }
 
        /*
@@ -582,26 +574,30 @@ ssize_t kernel_list_events(int tracer_fd, struct lttng_event **events)
 
        while ((size = fscanf(fp, "event { name = %m[^;]; };%n\n", &event, &pos)) == 1) {
                if (count > nbmem) {
-                       DBG("Reallocating event list from %zd to %zd bytes", nbmem,
+                       DBG("Reallocating event list from %zu to %zu bytes", nbmem,
                                        nbmem + KERNEL_EVENT_LIST_SIZE);
                        /* Adding the default size again */
                        nbmem += KERNEL_EVENT_LIST_SIZE;
                        elist = realloc(elist, nbmem);
                        if (elist == NULL) {
                                perror("realloc list events");
-                               goto error;
+                               count = -ENOMEM;
+                               goto end;
                        }
                }
-               strncpy(elist[count].name, event, strlen(event));
+               strncpy(elist[count].name, event, LTTNG_SYMBOL_NAME_LEN);
+               elist[count].name[LTTNG_SYMBOL_NAME_LEN - 1] = '\0';
                count++;
        }
 
        *events = elist;
-
-       DBG("Kernel list events done (%ld events)", count);
-
+       DBG("Kernel list events done (%zu events)", count);
+end:
+       fclose(fp);     /* closes both fp and fd */
        return count;
 
+error_fp:
+       close(fd);
 error:
        return -1;
 }
This page took 0.02697 seconds and 4 git commands to generate.