hotspot/src/share/vm/gc/g1/g1ConcurrentMark.inline.hpp
author tschatzl
Thu, 15 Sep 2016 16:44:19 +0200
changeset 41176 ff9f64534cff
parent 39966 43c7f35b800e
child 42597 a9611bab7578
permissions -rw-r--r--
8159422: Very high Concurrent Mark mark stack contention Summary: Decrease contention on mark stack by splitting locks, and minimizing the amount of time these locks are held. Improve mark stack management. Reviewed-by: kbarrett, mgerdin, eosterlund
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
9994
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
     1
/*
35862
411842d0c882 8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents: 34282
diff changeset
     2
 * Copyright (c) 2001, 2016, Oracle and/or its affiliates. All rights reserved.
9994
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
     4
 *
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
     7
 * published by the Free Software Foundation.
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
     8
 *
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
    13
 * accompanied this code).
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
    14
 *
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
    18
 *
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
    21
 * questions.
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
    22
 *
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
    23
 */
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
    24
35943
e726308008c0 8148973: Rename g1/concurrentMark.{hpp,cpp,inline.hpp} to g1/g1ConcurrentMark.{hpp,cpp,inline.hpp}
ehelin
parents: 35862
diff changeset
    25
#ifndef SHARE_VM_GC_G1_G1CONCURRENTMARK_INLINE_HPP
e726308008c0 8148973: Rename g1/concurrentMark.{hpp,cpp,inline.hpp} to g1/g1ConcurrentMark.{hpp,cpp,inline.hpp}
ehelin
parents: 35862
diff changeset
    26
#define SHARE_VM_GC_G1_G1CONCURRENTMARK_INLINE_HPP
9994
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
    27
30764
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30577
diff changeset
    28
#include "gc/g1/g1CollectedHeap.inline.hpp"
35943
e726308008c0 8148973: Rename g1/concurrentMark.{hpp,cpp,inline.hpp} to g1/g1ConcurrentMark.{hpp,cpp,inline.hpp}
ehelin
parents: 35862
diff changeset
    29
#include "gc/g1/g1ConcurrentMark.hpp"
37414
2672ba9af0dc 8151386: Extract card live data out of G1ConcurrentMark
tschatzl
parents: 37413
diff changeset
    30
#include "gc/g1/suspendibleThreadSet.hpp"
30764
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30577
diff changeset
    31
#include "gc/shared/taskqueue.inline.hpp"
9994
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
    32
37413
2f71679d06dd 8077144: Concurrent mark initialization takes too long
tschatzl
parents: 35943
diff changeset
    33
inline bool G1ConcurrentMark::par_mark(oop obj) {
2f71679d06dd 8077144: Concurrent mark initialization takes too long
tschatzl
parents: 35943
diff changeset
    34
  return _nextMarkBitMap->parMark((HeapWord*)obj);
11584
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11583
diff changeset
    35
}
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11583
diff changeset
    36
35943
e726308008c0 8148973: Rename g1/concurrentMark.{hpp,cpp,inline.hpp} to g1/g1ConcurrentMark.{hpp,cpp,inline.hpp}
ehelin
parents: 35862
diff changeset
    37
