hotspot/src/share/vm/gc/g1/g1EvacFailure.cpp
author tschatzl
Wed, 06 Jul 2016 11:22:55 +0200
changeset 39698 4016de4e596b
parent 38081 a3bcb7197d45
child 46258 d26ebd7e2f10
permissions -rw-r--r--
8159978: Use an array to store the collection set regions instead of linking through regions Summary: Fix a potential problem with memory visibility in the sampling thread in the collection set by changing the way we store the collection set. Reviewed-by: ehelin, jmasa
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
11451
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
     1
/*
35943
e726308008c0 8148973: Rename g1/concurrentMark.{hpp,cpp,inline.hpp} to g1/g1ConcurrentMark.{hpp,cpp,inline.hpp}
ehelin
parents: 35851
diff changeset
     2
 * Copyright (c) 2012, 2016, Oracle and/or its affiliates. All rights reserved.
11451
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
     4
 *
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
     7
 * published by the Free Software Foundation.
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
     8
 *
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    13
 * accompanied this code).
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    14
 *
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    18
 *
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    21
 * questions.
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    22
 *
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    23
 */
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    24
29080
a16b0d9a08f5 8072910: Move the g1EvacFailure.hpp implementation to g1EvacFailure.cpp
stefank
parents: 27009
diff changeset
    25
#include "precompiled.hpp"
30764
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 29080
diff changeset
    26
#include "gc/g1/dirtyCardQueue.hpp"
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 29080
diff changeset
    27
#include "gc/g1/g1CollectedHeap.inline.hpp"
31331
a7c714b6cfb3 7097567: G1: abstract and encapsulate collector phases and transitions between them
drwhite
parents: 30764
diff changeset
    28
#include "gc/g1/g1CollectorState.hpp"
35943
e726308008c0 8148973: Rename g1/concurrentMark.{hpp,cpp,inline.hpp} to g1/g1ConcurrentMark.{hpp,cpp,inline.hpp}
ehelin
parents: 35851
diff changeset
    29
#include "gc/g1/g1ConcurrentMark.inline.hpp"
30764
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 29080
diff changeset
    30
#include "gc/g1/g1EvacFailure.hpp"
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents: 34614
diff changeset
    31
#include "gc/g1/g1HeapVerifier.hpp"
30764
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 29080
diff changeset
    32
#include "gc/g1/g1OopClosures.inline.hpp"
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 29080
diff changeset
    33
#include "gc/g1/g1_globals.hpp"
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 29080
diff changeset
    34
#include "gc/g1/heapRegion.hpp"
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 29080
diff changeset
    35
#include "gc/g1/heapRegionRemSet.hpp"
38081
a3bcb7197d45 8151556: Use the PreservedMarks* classes for the G1 preserved mark stacks
tonyp
parents: 37988
diff changeset
    36
