Reland of "[cctest] Add tests for aborting compaction of pages"

src/heap/mark-compact.cc

 // Copyright 2012 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "src/heap/mark-compact.h"
 
 #include "src/base/atomicops.h"
 #include "src/base/bits.h"
 #include "src/base/sys-info.h"
 #include "src/code-stubs.h"
@@ skipping 3756 matching lines @@
     // Clear sweeping flags indicating that marking bits are still intact.
     p->ClearWasSwept();
 
     if (p->IsFlagSet(Page::RESCAN_ON_EVACUATION) ||
         p->IsEvacuationCandidate()) {
       // Will be processed in EvacuateNewSpaceAndCandidates.
       DCHECK(evacuation_candidates_.length() > 0);
       continue;
     }
 
+    if (p->IsFlagSet(Page::NEVER_ALLOCATE_ON_PAGE)) {
+      // We do not sweep pages that are marked to be ignored for allocation.
+      p->SetWasSwept();
+      Bitmap::Clear(p);
+      p->ResetLiveBytes();

[Hannes Payer (out of office), 2015/12/16 12:07:14]

Should we really reset live bytes here? That would indicate that everything is
dead on this page.

[Michael Lippautz, 2015/12/16 12:13:14]

We reset live bytes after sweeping a page. I think we want to do that here since
we don't sweep them but want them to be in the same state as regular pages after
sweeping (i.e. cleared mark bits, reset live byte counter).

[Hannes Payer (out of office), 2015/12/16 12:44:00]

Acknowledged.

+      continue;
+    }
+
     // One unused page is kept, all further are released before sweeping them.
     if (p->LiveBytes() == 0) {
       if (unused_page_present) {
         if (FLAG_gc_verbose) {
           PrintIsolate(isolate(), "sweeping: released page: %p", p);
         }
         space->ReleasePage(p);
         continue;
       }
       unused_page_present = true;
@@ skipping 172 matching lines @@
     MarkBit mark_bit = Marking::MarkBitFrom(host);
     if (Marking::IsBlack(mark_bit)) {
       RelocInfo rinfo(isolate(), pc, RelocInfo::CODE_TARGET, 0, host);
       RecordRelocSlot(&rinfo, target);
     }
   }
 }
 
 }  // namespace internal
 }  // namespace v8