inline bool G1CMBitMapRO::iterate(BitMapClosure* cl, MemRegion mr) {
11574
8a7fe61966c0 7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents: 11455
diff changeset
    38
  HeapWord* start_addr = MAX2(startWord(), mr.start());
8a7fe61966c0 7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents: 11455
diff changeset
    39
  HeapWord* end_addr = MIN2(endWord(), mr.end());
8a7fe61966c0 7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents: 11455
diff changeset
    40
8a7fe61966c0 7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents: 11455
diff changeset
    41
  if (end_addr > start_addr) {
8a7fe61966c0 7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents: 11455
diff changeset
    42
    // Right-open interval [start-offset, end-offset).
8a7fe61966c0 7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents: 11455
diff changeset
    43
    BitMap::idx_t start_offset = heapWordToOffset(start_addr);
8a7fe61966c0 7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents: 11455
diff changeset
    44
    BitMap::idx_t end_offset = heapWordToOffset(end_addr);
8a7fe61966c0 7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents: 11455
diff changeset
    45
8a7fe61966c0 7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents: 11455
diff changeset
    46
    start_offset = _bm.get_next_one_offset(start_offset, end_offset);
8a7fe61966c0 7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents: 11455
diff changeset
    47
    while (start_offset < end_offset) {
8a7fe61966c0 7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents: 11455
diff changeset
    48
      if (!cl->do_bit(start_offset)) {
8a7fe61966c0 7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents: 11455
diff changeset
    49
        return false;
8a7fe61966c0 7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents: 11455
diff changeset
    50
      }
15949
a55dea304e8c 8008079: G1: Add nextObject routine to CMBitMapRO and replace nextWord
tamao
parents: 14073
diff changeset
    51
      HeapWord* next_addr = MIN2(nextObject(offsetToHeapWord(start_offset)), end_addr);
11574
8a7fe61966c0 7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents: 11455
diff changeset
    52
      BitMap::idx_t next_offset = heapWordToOffset(next_addr);
8a7fe61966c0 7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents: 11455
diff changeset
    53
      start_offset = _bm.get_next_one_offset(next_offset, end_offset);
8a7fe61966c0 7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents: 11455
diff changeset
    54
    }
8a7fe61966c0 7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents: 11455
diff changeset
    55
  }
8a7fe61966c0 7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents: 11455
diff changeset
    56
  return true;
8a7fe61966c0 7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents: 11455
diff changeset
    57
}
8a7fe61966c0 7121547: G1: High number mispredicted branches while iterating over the marking bitmap
johnc
parents: 11455
diff changeset
    58
35862
411842d0c882 8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents: 34282
diff changeset
    59
// The argument addr should be the start address of a valid object
35943
e726308008c0 8148973: Rename g1/concurrentMark.{hpp,cpp,inline.hpp} to g1/g1ConcurrentMark.{hpp,cpp,inline.hpp}
ehelin
parents: 35862
diff changeset
    60
HeapWord* G1CMBitMapRO::nextObject(HeapWord* addr) {
35862
411842d0c882 8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents: 34282
diff changeset
    61
  oop obj = (oop) addr;
411842d0c882 8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents: 34282
diff changeset
    62
  HeapWord* res =  addr + obj->size();
411842d0c882 8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents: 34282
diff changeset
    63
  assert(offsetToHeapWord(heapWordToOffset(res)) == res, "sanity");
411842d0c882 8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents: 34282
diff changeset
    64
  return res;
411842d0c882 8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents: 34282
diff changeset
    65
}
411842d0c882 8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents: 34282
diff changeset
    66
26160
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 24424
diff changeset
    67
#define check_mark(addr)                                                       \
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 24424
diff changeset
    68
  assert(_bmStartWord <= (addr) && (addr) < (_bmStartWord + _bmWordSize),      \
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 24424
diff changeset
    69
         "outside underlying space?");                                         \
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 24424
diff changeset
    70
  assert(G1CollectedHeap::heap()->is_in_exact(addr),                           \
33105
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 32350
diff changeset
    71
         "Trying to access not available bitmap " PTR_FORMAT                   \
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 32350
diff changeset
    72
         " corresponding to " PTR_FORMAT " (%u)",                              \
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 32350
diff changeset
    73
         p2i(this), p2i(addr), G1CollectedHeap::heap()->addr_to_region(addr));
26160
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 24424
diff changeset
    74
35943
e726308008c0 8148973: Rename g1/concurrentMark.{hpp,cpp,inline.hpp} to g1/g1ConcurrentMark.{hpp,cpp,inline.hpp}
ehelin
parents: 35862
diff changeset
    75
inline void G1CMBitMap::mark(HeapWord* addr) {
26160
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 24424
diff changeset
    76
  check_mark(addr);
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 24424
diff changeset
    77
  _bm.set_bit(heapWordToOffset(addr));
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 24424
diff changeset
    78
}
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 24424
diff changeset
    79
