Revision: 22545
Author: [email protected]
Date: Wed Jul 23 09:17:21 2014 UTC
Log: Wait for sweeper threads when a scan on scavenge page is not
swept.
BUG=
[email protected]
Review URL: https://codereview.chromium.org/404083002
http://code.google.com/p/v8/source/detail?r=22545
Modified:
/branches/bleeding_edge/src/mark-compact.cc
/branches/bleeding_edge/src/mark-compact.h
/branches/bleeding_edge/src/store-buffer.cc
=======================================
--- /branches/bleeding_edge/src/mark-compact.cc Tue Jul 22 16:30:23 2014 UTC
+++ /branches/bleeding_edge/src/mark-compact.cc Wed Jul 23 09:17:21 2014 UTC
@@ -4098,37 +4098,42 @@
int MarkCompactCollector::SweepInParallel(PagedSpace* space,
int required_freed_bytes) {
- PageIterator it(space);
- FreeList* free_list = space == heap()->old_pointer_space()
- ? free_list_old_pointer_space_.get()
- : free_list_old_data_space_.get();
- FreeList private_free_list(space);
int max_freed = 0;
int max_freed_overall = 0;
+ PageIterator it(space);
while (it.has_next()) {
Page* p = it.next();
- if (p->TryParallelSweeping()) {
- if (space->swept_precisely()) {
- max_freed = SweepPrecisely<SWEEP_ONLY,
- SWEEP_IN_PARALLEL,
- IGNORE_SKIP_LIST,
- IGNORE_FREE_SPACE>(
- space, &private_free_list, p, NULL);
- } else {
- max_freed = SweepConservatively<SWEEP_IN_PARALLEL>(
- space, &private_free_list, p);
- }
- ASSERT(max_freed >= 0);
- free_list->Concatenate(&private_free_list);
- if (required_freed_bytes > 0 && max_freed >= required_freed_bytes) {
- return max_freed;
- }
- max_freed_overall = Max(max_freed, max_freed_overall);
+ max_freed = SweepInParallel(p, space);
+ ASSERT(max_freed >= 0);
+ if (required_freed_bytes > 0 && max_freed >= required_freed_bytes) {
+ return max_freed;
}
+ max_freed_overall = Max(max_freed, max_freed_overall);
if (p == space->end_of_unswept_pages()) break;
}
return max_freed_overall;
}
+
+
+int MarkCompactCollector::SweepInParallel(Page* page, PagedSpace* space) {
+ int max_freed = 0;
+ if (page->TryParallelSweeping()) {
+ FreeList* free_list = space == heap()->old_pointer_space()
+ ? free_list_old_pointer_space_.get()
+ : free_list_old_data_space_.get();
+ FreeList private_free_list(space);
+ if (space->swept_precisely()) {
+ max_freed = SweepPrecisely<SWEEP_ONLY, SWEEP_IN_PARALLEL,
+ IGNORE_SKIP_LIST, IGNORE_FREE_SPACE>(
+ space, &private_free_list, page, NULL);
+ } else {
+ max_freed = SweepConservatively<SWEEP_IN_PARALLEL>(
+ space, &private_free_list, page);
+ }
+ free_list->Concatenate(&private_free_list);
+ }
+ return max_freed;
+}
void MarkCompactCollector::SweepSpace(PagedSpace* space, SweeperType
sweeper) {
=======================================
--- /branches/bleeding_edge/src/mark-compact.h Fri Jul 18 11:04:20 2014 UTC
+++ /branches/bleeding_edge/src/mark-compact.h Wed Jul 23 09:17:21 2014 UTC
@@ -658,9 +658,14 @@
// Concurrent and parallel sweeping support. If required_freed_bytes was
set
// to a value larger than 0, then sweeping returns after a block of at
least
// required_freed_bytes was freed. If required_freed_bytes was set to
zero
- // then the whole given space is swept.
+ // then the whole given space is swept. It returns the size of the
maximum
+ // continuous freed memory chunk.
int SweepInParallel(PagedSpace* space, int required_freed_bytes);
+ // Sweeps a given page concurrently to the sweeper threads. It returns
the
+ // size of the maximum continuous freed memory chunk.
+ int SweepInParallel(Page* page, PagedSpace* space);
+
void EnsureSweepingCompleted();
// If sweeper threads are not active this method will return true. If
=======================================
--- /branches/bleeding_edge/src/store-buffer.cc Thu Jul 10 12:22:01 2014 UTC
+++ /branches/bleeding_edge/src/store-buffer.cc Wed Jul 23 09:17:21 2014 UTC
@@ -505,6 +505,16 @@
}
}
} else {
+ if (page->parallel_sweeping() >
MemoryChunk::SWEEPING_FINALIZE) {
+ heap_->mark_compact_collector()->SweepInParallel(page,
owner);
+ if (page->parallel_sweeping() >
MemoryChunk::SWEEPING_FINALIZE) {
+ // We were not able to sweep that page, i.e., a concurrent
+ // sweeper thread currently owns this page.
+ // TODO(hpayer): This may introduce a huge pause here. We
+ // just care about finish sweeping of the scan on scavenge
page.
+ heap_->mark_compact_collector()->EnsureSweepingCompleted();
+ }
+ }
FindPointersToNewSpaceInRegion(
start, end, slot_callback, clear_maps);
}
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
