X-Git-Url: http://demsky.eecs.uci.edu/git/?a=blobdiff_plain;f=threads.cc;h=792014e62b4d3c88c1ecc052a4de120c25cace52;hb=84e24d516d4e9dbd30f1fff7e9a185d1540d20eb;hp=35000dc17d2ad40f73ea3652f0b634dd9f95678b;hpb=9ab763adc965ca76a8d65b9898d20c60cdb44445;p=model-checker.git diff --git a/threads.cc b/threads.cc index 35000dc..792014e 100644 --- a/threads.cc +++ b/threads.cc @@ -1,5 +1,3 @@ -/* -*- Mode: C; indent-tabs-mode: t -*- */ - #include "libthreads.h" #include "common.h" #include "threads.h" @@ -11,23 +9,35 @@ static void * stack_allocate(size_t size) { - return userMalloc(size); + return malloc(size); } static void stack_free(void *stack) { - userFree(stack); + free(stack); } Thread * thread_current(void) { + ASSERT(model); return model->scheduler->get_current_thread(); } -/* This method just gets around makecontext not being 64-bit clean */ - +/** + * Provides a startup wrapper for each thread, allowing some initial + * model-checking data to be recorded. This method also gets around makecontext + * not being 64-bit clean + */ void thread_startup() { - Thread * curr_thread=thread_current(); + Thread * curr_thread = thread_current(); + + /* TODO -- we should make this event always immediately follow the + CREATE event, so we don't get redundant traces... */ + + /* Add dummy "start" action, just to create a first clock vector */ + model->switch_to_master(new ModelAction(THREAD_START, std::memory_order_seq_cst, curr_thread)); + + /* Call the actual thread function */ curr_thread->start_routine(curr_thread->arg); } @@ -70,19 +80,20 @@ void Thread::complete() } } -Thread::Thread(thrd_t *t, void (*func)(void *), void *a) { +Thread::Thread(thrd_t *t, void (*func)(void *), void *a) : + start_routine(func), + arg(a), + user_thread(t), + state(THREAD_CREATED), + last_action_val(VALUE_NONE) +{ int ret; - user_thread = t; - start_routine = func; - arg = a; - /* Initialize state */ ret = create_context(); if (ret) printf("Error in create_context\n"); - state = THREAD_CREATED; id = model->get_next_id(); *user_thread = id; parent = thread_current();