diff options
| author | 2024-02-29 20:36:22 +0000 | |
|---|---|---|
| committer | 2024-02-29 20:36:22 +0000 | |
| commit | 2e553f12ac2670eecedeca3cf85a3e24f19ba04c (patch) | |
| tree | 87cc79959eaa1f534b62b8291afd3cc37e6da5f2 /runtime/thread_list.cc | |
| parent | 940428ea116dadb3e04a255a7c586a34b6245979 (diff) | |
| parent | 0fb2079920ddd0b9d213fbd28f14f804fada87e5 (diff) | |
Fix implicit suspend trigger memory ordering am: 0fb2079920
Original change: https://android-review.googlesource.com/c/platform/art/+/2982020
Change-Id: Ib261dd6a0bd909fa19d4fb4810d3be0f8f3096d7
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
Diffstat (limited to 'runtime/thread_list.cc')
| -rw-r--r-- | runtime/thread_list.cc | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/runtime/thread_list.cc b/runtime/thread_list.cc index 5e63b27b20..b841eaaccb 100644 --- a/runtime/thread_list.cc +++ b/runtime/thread_list.cc @@ -1127,9 +1127,6 @@ bool ThreadList::SuspendThread(Thread* self, // Now wait for target to decrement suspend barrier. std::optional<std::string> failure_info; if (!is_suspended) { - // As an experiment, redundantly trigger suspension. TODO: Remove this. - std::atomic_thread_fence(std::memory_order_seq_cst); - thread->TriggerSuspend(); failure_info = WaitForSuspendBarrier(&wrapped_barrier.barrier_, tid, attempt_of_4); if (!failure_info.has_value()) { is_suspended = true; |