#include "gc/shared/preservedMarks.inline.hpp"
11451
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    37
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    38
class UpdateRSetDeferred : public OopsInHeapRegionClosure {
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    39
private:
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    40
  G1CollectedHeap* _g1;
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    41
  DirtyCardQueue *_dcq;
20309
7445302daff6 8025305: Cleanup CardTableModRefBS usage in G1
mgerdin
parents: 11584
diff changeset
    42
  G1SATBCardTableModRefBS* _ct_bs;
11451
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    43
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    44
public:
31631
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
    45
  UpdateRSetDeferred(DirtyCardQueue* dcq) :
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
    46
    _g1(G1CollectedHeap::heap()), _ct_bs(_g1->g1_barrier_set()), _dcq(dcq) {}
11451
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    47
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    48
  virtual void do_oop(narrowOop* p) { do_oop_work(p); }
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    49
  virtual void do_oop(      oop* p) { do_oop_work(p); }
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    50
  template <class T> void do_oop_work(T* p) {
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    51
    assert(_from->is_in_reserved(p), "paranoia");
34614
5f1e98f98bb7 8144040: UpdateRSetDeferred in G1EvacFailure will never visit survivor regions
ehelin
parents: 34311
diff changeset
    52
    assert(!_from->is_survivor(), "Unexpected evac failure in survivor region");
5f1e98f98bb7 8144040: UpdateRSetDeferred in G1EvacFailure will never visit survivor regions
ehelin
parents: 34311
diff changeset
    53
5f1e98f98bb7 8144040: UpdateRSetDeferred in G1EvacFailure will never visit survivor regions
ehelin
parents: 34311
diff changeset
    54
    if (!_from->is_in_reserved(oopDesc::load_decode_heap_oop(p))) {
11451
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    55
      size_t card_index = _ct_bs->index_for(p);
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    56
      if (_ct_bs->mark_card_deferred(card_index)) {
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    57
        _dcq->enqueue((jbyte*)_ct_bs->byte_for_index(card_index));
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    58
      }
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    59
    }
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    60
  }
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    61
};
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    62
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    63
class RemoveSelfForwardPtrObjClosure: public ObjectClosure {
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    64
private:
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    65
  G1CollectedHeap* _g1;
35943
e726308008c0 8148973: Rename g1/concurrentMark.{hpp,cpp,inline.hpp} to g1/g1ConcurrentMark.{hpp,cpp,inline.hpp}
ehelin
parents: 35851
diff changeset
    66
  G1ConcurrentMark* _cm;
11451
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    67
  HeapRegion* _hr;
11455
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 11451
diff changeset
    68
  size_t _marked_bytes;
11451
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    69
  OopsInHeapRegionClosure *_update_rset_cl;
11455
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 11451
diff changeset
    70
  bool _during_initial_mark;
11583
83a7383de44c 6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents: 11455
diff changeset
    71
  uint _worker_id;
31631
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
    72
  HeapWord* _last_forwarded_object_end;
11583
83a7383de44c 6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents: 11455
diff changeset
    73
11451
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    74
public:
31631
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
    75
  RemoveSelfForwardPtrObjClosure(HeapRegion* hr,
11455
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 11451
diff changeset
    76
                                 OopsInHeapRegionClosure* update_rset_cl,
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 11451
diff changeset
    77
                                 bool during_initial_mark,
11583
83a7383de44c 6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents: 11455
diff changeset
    78
                                 uint worker_id) :
31631
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
    79
    _g1(G1CollectedHeap::heap()),
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
    80
    _cm(_g1->concurrent_mark()),
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
    81
    _hr(hr),
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
    82
    _marked_bytes(0),
11451
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    83
    _update_rset_cl(update_rset_cl),
11455
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 11451
diff changeset
    84
    _during_initial_mark(during_initial_mark),
25492
d27050bdfb04 8049421: G1 Class Unloading after completing a concurrent mark cycle
stefank
parents: 25491
diff changeset
    85
    _worker_id(worker_id),
31631
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
    86
    _last_forwarded_object_end(hr->bottom()) { }
11451
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    87
11455
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 11451
diff changeset
    88
  size_t marked_bytes() { return _marked_bytes; }
11451
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    89
31631
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
    90
  // Iterate over the live objects in the region to find self-forwarded objects
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
    91
  // that need to be kept live. We need to update the remembered sets of these
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
    92
  // objects. Further update the BOT and marks.
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
    93
  // We can coalesce and overwrite the remaining heap contents with dummy objects
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
    94
  // as they have either been dead or evacuated (which are unreferenced now, i.e.
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
    95
  // dead too) already.
11451
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    96
  void do_object(oop obj) {
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    97
    HeapWord* obj_addr = (HeapWord*) obj;
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
    98
    assert(_hr->is_in(obj_addr), "sanity");
25492
d27050bdfb04 8049421: G1 Class Unloading after completing a concurrent mark cycle
stefank
parents: 25491
diff changeset
    99
11451
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
   100
    if (obj->is_forwarded() && obj->forwardee() == obj) {
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
   101
      // The object failed to move.
11455
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 11451
diff changeset
   102
31631
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   103
      zap_dead_objects(_last_forwarded_object_end, obj_addr);
11455
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 11451
diff changeset
   104
      // We consider all objects that we find self-forwarded to be
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 11451
diff changeset
   105
      // live. What we'll do is that we'll update the prev marking
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 11451
diff changeset
   106
      // info so that they are all under PTAMS and explicitly marked.
25492
d27050bdfb04 8049421: G1 Class Unloading after completing a concurrent mark cycle
stefank
parents: 25491
diff changeset
   107
      if (!_cm->isPrevMarked(obj)) {
d27050bdfb04 8049421: G1 Class Unloading after completing a concurrent mark cycle
stefank
parents: 25491
diff changeset
   108
        _cm->markPrev(obj);
d27050bdfb04 8049421: G1 Class Unloading after completing a concurrent mark cycle
stefank
parents: 25491
diff changeset
   109
      }
11455
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 11451
diff changeset
   110
      if (_during_initial_mark) {
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 11451
diff changeset
   111
        // For the next marking info we'll only mark the
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 11451
diff changeset
   112
        // self-forwarded objects explicitly if we are during
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 11451
diff changeset
   113
        // initial-mark (since, normally, we only mark objects pointed
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 11451
diff changeset
   114
        // to by roots if we succeed in copying them). By marking all
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 11451
diff changeset
   115
        // self-forwarded objects we ensure that we mark any that are
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 11451
diff changeset
   116
        // still pointed to be roots. During concurrent marking, and
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 11451
diff changeset
   117
        // after initial-mark, we don't need to mark any objects
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 11451
diff changeset
   118
        // explicitly and all objects in the CSet are considered
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 11451
diff changeset
   119
        // (implicitly) live. So, we won't mark them explicitly and
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 11451
diff changeset
   120
        // we'll leave them over NTAMS.
37413
2f71679d06dd 8077144: Concurrent mark initialization takes too long
tschatzl
parents: 35943
diff changeset
   121
        _cm->grayRoot(obj, _hr);
11451
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
   122
      }
37413
2f71679d06dd 8077144: Concurrent mark initialization takes too long
tschatzl
parents: 35943
diff changeset
   123
      size_t obj_size = obj->size();
2f71679d06dd 8077144: Concurrent mark initialization takes too long
tschatzl
parents: 35943
diff changeset
   124
11455
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 11451
diff changeset
   125
      _marked_bytes += (obj_size * HeapWordSize);
38081
a3bcb7197d45 8151556: Use the PreservedMarks* classes for the G1 preserved mark stacks
tonyp
parents: 37988
diff changeset
   126
      PreservedMarks::init_forwarded_mark(obj);
11451
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
   127
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
   128
      // While we were processing RSet buffers during the collection,
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
   129
      // we actually didn't scan any cards on the collection set,
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
   130
      // since we didn't want to update remembered sets with entries
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
   131
      // that point into the collection set, given that live objects
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
   132
      // from the collection set are about to move and such entries
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
   133
      // will be stale very soon.
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
   134
      // This change also dealt with a reliability issue which
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
   135
      // involved scanning a card in the collection set and coming
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
   136
      // across an array that was being chunked and looking malformed.
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
   137
      // The problem is that, if evacuation fails, we might have
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
   138
      // remembered set entries missing given that we skipped cards on
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
   139
      // the collection set. So, we'll recreate such entries now.
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
   140
      obj->oop_iterate(_update_rset_cl);
31631
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   141
37413
2f71679d06dd 8077144: Concurrent mark initialization takes too long
tschatzl
parents: 35943
diff changeset
   142
      HeapWord* obj_end = obj_addr + obj_size;
31631
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   143
      _last_forwarded_object_end = obj_end;
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   144
      _hr->cross_threshold(obj_addr, obj_end);
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   145
    }
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   146
  }
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   147
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   148
  // Fill the memory area from start to end with filler objects, and update the BOT
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   149
  // and the mark bitmap accordingly.
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   150
  void zap_dead_objects(HeapWord* start, HeapWord* end) {
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   151
    if (start == end) {
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   152
      return;
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   153
    }
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   154
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   155
    size_t gap_size = pointer_delta(end, start);
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   156
    MemRegion mr(start, gap_size);
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   157
    if (gap_size >= CollectedHeap::min_fill_size()) {
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   158
      CollectedHeap::fill_with_objects(start, gap_size);
25492
d27050bdfb04 8049421: G1 Class Unloading after completing a concurrent mark cycle
stefank
parents: 25491
diff changeset
   159
31631
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   160
      HeapWord* end_first_obj = start + ((oop)start)->size();
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   161
      _hr->cross_threshold(start, end_first_obj);
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   162
      // Fill_with_objects() may have created multiple (i.e. two)
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   163
      // objects, as the max_fill_size() is half a region.
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   164
      // After updating the BOT for the first object, also update the
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   165
      // BOT for the second object to make the BOT complete.
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   166
      if (end_first_obj != end) {
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   167
        _hr->cross_threshold(end_first_obj, end);
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   168
#ifdef ASSERT
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   169
        size_t size_second_obj = ((oop)end_first_obj)->size();
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   170
        HeapWord* end_of_second_obj = end_first_obj + size_second_obj;
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   171
        assert(end == end_of_second_obj,
33105
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 31631
diff changeset
   172
               "More than two objects were used to fill the area from " PTR_FORMAT " to " PTR_FORMAT ", "
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 31631
diff changeset
   173
               "second objects size " SIZE_FORMAT " ends at " PTR_FORMAT,
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 31631
diff changeset
   174
               p2i(start), p2i(end), size_second_obj, p2i(end_of_second_obj));
31631
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   175
#endif
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   176
      }
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   177
    }
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   178
    _cm->clearRangePrevBitmap(mr);
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   179
  }
25492
d27050bdfb04 8049421: G1 Class Unloading after completing a concurrent mark cycle
stefank
parents: 25491
diff changeset
   180
31631
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   181
  void zap_remainder() {
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   182
    zap_dead_objects(_last_forwarded_object_end, _hr->top());
11451
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
   183
  }
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
   184
};
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
   185
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
   186