35943
e726308008c0 8148973: Rename g1/concurrentMark.{hpp,cpp,inline.hpp} to g1/g1ConcurrentMark.{hpp,cpp,inline.hpp}
ehelin
parents: 35862
diff changeset
    80
inline void G1CMBitMap::clear(HeapWord* addr) {
26160
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 24424
diff changeset
    81
  check_mark(addr);
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 24424
diff changeset
    82
  _bm.clear_bit(heapWordToOffset(addr));
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 24424
diff changeset
    83
}
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 24424
diff changeset
    84
35943
e726308008c0 8148973: Rename g1/concurrentMark.{hpp,cpp,inline.hpp} to g1/g1ConcurrentMark.{hpp,cpp,inline.hpp}
ehelin
parents: 35862
diff changeset
    85
inline bool G1CMBitMap::parMark(HeapWord* addr) {
26160
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 24424
diff changeset
    86
  check_mark(addr);
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 24424
diff changeset
    87
  return _bm.par_set_bit(heapWordToOffset(addr));
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 24424
diff changeset
    88
}
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 24424
diff changeset
    89
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 24424
diff changeset
    90
#undef check_mark
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 24424
diff changeset
    91
41176
ff9f64534cff 8159422: Very high Concurrent Mark mark stack contention
tschatzl
parents: 39966
diff changeset
    92
#ifndef PRODUCT
31994
3721b7aa3a0d 8079082: VerifyNoCSetOopsClosure is derived twice from Closure
kbarrett
parents: 31592
diff changeset
    93
template<typename Fn>
41176
ff9f64534cff 8159422: Very high Concurrent Mark mark stack contention
tschatzl
parents: 39966
diff changeset
    94
inline void G1CMMarkStack::iterate(Fn fn) const {
39966
43c7f35b800e 8160897: Concurrent mark mark stack memory allocation leaks memory
tschatzl
parents: 37414
diff changeset
    95
  assert_at_safepoint(true);
41176
ff9f64534cff 8159422: Very high Concurrent Mark mark stack contention
tschatzl
parents: 39966
diff changeset
    96
ff9f64534cff 8159422: Very high Concurrent Mark mark stack contention
tschatzl
parents: 39966
diff changeset
    97
  size_t num_chunks = 0;
ff9f64534cff 8159422: Very high Concurrent Mark mark stack contention
tschatzl
parents: 39966
diff changeset
    98
ff9f64534cff 8159422: Very high Concurrent Mark mark stack contention
tschatzl
parents: 39966
diff changeset
    99
  OopChunk* cur = _chunk_list;
ff9f64534cff 8159422: Very high Concurrent Mark mark stack contention
tschatzl
parents: 39966
diff changeset
   100
  while (cur != NULL) {
ff9f64534cff 8159422: Very high Concurrent Mark mark stack contention
tschatzl
parents: 39966
diff changeset
   101
    guarantee(num_chunks <= _chunks_in_chunk_list, "Found " SIZE_FORMAT " oop chunks which is more than there should be", num_chunks);
ff9f64534cff 8159422: Very high Concurrent Mark mark stack contention
tschatzl
parents: 39966
diff changeset
   102
ff9f64534cff 8159422: Very high Concurrent Mark mark stack contention
tschatzl
parents: 39966
diff changeset
   103
    for (size_t i = 0; i < OopsPerChunk; ++i) {
ff9f64534cff 8159422: Very high Concurrent Mark mark stack contention
tschatzl
parents: 39966
diff changeset
   104
      if (cur->data[i] == NULL) {
ff9f64534cff 8159422: Very high Concurrent Mark mark stack contention
tschatzl
parents: 39966
diff changeset
   105
        break;
ff9f64534cff 8159422: Very high Concurrent Mark mark stack contention
tschatzl
parents: 39966
diff changeset
   106
      }
ff9f64534cff 8159422: Very high Concurrent Mark mark stack contention
tschatzl
parents: 39966
diff changeset
   107
      fn(cur->data[i]);
ff9f64534cff 8159422: Very high Concurrent Mark mark stack contention
tschatzl
parents: 39966
diff changeset
   108
    }
ff9f64534cff 8159422: Very high Concurrent Mark mark stack contention
tschatzl
parents: 39966
diff changeset
   109
    cur = cur->next;
ff9f64534cff 8159422: Very high Concurrent Mark mark stack contention
tschatzl
parents: 39966
diff changeset
   110
    num_chunks++;
31994
3721b7aa3a0d 8079082: VerifyNoCSetOopsClosure is derived twice from Closure
kbarrett
parents: 31592
diff changeset
   111
  }
3721b7aa3a0d 8079082: VerifyNoCSetOopsClosure is derived twice from Closure
kbarrett
parents: 31592
diff changeset
   112
}
41176
ff9f64534cff 8159422: Very high Concurrent Mark mark stack contention
tschatzl
parents: 39966
diff changeset
   113
