X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/374a4cbac85a61365f57a05fe8960b9fe94dcdfd..9cc1089f9aa108b9b8b32cc42c7afca7c45dc593:/src/xbt/parmap.c diff --git a/src/xbt/parmap.c b/src/xbt/parmap.c index 41bf72392f..90a0b8101f 100644 --- a/src/xbt/parmap.c +++ b/src/xbt/parmap.c @@ -3,7 +3,7 @@ /* This program is free software; you can redistribute it and/or modify it * under the terms of the license (GNU LGPL) which comes with this package. */ -#include "gras_config.h" +#include "internal_config.h" #include #ifndef _XBT_WIN32 @@ -12,6 +12,7 @@ #ifdef HAVE_FUTEX_H #include +#include #endif #include "xbt/parmap.h" @@ -20,9 +21,9 @@ #include "xbt/dynar.h" #include "xbt/xbt_os_thread.h" #include "xbt/sysdep.h" +#include "simix/smx_private.h" XBT_LOG_NEW_DEFAULT_SUBCATEGORY(xbt_parmap, xbt, "parmap: parallel map"); -XBT_LOG_NEW_SUBCATEGORY(xbt_parmap_unit, xbt_parmap, "parmap unit testing"); typedef enum { XBT_PARMAP_WORK, @@ -52,6 +53,11 @@ static void xbt_parmap_busy_worker_signal(xbt_parmap_t parmap); static void xbt_parmap_busy_master_signal(xbt_parmap_t parmap); static void xbt_parmap_busy_worker_wait(xbt_parmap_t parmap, unsigned round); +#ifdef HAVE_MC +static void xbt_parmap_mc_work(xbt_parmap_t parmap, int worker_id); +static void *xbt_parmap_mc_worker_main(void *arg); +#endif + /** * \brief Parallel map structure */ @@ -65,6 +71,14 @@ typedef struct s_xbt_parmap { xbt_dynar_t data; /**< parameters to pass to fun in parallel */ unsigned int index; /**< index of the next element of data to pick */ +#ifdef HAVE_MC + int finish; + void* ref_snapshot; + int_f_pvoid_pvoid_t snapshot_compare; + unsigned int length; + void* mc_data; +#endif + /* posix only */ xbt_os_cond_t ready_cond; xbt_os_mutex_t ready_mutex; @@ -79,6 +93,16 @@ typedef struct s_xbt_parmap { void (*worker_wait_f)(xbt_parmap_t, unsigned); /**< waits for more work */ } s_xbt_parmap_t; +/** + * \brief Thread data transmission structure + */ +typedef struct s_xbt_parmap_thread_data{ + xbt_parmap_t parmap; + int worker_id; +} s_xbt_parmap_thread_data_t; + +typedef s_xbt_parmap_thread_data_t *xbt_parmap_thread_data_t; + /** * \brief Creates a parallel map object * \param num_workers number of worker threads to create @@ -100,13 +124,51 @@ xbt_parmap_t xbt_parmap_new(unsigned int num_workers, e_xbt_parmap_mode_t mode) xbt_parmap_set_mode(parmap, mode); /* Create the pool of worker threads */ + xbt_parmap_thread_data_t data; for (i = 1; i < num_workers; i++) { - worker = xbt_os_thread_create(NULL, xbt_parmap_worker_main, parmap, NULL); + data = xbt_new0(s_xbt_parmap_thread_data_t, 1); + data->parmap = parmap; + data->worker_id = i; + worker = xbt_os_thread_create(NULL, xbt_parmap_worker_main, data, NULL); xbt_os_thread_detach(worker); } return parmap; } +#ifdef HAVE_MC +/** + * \brief Creates a parallel map object + * \param num_workers number of worker threads to create + * \param mode how to synchronize the worker threads + * \return the parmap created + */ +xbt_parmap_t xbt_parmap_mc_new(unsigned int num_workers, e_xbt_parmap_mode_t mode) +{ + unsigned int i; + xbt_os_thread_t worker = NULL; + + XBT_DEBUG("Create new parmap (%u workers)", num_workers); + + /* Initialize the thread pool data structure */ + xbt_parmap_t parmap = xbt_new0(s_xbt_parmap_t, 1); + + parmap->num_workers = num_workers; + parmap->status = XBT_PARMAP_WORK; + xbt_parmap_set_mode(parmap, mode); + + /* Create the pool of worker threads */ + xbt_parmap_thread_data_t data; + for (i = 1; i < num_workers; i++) { + data = xbt_new0(s_xbt_parmap_thread_data_t, 1); + data->parmap = parmap; + data->worker_id = i; + worker = xbt_os_thread_create(NULL, xbt_parmap_mc_worker_main, data, NULL); + xbt_os_thread_detach(worker); + } + return parmap; +} +#endif + /** * \brief Destroys a parmap * \param parmap the parmap to destroy @@ -242,8 +304,11 @@ static void xbt_parmap_work(xbt_parmap_t parmap) */ static void *xbt_parmap_worker_main(void *arg) { - xbt_parmap_t parmap = (xbt_parmap_t) arg; + xbt_parmap_thread_data_t data = (xbt_parmap_thread_data_t) arg; + xbt_parmap_t parmap = data->parmap; unsigned round = 0; + smx_context_t context = SIMIX_context_new(NULL, 0, NULL, NULL, NULL); + SIMIX_context_set_current(context); XBT_DEBUG("New worker thread created"); @@ -252,20 +317,104 @@ static void *xbt_parmap_worker_main(void *arg) parmap->worker_wait_f(parmap, ++round); if (parmap->status == XBT_PARMAP_WORK) { - XBT_DEBUG("Worker got a job"); + XBT_DEBUG("Worker %d got a job", data->worker_id); xbt_parmap_work(parmap); parmap->worker_signal_f(parmap); - XBT_DEBUG("Worker has finished"); + XBT_DEBUG("Worker %d has finished", data->worker_id); + + /* We are destroying the parmap */ + } else { + xbt_free(data); + parmap->worker_signal_f(parmap); + return NULL; + } + } +} + +#ifdef HAVE_MC + +/** + * \brief Applies a list of tasks in parallel. + * \param parmap a parallel map object + * \param fun the function to call in parallel + * \param data each element of this dynar will be passed as an argument to fun + */ +int xbt_parmap_mc_apply(xbt_parmap_t parmap, int_f_pvoid_pvoid_t fun, + void* data, unsigned int length, void* ref_snapshot) +{ + /* Assign resources to worker threads */ + parmap->snapshot_compare = fun; + parmap->mc_data = data; + parmap->index = 0; + parmap->finish = -1; + parmap->length = length; + parmap->ref_snapshot = ref_snapshot; + parmap->master_signal_f(parmap); + xbt_parmap_mc_work(parmap, 0); + parmap->master_wait_f(parmap); + XBT_DEBUG("Job done"); + return parmap->finish; +} + +static void xbt_parmap_mc_work(xbt_parmap_t parmap, int worker_id) +{ + unsigned int data_size = (parmap->length / parmap->num_workers) + + ((parmap->length % parmap->num_workers) ? 1 :0); + void* start = (char*)parmap->mc_data + (data_size*worker_id*sizeof(void*)); + void* end = MIN((char *)start + data_size* sizeof(void*), (char*)parmap->mc_data + parmap->length*sizeof(void*)); + + //XBT_CRITICAL("Worker %d : %p -> %p (%d)", worker_id, start, end, data_size); + + while ( start < end && parmap->finish == -1) { + //XBT_CRITICAL("Starting with %p", start); + int res = parmap->snapshot_compare(*(void**)start, parmap->ref_snapshot); + start = (char *)start + sizeof(start); + if (!res){ + + parmap->finish = ((char*)start - (char*)parmap->mc_data) / sizeof(void*); + //XBT_CRITICAL("Find good one %p (%p)", start, parmap->mc_data); + break; + } + } +} + +/** + * \brief Main function of a worker thread. + * \param arg the parmap + */ +static void *xbt_parmap_mc_worker_main(void *arg) +{ + xbt_parmap_thread_data_t data = (xbt_parmap_thread_data_t) arg; + xbt_parmap_t parmap = data->parmap; + unsigned round = 0; + /* smx_context_t context = SIMIX_context_new(NULL, 0, NULL, NULL, NULL); */ + /* SIMIX_context_set_current(context); */ + + XBT_DEBUG("New worker thread created"); + + /* Worker's main loop */ + while (1) { + parmap->worker_wait_f(parmap, ++round); + if (parmap->status == XBT_PARMAP_WORK) { + + XBT_DEBUG("Worker %d got a job", data->worker_id); + + xbt_parmap_mc_work(parmap, data->worker_id); + parmap->worker_signal_f(parmap); + + XBT_DEBUG("Worker %d has finished", data->worker_id); /* We are destroying the parmap */ } else { + xbt_free(data); parmap->worker_signal_f(parmap); return NULL; } } } +#endif #ifdef HAVE_FUTEX_H static void futex_wait(unsigned *uaddr, unsigned val) @@ -346,7 +495,7 @@ static void xbt_parmap_posix_worker_wait(xbt_parmap_t parmap, unsigned round) { xbt_os_mutex_acquire(parmap->ready_mutex); /* wait for more work */ - if (parmap->work < round) { + if (parmap->work != round) { xbt_os_cond_wait(parmap->ready_cond, parmap->ready_mutex); } xbt_os_mutex_release(parmap->ready_mutex); @@ -383,7 +532,7 @@ static void xbt_parmap_futex_worker_signal(xbt_parmap_t parmap) unsigned count = __sync_add_and_fetch(&parmap->thread_counter, 1); if (count == parmap->num_workers) { /* all workers have finished, wake the controller */ - futex_wake(&parmap->thread_counter, 1); + futex_wake(&parmap->thread_counter, INT_MAX); } } @@ -399,7 +548,7 @@ static void xbt_parmap_futex_master_signal(xbt_parmap_t parmap) parmap->thread_counter = 1; __sync_add_and_fetch(&parmap->work, 1); /* wake all workers */ - futex_wake(&parmap->work, parmap->num_workers - 1); + futex_wake(&parmap->work, INT_MAX); } /** @@ -415,8 +564,10 @@ static void xbt_parmap_futex_worker_wait(xbt_parmap_t parmap, unsigned round) { unsigned work = parmap->work; /* wait for more work */ - if (work < round) + while (work != round) { futex_wait(&parmap->work, work); + work = parmap->work; + } } #endif @@ -471,156 +622,7 @@ static void xbt_parmap_busy_master_signal(xbt_parmap_t parmap) static void xbt_parmap_busy_worker_wait(xbt_parmap_t parmap, unsigned round) { /* wait for more work */ - while (parmap->work < round) { + while (parmap->work != round) { xbt_os_thread_yield(); } } - -#ifdef SIMGRID_TEST -#include "xbt.h" -#include "xbt/ex.h" -#include "xbt/xbt_os_thread.h" -#include "xbt/xbt_os_time.h" -#include "gras_config.h" /* HAVE_FUTEX_H */ - -XBT_TEST_SUITE("parmap", "Parallel Map"); -XBT_LOG_EXTERNAL_DEFAULT_CATEGORY(xbt_parmap_unit); - -#ifdef HAVE_FUTEX_H -#define TEST_PARMAP_SKIP_TEST(mode) 0 -#else -#define TEST_PARMAP_SKIP_TEST(mode) ((mode) == XBT_PARMAP_FUTEX) -#endif - -#define TEST_PARMAP_VALIDATE_MODE(mode) \ - if (TEST_PARMAP_SKIP_TEST(mode)) { xbt_test_skip(); return; } else ((void)0) - -static void fun_double(void *arg) -{ - unsigned *u = arg; - *u = 2 * *u + 1; -} - -/* Check that the computations are correctly done. */ -static void test_parmap_basic(e_xbt_parmap_mode_t mode) -{ - unsigned num_workers; - - for (num_workers = 1 ; num_workers <= 16 ; num_workers *= 2) { - const unsigned len = 1033; - const unsigned num = 5; - unsigned *a; - xbt_dynar_t data; - xbt_parmap_t parmap; - unsigned i; - - xbt_test_add("Basic parmap usage (%u workers)", num_workers); - - TEST_PARMAP_VALIDATE_MODE(mode); - parmap = xbt_parmap_new(num_workers, mode); - - a = xbt_malloc(len * sizeof *a); - data = xbt_dynar_new(sizeof a, NULL); - for (i = 0; i < len; i++) { - a[i] = i; - xbt_dynar_push_as(data, void *, &a[i]); - } - - for (i = 0; i < num; i++) - xbt_parmap_apply(parmap, fun_double, data); - - for (i = 0; i < len; i++) { - unsigned expected = (1U << num) * (i + 1) - 1; - xbt_test_assert(a[i] == expected, - "a[%u]: expected %u, got %u", i, expected, a[i]); - } - - xbt_dynar_free(&data); - xbt_free(a); - xbt_parmap_destroy(parmap); - } -} - -XBT_TEST_UNIT("basic_posix", test_parmap_basic_posix, "Basic usage: posix") -{ - test_parmap_basic(XBT_PARMAP_POSIX); -} - -XBT_TEST_UNIT("basic_futex", test_parmap_basic_futex, "Basic usage: futex") -{ - test_parmap_basic(XBT_PARMAP_FUTEX); -} - -XBT_TEST_UNIT("basic_busy_wait", test_parmap_basic_busy_wait, "Basic usage: busy_wait") -{ - test_parmap_basic(XBT_PARMAP_BUSY_WAIT); -} - -static void fun_get_id(void *arg) -{ - *(uintptr_t *)arg = (uintptr_t)xbt_os_thread_self(); - xbt_os_sleep(0.5); -} - -static int fun_compare(const void *pa, const void *pb) -{ - uintptr_t a = *(uintptr_t *)pa; - uintptr_t b = *(uintptr_t *)pb; - return a < b ? -1 : a > b ? 1 : 0; -} - -/* Check that all threads are working. */ -static void test_parmap_extended(e_xbt_parmap_mode_t mode) -{ - unsigned num_workers; - - for (num_workers = 1 ; num_workers <= 16 ; num_workers *= 2) { - const unsigned len = 2 * num_workers; - uintptr_t *a; - xbt_parmap_t parmap; - xbt_dynar_t data; - unsigned i; - unsigned count; - - xbt_test_add("Extended parmap usage (%u workers)", num_workers); - - TEST_PARMAP_VALIDATE_MODE(mode); - parmap = xbt_parmap_new(num_workers, mode); - - a = xbt_malloc(len * sizeof *a); - data = xbt_dynar_new(sizeof a, NULL); - for (i = 0; i < len; i++) - xbt_dynar_push_as(data, void *, &a[i]); - - xbt_parmap_apply(parmap, fun_get_id, data); - - qsort(a, len, sizeof a[0], fun_compare); - count = 1; - for (i = 1; i < len; i++) - if (a[i] != a[i - 1]) - count++; - xbt_test_assert(count == num_workers, - "only %u/%u threads did some work", count, num_workers); - - xbt_dynar_free(&data); - xbt_free(a); - xbt_parmap_destroy(parmap); - } -} - -XBT_TEST_UNIT("extended_posix", test_parmap_extended_posix, "Extended usage: posix") -{ - test_parmap_extended(XBT_PARMAP_POSIX); -} - -XBT_TEST_UNIT("extended_futex", test_parmap_extended_futex, "Extended usage: futex") -{ - test_parmap_extended(XBT_PARMAP_FUTEX); -} - -XBT_TEST_UNIT("extended_busy_wait", test_parmap_extended_busy_wait, "Extended usage: busy_wait") -{ - test_parmap_extended(XBT_PARMAP_BUSY_WAIT); -} - -#endif /* SIMGRID_TEST */