class RemoveSelfForwardPtrHRClosure: public HeapRegionClosure {
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
   187
  G1CollectedHeap* _g1h;
11583
83a7383de44c 6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents: 11455
diff changeset
   188
  uint _worker_id;
27009
e7e723732b6b 8058298: Separate heap region iterator claim values from the data structures iterated over
mlarsson
parents: 26846
diff changeset
   189
  HeapRegionClaimer* _hrclaimer;
11451
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
   190
26701
f8ff74a6c058 8052172: Evacuation failure handling in G1 does not evacuate all objects if -XX:-G1DeferredRSUpdate is set
tschatzl
parents: 26422
diff changeset
   191
  DirtyCardQueue _dcq;
f8ff74a6c058 8052172: Evacuation failure handling in G1 does not evacuate all objects if -XX:-G1DeferredRSUpdate is set
tschatzl
parents: 26422
diff changeset
   192
  UpdateRSetDeferred _update_rset_cl;
f8ff74a6c058 8052172: Evacuation failure handling in G1 does not evacuate all objects if -XX:-G1DeferredRSUpdate is set
tschatzl
parents: 26422
diff changeset
   193
11451
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
   194
public:
31631
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   195
  RemoveSelfForwardPtrHRClosure(uint worker_id,
27009
e7e723732b6b 8058298: Separate heap region iterator claim values from the data structures iterated over
mlarsson
parents: 26846
diff changeset
   196
                                HeapRegionClaimer* hrclaimer) :
31631
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   197
    _g1h(G1CollectedHeap::heap()),
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   198
    _dcq(&_g1h->dirty_card_queue_set()),
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   199
    _update_rset_cl(&_dcq),
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   200
    _worker_id(worker_id),
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   201
    _hrclaimer(hrclaimer) {
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   202
  }
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   203
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   204
  size_t remove_self_forward_ptr_by_walking_hr(HeapRegion* hr,
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   205
                                               bool during_initial_mark) {
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   206
    RemoveSelfForwardPtrObjClosure rspc(hr,
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   207
                                        &_update_rset_cl,
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   208
                                        during_initial_mark,
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   209
                                        _worker_id);
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   210
    _update_rset_cl.set_region(hr);
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   211
    hr->object_iterate(&rspc);
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   212
    // Need to zap the remainder area of the processed region.
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   213
    rspc.zap_remainder();
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   214
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   215
    return rspc.marked_bytes();
27009
e7e723732b6b 8058298: Separate heap region iterator claim values from the data structures iterated over
mlarsson
parents: 26846
diff changeset
   216
  }
11451
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
   217
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
   218
  bool doHeapRegion(HeapRegion *hr) {
31331
a7c714b6cfb3 7097567: G1: abstract and encapsulate collector phases and transitions between them
drwhite
parents: 30764
diff changeset
   219
    bool during_initial_mark = _g1h->collector_state()->during_initial_mark_pause();
a7c714b6cfb3 7097567: G1: abstract and encapsulate collector phases and transitions between them
drwhite
parents: 30764
diff changeset
   220
    bool during_conc_mark = _g1h->collector_state()->mark_in_progress();
11455
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 11451
diff changeset
   221
33105
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 31631
diff changeset
   222
    assert(!hr->is_pinned(), "Unexpected pinned region at index %u", hr->hrm_index());
11451
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
   223
    assert(hr->in_collection_set(), "bad CS");
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
   224
27009
e7e723732b6b 8058298: Separate heap region iterator claim values from the data structures iterated over
mlarsson
parents: 26846
diff changeset
   225
    if (_hrclaimer->claim_region(hr->hrm_index())) {
11451
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
   226
      if (hr->evacuation_failed()) {
11455
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 11451
diff changeset
   227
        hr->note_self_forwarding_removal_start(during_initial_mark,
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 11451
diff changeset
   228
                                               during_conc_mark);
35851
5451ff646f8e 8147814: Move verification code out of g1collectedheap
david
parents: 34614
diff changeset
   229
        _g1h->verifier()->check_bitmaps("Self-Forwarding Ptr Removal", hr);
11451
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
   230
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
   231
        hr->reset_bot();
31631
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   232
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   233
        size_t live_bytes = remove_self_forward_ptr_by_walking_hr(hr, during_initial_mark);
11451
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
   234
26422
4ee5901e205e 8048268: G1 Code Root Migration performs poorly
mgerdin
parents: 25492
diff changeset
   235
        hr->rem_set()->clean_strong_code_roots(hr);
4ee5901e205e 8048268: G1 Code Root Migration performs poorly
mgerdin
parents: 25492
diff changeset
   236
11455
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 11451
diff changeset
   237
        hr->note_self_forwarding_removal_end(during_initial_mark,
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 11451
diff changeset
   238
                                             during_conc_mark,
31631
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   239
                                             live_bytes);
11451
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
   240
      }
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
   241
    }
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
   242
    return false;
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
   243
  }
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
   244
};
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
   245
