X-Git-Url: https://git.lttng.org/?p=lttng-tools.git;a=blobdiff_plain;f=src%2Fbin%2Flttng%2Fcommands%2Fenable_events.c;h=cdd137cc9193d3aed0efdfee037546b97c958402;hp=51388298954768d35990b38e540617fc008e4020;hb=178191b3899f114001f000c2e7f46909969f9c6f;hpb=2f70b271351fe2b7befc4e327503f7c13a57dcd5 diff --git a/src/bin/lttng/commands/enable_events.c b/src/bin/lttng/commands/enable_events.c index 513882989..cdd137cc9 100644 --- a/src/bin/lttng/commands/enable_events.c +++ b/src/bin/lttng/commands/enable_events.c @@ -301,6 +301,7 @@ int loglevel_str_to_value(const char *inputstr) static int enable_events(char *session_name) { int err, ret = CMD_SUCCESS, warn = 0; + unsigned int event_enabled = 0; char *event_name, *channel_name = NULL; struct lttng_event ev; struct lttng_domain dom; @@ -366,6 +367,9 @@ static int enable_events(char *session_name) } } + /* Reset flag before enabling a new event. */ + event_enabled = 0; + ret = lttng_enable_event(handle, &ev, channel_name); if (ret < 0) { switch (-ret) { @@ -380,6 +384,7 @@ static int enable_events(char *session_name) } goto end; } + event_enabled = 1; switch (opt_event_type) { case LTTNG_EVENT_TRACEPOINT: @@ -416,10 +421,11 @@ static int enable_events(char *session_name) */ goto error; } - if (opt_filter) { - ret = lttng_set_event_filter(handle, ev.name, channel_name, + if (opt_filter && event_enabled) { + ret = lttng_set_event_filter(handle, &ev, channel_name, opt_filter); if (ret < 0) { + fprintf(stderr, "Ret filter: %d\n", ret); switch (-ret) { case LTTNG_ERR_FILTER_EXIST: WARN("Filter on events is already enabled" @@ -558,6 +564,9 @@ static int enable_events(char *session_name) goto error; } + /* Reset flag before enabling a new event. */ + event_enabled = 0; + ret = lttng_enable_event(handle, &ev, channel_name); if (ret < 0) { /* Turn ret to positive value to handle the positive error code */ @@ -575,9 +584,10 @@ static int enable_events(char *session_name) } else { MSG("%s event %s created in channel %s", opt_kernel ? "kernel": "UST", event_name, channel_name); + event_enabled = 1; } - if (opt_filter) { - ret = lttng_set_event_filter(handle, ev.name, channel_name, + if (opt_filter && event_enabled) { + ret = lttng_set_event_filter(handle, &ev, channel_name, opt_filter); if (ret < 0) { switch (-ret) {