Set unswept free bytes for concurent sweeper.

src/sweeper-thread.cc

 // Copyright 2012 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 57 matching lines @@
     collector_->SweepInParallel(heap_->old_data_space(),
                                 &private_free_list_old_data_space_,
                                 &free_list_old_data_space_);
     collector_->SweepInParallel(heap_->old_pointer_space(),
                                 &private_free_list_old_pointer_space_,
                                 &free_list_old_pointer_space_);
     end_sweeping_semaphore_->Signal();
   }
 }
 
-intptr_t SweeperThread::StealMemory(PagedSpace* space) {
-  intptr_t free_bytes = 0;
-  if (space->identity() == OLD_POINTER_SPACE) {
-    free_bytes = space->free_list()->Concatenate(&free_list_old_pointer_space_);
-    space->AddToAccountingStats(free_bytes);
-  } else if (space->identity() == OLD_DATA_SPACE) {
-    free_bytes = space->free_list()->Concatenate(&free_list_old_data_space_);
-    space->AddToAccountingStats(free_bytes);
-  }
+
+intptr_t SweeperThread::StealMemory(PagedSpace* space, FreeList* free_list) {
+  intptr_t free_bytes = space->free_list()->Concatenate(free_list);
+  space->AddToAccountingStats(free_bytes);
+  space->DecreaseUnsweptFreeBytes(free_bytes);

[Michael Starzinger, 2013/02/28 13:09:52]

As discussed offline: Let's move both (AddToAccountingStats and
DecreaseUnsweptFreeBytes) into
MarkCompactCollector::StealMemoryFromSweeperThreads, it should be part the
mark-compact.cc file.

[Hannes Payer (out of office), 2013/02/28 14:42:34]

Done.

   return free_bytes;
 }
 
+
+intptr_t SweeperThread::StealMemory(PagedSpace* space) {
+  if (space->identity() == OLD_POINTER_SPACE) {
+    return StealMemory(space, &free_list_old_pointer_space_);
+  } else if (space->identity() == OLD_DATA_SPACE) {
+    return StealMemory(space, &free_list_old_data_space_);
+  }
+  return 0;
+}
+
 void SweeperThread::Stop() {
   Release_Store(&stop_thread_, static_cast<AtomicWord>(true));
   start_sweeping_semaphore_->Signal();
   stop_semaphore_->Wait();
 }
 
 
 void SweeperThread::StartSweeping() {
   start_sweeping_semaphore_->Signal();
 }
 
 
 void SweeperThread::WaitForSweeperThread() {
   end_sweeping_semaphore_->Wait();
 }
 } }  // namespace v8::internal