#endif
31994
3721b7aa3a0d 8079082: VerifyNoCSetOopsClosure is derived twice from Closure
kbarrett
parents: 31592
diff changeset
   114
32350
fec57eb3f599 8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents: 31994
diff changeset
   115
// It scans an object and visits its children.
35943
e726308008c0 8148973: Rename g1/concurrentMark.{hpp,cpp,inline.hpp} to g1/g1ConcurrentMark.{hpp,cpp,inline.hpp}
ehelin
parents: 35862
diff changeset
   116
inline void G1CMTask::scan_object(oop obj) { process_grey_object<true>(obj); }
32350
fec57eb3f599 8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents: 31994
diff changeset
   117
35943
e726308008c0 8148973: Rename g1/concurrentMark.{hpp,cpp,inline.hpp} to g1/g1ConcurrentMark.{hpp,cpp,inline.hpp}
ehelin
parents: 35862
diff changeset
   118
inline void G1CMTask::push(oop obj) {
9994
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
   119
  HeapWord* objAddr = (HeapWord*) obj;
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
   120
  assert(_g1h->is_in_g1_reserved(objAddr), "invariant");
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
   121
  assert(!_g1h->is_on_master_free_list(
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
   122
              _g1h->heap_region_containing((HeapWord*) objAddr)), "invariant");
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
   123
  assert(!_g1h->is_obj_ill(obj), "invariant");
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
   124
  assert(_nextMarkBitMap->isMarked(objAddr), "invariant");
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
   125
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
   126
  if (!_task_queue->push(obj)) {
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
   127
    // The local task queue looks full. We need to push some entries
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
   128
    // to the global stack.
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
   129
    move_entries_to_global_stack();
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
   130
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
   131
    // this should succeed since, even if we overflow the global
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
   132
    // stack, we should have definitely removed some entries from the
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
   133
    // local queue. So, there must be space on it.
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
   134
    bool success = _task_queue->push(obj);
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
   135
    assert(success, "invariant");
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
   136
  }
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
   137
}
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
   138
35943
e726308008c0 8148973: Rename g1/concurrentMark.{hpp,cpp,inline.hpp} to g1/g1ConcurrentMark.{hpp,cpp,inline.hpp}
ehelin
parents: 35862
diff changeset
   139
