Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(352)

Unified Diff: src/heap/spaces.h

Issue 1341293002: [heap] Extend mutex guards for CodeRange. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 5 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | src/heap/spaces.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/heap/spaces.h
diff --git a/src/heap/spaces.h b/src/heap/spaces.h
index b11f926fde3cf373b1144b0dc67ebc5f859c0a02..6e1750ef63a1a5a9fa51bf96fc8309a3c89c289f 100644
--- a/src/heap/spaces.h
+++ b/src/heap/spaces.h
@@ -1013,9 +1013,9 @@ class CodeRange {
size_t size;
};
- // All access to free_list_ require to take the free_list_mutex_. GC threads
- // may access the free_list_ concurrently to the main thread.
- base::Mutex free_list_mutex_;
+ // The global mutex guards free_list_ and allocation_list_ as GC threads may
+ // access both lists concurrently to the main thread.
+ base::Mutex code_range_mutex_;
// Freed blocks of memory are added to the free list. When the allocation
// list is exhausted, the free list is sorted and merged to make the new
« no previous file with comments | « no previous file | src/heap/spaces.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698