Remove lttng_event_rule_tracepoint
[lttng-tools.git] / tests / regression / tools / notification / notification.c
index c097c102698d88871a3c089a8d5558324a9be2b3..556a0e0b1b8807618f2bd7bb02ac5a330d7f0772 100644 (file)
@@ -1220,7 +1220,8 @@ static void test_subscription_twice(const char *session_name,
                        domain_type, BUFFER_USAGE_TYPE_LOW, 0.99, &condition,
                        &action, &trigger);
        if (ret) {
-               fail("Setup error on trigger registration");
+               fail("Setup error on trigger registration in %s()",
+                               __FUNCTION__);
                goto end;
        }
 
@@ -1245,7 +1246,11 @@ static void test_subscription_twice(const char *session_name,
                        "Subscribe to a condition for which subscription was already done");
 
 end:
-       lttng_unregister_trigger(trigger);
+       ret = lttng_unregister_trigger(trigger);
+       if (ret) {
+               fail("Failed to unregister trigger in %s()", __FUNCTION__);
+       }
+
        lttng_trigger_destroy(trigger);
        lttng_notification_channel_destroy(notification_channel);
        lttng_action_destroy(action);
@@ -1448,32 +1453,64 @@ static void create_tracepoint_event_rule_trigger(const char *event_pattern,
                struct lttng_condition **condition,
                struct lttng_trigger **trigger)
 {
-       enum lttng_event_rule_status event_rule_status;
-       enum lttng_trigger_status trigger_status;
+       typedef struct lttng_event_rule *(*event_rule_create)(void);
+       typedef enum lttng_event_rule_status (
+                       *event_rule_set_name_pattern)(
+                       struct lttng_event_rule *rule,
+                       const char *pattern);
+       typedef enum lttng_event_rule_status (*event_rule_set_filter)(
+                       struct lttng_event_rule *rule,
+                       const char *expression);
+       typedef enum lttng_event_rule_status (
+                       *event_rule_add_name_pattern_exclusion)(
+                       struct lttng_event_rule * rule, const char *exclusion);
 
+       enum lttng_event_rule_status event_rule_status;
        struct lttng_action *tmp_action = NULL;
        struct lttng_event_rule *event_rule = NULL;
        struct lttng_condition *tmp_condition = NULL;
        struct lttng_trigger *tmp_trigger = NULL;
        int ret;
+       enum lttng_error_code ret_code;
+       event_rule_create create;
+       event_rule_set_name_pattern set_name_pattern;
+       event_rule_set_filter set_filter;
+       event_rule_add_name_pattern_exclusion add_name_pattern_exclusion;
 
        assert(event_pattern);
        assert(trigger_name);
        assert(condition);
        assert(trigger);
 
-       event_rule = lttng_event_rule_tracepoint_create(domain_type);
+       /* Set the function pointers based on the domain type. */
+       switch (domain_type) {
+       case LTTNG_DOMAIN_UST:
+               create = lttng_event_rule_user_tracepoint_create;
+               set_name_pattern = lttng_event_rule_user_tracepoint_set_name_pattern;
+               set_filter = lttng_event_rule_user_tracepoint_set_filter;
+               add_name_pattern_exclusion = lttng_event_rule_user_tracepoint_add_name_pattern_exclusion;
+               break;
+       case LTTNG_DOMAIN_KERNEL:
+               create = lttng_event_rule_kernel_tracepoint_create;
+               set_name_pattern = lttng_event_rule_kernel_tracepoint_set_name_pattern;
+               set_filter = lttng_event_rule_kernel_tracepoint_set_filter;
+               add_name_pattern_exclusion = NULL;
+               break;
+       default:
+               abort();
+               break;
+       }
+
+       event_rule = create();
        ok(event_rule, "Tracepoint event rule object creation");
 
-       event_rule_status = lttng_event_rule_tracepoint_set_pattern(
-                       event_rule, event_pattern);
+       event_rule_status = set_name_pattern(event_rule, event_pattern);
        ok(event_rule_status == LTTNG_EVENT_RULE_STATUS_OK,
                        "Setting tracepoint event rule pattern: '%s'",
                        event_pattern);
 
        if (filter) {
-               event_rule_status = lttng_event_rule_tracepoint_set_filter(
-                               event_rule, filter);
+               event_rule_status = set_filter(event_rule, filter);
                ok(event_rule_status == LTTNG_EVENT_RULE_STATUS_OK,
                                "Setting tracepoint event rule filter: '%s'",
                                filter);
@@ -1484,13 +1521,12 @@ static void create_tracepoint_event_rule_trigger(const char *event_pattern,
                bool success = true;
 
                assert(domain_type == LTTNG_DOMAIN_UST);
+               assert(add_name_pattern_exclusion != NULL);
                assert(exclusion_count > 0);
 
                for (i = 0; i < exclusion_count; i++) {
-                       event_rule_status =
-                                       lttng_event_rule_tracepoint_add_exclusion(
-                                                       event_rule,
-                                                       exclusions[i]);
+                       event_rule_status = add_name_pattern_exclusion(
+                                       event_rule, exclusions[i]);
                        if (event_rule_status != LTTNG_EVENT_RULE_STATUS_OK) {
                                fail("Setting tracepoint event rule exclusion '%s'.",
                                                exclusions[i]);
@@ -1518,12 +1554,8 @@ static void create_tracepoint_event_rule_trigger(const char *event_pattern,
        tmp_trigger = lttng_trigger_create(tmp_condition, tmp_action);
        ok(tmp_trigger, "Trigger object creation %s", trigger_name);
 
-       trigger_status = lttng_trigger_set_name(tmp_trigger, trigger_name);
-       ok(trigger_status == LTTNG_TRIGGER_STATUS_OK,
-                       "Setting name to trigger %s", trigger_name);
-
-       ret = lttng_register_trigger(tmp_trigger);
-       ok(ret == 0, "Trigger registration %s", trigger_name);
+       ret_code = lttng_register_trigger_with_name(tmp_trigger, trigger_name);
+       ok(ret_code == LTTNG_OK, "Trigger registration %s", trigger_name);
 
        lttng_event_rule_destroy(event_rule);
 
@@ -1832,10 +1864,10 @@ static void test_kprobe_event_rule_notification(
                enum lttng_domain_type domain_type)
 {
        int i, ret;
+       enum lttng_error_code ret_code;
        const int notification_count = 3;
        enum lttng_notification_channel_status nc_status;
        enum lttng_event_rule_status event_rule_status;
-       enum lttng_trigger_status trigger_status;
        struct lttng_notification_channel *notification_channel = NULL;
        struct lttng_condition *condition = NULL;
        struct lttng_kernel_probe_location *location = NULL;
@@ -1861,10 +1893,10 @@ static void test_kprobe_event_rule_notification(
                        lttng_session_daemon_notification_endpoint);
        ok(notification_channel, "Notification channel object creation");
 
-       event_rule = lttng_event_rule_kernel_probe_create(location);
+       event_rule = lttng_event_rule_kernel_kprobe_create(location);
        ok(event_rule, "kprobe event rule object creation");
 
-       event_rule_status = lttng_event_rule_kernel_probe_set_event_name(
+       event_rule_status = lttng_event_rule_kernel_kprobe_set_event_name(
                        event_rule, trigger_name);
        ok(event_rule_status == LTTNG_EVENT_RULE_STATUS_OK,
                        "Setting kprobe event rule name: '%s'", trigger_name);
@@ -1879,12 +1911,8 @@ static void test_kprobe_event_rule_notification(
                goto end;
        }
 
-       trigger_status = lttng_trigger_set_name(trigger, trigger_name);
-       ok(trigger_status == LTTNG_TRIGGER_STATUS_OK,
-                       "Setting trigger name to '%s'", trigger_name);
-
-       ret = lttng_register_trigger(trigger);
-       if (ret) {
+       ret_code = lttng_register_trigger_with_name(trigger, trigger_name);
+       if (ret_code != LTTNG_OK) {
                fail("Failed to register trigger with kernel probe event rule condition and notify action");
                goto end;
        }
@@ -1933,10 +1961,10 @@ static void test_uprobe_event_rule_notification(
                const char *test_symbol_name)
 {
        int i, ret;
+       enum lttng_error_code ret_code;
        const int notification_count = 3;
        enum lttng_notification_channel_status nc_status;
        enum lttng_event_rule_status event_rule_status;
-       enum lttng_trigger_status trigger_status;
        struct lttng_notification_channel *notification_channel = NULL;
        struct lttng_userspace_probe_location *probe_location = NULL;
        struct lttng_userspace_probe_location_lookup_method *lookup_method =
@@ -1970,10 +1998,10 @@ static void test_uprobe_event_rule_notification(
                        lttng_session_daemon_notification_endpoint);
        ok(notification_channel, "Notification channel object creation");
 
-       event_rule = lttng_event_rule_userspace_probe_create(probe_location);
+       event_rule = lttng_event_rule_kernel_uprobe_create(probe_location);
        ok(event_rule, "kprobe event rule object creation");
 
-       event_rule_status = lttng_event_rule_userspace_probe_set_event_name(
+       event_rule_status = lttng_event_rule_kernel_uprobe_set_event_name(
                        event_rule, trigger_name);
        ok(event_rule_status == LTTNG_EVENT_RULE_STATUS_OK,
                        "Setting uprobe event rule name: '%s'", trigger_name);
@@ -1988,12 +2016,8 @@ static void test_uprobe_event_rule_notification(
                goto end;
        }
 
-       trigger_status = lttng_trigger_set_name(trigger, trigger_name);
-       ok(trigger_status == LTTNG_TRIGGER_STATUS_OK,
-                       "Setting name to trigger '%s'", trigger_name);
-
-       ret = lttng_register_trigger(trigger);
-       if (ret) {
+       ret_code = lttng_register_trigger_with_name(trigger, trigger_name);
+       if (ret_code != LTTNG_OK) {
                fail("Failed to register trigger with userspace probe event rule condition and notify action");
                goto end;
        }
@@ -2040,10 +2064,10 @@ static void test_syscall_event_rule_notification(
                enum lttng_domain_type domain_type)
 {
        int i, ret;
+       enum lttng_error_code ret_code;
        const int notification_count = 3;
        enum lttng_notification_channel_status nc_status;
        enum lttng_event_rule_status event_rule_status;
-       enum lttng_trigger_status trigger_status;
        struct lttng_notification_channel *notification_channel = NULL;
        struct lttng_condition *condition = NULL;
        struct lttng_event_rule *event_rule = NULL;
@@ -2062,10 +2086,10 @@ static void test_syscall_event_rule_notification(
                        lttng_session_daemon_notification_endpoint);
        ok(notification_channel, "Notification channel object creation");
 
-       event_rule = lttng_event_rule_syscall_create(LTTNG_EVENT_RULE_SYSCALL_EMISSION_SITE_ENTRY);
+       event_rule = lttng_event_rule_kernel_syscall_create(LTTNG_EVENT_RULE_KERNEL_SYSCALL_EMISSION_SITE_ENTRY);
        ok(event_rule, "syscall event rule object creation");
 
-       event_rule_status = lttng_event_rule_syscall_set_pattern(
+       event_rule_status = lttng_event_rule_kernel_syscall_set_name_pattern(
                        event_rule, syscall_name);
        ok(event_rule_status == LTTNG_EVENT_RULE_STATUS_OK,
                        "Setting syscall event rule pattern: '%s'", syscall_name);
@@ -2080,12 +2104,8 @@ static void test_syscall_event_rule_notification(
                goto end;
        }
 
-       trigger_status = lttng_trigger_set_name(trigger, trigger_name);
-       ok(trigger_status == LTTNG_TRIGGER_STATUS_OK,
-                       "Setting name to trigger '%s'", trigger_name);
-
-       ret = lttng_register_trigger(trigger);
-       if (ret) {
+       ret_code = lttng_register_trigger_with_name(trigger, trigger_name);
+       if (ret_code != LTTNG_OK) {
                fail("Failed to register trigger with syscall event rule condition and notify action");
                goto end;
        }
@@ -2129,10 +2149,10 @@ static void test_syscall_event_rule_notification_filter(
                enum lttng_domain_type domain_type)
 {
        int i, ret;
+       enum lttng_error_code ret_code;
        const int notification_count = 3;
        enum lttng_notification_channel_status nc_status;
        enum lttng_event_rule_status event_rule_status;
-       enum lttng_trigger_status trigger_status;
        struct lttng_notification_channel *notification_channel = NULL;
        struct lttng_condition *condition = NULL;
        struct lttng_event_rule *event_rule = NULL;
@@ -2152,15 +2172,15 @@ static void test_syscall_event_rule_notification_filter(
                        lttng_session_daemon_notification_endpoint);
        ok(notification_channel, "Notification channel object creation");
 
-       event_rule = lttng_event_rule_syscall_create(LTTNG_EVENT_RULE_SYSCALL_EMISSION_SITE_ENTRY);
+       event_rule = lttng_event_rule_kernel_syscall_create(LTTNG_EVENT_RULE_KERNEL_SYSCALL_EMISSION_SITE_ENTRY);
        ok(event_rule, "syscall event rule object creation");
 
-       event_rule_status = lttng_event_rule_syscall_set_pattern(
+       event_rule_status = lttng_event_rule_kernel_syscall_set_name_pattern(
                        event_rule, syscall_name);
        ok(event_rule_status == LTTNG_EVENT_RULE_STATUS_OK,
                        "Setting syscall event rule pattern: '%s'", syscall_name);
 
-       event_rule_status = lttng_event_rule_syscall_set_filter(
+       event_rule_status = lttng_event_rule_kernel_syscall_set_filter(
                        event_rule, filter_pattern);
        ok(event_rule_status == LTTNG_EVENT_RULE_STATUS_OK,
                        "Setting filter: '%s'", filter_pattern);
@@ -2175,12 +2195,8 @@ static void test_syscall_event_rule_notification_filter(
                goto end;
        }
 
-       trigger_status = lttng_trigger_set_name(trigger, trigger_name);
-       ok(trigger_status == LTTNG_TRIGGER_STATUS_OK,
-                       "Setting name to trigger '%s'", trigger_name);
-
-       ret = lttng_register_trigger(trigger);
-       if (ret) {
+       ret_code = lttng_register_trigger_with_name(trigger, trigger_name);
+       if (ret_code != LTTNG_OK) {
                fail("Failed to register trigger with syscall filtering event rule condition and notify action");
                goto end;
        }
@@ -2509,7 +2525,7 @@ int main(int argc, const char *argv[])
        switch (test_scenario) {
        case 1:
        {
-               plan_tests(44);
+               plan_tests(41);
 
                /* Test cases that need gen-ust-event testapp. */
                diag("Test basic notification error paths for %s domain",
@@ -2575,7 +2591,7 @@ int main(int argc, const char *argv[])
                 * Test cases that need a test app with more than one event
                 * type.
                 */
-               plan_tests(25);
+               plan_tests(23);
 
                /*
                 * At the moment, the only test case of this scenario is
@@ -2590,7 +2606,7 @@ int main(int argc, const char *argv[])
        }
        case 4:
        {
-               plan_tests(12);
+               plan_tests(11);
                /* Test cases that need the kernel tracer. */
                assert(domain_type == LTTNG_DOMAIN_KERNEL);
 
@@ -2603,7 +2619,7 @@ int main(int argc, const char *argv[])
        }
        case 5:
        {
-               plan_tests(25);
+               plan_tests(23);
                /* Test cases that need the kernel tracer. */
                assert(domain_type == LTTNG_DOMAIN_KERNEL);
 
@@ -2623,7 +2639,7 @@ int main(int argc, const char *argv[])
        {
                const char *testapp_path, *test_symbol_name;
 
-               plan_tests(12);
+               plan_tests(11);
 
                if (argc < 7) {
                        fail("Missing parameter for tests to run %d", argc);
@@ -2647,10 +2663,10 @@ int main(int argc, const char *argv[])
        {
                switch(domain_type) {
                case LTTNG_DOMAIN_UST:
-                       plan_tests(222);
+                       plan_tests(221);
                        break;
                case LTTNG_DOMAIN_KERNEL:
-                       plan_tests(216);
+                       plan_tests(215);
                        break;
                default:
                        assert(0);
This page took 0.028251 seconds and 4 git commands to generate.