inline bool G1CMTask::is_below_finger(oop obj, HeapWord* global_finger) const {
30577
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30566
diff changeset
   140
  // If obj is above the global finger, then the mark bitmap scan
30162
ba9738750a74 8076265: Simplify deal_with_reference
kbarrett
parents: 29472
diff changeset
   141
  // will find it later, and no push is needed.  Similarly, if we have
30577
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30566
diff changeset
   142
  // a current region and obj is between the local finger and the
30162
ba9738750a74 8076265: Simplify deal_with_reference
kbarrett
parents: 29472
diff changeset
   143
  // end of the current region, then no push is needed.  The tradeoff
ba9738750a74 8076265: Simplify deal_with_reference
kbarrett
parents: 29472
diff changeset
   144
  // of checking both vs only checking the global finger is that the
ba9738750a74 8076265: Simplify deal_with_reference
kbarrett
parents: 29472
diff changeset
   145
  // local check will be more accurate and so result in fewer pushes,
ba9738750a74 8076265: Simplify deal_with_reference
kbarrett
parents: 29472
diff changeset
   146
  // but may also be a little slower.
30577
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30566
diff changeset
   147
  HeapWord* objAddr = (HeapWord*)obj;
30162
ba9738750a74 8076265: Simplify deal_with_reference
kbarrett
parents: 29472
diff changeset
   148
  if (_finger != NULL) {
ba9738750a74 8076265: Simplify deal_with_reference
kbarrett
parents: 29472
diff changeset
   149
    // We have a current region.
9994
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
   150
30162
ba9738750a74 8076265: Simplify deal_with_reference
kbarrett
parents: 29472
diff changeset
   151
    // Finger and region values are all NULL or all non-NULL.  We
ba9738750a74 8076265: Simplify deal_with_reference
kbarrett
parents: 29472
diff changeset
   152
    // use _finger to check since we immediately use its value.
ba9738750a74 8076265: Simplify deal_with_reference
kbarrett
parents: 29472
diff changeset
   153
    assert(_curr_region != NULL, "invariant");
ba9738750a74 8076265: Simplify deal_with_reference
kbarrett
parents: 29472
diff changeset
   154
    assert(_region_limit != NULL, "invariant");
ba9738750a74 8076265: Simplify deal_with_reference
kbarrett
parents: 29472
diff changeset
   155
    assert(_region_limit <= global_finger, "invariant");
ba9738750a74 8076265: Simplify deal_with_reference
kbarrett
parents: 29472
diff changeset
   156
30577
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30566
diff changeset
   157
    // True if obj is less than the local finger, or is between
30162
ba9738750a74 8076265: Simplify deal_with_reference
kbarrett
parents: 29472
diff changeset
   158
    // the region limit and the global finger.
ba9738750a74 8076265: Simplify deal_with_reference
kbarrett
parents: 29472
diff changeset
   159
    if (objAddr < _finger) {
ba9738750a74 8076265: Simplify deal_with_reference
kbarrett
parents: 29472
diff changeset
   160
      return true;
ba9738750a74 8076265: Simplify deal_with_reference
kbarrett
parents: 29472
diff changeset
   161
    } else if (objAddr < _region_limit) {
ba9738750a74 8076265: Simplify deal_with_reference
kbarrett
parents: 29472
diff changeset
   162
      return false;
ba9738750a74 8076265: Simplify deal_with_reference
kbarrett
parents: 29472
diff changeset
   163
    } // Else check global finger.
ba9738750a74 8076265: Simplify deal_with_reference
kbarrett
parents: 29472
diff changeset
   164
  }
ba9738750a74 8076265: Simplify deal_with_reference
kbarrett
parents: 29472
diff changeset
   165
  // Check global finger.
ba9738750a74 8076265: Simplify deal_with_reference
kbarrett
parents: 29472
diff changeset
   166
  return objAddr < global_finger;
ba9738750a74 8076265: Simplify deal_with_reference
kbarrett
parents: 29472
diff changeset
   167
}
9994
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
   168
32350
fec57eb3f599 8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents: 31994
diff changeset
   169
template<bool scan>
35943
e726308008c0 8148973: Rename g1/concurrentMark.{hpp,cpp,inline.hpp} to g1/g1ConcurrentMark.{hpp,cpp,inline.hpp}
ehelin
parents: 35862
diff changeset
   170
