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

Unified Diff: src/heap/heap.cc

Issue 2781363002: [heap] Take page lock when scavenging old to new references in Scavenger. (Closed)
Patch Set: enum Created 3 years, 9 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/mark-compact.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/heap/heap.cc
diff --git a/src/heap/heap.cc b/src/heap/heap.cc
index 7dfdb9c8d25a01cf713e0f5af5d87a4336b31285..57b4cea6cf0cb88f5d8318c837d3e11707ffcb6c 100644
--- a/src/heap/heap.cc
+++ b/src/heap/heap.cc
@@ -1723,12 +1723,14 @@ void Heap::Scavenge() {
{
// Copy objects reachable from the old generation.
TRACE_GC(tracer(), GCTracer::Scope::SCAVENGER_OLD_TO_NEW_POINTERS);
- RememberedSet<OLD_TO_NEW>::Iterate(this, [this](Address addr) {
- return Scavenger::CheckAndScavengeObject(this, addr);
- });
+ RememberedSet<OLD_TO_NEW>::Iterate(
+ this, SYNCHRONIZED, [this](Address addr) {
+ return Scavenger::CheckAndScavengeObject(this, addr);
+ });
RememberedSet<OLD_TO_NEW>::IterateTyped(
- this, [this](SlotType type, Address host_addr, Address addr) {
+ this, SYNCHRONIZED,
+ [this](SlotType type, Address host_addr, Address addr) {
return UpdateTypedSlotHelper::UpdateTypedSlot(
isolate(), type, addr, [this](Object** addr) {
// We expect that objects referenced by code are long living.
« no previous file with comments | « no previous file | src/heap/mark-compact.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698