31631
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   246
G1ParRemoveSelfForwardPtrsTask::G1ParRemoveSelfForwardPtrsTask() :
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   247
  AbstractGangTask("G1 Remove Self-forwarding Pointers"),
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   248
  _g1h(G1CollectedHeap::heap()),
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   249
  _hrclaimer(_g1h->workers()->active_workers()) { }
11451
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
   250
29080
a16b0d9a08f5 8072910: Move the g1EvacFailure.hpp implementation to g1EvacFailure.cpp
stefank
parents: 27009
diff changeset
   251
void G1ParRemoveSelfForwardPtrsTask::work(uint worker_id) {
31631
4536cf7bc89f 8129558: Coalesce dead objects during removal of self-forwarded pointers
tschatzl
parents: 31346
diff changeset
   252
  RemoveSelfForwardPtrHRClosure rsfp_cl(worker_id, &_hrclaimer);
11451
9c9aa4d85660 7121496: G1: do the per-region evacuation failure handling work in parallel
johnc
parents:
diff changeset
   253
39698
4016de4e596b 8159978: Use an array to store the collection set regions instead of linking through regions
tschatzl
parents: 38081
diff changeset
   254
  _g1h->collection_set_iterate_from(&rsfp_cl, worker_id);
29080
a16b0d9a08f5 8072910: Move the g1EvacFailure.hpp implementation to g1EvacFailure.cpp
stefank
parents: 27009
diff changeset
   255
}