inline void G1CMTask::process_grey_object(oop obj) {
32350
fec57eb3f599 8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents: 31994
diff changeset
   171
  assert(scan || obj->is_typeArray(), "Skipping scan of grey non-typeArray");
fec57eb3f599 8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents: 31994
diff changeset
   172
  assert(_nextMarkBitMap->isMarked((HeapWord*) obj), "invariant");
fec57eb3f599 8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents: 31994
diff changeset
   173
fec57eb3f599 8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents: 31994
diff changeset
   174
  size_t obj_size = obj->size();
fec57eb3f599 8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents: 31994
diff changeset
   175
  _words_scanned += obj_size;
fec57eb3f599 8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents: 31994
diff changeset
   176
fec57eb3f599 8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents: 31994
diff changeset
   177
  if (scan) {
fec57eb3f599 8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents: 31994
diff changeset
   178
    obj->oop_iterate(_cm_oop_closure);
fec57eb3f599 8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents: 31994
diff changeset
   179
  }
fec57eb3f599 8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents: 31994
diff changeset
   180
  check_limits();
fec57eb3f599 8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents: 31994
diff changeset
   181
}
fec57eb3f599 8133121: Move implementation of process_grey_object to concurrentMark.inline.hpp
asiebenborn
parents: 31994
diff changeset
   182
37413
2f71679d06dd 8077144: Concurrent mark initialization takes too long
tschatzl
parents: 35943
diff changeset
   183
inline void G1CMTask::make_reference_grey(oop obj) {
2f71679d06dd 8077144: Concurrent mark initialization takes too long
tschatzl
parents: 35943
diff changeset
   184
  if (_cm->par_mark(obj)) {
30577
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30566
diff changeset
   185
    // No OrderAccess:store_load() is needed. It is implicit in the
35943
e726308008c0 8148973: Rename g1/concurrentMark.{hpp,cpp,inline.hpp} to g1/g1ConcurrentMark.{hpp,cpp,inline.hpp}
ehelin
parents: 35862
diff changeset
   186
    // CAS done in G1CMBitMap::parMark() call in the routine above.
30577
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30566
diff changeset
   187
    HeapWord* global_finger = _cm->finger();
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30566
diff changeset
   188
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30566
diff changeset
   189
    // We only need to push a newly grey object on the mark
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30566
diff changeset
   190
    // stack if it is in a section of memory the mark bitmap
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30566
diff changeset
   191
    // scan has already examined.  Mark bitmap scanning
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30566
diff changeset
   192
    // maintains progress "fingers" for determining that.
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30566
diff changeset
   193
    //
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30566
diff changeset
   194
    // Notice that the global finger might be moving forward
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30566
diff changeset
   195
    // concurrently. This is not a problem. In the worst case, we
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30566
diff changeset
   196
    // mark the object while it is above the global finger and, by
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30566
diff changeset
   197
    // the time we read the global finger, it has moved forward
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30566
diff changeset
   198
    // past this object. In this case, the object will probably
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30566
diff changeset
   199
    // be visited when a task is scanning the region and will also
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30566
diff changeset
   200
    // be pushed on the stack. So, some duplicate work, but no
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30566
diff changeset
   201
    // correctness problems.
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30566
diff changeset
   202
    if (is_below_finger(obj, global_finger)) {
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30566
diff changeset
   203
      if (obj->is_typeArray()) {
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30566
diff changeset
   204
        // Immediately process arrays of primitive types, rather
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30566
diff changeset
   205
        // than pushing on the mark stack.  This keeps us from
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30566
diff changeset
   206
        // adding humongous objects to the mark stack that might
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30566
diff changeset
   207
        // be reclaimed before the entry is processed - see
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30566
diff changeset
   208
        // selection of candidates for eager reclaim of humongous
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30566
diff changeset
   209
        // objects.  The cost of the additional type test is
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30566
diff changeset
   210
        // mitigated by avoiding a trip through the mark stack,
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30566
diff changeset
   211
        // by only doing a bookkeeping update and avoiding the
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30566
diff changeset
   212
        // actual scan of the object - a typeArray contains no
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30566
diff changeset
   213
        // references, and the metadata is built-in.
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30566
diff changeset
   214
        process_grey_object<false>(obj);
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30566
diff changeset
   215
      } else {
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30566
diff changeset
   216
        push(obj);
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30566
diff changeset
   217
      }
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30566
diff changeset
   218
    }
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30566
diff changeset
   219
  }
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30566
diff changeset
   220
}
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30566
diff changeset
   221
