uevent = trace_ust_find_event(uchan->events, event->name, filter,
event->loglevel_type, event->loglevel, exclusion);
if (!uevent) {
- uevent = trace_ust_create_event(event, filter_expression,
- filter, exclusion, internal_event);
+ ret = trace_ust_create_event(event, filter_expression,
+ filter, exclusion, internal_event, &uevent);
/* We have passed ownership */
filter_expression = NULL;
filter = NULL;
exclusion = NULL;
- if (uevent == NULL) {
- ret = LTTNG_ERR_UST_ENABLE_FAIL;
+ if (ret != LTTNG_OK) {
goto error;
}
}
uevent->enabled = 1;
+ if (to_create) {
+ /* Add ltt ust event to channel */
+ add_unique_ust_event(uchan->events, uevent);
+ }
+
+ if (!usess->active) {
+ goto end;
+ }
if (to_create) {
/* Create event on all UST registered apps for session */
}
}
- if (to_create) {
- /* Add ltt ust event to channel */
- add_unique_ust_event(uchan->events, uevent);
- }
-
DBG("Event UST %s %s in channel %s", uevent->attr.name,
to_create ? "created" : "enabled", uchan->name);
/* It's already disabled so everything is OK */
goto next;
}
+ uevent->enabled = 0;
+ DBG2("Event UST %s disabled in channel %s", uevent->attr.name,
+ uchan->name);
+ if (!usess->active) {
+ goto next;
+ }
ret = ust_app_disable_event_glb(usess, uchan, uevent);
if (ret < 0 && ret != -LTTNG_UST_ERR_EXIST) {
ret = LTTNG_ERR_UST_DISABLE_FAIL;
goto error;
}
- uevent->enabled = 0;
-
- DBG2("Event UST %s disabled in channel %s", uevent->attr.name,
- uchan->name);
-
next:
/* Get next duplicate event by name. */
cds_lfht_next_duplicate(ht->ht, trace_ust_ht_match_event_by_name,
ret = LTTNG_ERR_NOMEM;
goto error;
}
-
+ filter = NULL;
+ filter_expression = NULL;
created = 1;
}
- /* Already enabled? */
- if (aevent->enabled) {
- goto end;
- }
-
if (created && filter) {
ret = add_filter_app_ctx(filter, filter_expression, agt);
if (ret != LTTNG_OK) {
}
}
+ /* Already enabled? */
+ if (aevent->enabled) {
+ ret = LTTNG_OK;
+ goto end;
+ }
+
ret = agent_enable_event(aevent, agt->domain);
if (ret != LTTNG_OK) {
goto error;
agent_add_event(aevent, agt);
}
-end:
- return LTTNG_OK;
+ ret = LTTNG_OK;
+ goto end;
error:
if (created) {
agent_destroy_event(aevent);
}
+end:
+ free(filter);
+ free(filter_expression);
return ret;
}
/* If the agent event exists, it must be available on the UST side. */
assert(uevent);
- ret = ust_app_disable_event_glb(usess, uchan, uevent);
- if (ret < 0 && ret != -LTTNG_UST_ERR_EXIST) {
- ret = LTTNG_ERR_UST_DISABLE_FAIL;
- goto error;
+ if (usess->active) {
+ ret = ust_app_disable_event_glb(usess, uchan, uevent);
+ if (ret < 0 && ret != -LTTNG_UST_ERR_EXIST) {
+ ret = LTTNG_ERR_UST_DISABLE_FAIL;
+ goto error;
+ }
}
/*