diff options
| author | Brian Anderson <banderson@mozilla.com> | 2012-03-02 21:22:42 -0800 |
|---|---|---|
| committer | Brian Anderson <banderson@mozilla.com> | 2012-03-05 19:39:55 -0800 |
| commit | b2a075e20d07994c960a7598fee2771d10cb9989 (patch) | |
| tree | 4e7a2fd278e8fb43ea692c15a89b7daac158adf5 /src/rt/rust_task.cpp | |
| parent | d7298a797b1041e9e997378bdb3cd4923567b2d4 (diff) | |
| download | rust-b2a075e20d07994c960a7598fee2771d10cb9989.tar.gz rust-b2a075e20d07994c960a7598fee2771d10cb9989.zip | |
rt: Protect rust_task::state with a lock
Diffstat (limited to 'src/rt/rust_task.cpp')
| -rw-r--r-- | src/rt/rust_task.cpp | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/src/rt/rust_task.cpp b/src/rt/rust_task.cpp index 92637ace632..dd0ae7846bc 100644 --- a/src/rt/rust_task.cpp +++ b/src/rt/rust_task.cpp @@ -74,7 +74,6 @@ rust_task::rust_task(rust_task_thread *thread, rust_task_list *state, cache(NULL), kernel(thread->kernel), name(name), - state(state), cond(NULL), cond_name("none"), list_index(-1), @@ -87,6 +86,7 @@ rust_task::rust_task(rust_task_thread *thread, rust_task_list *state, dynastack(this), cc_counter(0), total_stack_sz(0), + state(state), killed(false), reentered_rust_stack(false), c_stack(NULL), @@ -355,12 +355,14 @@ rust_task::get_frame_glue_fns(uintptr_t fp) { bool rust_task::running() { + scoped_lock with(state_lock); return state == &thread->running_tasks; } bool rust_task::blocked() { + scoped_lock with(state_lock); return state == &thread->blocked_tasks; } @@ -373,6 +375,7 @@ rust_task::blocked_on(rust_cond *on) bool rust_task::dead() { + scoped_lock with(state_lock); return state == &thread->dead_tasks; } @@ -407,7 +410,10 @@ rust_task::transition(rust_task_list *src, rust_task_list *dst) { I(thread, state == src); src->remove(this); dst->append(this); - state = dst; + { + scoped_lock with(state_lock); + state = dst; + } thread->lock.signal(); if(unlock) thread->lock.unlock(); |