35943
e726308008c0 8148973: Rename g1/concurrentMark.{hpp,cpp,inline.hpp} to g1/g1ConcurrentMark.{hpp,cpp,inline.hpp}
ehelin
parents: 35862
diff changeset
   222
inline void G1CMTask::deal_with_reference(oop obj) {
30577
de9fa7ccc1bc 8075215: SATB buffer processing found reclaimed humongous object
kbarrett
parents: 30566
diff changeset
   223
  increment_refs_reached();
9994
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
   224
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
   225
  HeapWord* objAddr = (HeapWord*) obj;
33105
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 32350
diff changeset
   226
  assert(obj->is_oop_or_null(true /* ignore mark word */), "Expected an oop or NULL at " PTR_FORMAT, p2i(obj));
11583
83a7383de44c 6484965: G1: piggy-back liveness accounting phase on marking
johnc
parents: 11574
diff changeset
   227
  if (_g1h->is_in_g1_reserved(objAddr)) {
9994
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
   228
    assert(obj != NULL, "null check is implicit");
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
   229
    if (!_nextMarkBitMap->isMarked(objAddr)) {
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
   230
      // Only get the containing region if the object is not marked on the
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
   231
      // bitmap (otherwise, it's a waste of time since we won't do
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
   232
      // anything with it).
33786
ac8da6513351 8139867: Change how startsHumongous and continuesHumongous regions work in G1.
david
parents: 33740
diff changeset
   233
      HeapRegion* hr = _g1h->heap_region_containing(obj);
9994
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
   234
      if (!hr->obj_allocated_since_next_marking(obj)) {
37413
2f71679d06dd 8077144: Concurrent mark initialization takes too long
tschatzl
parents: 35943
diff changeset
   235
        make_reference_grey(obj);
9994
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
   236
      }
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
   237
    }
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
   238
  }
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
   239
}
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents:
diff changeset
   240
35943
e726308008c0 8148973: Rename g1/concurrentMark.{hpp,cpp,inline.hpp} to g1/g1ConcurrentMark.{hpp,cpp,inline.hpp}
ehelin
parents: 35862
diff changeset
   241
inline void G1ConcurrentMark::markPrev(oop p) {
11455
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 9999
diff changeset
   242
  assert(!_prevMarkBitMap->isMarked((HeapWord*) p), "sanity");
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 9999
diff changeset
   243
  // Note we are overriding the read-only view of the prev map here, via
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 9999
diff changeset
   244
  // the cast.
35943
e726308008c0 8148973: Rename g1/concurrentMark.{hpp,cpp,inline.hpp} to g1/g1ConcurrentMark.{hpp,cpp,inline.hpp}
ehelin
parents: 35862
diff changeset
   245
  ((G1CMBitMap*)_prevMarkBitMap)->mark((HeapWord*) p);
11455
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 9999
diff changeset
   246
}
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 9999
diff changeset
   247
35943
e726308008c0 8148973: Rename g1/concurrentMark.{hpp,cpp,inline.hpp} to g1/g1ConcurrentMark.{hpp,cpp,inline.hpp}
ehelin
parents: 35862
diff changeset
   248
bool G1ConcurrentMark::isPrevMarked(oop p) const {
35862
411842d0c882 8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents: 34282
diff changeset
   249
  assert(p != NULL && p->is_oop(), "expected an oop");
411842d0c882 8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents: 34282
diff changeset
   250
  HeapWord* addr = (HeapWord*)p;
411842d0c882 8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents: 34282
diff changeset
   251
  assert(addr >= _prevMarkBitMap->startWord() ||
411842d0c882 8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents: 34282
diff changeset
   252
         addr < _prevMarkBitMap->endWord(), "in a region");
411842d0c882 8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents: 34282
diff changeset
   253
411842d0c882 8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents: 34282
diff changeset
   254
  return _prevMarkBitMap->isMarked(addr);
411842d0c882 8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents: 34282
diff changeset
   255
}
411842d0c882 8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents: 34282
diff changeset
   256
