X-Git-Url: http://git.efficios.com/?p=lttng-tools.git;a=blobdiff_plain;f=tests%2Funit%2Ftest_session.c;h=4e659d5242b6af9f1fdf5ee91ac5e711159d33e7;hp=196124bd3c33e5b50beebcb1b49215c89450d879;hb=c109c263a5bf6c52d3a1cf27e26a784fb28fac07;hpb=7972aab22f74b18faa168c0482216a3dd711a075 diff --git a/tests/unit/test_session.c b/tests/unit/test_session.c index 196124bd3..4e659d524 100644 --- a/tests/unit/test_session.c +++ b/tests/unit/test_session.c @@ -16,7 +16,6 @@ * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. */ -#define _GNU_SOURCE #include #include #include @@ -25,30 +24,33 @@ #include #include #include +#include #include #include #include +#include +#include +#include #include #include #define SESSION1 "test1" -/* This path will NEVER be created in this test */ -#define PATH1 "/tmp/.test-junk-lttng" - #define MAX_SESSIONS 10000 #define RANDOM_STRING_LEN 11 /* Number of TAP tests in this file */ -#define NUM_TESTS 12 +#define NUM_TESTS 11 +struct health_app *health_sessiond; static struct ltt_session_list *session_list; -/* For lttngerr.h */ +/* For error.h */ int lttng_opt_quiet = 1; int lttng_opt_verbose = 0; +int lttng_opt_mi; int ust_consumerd32_fd; int ust_consumerd64_fd; @@ -110,10 +112,11 @@ static void empty_session_list(void) { struct ltt_session *iter, *tmp; + session_lock_list(); cds_list_for_each_entry_safe(iter, tmp, &session_list->head, list) { - cds_list_del(&iter->list); - free(iter); + session_destroy(iter); } + session_unlock_list(); /* Session list must be 0 */ assert(!session_list_count()); @@ -122,30 +125,35 @@ static void empty_session_list(void) /* * Test creation of 1 session */ -static int create_one_session(char *name, char *path) +static int create_one_session(char *name) { int ret; + enum lttng_error_code ret_code; + struct ltt_session *session = NULL; - ret = session_create(name, path, geteuid(), getegid()); - if (ret == LTTNG_OK) { + session_lock_list(); + ret_code = session_create(name, geteuid(), getegid(), &session); + session_put(session); + if (ret_code == LTTNG_OK) { /* Validate */ ret = find_session_name(name); if (ret < 0) { /* Session not found by name */ printf("session not found after creation\n"); - return -1; + ret = -1; } else { /* Success */ - return 0; + ret = 0; } } else { - if (ret == LTTNG_ERR_EXIST_SESS) { + if (ret_code == LTTNG_ERR_EXIST_SESS) { printf("(session already exists) "); } - return -1; + ret = -1; } - return 0; + session_unlock_list(); + return ret; } /* @@ -154,25 +162,23 @@ static int create_one_session(char *name, char *path) static int destroy_one_session(struct ltt_session *session) { int ret; + char session_name[NAME_MAX]; - ret = session_destroy(session); + strncpy(session_name, session->name, sizeof(session_name)); + session_name[sizeof(session_name) - 1] = '\0'; - if (ret == LTTNG_OK) { - /* Validate */ - if (session == NULL) { - return 0; - } - ret = find_session_name(session->name); - if (ret < 0) { - /* Success, -1 means that the sesion is NOT found */ - return 0; - } else { - /* Fail */ - return -1; - } - } + session_destroy(session); + session_put(session); - return 0; + ret = find_session_name(session_name); + if (ret < 0) { + /* Success, -1 means that the sesion is NOT found */ + ret = 0; + } else { + /* Fail */ + ret = -1; + } + return ret; } /* @@ -184,94 +190,130 @@ static int two_session_same_name(void) int ret; struct ltt_session *sess; - ret = create_one_session(SESSION1, PATH1); + ret = create_one_session(SESSION1); if (ret < 0) { /* Fail */ - return -1; + ret = -1; + goto end; } + session_lock_list(); sess = session_find_by_name(SESSION1); if (sess) { /* Success */ - return 0; + session_put(sess); + session_unlock_list(); + ret = 0; + goto end_unlock; + } else { + /* Fail */ + ret = -1; + goto end_unlock; } - - /* Fail */ - return -1; +end_unlock: + session_unlock_list(); +end: + return ret; } -void test_session_list(void) +static void test_session_list(void) { session_list = session_get_list(); ok(session_list != NULL, "Session list: not NULL"); } -void test_create_one_session(void) +static void test_create_one_session(void) { - ok(create_one_session(SESSION1, PATH1) == 0, + ok(create_one_session(SESSION1) == 0, "Create session: %s", SESSION1); } -void test_validate_session(void) +static void test_validate_session(void) { struct ltt_session *tmp; + session_lock_list(); tmp = session_find_by_name(SESSION1); ok(tmp != NULL, "Validating session: session found"); - ok(tmp->kernel_session == NULL && - strlen(tmp->path) && - strlen(tmp->name), - "Validating session: basic sanity check"); + if (tmp) { + ok(tmp->kernel_session == NULL && + strlen(tmp->name), + "Validating session: basic sanity check"); + } else { + skip(1, "Skipping session validation check as session was not found"); + goto end; + } session_lock(tmp); session_unlock(tmp); + session_put(tmp); +end: + session_unlock_list(); } -void test_destroy_session(void) +static void test_destroy_session(void) { struct ltt_session *tmp; + session_lock_list(); tmp = session_find_by_name(SESSION1); ok(tmp != NULL, "Destroying session: session found"); - ok(destroy_one_session(tmp) == 0, - "Destroying session: %s destroyed", - SESSION1); + if (tmp) { + ok(destroy_one_session(tmp) == 0, + "Destroying session: %s destroyed", + SESSION1); + } else { + skip(1, "Skipping session destruction as it was not found"); + } + session_unlock_list(); } -void test_duplicate_session(void) +static void test_duplicate_session(void) { ok(two_session_same_name() == 0, "Duplicate session creation"); } -void test_bogus_session_param(void) +static void test_session_name_generation(void) { - ok(create_one_session(NULL, NULL) < 0, - "Create session with bogus param: NULL, NULL should fail"); - - ok(create_one_session(NULL, PATH1) < 0, - "Create session with bogus param: NULL, %s should fail", - PATH1); - - ok(session_list_count() == 0, - "Create session with bogus param: session list empty"); + struct ltt_session *session = NULL; + enum lttng_error_code ret_code; + const char *expected_session_name_prefix = DEFAULT_SESSION_NAME; + + session_lock_list(); + ret_code = session_create(NULL, geteuid(), getegid(), &session); + ok(ret_code == LTTNG_OK, + "Create session with a NULL name (auto-generate a name)"); + if (!session) { + skip(1, "Skipping session name generation tests as session_create() failed."); + goto end; + } + diag("Automatically-generated session name: %s", *session->name ? + session->name : "ERROR"); + ok(*session->name && !strncmp(expected_session_name_prefix, session->name, + sizeof(DEFAULT_SESSION_NAME) - 1), + "Auto-generated session name starts with %s", + DEFAULT_SESSION_NAME); +end: + session_put(session); + session_unlock_list(); } -void test_large_session_number(void) +static void test_large_session_number(void) { int ret, i, failed = 0; struct ltt_session *iter, *tmp; for (i = 0; i < MAX_SESSIONS; i++) { char *tmp_name = get_random_string(); - ret = create_one_session(tmp_name, PATH1); + ret = create_one_session(tmp_name); if (ret < 0) { diag("session %d (name: %s) creation failed", i, tmp_name); ++failed; @@ -284,15 +326,18 @@ void test_large_session_number(void) failed = 0; + session_lock_list(); for (i = 0; i < MAX_SESSIONS; i++) { cds_list_for_each_entry_safe(iter, tmp, &session_list->head, list) { + assert(session_get(iter)); ret = destroy_one_session(iter); if (ret < 0) { - diag("session %d (name: %s) destroy failed", i, iter->name); + diag("session %d destroy failed", i); ++failed; } } } + session_unlock_list(); ok(failed == 0 && session_list_count() == 0, "Large sessions number: destroyed %u sessions", @@ -301,10 +346,19 @@ void test_large_session_number(void) int main(int argc, char **argv) { - diag("Sessions unit tests"); + struct lttng_thread *ht_cleanup_thread; plan_tests(NUM_TESTS); + health_sessiond = health_app_create(NR_HEALTH_SESSIOND_TYPES); + ht_cleanup_thread = launch_ht_cleanup_thread(); + assert(ht_cleanup_thread); + lttng_thread_put(ht_cleanup_thread); + + diag("Sessions unit tests"); + + rcu_register_thread(); + test_session_list(); test_create_one_session(); @@ -317,9 +371,12 @@ int main(int argc, char **argv) empty_session_list(); - test_bogus_session_param(); + test_session_name_generation(); test_large_session_number(); + rcu_unregister_thread(); + lttng_thread_list_shutdown_orphans(); + return exit_status(); }