Mi enable-event command: support for enable-event command
authorJonathan Rajotte Julien <jonathan.r.julien@gmail.com>
Wed, 18 Jun 2014 21:12:41 +0000 (17:12 -0400)
committerJonathan Rajotte Julien <jonathan.r.julien@gmail.com>
Tue, 22 Jul 2014 20:14:57 +0000 (16:14 -0400)
Signed-off-by: Jonathan Rajotte Julien <jonathan.r.julien@gmail.com>
src/bin/lttng/commands/enable_events.c
src/common/mi-lttng.c
src/common/mi-lttng.h
src/common/mi_lttng.xsd

index 9b16d7c86ec7959230e83ac97160987e5b30d719..1f4e02470bb6a176a350aab16ecbffdced00b2b5 100644 (file)
 #include <inttypes.h>
 #include <ctype.h>
 
-#include "../command.h"
 #include <src/common/sessiond-comm/sessiond-comm.h>
 
+/* Mi dependancy */
+#include <common/mi-lttng.h>
+
+#include "../command.h"
+
 #if (LTTNG_SYMBOL_NAME_LEN == 256)
 #define LTTNG_SYMBOL_NAME_LEN_SCANF_IS_A_BROKEN_API    "255"
 #endif
@@ -71,6 +75,7 @@ enum {
 };
 
 static struct lttng_handle *handle;
+static struct mi_writer *writer;
 
 static struct poptOption long_options[] = {
        /* longName, shortName, argInfo, argPtr, value, descrip, argDesc */
@@ -399,6 +404,40 @@ const char *print_raw_channel_name(const char *name)
        return name ? : "<default>";
 }
 
+/*
+ * Mi print exlcusion list
+ */
+static
+int mi_print_exclusion(int count, char **names)
+{
+       int i, ret;
+
+       assert(writer);
+
+       if (count == 0) {
+               ret = 0;
+               goto end;
+       }
+       ret = mi_lttng_writer_open_element(writer, config_element_exclusions);
+       if (ret) {
+               goto end;
+       }
+
+       for (i = 0; i < count; i++) {
+               ret = mi_lttng_writer_write_element_string(writer,
+                               config_element_exclusion, names[i]);
+               if (ret) {
+                       goto end;
+               }
+       }
+
+       /* Close exclusions element */
+       ret = mi_lttng_writer_close_element(writer);
+
+end:
+       return ret;
+}
+
 /*
  * Return allocated string for pretty-printing exclusion names.
  */
@@ -429,6 +468,7 @@ char *print_exclusions(int count, char **names)
                        strcat(ret, ",");
                }
        }
+
        return ret;
 }
 
@@ -522,10 +562,12 @@ end:
 }
 /*
  * Enabling event using the lttng API.
+ * Note: in case of error only the last error code will be return.
  */
 static int enable_events(char *session_name)
 {
-       int ret = CMD_SUCCESS, warn = 0;
+       int ret = CMD_SUCCESS, command_ret = CMD_SUCCESS;
+       int error_holder = CMD_SUCCESS, warn = 0, error = 0, success = 1;
        char *event_name, *channel_name = NULL;
        struct lttng_event ev;
        struct lttng_domain dom;
@@ -578,6 +620,16 @@ static int enable_events(char *session_name)
                goto error;
        }
 
+       /* Prepare Mi */
+       if (lttng_opt_mi) {
+               /* Open a events element */
+               ret = mi_lttng_writer_open_element(writer, config_element_events);
+               if (ret) {
+                       ret = CMD_ERROR;
+                       goto error;
+               }
+       }
+
        if (opt_enable_all) {
                /* Default setup for enable all */
                if (opt_kernel) {
@@ -617,6 +669,7 @@ static int enable_events(char *session_name)
                        if (ret == CMD_ERROR) {
                                goto error;
                        }
+                       ev.exclusion = 1;
                }
                if (!opt_filter) {
                        ret = lttng_enable_event_with_exclusions(handle,
@@ -628,6 +681,7 @@ static int enable_events(char *session_name)
                                case LTTNG_ERR_KERN_EVENT_EXIST:
                                        WARN("Kernel events already enabled (channel %s, session %s)",
                                                        print_channel_name(channel_name), session_name);
+                                       warn = 1;
                                        break;
                                default:
                                        ERR("Events: %s (channel %s, session %s)",
@@ -636,6 +690,7 @@ static int enable_events(char *session_name)
                                                                ? print_raw_channel_name(channel_name)
                                                                : print_channel_name(channel_name),
                                                        session_name);
+                                       error = 1;
                                        break;
                                }
                                goto end;
@@ -692,15 +747,17 @@ static int enable_events(char *session_name)
                                goto error;
                        }
                }