37413
2f71679d06dd 8077144: Concurrent mark initialization takes too long
tschatzl
parents: 35943
diff changeset
   257
inline void G1ConcurrentMark::grayRoot(oop obj, HeapRegion* hr) {
11584
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11583
diff changeset
   258
  assert(obj != NULL, "pre-condition");
11455
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 9999
diff changeset
   259
  HeapWord* addr = (HeapWord*) obj;
11584
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11583
diff changeset
   260
  if (hr == NULL) {
33786
ac8da6513351 8139867: Change how startsHumongous and continuesHumongous regions work in G1.
david
parents: 33740
diff changeset
   261
    hr = _g1h->heap_region_containing(addr);
11584
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11583
diff changeset
   262
  } else {
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11583
diff changeset
   263
    assert(hr->is_in(addr), "pre-condition");
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11583
diff changeset
   264
  }
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11583
diff changeset
   265
  assert(hr != NULL, "sanity");
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11583
diff changeset
   266
  // Given that we're looking for a region that contains an object
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11583
diff changeset
   267
  // header it's impossible to get back a HC region.
26846
7d4376f8560e 8058495: G1: normalize names for isHumongous() and friends
tonyp
parents: 26831
diff changeset
   268
  assert(!hr->is_continues_humongous(), "sanity");
11455
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 9999
diff changeset
   269
11584
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11583
diff changeset
   270
  if (addr < hr->next_top_at_mark_start()) {
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11583
diff changeset
   271
    if (!_nextMarkBitMap->isMarked(addr)) {
37413
2f71679d06dd 8077144: Concurrent mark initialization takes too long
tschatzl
parents: 35943
diff changeset
   272
      par_mark(obj);
11584
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11583
diff changeset
   273
    }
11455
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 9999
diff changeset
   274
  }
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 9999
diff changeset
   275
}
a6ab3d8b9a4c 6888336: G1: avoid explicitly marking and pushing objects in survivor spaces
tonyp
parents: 9999
diff changeset
   276
37414
2672ba9af0dc 8151386: Extract card live data out of G1ConcurrentMark
tschatzl
parents: 37413
diff changeset
   277
inline bool G1ConcurrentMark::do_yield_check() {
2672ba9af0dc 8151386: Extract card live data out of G1ConcurrentMark
tschatzl
parents: 37413
diff changeset
   278
  if (SuspendibleThreadSet::should_yield()) {
2672ba9af0dc 8151386: Extract card live data out of G1ConcurrentMark
tschatzl
parents: 37413
diff changeset
   279
    SuspendibleThreadSet::yield();
2672ba9af0dc 8151386: Extract card live data out of G1ConcurrentMark
tschatzl
parents: 37413
diff changeset
   280
    return true;
2672ba9af0dc 8151386: Extract card live data out of G1ConcurrentMark
tschatzl
parents: 37413
diff changeset
   281
  } else {
2672ba9af0dc 8151386: Extract card live data out of G1ConcurrentMark
tschatzl
parents: 37413
diff changeset
   282
    return false;
2672ba9af0dc 8151386: Extract card live data out of G1ConcurrentMark
tschatzl
parents: 37413
diff changeset
   283
  }
2672ba9af0dc 8151386: Extract card live data out of G1ConcurrentMark
tschatzl
parents: 37413
diff changeset
   284
}
2672ba9af0dc 8151386: Extract card live data out of G1ConcurrentMark
tschatzl
parents: 37413
diff changeset
   285
35943
e726308008c0 8148973: Rename g1/concurrentMark.{hpp,cpp,inline.hpp} to g1/g1ConcurrentMark.{hpp,cpp,inline.hpp}
ehelin
parents: 35862
diff changeset
   286
#endif // SHARE_VM_GC_G1_G1CONCURRENTMARK_INLINE_HPP