+
                if (opt_filter) {
-                       ret = lttng_enable_event_with_exclusions(handle, &ev, channel_name,
+                       command_ret = lttng_enable_event_with_exclusions(handle, &ev, channel_name,
                                                opt_filter, exclusion_count, exclusion_list);
-                       if (ret < 0) {
-                               switch (-ret) {
+                       if (command_ret < 0) {
+                               switch (-command_ret) {
                                case LTTNG_ERR_FILTER_EXIST:
                                        WARN("Filter on all events is already enabled"
                                                        " (channel %s, session %s)",
                                                print_channel_name(channel_name), session_name);
+                                       warn = 1;
                                        break;
                                default:
                                        ERR("All events: %s (channel %s, session %s, filter \'%s\')",
@@ -709,13 +766,62 @@ static int enable_events(char *session_name)
                                                                ? print_raw_channel_name(channel_name)
                                                                : print_channel_name(channel_name),
                                                        session_name, opt_filter);
+                                       error = 1;
                                        break;
                                }
-                               goto error;
+                               error_holder = command_ret;
                        } else {
+                               ev.filter = 1;
                                MSG("Filter '%s' successfully set", opt_filter);
                        }
                }
+
+               if (lttng_opt_mi) {
+                       /* The wildcard * is used for kernel and ust domain to
+                        * represent ALL. We copy * in event name to force the wildcard use
+                        * for kernel domain
+                        *
+                        * Note: this is strictly for semantic and printing while in
+                        * machine interface mode.
+                        */
+                       strcpy(ev.name, "*");
+
+                       /* If we reach here the events are enabled */
+                       if (!error && !warn) {
+                               ev.enabled = 1;
+                       } else {
+                               ev.enabled = 0;
+                               success = 0;
+                       }
+                       ret = mi_lttng_event(writer, &ev, 1);
+                       if (ret) {
+                               ret = CMD_ERROR;
+                               goto error;
+                       }
+
+                       /* print exclusion */
+                       ret = mi_print_exclusion(exclusion_count, exclusion_list);
+                       if (ret) {
+                               ret = CMD_ERROR;
+                               goto error;
+                       }
+
+                       /* Success ? */
+                       ret = mi_lttng_writer_write_element_bool(writer,
+                                       mi_lttng_element_command_success, success);
+                       if (ret) {
+                               ret = CMD_ERROR;
+                               goto error;
+                       }
+
+                       /* Close event element */
+                       ret = mi_lttng_writer_close_element(writer);
+                       if (ret) {
+                               ret = CMD_ERROR;
+                               goto error;
+                       }
+               }
+
                goto end;
        }
 
@@ -802,6 +908,7 @@ static int enable_events(char *session_name)
                        }
 
                        if (opt_exclude) {
+                               ev.exclusion = 1;
                                if (opt_event_type != LTTNG_EVENT_ALL && opt_event_type != LTTNG_EVENT_TRACEPOINT) {
                                        ERR("Exclusion option can only be used with tracepoint events");
                                        ret = CMD_ERROR;
@@ -865,18 +972,19 @@ static int enable_events(char *session_name)
                if (!opt_filter) {
                        char *exclusion_string;
 
-                       ret = lttng_enable_event_with_exclusions(handle,
+                       command_ret = lttng_enable_event_with_exclusions(handle,
                                        &ev, channel_name,
                                        NULL, exclusion_count, exclusion_list);
                        exclusion_string = print_exclusions(exclusion_count, exclusion_list);
-                       if (ret < 0) {
+                       if (command_ret < 0) {
                                /* Turn ret to positive value to handle the positive error code */
-                               switch (-ret) {
+                               switch (-command_ret) {
                                case LTTNG_ERR_KERN_EVENT_EXIST:
                                        WARN("Kernel event %s%s already enabled (channel %s, session %s)",
                                                        event_name,
                                                        exclusion_string,
                                                        print_channel_name(channel_name), session_name);
+                                       warn = 1;
                                        break;
                                default:
                                        ERR("Event %s%s: %s (channel %s, session %s)", event_name,
@@ -886,9 +994,10 @@ static int enable_events(char *session_name)
                                                                ? print_raw_channel_name(channel_name)
                                                                : print_channel_name(channel_name),
                                                        session_name);
+                                       error = 1;
                                        break;
                                }
-                               warn = 1;
+                               error_holder = command_ret;
                        } else {
                                /* So we don't print the default channel name for JUL. */
                                if (dom.type == LTTNG_DOMAIN_JUL) {
@@ -908,18 +1017,22 @@ static int enable_events(char *session_name)
                if (opt_filter) {
                        char *exclusion_string;
 
-                       ret = lttng_enable_event_with_exclusions(handle, &ev, channel_name,
+                       /* Filter present */
+                       ev.filter = 1;
+
+                       command_ret = lttng_enable_event_with_exclusions(handle, &ev, channel_name,
                                        opt_filter, exclusion_count, exclusion_list);
                        exclusion_string = print_exclusions(exclusion_count, exclusion_list);
 
-                       if (ret < 0) {
-                               switch (-ret) {
+                       if (command_ret < 0) {
+                               switch (-command_ret) {
                                case LTTNG_ERR_FILTER_EXIST:
                                        WARN("Filter on event %s%s is already enabled"
                                                        " (channel %s, session %s)",
                                                event_name,
                                                exclusion_string,
                                                print_channel_name(channel_name), session_name);
+                                       warn = 1;
                                        break;
                                default:
                                        ERR("Event %s%s: %s (channel %s, session %s, filter \'%s\')", ev.name,
@@ -929,10 +1042,11 @@ static int enable_events(char *session_name)
                                                                ? print_raw_channel_name(channel_name)
                                                                : print_channel_name(channel_name),
                                                        session_name, opt_filter);
+                                       error = 1;
                                        break;
                                }
-                               free(exclusion_string);
-                               goto error;
+                               error_holder = command_ret;
+
                        } else {
                                MSG("Event %s%s: Filter '%s' successfully set",
                                                event_name, exclusion_string,
@@ -941,15 +1055,66 @@ static int enable_events(char *session_name)
                        free(exclusion_string);
                }
 
+               if (lttng_opt_mi) {
+                       if (command_ret) {
+                               success = 0;
+                               ev.enabled = 0;
+                       } else {
+                               ev.enabled = 1;
+                       }
+
+                       ret = mi_lttng_event(writer, &ev, 1);
+                       if (ret) {
+                               ret = CMD_ERROR;
+                               goto error;
+                       }
+
+                       /* print exclusion */
+                       ret = mi_print_exclusion(exclusion_count, exclusion_list);
+                       if (ret) {
+                               ret = CMD_ERROR;
+                               goto error;
+                       }
+
+                       /* Success ? */
+                       ret = mi_lttng_writer_write_element_bool(writer,
+                                       mi_lttng_element_command_success, success);
+                       if (ret) {
+                               ret = CMD_ERROR;
+                               goto end;
+                       }
+
+                       /* Close event element */
+                       ret = mi_lttng_writer_close_element(writer);
+                       if (ret) {
+                               ret = CMD_ERROR;
+                               goto end;
+                       }
+               }
+
                /* Next event */
                event_name = strtok(NULL, ",");
+               /* Reset warn, error and success */
+               success = 1;
        }
 
 end:
+       /* Close Mi */
+       if (lttng_opt_mi) {
+               /* Close events element */
+               ret = mi_lttng_writer_close_element(writer);
+               if (ret) {
+                       ret = CMD_ERROR;
+                       goto error;
+               }
+       }
 error:
        if (warn) {
                ret = CMD_WARNING;
        }
+       if (error) {
+               ret = CMD_ERROR;
+       }
        lttng_destroy_handle(handle);
 
        if (exclusion_list != NULL) {
@@ -959,6 +1124,11 @@ error:
                free(exclusion_list);
        }
 
+       /* Overwrite ret with error_holder if there was an actual error with
+        * enabling an event.
+        */
+       ret = error_holder ? error_holder : ret;
+
        return ret;
 }
 
@@ -967,7 +1137,7 @@ error:
  */
 int cmd_enable_events(int argc, const char **argv)
 {
-       int opt, ret = CMD_SUCCESS;
+       int opt, ret = CMD_SUCCESS, command_ret = CMD_SUCCESS, success = 1;
        static poptContext pc;
        char *session_name = NULL;
        int event_type = -1;
@@ -975,13 +1145,6 @@ int cmd_enable_events(int argc, const char **argv)
        pc = poptGetContext(NULL, argc, argv, long_options, 0);
        poptReadDefaultConfig(pc, 0);
 
-       /* TODO: mi support */
-       if (lttng_opt_mi) {
-               ret = -LTTNG_ERR_MI_NOT_IMPLEMENTED;
-               ERR("mi option not supported");
-               goto end;
-       }
-
        /* Default event type */
        opt_event_type = LTTNG_EVENT_ALL;
 
@@ -1041,6 +1204,31 @@ int cmd_enable_events(int argc, const char **argv)
                }
        }
 
+       /* Mi check */
+       if (lttng_opt_mi) {
+               writer = mi_lttng_writer_create(fileno(stdout), lttng_opt_mi);
+               if (!writer) {
+                       ret = -LTTNG_ERR_NOMEM;
+                       goto end;
+               }
+
+               /* Open command element */
+               ret = mi_lttng_writer_command_open(writer,
+                               mi_lttng_element_command_enable_event);
+               if (ret) {
+                       ret = CMD_ERROR;
+                       goto end;
+               }
+
+               /* Open output element */
+               ret = mi_lttng_writer_open_element(writer,
+                               mi_lttng_element_command_output);
+               if (ret) {
+                       ret = CMD_ERROR;
+                       goto end;
+               }
+       }
+
        opt_event_list = (char*) poptGetArg(pc);
        if (opt_event_list == NULL && opt_enable_all == 0) {
                ERR("Missing event name(s).\n");
@@ -1052,20 +1240,59 @@ int cmd_enable_events(int argc, const char **argv)
        if (!opt_session_name) {
                session_name = get_session_name();
                if (session_name == NULL) {
-                       ret = CMD_ERROR;
-                       goto end;
+                       command_ret = CMD_ERROR;
+                       success = 0;
+                       goto mi_closing;
                }
        } else {
                session_name = opt_session_name;
        }
 
-       ret = enable_events(session_name);
+       command_ret = enable_events(session_name);
+       if (command_ret) {
+               success = 0;
+               goto mi_closing;
+       }
+
+mi_closing:
+       /* Mi closing */
+       if (lttng_opt_mi) {
+               /* Close  output element */
+               ret = mi_lttng_writer_close_element(writer);
+               if (ret) {
+                       ret = CMD_ERROR;
+                       goto end;
+               }
+
+               ret = mi_lttng_writer_write_element_bool(writer,
+                               mi_lttng_element_command_success, success);
+               if (ret) {
+                       ret = CMD_ERROR;
+                       goto end;
+               }
+
+               /* Command element close */
+               ret = mi_lttng_writer_command_close(writer);
+               if (ret) {
+                       ret = CMD_ERROR;
+                       goto end;
+               }
+       }
 
 end:
+       /* Mi clean-up */
+       if (writer && mi_lttng_writer_destroy(writer)) {
+               /* Preserve original error code */
+               ret = ret ? ret : LTTNG_ERR_MI_IO_FAIL;
+       }
+
        if (opt_session_name == NULL) {
                free(session_name);
        }
 
+       /* Overwrite ret if an error occurred in enable_events */
+       ret = command_ret ? command_ret : ret;
+
        poptFreeContext(pc);
        return ret;
 }
index d13c2a6f7eeb70255d1d584c6be84aaebe294af1..0149204f53074e232807e18af66bce0577203f1c 100644 (file)
@@ -27,6 +27,7 @@
 const char * const mi_lttng_element_command = "command";
 const char * const mi_lttng_element_command_name = "name";
 const char * const mi_lttng_element_command_version = "version";
+const char * const mi_lttng_element_command_enable_event = "enable-event";
 const char * const mi_lttng_element_command_list = "list";
 const char * const mi_lttng_element_command_save = "save";
 const char * const mi_lttng_element_command_load = "load";
@@ -908,9 +909,7 @@ int mi_lttng_event(struct mi_writer *writer,
        }
 
        switch (event->type) {
-       case LTTNG_EVENT_ALL:
                /* We should never have "all" events in list. */
-               assert(0);
                break;
        case LTTNG_EVENT_TRACEPOINT:
        {
@@ -927,6 +926,8 @@ int mi_lttng_event(struct mi_writer *writer,
        case LTTNG_EVENT_FUNCTION_ENTRY:
                ret = mi_lttng_event_function_entry(writer, event);
                break;
+       case LTTNG_EVENT_ALL:
+               /* Fallthrough */
        default:
                break;
        }
index 4b2b035e04a66d1d55c9036103975e3ce90a3612..d621f36a56839aab527643d06398e56a0fe73b8d 100644 (file)
@@ -49,6 +49,7 @@ struct mi_lttng_version {
 const char * const mi_lttng_element_command;
 const char * const mi_lttng_element_command_name;
 const char * const mi_lttng_element_command_version;
+const char * const mi_lttng_element_command_enable_event;
 const char * const mi_lttng_element_command_list;
 const char * const mi_lttng_element_command_save;
 const char * const mi_lttng_element_command_load;
index 08184aca687057f4e1ae174ed9c3b2b3584b6ab9..02aab41067320e27ea4d579424f23fcc1a0eff1c 100644 (file)
@@ -158,6 +158,12 @@ THE SOFTWARE.
                </xs:choice>
        </xs:complexType>
 
+       <xs:complexType name="event_exclusion_list_type">
+               <xs:sequence>
+                       <xs:element name="exclusion" type="xs:string" minOccurs="0" maxOccurs="unbounded"/>
+               </xs:sequence>
+       </xs:complexType>
+
        <!-- Maps to lttng_event struct -->
        <xs:complexType name="event_type">
                <xs:all>
@@ -168,8 +174,10 @@ THE SOFTWARE.
                        <xs:element name="enabled" type="xs:boolean" default="false" minOccurs="0" />
                        <xs:element name="filter" type="xs:boolean" minOccurs="0" />
                        <xs:element name="exclusion" type="xs:boolean" minOccurs="0" />
+                       <xs:element name="exclusions" type="event_exclusion_list_type" minOccurs="0"/>
                        <xs:element name="attributes" type="event_attributes_type" minOccurs="0" />
                        <xs:element name="event_fields" type="event_fields_type" minOccurs="0" />
+                       <xs:element name="success" type="xs:boolean" minOccurs="0" />
                </xs:all>
        </xs:complexType>
 
@@ -380,6 +388,7 @@ THE SOFTWARE.
                        <xs:element name="calibrate" type="calibrate_type" minOccurs="0" />
                        <xs:element name="contexts" type="contexts_type" minOccurs="0" />
                        <xs:element name="channels" type="channels_type" minOccurs="0" />
+                       <xs:element name="events" type="event_list_type" minOccurs="0" />
                </xs:choice>
        </xs:complexType>
 
@@ -397,6 +406,7 @@ THE SOFTWARE.
                        <xs:enumeration value="calibrate" />
                        <xs:enumeration value="add-context" />
                        <xs:enumeration value="enable-channel" />
+                       <xs:enumeration value="enable-event" />
                </xs:restriction>
        </xs:simpleType>
 
This page took 0.03476 seconds and 5 git commands to generate.