src/hotspot/share/gc/shared/cardTableRS.cpp
author chegar
Thu, 17 Oct 2019 20:54:25 +0100
branchdatagramsocketimpl-branch
changeset 58679 9c3209ff7550
parent 58678 9cf78a70fa4f
parent 58084 cddef3bde924
permissions -rw-r--r--
datagramsocketimpl-branch: merge with default
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     1
/*
58083
9046db64ca39 8227168: Cleanup usage of NEW_C_HEAP_ARRAY
lkorinth
parents: 54110
diff changeset
     2
 * Copyright (c) 2001, 2019, Oracle and/or its affiliates. All rights reserved.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
489c9b5090e2 Initial load
duke
parents:
diff changeset
     4
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
489c9b5090e2 Initial load
duke
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
489c9b5090e2 Initial load
duke
parents:
diff changeset
     7
 * published by the Free Software Foundation.
489c9b5090e2 Initial load
duke
parents:
diff changeset
     8
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
489c9b5090e2 Initial load
duke
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
489c9b5090e2 Initial load
duke
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
489c9b5090e2 Initial load
duke
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
489c9b5090e2 Initial load
duke
parents:
diff changeset
    13
 * accompanied this code).
489c9b5090e2 Initial load
duke
parents:
diff changeset
    14
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
489c9b5090e2 Initial load
duke
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
489c9b5090e2 Initial load
duke
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    18
 *
5547
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 3261
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 3261
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 3261
diff changeset
    21
 * questions.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    22
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
    23
 */
489c9b5090e2 Initial load
duke
parents:
diff changeset
    24
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    25
#include "precompiled.hpp"
51959
db0c3952de52 8209645: Split ClassLoaderData and ClassLoaderDataGraph into separate files
coleenp
parents: 50800
diff changeset
    26
#include "classfile/classLoaderDataGraph.hpp"
30764
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30173
diff changeset
    27
#include "gc/shared/cardTableRS.hpp"
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30173
diff changeset
    28
#include "gc/shared/genCollectedHeap.hpp"
49982
9042ffe5b7fe 8200729: Conditional compilation of GCs
stefank
parents: 49733
diff changeset
    29
#include "gc/shared/genOopClosures.hpp"
30764
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30173
diff changeset
    30
#include "gc/shared/generation.hpp"
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30173
diff changeset
    31
#include "gc/shared/space.inline.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    32
#include "memory/allocation.inline.hpp"
50752
9d62da00bf15 8204540: Automatic oop closure devirtualization
stefank
parents: 49982
diff changeset
    33
#include "memory/iterator.inline.hpp"
49592
77fb0be7d19f 8199946: Move load/store and encode/decode out of oopDesc
stefank
parents: 49455
diff changeset
    34
#include "oops/access.inline.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    35
#include "oops/oop.inline.hpp"
40655
9f644073d3a0 8157907: Incorrect inclusion of atomic.hpp instead of atomic.inline.hpp
dholmes
parents: 37073
diff changeset
    36
#include "runtime/atomic.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    37
#include "runtime/java.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    38
#include "runtime/os.hpp"
15482
470d0b0c09f1 8005915: Unify SERIALGC and INCLUDE_ALTERNATE_GCS
jprovino
parents: 13728
diff changeset
    39
#include "utilities/macros.hpp"
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    40
47580
96392e113a0a 8186777: Make Klass::_java_mirror an OopHandle
coleenp
parents: 47216
diff changeset
    41
class HasAccumulatedModifiedOopsClosure : public CLDClosure {
33212
906b3d079b13 8139434: Unify GenRemSet and CardTableRS
david
parents: 33105
diff changeset
    42
  bool _found;
906b3d079b13 8139434: Unify GenRemSet and CardTableRS
david
parents: 33105
diff changeset
    43
 public:
906b3d079b13 8139434: Unify GenRemSet and CardTableRS
david
parents: 33105
diff changeset
    44
  HasAccumulatedModifiedOopsClosure() : _found(false) {}
47580
96392e113a0a 8186777: Make Klass::_java_mirror an OopHandle
coleenp
parents: 47216
diff changeset
    45
  void do_cld(ClassLoaderData* cld) {
33212
906b3d079b13 8139434: Unify GenRemSet and CardTableRS
david
parents: 33105
diff changeset
    46
    if (_found) {
906b3d079b13 8139434: Unify GenRemSet and CardTableRS
david
parents: 33105
diff changeset
    47
      return;
906b3d079b13 8139434: Unify GenRemSet and CardTableRS
david
parents: 33105
diff changeset
    48
    }
906b3d079b13 8139434: Unify GenRemSet and CardTableRS
david
parents: 33105
diff changeset
    49
47580
96392e113a0a 8186777: Make Klass::_java_mirror an OopHandle
coleenp
parents: 47216
diff changeset
    50
    if (cld->has_accumulated_modified_oops()) {
33212
906b3d079b13 8139434: Unify GenRemSet and CardTableRS
david
parents: 33105
diff changeset
    51
      _found = true;
906b3d079b13 8139434: Unify GenRemSet and CardTableRS
david
parents: 33105
diff changeset
    52
    }
906b3d079b13 8139434: Unify GenRemSet and CardTableRS
david
parents: 33105
diff changeset
    53
  }
906b3d079b13 8139434: Unify GenRemSet and CardTableRS
david
parents: 33105
diff changeset
    54
  bool found() {
906b3d079b13 8139434: Unify GenRemSet and CardTableRS
david
parents: 33105
diff changeset
    55
    return _found;
906b3d079b13 8139434: Unify GenRemSet and CardTableRS
david
parents: 33105
diff changeset
    56
  }
906b3d079b13 8139434: Unify GenRemSet and CardTableRS
david
parents: 33105
diff changeset
    57
};
906b3d079b13 8139434: Unify GenRemSet and CardTableRS
david
parents: 33105
diff changeset
    58
47580
96392e113a0a 8186777: Make Klass::_java_mirror an OopHandle
coleenp
parents: 47216
diff changeset
    59
bool CLDRemSet::mod_union_is_clear() {
33212
906b3d079b13 8139434: Unify GenRemSet and CardTableRS
david
parents: 33105
diff changeset
    60
  HasAccumulatedModifiedOopsClosure closure;
47580
96392e113a0a 8186777: Make Klass::_java_mirror an OopHandle
coleenp
parents: 47216
diff changeset
    61
  ClassLoaderDataGraph::cld_do(&closure);
33212
906b3d079b13 8139434: Unify GenRemSet and CardTableRS
david
parents: 33105
diff changeset
    62
906b3d079b13 8139434: Unify GenRemSet and CardTableRS
david
parents: 33105
diff changeset
    63
  return !closure.found();
906b3d079b13 8139434: Unify GenRemSet and CardTableRS
david
parents: 33105
diff changeset
    64
}
906b3d079b13 8139434: Unify GenRemSet and CardTableRS
david
parents: 33105
diff changeset
    65
906b3d079b13 8139434: Unify GenRemSet and CardTableRS
david
parents: 33105
diff changeset
    66
47580
96392e113a0a 8186777: Make Klass::_java_mirror an OopHandle
coleenp
parents: 47216
diff changeset
    67
class ClearCLDModUnionClosure : public CLDClosure {
33212
906b3d079b13 8139434: Unify GenRemSet and CardTableRS
david
parents: 33105
diff changeset
    68
 public:
47580
96392e113a0a 8186777: Make Klass::_java_mirror an OopHandle
coleenp
parents: 47216
diff changeset
    69
  void do_cld(ClassLoaderData* cld) {
96392e113a0a 8186777: Make Klass::_java_mirror an OopHandle
coleenp
parents: 47216
diff changeset
    70
    if (cld->has_accumulated_modified_oops()) {
96392e113a0a 8186777: Make Klass::_java_mirror an OopHandle
coleenp
parents: 47216
diff changeset
    71
      cld->clear_accumulated_modified_oops();
33212
906b3d079b13 8139434: Unify GenRemSet and CardTableRS
david
parents: 33105
diff changeset
    72
    }
906b3d079b13 8139434: Unify GenRemSet and CardTableRS
david
parents: 33105
diff changeset
    73
  }
906b3d079b13 8139434: Unify GenRemSet and CardTableRS
david
parents: 33105
diff changeset
    74
};
906b3d079b13 8139434: Unify GenRemSet and CardTableRS
david
parents: 33105
diff changeset
    75
47580
96392e113a0a 8186777: Make Klass::_java_mirror an OopHandle
coleenp
parents: 47216
diff changeset
    76
void CLDRemSet::clear_mod_union() {
96392e113a0a 8186777: Make Klass::_java_mirror an OopHandle
coleenp
parents: 47216
diff changeset
    77
  ClearCLDModUnionClosure closure;
96392e113a0a 8186777: Make Klass::_java_mirror an OopHandle
coleenp
parents: 47216
diff changeset
    78
  ClassLoaderDataGraph::cld_do(&closure);
33212
906b3d079b13 8139434: Unify GenRemSet and CardTableRS
david
parents: 33105
diff changeset
    79
}
906b3d079b13 8139434: Unify GenRemSet and CardTableRS
david
parents: 33105
diff changeset
    80
54110
f4f0dce5d0bb 8220301: Remove jbyte use in CardTable
tschatzl
parents: 51959
diff changeset
    81
CardTable::CardValue CardTableRS::find_unused_youngergenP_card_value() {
f4f0dce5d0bb 8220301: Remove jbyte use in CardTable
tschatzl
parents: 51959
diff changeset
    82
  for (CardValue v = youngergenP1_card;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    83
       v < cur_youngergen_and_prev_nonclean_card;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    84
       v++) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    85
    bool seen = false;
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents: 360
diff changeset
    86
    for (int g = 0; g < _regions_to_iterate; g++) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    87
      if (_last_cur_val_in_gen[g] == v) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    88
        seen = true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    89
        break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    90
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    91
    }
32623
390a27af5657 8134626: Misc cleanups after generation array removal
jwilhelm
parents: 31964
diff changeset
    92
    if (!seen) {
390a27af5657 8134626: Misc cleanups after generation array removal
jwilhelm
parents: 31964
diff changeset
    93
      return v;
390a27af5657 8134626: Misc cleanups after generation array removal
jwilhelm
parents: 31964
diff changeset
    94
    }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    95
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    96
  ShouldNotReachHere();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    97
  return 0;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    98
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    99
489c9b5090e2 Initial load
duke
parents:
diff changeset
   100
void CardTableRS::prepare_for_younger_refs_iterate(bool parallel) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   101
  // Parallel or sequential, we must always set the prev to equal the
489c9b5090e2 Initial load
duke
parents:
diff changeset
   102
  // last one written.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   103
  if (parallel) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   104
    // Find a parallel value to be used next.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   105
    jbyte next_val = find_unused_youngergenP_card_value();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   106
    set_cur_youngergen_card_val(next_val);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   107
489c9b5090e2 Initial load
duke
parents:
diff changeset
   108
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   109
    // In an sequential traversal we will always write youngergen, so that
489c9b5090e2 Initial load
duke
parents:
diff changeset
   110
    // the inline barrier is  correct.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   111
    set_cur_youngergen_card_val(youngergen_card);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   112
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   113
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   114
489c9b5090e2 Initial load
duke
parents:
diff changeset
   115
void CardTableRS::younger_refs_iterate(Generation* g,
30870
3050fdcdc60b 8080112: Replace and remove the last usages of CollectedHeap::n_par_threads()
stefank
parents: 30764
diff changeset
   116
                                       OopsInGenClosure* blk,
3050fdcdc60b 8080112: Replace and remove the last usages of CollectedHeap::n_par_threads()
stefank
parents: 30764
diff changeset
   117
                                       uint n_threads) {
31358
693058672cc6 8077842: Remove the level parameter passed around in GenCollectedHeap
jwilhelm
parents: 30870
diff changeset
   118
  // The indexing in this array is slightly odd. We want to access
693058672cc6 8077842: Remove the level parameter passed around in GenCollectedHeap
jwilhelm
parents: 30870
diff changeset
   119
  // the old generation record here, which is at index 2.
693058672cc6 8077842: Remove the level parameter passed around in GenCollectedHeap
jwilhelm
parents: 30870
diff changeset
   120
  _last_cur_val_in_gen[2] = cur_youngergen_card_val();
30870
3050fdcdc60b 8080112: Replace and remove the last usages of CollectedHeap::n_par_threads()
stefank
parents: 30764
diff changeset
   121
  g->younger_refs_iterate(blk, n_threads);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   122
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   123
54110
f4f0dce5d0bb 8220301: Remove jbyte use in CardTable
tschatzl
parents: 51959
diff changeset
   124
inline bool ClearNoncleanCardWrapper::clear_card(CardValue* entry) {
9336
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   125
  if (_is_par) {
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   126
    return clear_card_parallel(entry);
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   127
  } else {
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   128
    return clear_card_serial(entry);
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   129
  }
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   130
}
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   131
54110
f4f0dce5d0bb 8220301: Remove jbyte use in CardTable
tschatzl
parents: 51959
diff changeset
   132
inline bool ClearNoncleanCardWrapper::clear_card_parallel(CardValue* entry) {
9336
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   133
  while (true) {
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   134
    // In the parallel case, we may have to do this several times.
54110
f4f0dce5d0bb 8220301: Remove jbyte use in CardTable
tschatzl
parents: 51959
diff changeset
   135
    CardValue entry_val = *entry;
9336
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   136
    assert(entry_val != CardTableRS::clean_card_val(),
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   137
           "We shouldn't be looking at clean cards, and this should "
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   138
           "be the only place they get cleaned.");
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   139
    if (CardTableRS::card_is_dirty_wrt_gen_iter(entry_val)
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   140
        || _ct->is_prev_youngergen_card_val(entry_val)) {
54110
f4f0dce5d0bb 8220301: Remove jbyte use in CardTable
tschatzl
parents: 51959
diff changeset
   141
      CardValue res =
9336
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   142
        Atomic::cmpxchg(CardTableRS::clean_card_val(), entry, entry_val);
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   143
      if (res == entry_val) {
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   144
        break;
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   145
      } else {
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   146
        assert(res == CardTableRS::cur_youngergen_and_prev_nonclean_card,
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   147
               "The CAS above should only fail if another thread did "
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   148
               "a GC write barrier.");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   149
      }
9336
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   150
    } else if (entry_val ==
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   151
               CardTableRS::cur_youngergen_and_prev_nonclean_card) {
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   152
      // Parallelism shouldn't matter in this case.  Only the thread
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   153
      // assigned to scan the card should change this value.
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   154
      *entry = _ct->cur_youngergen_card_val();
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   155
      break;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   156
    } else {
9336
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   157
      assert(entry_val == _ct->cur_youngergen_card_val(),
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   158
             "Should be the only possibility.");
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   159
      // In this case, the card was clean before, and become
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   160
      // cur_youngergen only because of processing of a promoted object.
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   161
      // We don't have to look at the card.
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   162
      return false;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   163
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   164
  }
9336
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   165
  return true;
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   166
}
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   167
9336
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   168
54110
f4f0dce5d0bb 8220301: Remove jbyte use in CardTable
tschatzl
parents: 51959
diff changeset
   169
inline bool ClearNoncleanCardWrapper::clear_card_serial(CardValue* entry) {
f4f0dce5d0bb 8220301: Remove jbyte use in CardTable
tschatzl
parents: 51959
diff changeset
   170
  CardValue entry_val = *entry;
9336
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   171
  assert(entry_val != CardTableRS::clean_card_val(),
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   172
         "We shouldn't be looking at clean cards, and this should "
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   173
         "be the only place they get cleaned.");
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   174
  assert(entry_val != CardTableRS::cur_youngergen_and_prev_nonclean_card,
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   175
         "This should be possible in the sequential case.");
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   176
  *entry = CardTableRS::clean_card_val();
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   177
  return true;
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   178
}
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   179
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   180
ClearNoncleanCardWrapper::ClearNoncleanCardWrapper(
30870
3050fdcdc60b 8080112: Replace and remove the last usages of CollectedHeap::n_par_threads()
stefank
parents: 30764
diff changeset
   181
  DirtyCardToOopClosure* dirty_card_closure, CardTableRS* ct, bool is_par) :
3050fdcdc60b 8080112: Replace and remove the last usages of CollectedHeap::n_par_threads()
stefank
parents: 30764
diff changeset
   182
    _dirty_card_closure(dirty_card_closure), _ct(ct), _is_par(is_par) {
9336
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   183
}
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   184
54110
f4f0dce5d0bb 8220301: Remove jbyte use in CardTable
tschatzl
parents: 51959
diff changeset
   185
bool ClearNoncleanCardWrapper::is_word_aligned(CardTable::CardValue* entry) {
12118
36458db4ccf5 7068625: Testing 8 bytes of card table entries at a time speeds up card-scanning
brutisso
parents: 11174
diff changeset
   186
  return (((intptr_t)entry) & (BytesPerWord-1)) == 0;
36458db4ccf5 7068625: Testing 8 bytes of card table entries at a time speeds up card-scanning
brutisso
parents: 11174
diff changeset
   187
}
36458db4ccf5 7068625: Testing 8 bytes of card table entries at a time speeds up card-scanning
brutisso
parents: 11174
diff changeset
   188
30165
c4ff729cb74a 8076614: Add comment to ClearNoncleanCardWrapper::do_MemRegion()
brutisso
parents: 30155
diff changeset
   189
// The regions are visited in *decreasing* address order.
c4ff729cb74a 8076614: Add comment to ClearNoncleanCardWrapper::do_MemRegion()
brutisso
parents: 30155
diff changeset
   190
// This order aids with imprecise card marking, where a dirty
c4ff729cb74a 8076614: Add comment to ClearNoncleanCardWrapper::do_MemRegion()
brutisso
parents: 30155
diff changeset
   191
// card may cause scanning, and summarization marking, of objects
c4ff729cb74a 8076614: Add comment to ClearNoncleanCardWrapper::do_MemRegion()
brutisso
parents: 30155
diff changeset
   192
// that extend onto subsequent cards.
9336
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   193
void ClearNoncleanCardWrapper::do_MemRegion(MemRegion mr) {
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   194
  assert(mr.word_size() > 0, "Error");
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   195
  assert(_ct->is_aligned(mr.start()), "mr.start() should be card aligned");
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   196
  // mr.end() may not necessarily be card aligned.
54110
f4f0dce5d0bb 8220301: Remove jbyte use in CardTable
tschatzl
parents: 51959
diff changeset
   197
  CardValue* cur_entry = _ct->byte_for(mr.last());
f4f0dce5d0bb 8220301: Remove jbyte use in CardTable
tschatzl
parents: 51959
diff changeset
   198
  const CardValue* limit = _ct->byte_for(mr.start());
9336
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   199
  HeapWord* end_of_non_clean = mr.end();
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   200
  HeapWord* start_of_non_clean = end_of_non_clean;
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   201
  while (cur_entry >= limit) {
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   202
    HeapWord* cur_hw = _ct->addr_for(cur_entry);
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   203
    if ((*cur_entry != CardTableRS::clean_card_val()) && clear_card(cur_entry)) {
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   204
      // Continue the dirty range by opening the
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   205
      // dirty window one card to the left.
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   206
      start_of_non_clean = cur_hw;
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   207
    } else {
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   208
      // We hit a "clean" card; process any non-empty
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   209
      // "dirty" range accumulated so far.
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   210
      if (start_of_non_clean < end_of_non_clean) {
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   211
        const MemRegion mrd(start_of_non_clean, end_of_non_clean);
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   212
        _dirty_card_closure->do_MemRegion(mrd);
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   213
      }
12118
36458db4ccf5 7068625: Testing 8 bytes of card table entries at a time speeds up card-scanning
brutisso
parents: 11174
diff changeset
   214
36458db4ccf5 7068625: Testing 8 bytes of card table entries at a time speeds up card-scanning
brutisso
parents: 11174
diff changeset
   215
      // fast forward through potential continuous whole-word range of clean cards beginning at a word-boundary
36458db4ccf5 7068625: Testing 8 bytes of card table entries at a time speeds up card-scanning
brutisso
parents: 11174
diff changeset
   216
      if (is_word_aligned(cur_entry)) {
54110
f4f0dce5d0bb 8220301: Remove jbyte use in CardTable
tschatzl
parents: 51959
diff changeset
   217
        CardValue* cur_row = cur_entry - BytesPerWord;
49164
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   218
        while (cur_row >= limit && *((intptr_t*)cur_row) ==  CardTableRS::clean_card_row_val()) {
12118
36458db4ccf5 7068625: Testing 8 bytes of card table entries at a time speeds up card-scanning
brutisso
parents: 11174
diff changeset
   219
          cur_row -= BytesPerWord;
36458db4ccf5 7068625: Testing 8 bytes of card table entries at a time speeds up card-scanning
brutisso
parents: 11174
diff changeset
   220
        }
36458db4ccf5 7068625: Testing 8 bytes of card table entries at a time speeds up card-scanning
brutisso
parents: 11174
diff changeset
   221
        cur_entry = cur_row + BytesPerWord;
36458db4ccf5 7068625: Testing 8 bytes of card table entries at a time speeds up card-scanning
brutisso
parents: 11174
diff changeset
   222
        cur_hw = _ct->addr_for(cur_entry);
36458db4ccf5 7068625: Testing 8 bytes of card table entries at a time speeds up card-scanning
brutisso
parents: 11174
diff changeset
   223
      }
36458db4ccf5 7068625: Testing 8 bytes of card table entries at a time speeds up card-scanning
brutisso
parents: 11174
diff changeset
   224
9336
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   225
      // Reset the dirty window, while continuing to look
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   226
      // for the next dirty card that will start a
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   227
      // new dirty window.
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   228
      end_of_non_clean = cur_hw;
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   229
      start_of_non_clean = cur_hw;
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   230
    }
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   231
    // Note that "cur_entry" leads "start_of_non_clean" in
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   232
    // its leftward excursion after this point
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   233
    // in the loop and, when we hit the left end of "mr",
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   234
    // will point off of the left end of the card-table
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   235
    // for "mr".
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   236
    cur_entry--;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   237
  }
9336
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   238
  // If the first card of "mr" was dirty, we will have
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   239
  // been left with a dirty window, co-initial with "mr",
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   240
  // which we now process.
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   241
  if (start_of_non_clean < end_of_non_clean) {
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   242
    const MemRegion mrd(start_of_non_clean, end_of_non_clean);
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   243
    _dirty_card_closure->do_MemRegion(mrd);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   244
  }
9336
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   245
}
413920193f83 7037276: Unnecessary double traversal of dirty card windows
ysr
parents: 9183
diff changeset
   246
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   247
// clean (by dirty->clean before) ==> cur_younger_gen
489c9b5090e2 Initial load
duke
parents:
diff changeset
   248
// dirty                          ==> cur_youngergen_and_prev_nonclean_card
489c9b5090e2 Initial load
duke
parents:
diff changeset
   249
// precleaned                     ==> cur_youngergen_and_prev_nonclean_card
489c9b5090e2 Initial load
duke
parents:
diff changeset
   250
// prev-younger-gen               ==> cur_youngergen_and_prev_nonclean_card
489c9b5090e2 Initial load
duke
parents:
diff changeset
   251
// cur-younger-gen                ==> cur_younger_gen
489c9b5090e2 Initial load
duke
parents:
diff changeset
   252
// cur_youngergen_and_prev_nonclean_card ==> no change.
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 179
diff changeset
   253
void CardTableRS::write_ref_field_gc_par(void* field, oop new_val) {
54110
f4f0dce5d0bb 8220301: Remove jbyte use in CardTable
tschatzl
parents: 51959
diff changeset
   254
  volatile CardValue* entry = byte_for(field);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   255
  do {
54110
f4f0dce5d0bb 8220301: Remove jbyte use in CardTable
tschatzl
parents: 51959
diff changeset
   256
    CardValue entry_val = *entry;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   257
    // We put this first because it's probably the most common case.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   258
    if (entry_val == clean_card_val()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   259
      // No threat of contention with cleaning threads.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   260
      *entry = cur_youngergen_card_val();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   261
      return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   262
    } else if (card_is_dirty_wrt_gen_iter(entry_val)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   263
               || is_prev_youngergen_card_val(entry_val)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   264
      // Mark it as both cur and prev youngergen; card cleaning thread will
489c9b5090e2 Initial load
duke
parents:
diff changeset
   265
      // eventually remove the previous stuff.
54110
f4f0dce5d0bb 8220301: Remove jbyte use in CardTable
tschatzl
parents: 51959
diff changeset
   266
      CardValue new_val = cur_youngergen_and_prev_nonclean_card;
f4f0dce5d0bb 8220301: Remove jbyte use in CardTable
tschatzl
parents: 51959
diff changeset
   267
      CardValue res = Atomic::cmpxchg(new_val, entry, entry_val);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   268
      // Did the CAS succeed?
489c9b5090e2 Initial load
duke
parents:
diff changeset
   269
      if (res == entry_val) return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   270
      // Otherwise, retry, to see the new value.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   271
      continue;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   272
    } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   273
      assert(entry_val == cur_youngergen_and_prev_nonclean_card
489c9b5090e2 Initial load
duke
parents:
diff changeset
   274
             || entry_val == cur_youngergen_card_val(),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   275
             "should be only possibilities.");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   276
      return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   277
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   278
  } while (true);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   279
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   280
489c9b5090e2 Initial load
duke
parents:
diff changeset
   281
void CardTableRS::younger_refs_in_space_iterate(Space* sp,
30870
3050fdcdc60b 8080112: Replace and remove the last usages of CollectedHeap::n_par_threads()
stefank
parents: 30764
diff changeset
   282
                                                OopsInGenClosure* cl,
3050fdcdc60b 8080112: Replace and remove the last usages of CollectedHeap::n_par_threads()
stefank
parents: 30764
diff changeset
   283
                                                uint n_threads) {
49733
6f0a3ea5ab75 8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents: 49592
diff changeset
   284
  verify_used_region_at_save_marks(sp);
6f0a3ea5ab75 8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents: 49592
diff changeset
   285
9342
456b8d0486b5 7039089: G1: changeset for 7037276 broke heap verification, and related cleanups
ysr
parents: 9336
diff changeset
   286
  const MemRegion urasm = sp->used_region_at_save_marks();
49733
6f0a3ea5ab75 8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents: 49592
diff changeset
   287
  non_clean_card_iterate_possibly_parallel(sp, urasm, cl, this, n_threads);
6f0a3ea5ab75 8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents: 49592
diff changeset
   288
}
6f0a3ea5ab75 8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents: 49592
diff changeset
   289
9342
456b8d0486b5 7039089: G1: changeset for 7037276 broke heap verification, and related cleanups
ysr
parents: 9336
diff changeset
   290
#ifdef ASSERT
49733
6f0a3ea5ab75 8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents: 49592
diff changeset
   291
void CardTableRS::verify_used_region_at_save_marks(Space* sp) const {
9342
456b8d0486b5 7039089: G1: changeset for 7037276 broke heap verification, and related cleanups
ysr
parents: 9336
diff changeset
   292
  MemRegion ur    = sp->used_region();
49733
6f0a3ea5ab75 8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents: 49592
diff changeset
   293
  MemRegion urasm = sp->used_region_at_save_marks();
6f0a3ea5ab75 8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents: 49592
diff changeset
   294
6f0a3ea5ab75 8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents: 49592
diff changeset
   295
  assert(ur.contains(urasm),
33105
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 32623
diff changeset
   296
         "Did you forget to call save_marks()? "
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 32623
diff changeset
   297
         "[" PTR_FORMAT ", " PTR_FORMAT ") is not contained in "
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 32623
diff changeset
   298
         "[" PTR_FORMAT ", " PTR_FORMAT ")",
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 32623
diff changeset
   299
         p2i(urasm.start()), p2i(urasm.end()), p2i(ur.start()), p2i(ur.end()));
49733
6f0a3ea5ab75 8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents: 49592
diff changeset
   300
}
9342
456b8d0486b5 7039089: G1: changeset for 7037276 broke heap verification, and related cleanups
ysr
parents: 9336
diff changeset
   301
#endif
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   302
19289
213031f03f61 8023021: Unnecessary clearing of the card table introduced by the fix for JDK-8023013
brutisso
parents: 19286
diff changeset
   303
void CardTableRS::clear_into_younger(Generation* old_gen) {
31358
693058672cc6 8077842: Remove the level parameter passed around in GenCollectedHeap
jwilhelm
parents: 30870
diff changeset
   304
  assert(GenCollectedHeap::heap()->is_old_gen(old_gen),
693058672cc6 8077842: Remove the level parameter passed around in GenCollectedHeap
jwilhelm
parents: 30870
diff changeset
   305
         "Should only be called for the old generation");
19289
213031f03f61 8023021: Unnecessary clearing of the card table introduced by the fix for JDK-8023013
brutisso
parents: 19286
diff changeset
   306
  // The card tables for the youngest gen need never be cleared.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   307
  // There's a bit of subtlety in the clear() and invalidate()
489c9b5090e2 Initial load
duke
parents:
diff changeset
   308
  // methods that we exploit here and in invalidate_or_clear()
489c9b5090e2 Initial load
duke
parents:
diff changeset
   309
  // below to avoid missing cards at the fringes. If clear() or
489c9b5090e2 Initial load
duke
parents:
diff changeset
   310
  // invalidate() are changed in the future, this code should
489c9b5090e2 Initial load
duke
parents:
diff changeset
   311
  // be revisited. 20040107.ysr
19286
48394008c803 8022800: Use specific generations rather than generation iteration
brutisso
parents: 17376
diff changeset
   312
  clear(old_gen->prev_used_region());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   313
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   314
19289
213031f03f61 8023021: Unnecessary clearing of the card table introduced by the fix for JDK-8023013
brutisso
parents: 19286
diff changeset
   315
void CardTableRS::invalidate_or_clear(Generation* old_gen) {
31358
693058672cc6 8077842: Remove the level parameter passed around in GenCollectedHeap
jwilhelm
parents: 30870
diff changeset
   316
  assert(GenCollectedHeap::heap()->is_old_gen(old_gen),
693058672cc6 8077842: Remove the level parameter passed around in GenCollectedHeap
jwilhelm
parents: 30870
diff changeset
   317
         "Should only be called for the old generation");
19289
213031f03f61 8023021: Unnecessary clearing of the card table introduced by the fix for JDK-8023013
brutisso
parents: 19286
diff changeset
   318
  // Invalidate the cards for the currently occupied part of
213031f03f61 8023021: Unnecessary clearing of the card table introduced by the fix for JDK-8023013
brutisso
parents: 19286
diff changeset
   319
  // the old generation and clear the cards for the
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   320
  // unoccupied part of the generation (if any, making use
489c9b5090e2 Initial load
duke
parents:
diff changeset
   321
  // of that generation's prev_used_region to determine that
489c9b5090e2 Initial load
duke
parents:
diff changeset
   322
  // region). No need to do anything for the youngest
489c9b5090e2 Initial load
duke
parents:
diff changeset
   323
  // generation. Also see note#20040107.ysr above.
19289
213031f03f61 8023021: Unnecessary clearing of the card table introduced by the fix for JDK-8023013
brutisso
parents: 19286
diff changeset
   324
  MemRegion used_mr = old_gen->used_region();
213031f03f61 8023021: Unnecessary clearing of the card table introduced by the fix for JDK-8023013
brutisso
parents: 19286
diff changeset
   325
  MemRegion to_be_cleared_mr = old_gen->prev_used_region().minus(used_mr);
19286
48394008c803 8022800: Use specific generations rather than generation iteration
brutisso
parents: 17376
diff changeset
   326
  if (!to_be_cleared_mr.is_empty()) {
48394008c803 8022800: Use specific generations rather than generation iteration
brutisso
parents: 17376
diff changeset
   327
    clear(to_be_cleared_mr);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   328
  }
19286
48394008c803 8022800: Use specific generations rather than generation iteration
brutisso
parents: 17376
diff changeset
   329
  invalidate(used_mr);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   330
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   331
489c9b5090e2 Initial load
duke
parents:
diff changeset
   332
50800
6da12aa23b88 8205607: Use oop_iterate instead of oop_iterate_no_header
stefank
parents: 50752
diff changeset
   333
class VerifyCleanCardClosure: public BasicOopIterateClosure {
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 179
diff changeset
   334
private:
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 179
diff changeset
   335
  HeapWord* _boundary;
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 179
diff changeset
   336
  HeapWord* _begin;
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 179
diff changeset
   337
  HeapWord* _end;
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 179
diff changeset
   338
protected:
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 179
diff changeset
   339
  template <class T> void do_oop_work(T* p) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   340
    HeapWord* jp = (HeapWord*)p;
8923
67b8a1ea66e2 7029036: Card-table verification hangs with all framework collectors, except G1, even before the first GC
ysr
parents: 7397
diff changeset
   341
    assert(jp >= _begin && jp < _end,
33105
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 32623
diff changeset
   342
           "Error: jp " PTR_FORMAT " should be within "
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 32623
diff changeset
   343
           "[_begin, _end) = [" PTR_FORMAT "," PTR_FORMAT ")",
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 32623
diff changeset
   344
           p2i(jp), p2i(_begin), p2i(_end));
49592
77fb0be7d19f 8199946: Move load/store and encode/decode out of oopDesc
stefank
parents: 49455
diff changeset
   345
    oop obj = RawAccess<>::oop_load(p);
8923
67b8a1ea66e2 7029036: Card-table verification hangs with all framework collectors, except G1, even before the first GC
ysr
parents: 7397
diff changeset
   346
    guarantee(obj == NULL || (HeapWord*)obj >= _boundary,
33105
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 32623
diff changeset
   347
              "pointer " PTR_FORMAT " at " PTR_FORMAT " on "
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 32623
diff changeset
   348
              "clean card crosses boundary" PTR_FORMAT,
33198
dlong
parents: 33105 33160
diff changeset
   349
              p2i(obj), p2i(jp), p2i(_boundary));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   350
  }
8923
67b8a1ea66e2 7029036: Card-table verification hangs with all framework collectors, except G1, even before the first GC
ysr
parents: 7397
diff changeset
   351
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 179
diff changeset
   352
public:
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 179
diff changeset
   353
  VerifyCleanCardClosure(HeapWord* b, HeapWord* begin, HeapWord* end) :
8923
67b8a1ea66e2 7029036: Card-table verification hangs with all framework collectors, except G1, even before the first GC
ysr
parents: 7397
diff changeset
   354
    _boundary(b), _begin(begin), _end(end) {
67b8a1ea66e2 7029036: Card-table verification hangs with all framework collectors, except G1, even before the first GC
ysr
parents: 7397
diff changeset
   355
    assert(b <= begin,
33105
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 32623
diff changeset
   356
           "Error: boundary " PTR_FORMAT " should be at or below begin " PTR_FORMAT,
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 32623
diff changeset
   357
           p2i(b), p2i(begin));
8923
67b8a1ea66e2 7029036: Card-table verification hangs with all framework collectors, except G1, even before the first GC
ysr
parents: 7397
diff changeset
   358
    assert(begin <= end,
33105
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 32623
diff changeset
   359
           "Error: begin " PTR_FORMAT " should be strictly below end " PTR_FORMAT,
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 32623
diff changeset
   360
           p2i(begin), p2i(end));
8923
67b8a1ea66e2 7029036: Card-table verification hangs with all framework collectors, except G1, even before the first GC
ysr
parents: 7397
diff changeset
   361
  }
67b8a1ea66e2 7029036: Card-table verification hangs with all framework collectors, except G1, even before the first GC
ysr
parents: 7397
diff changeset
   362
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 179
diff changeset
   363
  virtual void do_oop(oop* p)       { VerifyCleanCardClosure::do_oop_work(p); }
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 179
diff changeset
   364
  virtual void do_oop(narrowOop* p) { VerifyCleanCardClosure::do_oop_work(p); }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   365
};
489c9b5090e2 Initial load
duke
parents:
diff changeset
   366
489c9b5090e2 Initial load
duke
parents:
diff changeset
   367
class VerifyCTSpaceClosure: public SpaceClosure {
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 179
diff changeset
   368
private:
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   369
  CardTableRS* _ct;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   370
  HeapWord* _boundary;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   371
public:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   372
  VerifyCTSpaceClosure(CardTableRS* ct, HeapWord* boundary) :
489c9b5090e2 Initial load
duke
parents:
diff changeset
   373
    _ct(ct), _boundary(boundary) {}
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 179
diff changeset
   374
  virtual void do_space(Space* s) { _ct->verify_space(s, _boundary); }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   375
};
489c9b5090e2 Initial load
duke
parents:
diff changeset
   376
489c9b5090e2 Initial load
duke
parents:
diff changeset
   377
class VerifyCTGenClosure: public GenCollectedHeap::GenClosure {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   378
  CardTableRS* _ct;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   379
public:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   380
  VerifyCTGenClosure(CardTableRS* ct) : _ct(ct) {}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   381
  void do_generation(Generation* gen) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   382
    // Skip the youngest generation.
31358
693058672cc6 8077842: Remove the level parameter passed around in GenCollectedHeap
jwilhelm
parents: 30870
diff changeset
   383
    if (GenCollectedHeap::heap()->is_young_gen(gen)) {
693058672cc6 8077842: Remove the level parameter passed around in GenCollectedHeap
jwilhelm
parents: 30870
diff changeset
   384
      return;
693058672cc6 8077842: Remove the level parameter passed around in GenCollectedHeap
jwilhelm
parents: 30870
diff changeset
   385
    }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   386
    // Normally, we're interested in pointers to younger generations.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   387
    VerifyCTSpaceClosure blk(_ct, gen->reserved().start());
489c9b5090e2 Initial load
duke
parents:
diff changeset
   388
    gen->space_iterate(&blk, true);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   389
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   390
};
489c9b5090e2 Initial load
duke
parents:
diff changeset
   391
489c9b5090e2 Initial load
duke
parents:
diff changeset
   392
void CardTableRS::verify_space(Space* s, HeapWord* gen_boundary) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   393
  // We don't need to do young-gen spaces.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   394
  if (s->end() <= gen_boundary) return;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   395
  MemRegion used = s->used_region();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   396
54110
f4f0dce5d0bb 8220301: Remove jbyte use in CardTable
tschatzl
parents: 51959
diff changeset
   397
  CardValue* cur_entry = byte_for(used.start());
f4f0dce5d0bb 8220301: Remove jbyte use in CardTable
tschatzl
parents: 51959
diff changeset
   398
  CardValue* limit = byte_after(used.last());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   399
  while (cur_entry < limit) {
31964
d34ad1715d96 8130931: Refactor CardTableModRefBS[ForCTRS]
kbarrett
parents: 31358
diff changeset
   400
    if (*cur_entry == clean_card_val()) {
54110
f4f0dce5d0bb 8220301: Remove jbyte use in CardTable
tschatzl
parents: 51959
diff changeset
   401
      CardValue* first_dirty = cur_entry+1;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   402
      while (first_dirty < limit &&
31964
d34ad1715d96 8130931: Refactor CardTableModRefBS[ForCTRS]
kbarrett
parents: 31358
diff changeset
   403
             *first_dirty == clean_card_val()) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   404
        first_dirty++;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   405
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   406
      // If the first object is a regular object, and it has a
489c9b5090e2 Initial load
duke
parents:
diff changeset
   407
      // young-to-old field, that would mark the previous card.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   408
      HeapWord* boundary = addr_for(cur_entry);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   409
      HeapWord* end = (first_dirty >= limit) ? used.end() : addr_for(first_dirty);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   410
      HeapWord* boundary_block = s->block_start(boundary);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   411
      HeapWord* begin = boundary;             // Until proven otherwise.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   412
      HeapWord* start_block = boundary_block; // Until proven otherwise.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   413
      if (boundary_block < boundary) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   414
        if (s->block_is_obj(boundary_block) && s->obj_is_alive(boundary_block)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   415
          oop boundary_obj = oop(boundary_block);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   416
          if (!boundary_obj->is_objArray() &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
   417
              !boundary_obj->is_typeArray()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   418
            guarantee(cur_entry > byte_for(used.start()),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   419
                      "else boundary would be boundary_block");
31964
d34ad1715d96 8130931: Refactor CardTableModRefBS[ForCTRS]
kbarrett
parents: 31358
diff changeset
   420
            if (*byte_for(boundary_block) != clean_card_val()) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   421
              begin = boundary_block + s->block_size(boundary_block);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   422
              start_block = begin;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   423
            }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   424
          }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   425
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   426
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   427
      // Now traverse objects until end.
8923
67b8a1ea66e2 7029036: Card-table verification hangs with all framework collectors, except G1, even before the first GC
ysr
parents: 7397
diff changeset
   428
      if (begin < end) {
67b8a1ea66e2 7029036: Card-table verification hangs with all framework collectors, except G1, even before the first GC
ysr
parents: 7397
diff changeset
   429
        MemRegion mr(begin, end);
67b8a1ea66e2 7029036: Card-table verification hangs with all framework collectors, except G1, even before the first GC
ysr
parents: 7397
diff changeset
   430
        VerifyCleanCardClosure verify_blk(gen_boundary, begin, end);
67b8a1ea66e2 7029036: Card-table verification hangs with all framework collectors, except G1, even before the first GC
ysr
parents: 7397
diff changeset
   431
        for (HeapWord* cur = start_block; cur < end; cur += s->block_size(cur)) {
67b8a1ea66e2 7029036: Card-table verification hangs with all framework collectors, except G1, even before the first GC
ysr
parents: 7397
diff changeset
   432
          if (s->block_is_obj(cur) && s->obj_is_alive(cur)) {
50800
6da12aa23b88 8205607: Use oop_iterate instead of oop_iterate_no_header
stefank
parents: 50752
diff changeset
   433
            oop(cur)->oop_iterate(&verify_blk, mr);
8923
67b8a1ea66e2 7029036: Card-table verification hangs with all framework collectors, except G1, even before the first GC
ysr
parents: 7397
diff changeset
   434
          }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   435
        }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   436
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   437
      cur_entry = first_dirty;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   438
    } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   439
      // We'd normally expect that cur_youngergen_and_prev_nonclean_card
489c9b5090e2 Initial load
duke
parents:
diff changeset
   440
      // is a transient value, that cannot be in the card table
489c9b5090e2 Initial load
duke
parents:
diff changeset
   441
      // except during GC, and thus assert that:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   442
      // guarantee(*cur_entry != cur_youngergen_and_prev_nonclean_card,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   443
      //        "Illegal CT value");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   444
      // That however, need not hold, as will become clear in the
489c9b5090e2 Initial load
duke
parents:
diff changeset
   445
      // following...
489c9b5090e2 Initial load
duke
parents:
diff changeset
   446
489c9b5090e2 Initial load
duke
parents:
diff changeset
   447
      // We'd normally expect that if we are in the parallel case,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   448
      // we can't have left a prev value (which would be different
489c9b5090e2 Initial load
duke
parents:
diff changeset
   449
      // from the current value) in the card table, and so we'd like to
489c9b5090e2 Initial load
duke
parents:
diff changeset
   450
      // assert that:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   451
      // guarantee(cur_youngergen_card_val() == youngergen_card
489c9b5090e2 Initial load
duke
parents:
diff changeset
   452
      //           || !is_prev_youngergen_card_val(*cur_entry),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   453
      //           "Illegal CT value");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   454
      // That, however, may not hold occasionally, because of
489c9b5090e2 Initial load
duke
parents:
diff changeset
   455
      // CMS or MSC in the old gen. To wit, consider the
489c9b5090e2 Initial load
duke
parents:
diff changeset
   456
      // following two simple illustrative scenarios:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   457
      // (a) CMS: Consider the case where a large object L
489c9b5090e2 Initial load
duke
parents:
diff changeset
   458
      //     spanning several cards is allocated in the old
489c9b5090e2 Initial load
duke
parents:
diff changeset
   459
      //     gen, and has a young gen reference stored in it, dirtying
489c9b5090e2 Initial load
duke
parents:
diff changeset
   460
      //     some interior cards. A young collection scans the card,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   461
      //     finds a young ref and installs a youngergenP_n value.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   462
      //     L then goes dead. Now a CMS collection starts,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   463
      //     finds L dead and sweeps it up. Assume that L is
489c9b5090e2 Initial load
duke
parents:
diff changeset
   464
      //     abutting _unallocated_blk, so _unallocated_blk is
489c9b5090e2 Initial load
duke
parents:
diff changeset
   465
      //     adjusted down to (below) L. Assume further that
489c9b5090e2 Initial load
duke
parents:
diff changeset
   466
      //     no young collection intervenes during this CMS cycle.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   467
      //     The next young gen cycle will not get to look at this
489c9b5090e2 Initial load
duke
parents:
diff changeset
   468
      //     youngergenP_n card since it lies in the unoccupied
489c9b5090e2 Initial load
duke
parents:
diff changeset
   469
      //     part of the space.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   470
      //     Some young collections later the blocks on this
489c9b5090e2 Initial load
duke
parents:
diff changeset
   471
      //     card can be re-allocated either due to direct allocation
489c9b5090e2 Initial load
duke
parents:
diff changeset
   472
      //     or due to absorbing promotions. At this time, the
489c9b5090e2 Initial load
duke
parents:
diff changeset
   473
      //     before-gc verification will fail the above assert.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   474
      // (b) MSC: In this case, an object L with a young reference
489c9b5090e2 Initial load
duke
parents:
diff changeset
   475
      //     is on a card that (therefore) holds a youngergen_n value.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   476
      //     Suppose also that L lies towards the end of the used
489c9b5090e2 Initial load
duke
parents:
diff changeset
   477
      //     the used space before GC. An MSC collection
489c9b5090e2 Initial load
duke
parents:
diff changeset
   478
      //     occurs that compacts to such an extent that this
489c9b5090e2 Initial load
duke
parents:
diff changeset
   479
      //     card is no longer in the occupied part of the space.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   480
      //     Since current code in MSC does not always clear cards
489c9b5090e2 Initial load
duke
parents:
diff changeset
   481
      //     in the unused part of old gen, this stale youngergen_n
489c9b5090e2 Initial load
duke
parents:
diff changeset
   482
      //     value is left behind and can later be covered by
489c9b5090e2 Initial load
duke
parents:
diff changeset
   483
      //     an object when promotion or direct allocation
489c9b5090e2 Initial load
duke
parents:
diff changeset
   484
      //     re-allocates that part of the heap.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   485
      //
489c9b5090e2 Initial load
duke
parents:
diff changeset
   486
      // Fortunately, the presence of such stale card values is
489c9b5090e2 Initial load
duke
parents:
diff changeset
   487
      // "only" a minor annoyance in that subsequent young collections
489c9b5090e2 Initial load
duke
parents:
diff changeset
   488
      // might needlessly scan such cards, but would still never corrupt
489c9b5090e2 Initial load
duke
parents:
diff changeset
   489
      // the heap as a result. However, it's likely not to be a significant
489c9b5090e2 Initial load
duke
parents:
diff changeset
   490
      // performance inhibitor in practice. For instance,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   491
      // some recent measurements with unoccupied cards eagerly cleared
489c9b5090e2 Initial load
duke
parents:
diff changeset
   492
      // out to maintain this invariant, showed next to no
489c9b5090e2 Initial load
duke
parents:
diff changeset
   493
      // change in young collection times; of course one can construct
489c9b5090e2 Initial load
duke
parents:
diff changeset
   494
      // degenerate examples where the cost can be significant.)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   495
      // Note, in particular, that if the "stale" card is modified
489c9b5090e2 Initial load
duke
parents:
diff changeset
   496
      // after re-allocation, it would be dirty, not "stale". Thus,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   497
      // we can never have a younger ref in such a card and it is
489c9b5090e2 Initial load
duke
parents:
diff changeset
   498
      // safe not to scan that card in any collection. [As we see
489c9b5090e2 Initial load
duke
parents:
diff changeset
   499
      // below, we do some unnecessary scanning
489c9b5090e2 Initial load
duke
parents:
diff changeset
   500
      // in some cases in the current parallel scanning algorithm.]
489c9b5090e2 Initial load
duke
parents:
diff changeset
   501
      //
489c9b5090e2 Initial load
duke
parents:
diff changeset
   502
      // The main point below is that the parallel card scanning code
489c9b5090e2 Initial load
duke
parents:
diff changeset
   503
      // deals correctly with these stale card values. There are two main
32623
390a27af5657 8134626: Misc cleanups after generation array removal
jwilhelm
parents: 31964
diff changeset
   504
      // cases to consider where we have a stale "young gen" value and a
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   505
      // "derivative" case to consider, where we have a stale
489c9b5090e2 Initial load
duke
parents:
diff changeset
   506
      // "cur_younger_gen_and_prev_non_clean" value, as will become
489c9b5090e2 Initial load
duke
parents:
diff changeset
   507
      // apparent in the case analysis below.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   508
      // o Case 1. If the stale value corresponds to a younger_gen_n
489c9b5090e2 Initial load
duke
parents:
diff changeset
   509
      //   value other than the cur_younger_gen value then the code
489c9b5090e2 Initial load
duke
parents:
diff changeset
   510
      //   treats this as being tantamount to a prev_younger_gen
489c9b5090e2 Initial load
duke
parents:
diff changeset
   511
      //   card. This means that the card may be unnecessarily scanned.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   512
      //   There are two sub-cases to consider:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   513
      //   o Case 1a. Let us say that the card is in the occupied part
489c9b5090e2 Initial load
duke
parents:
diff changeset
   514
      //     of the generation at the time the collection begins. In
489c9b5090e2 Initial load
duke
parents:
diff changeset
   515
      //     that case the card will be either cleared when it is scanned
489c9b5090e2 Initial load
duke
parents:
diff changeset
   516
      //     for young pointers, or will be set to cur_younger_gen as a
489c9b5090e2 Initial load
duke
parents:
diff changeset
   517
      //     result of promotion. (We have elided the normal case where
489c9b5090e2 Initial load
duke
parents:
diff changeset
   518
      //     the scanning thread and the promoting thread interleave
489c9b5090e2 Initial load
duke
parents:
diff changeset
   519
      //     possibly resulting in a transient
489c9b5090e2 Initial load
duke
parents:
diff changeset
   520
      //     cur_younger_gen_and_prev_non_clean value before settling
489c9b5090e2 Initial load
duke
parents:
diff changeset
   521
      //     to cur_younger_gen. [End Case 1a.]
489c9b5090e2 Initial load
duke
parents:
diff changeset
   522
      //   o Case 1b. Consider now the case when the card is in the unoccupied
489c9b5090e2 Initial load
duke
parents:
diff changeset
   523
      //     part of the space which becomes occupied because of promotions
489c9b5090e2 Initial load
duke
parents:
diff changeset
   524
      //     into it during the current young GC. In this case the card
489c9b5090e2 Initial load
duke
parents:
diff changeset
   525
      //     will never be scanned for young references. The current
489c9b5090e2 Initial load
duke
parents:
diff changeset
   526
      //     code will set the card value to either
489c9b5090e2 Initial load
duke
parents:
diff changeset
   527
      //     cur_younger_gen_and_prev_non_clean or leave
489c9b5090e2 Initial load
duke
parents:
diff changeset
   528
      //     it with its stale value -- because the promotions didn't
489c9b5090e2 Initial load
duke
parents:
diff changeset
   529
      //     result in any younger refs on that card. Of these two
489c9b5090e2 Initial load
duke
parents:
diff changeset
   530
      //     cases, the latter will be covered in Case 1a during
489c9b5090e2 Initial load
duke
parents:
diff changeset
   531
      //     a subsequent scan. To deal with the former case, we need
489c9b5090e2 Initial load
duke
parents:
diff changeset
   532
      //     to further consider how we deal with a stale value of
489c9b5090e2 Initial load
duke
parents:
diff changeset
   533
      //     cur_younger_gen_and_prev_non_clean in our case analysis
489c9b5090e2 Initial load
duke
parents:
diff changeset
   534
      //     below. This we do in Case 3 below. [End Case 1b]
489c9b5090e2 Initial load
duke
parents:
diff changeset
   535
      //   [End Case 1]
489c9b5090e2 Initial load
duke
parents:
diff changeset
   536
      // o Case 2. If the stale value corresponds to cur_younger_gen being
489c9b5090e2 Initial load
duke
parents:
diff changeset
   537
      //   a value not necessarily written by a current promotion, the
489c9b5090e2 Initial load
duke
parents:
diff changeset
   538
      //   card will not be scanned by the younger refs scanning code.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   539
      //   (This is OK since as we argued above such cards cannot contain
489c9b5090e2 Initial load
duke
parents:
diff changeset
   540
      //   any younger refs.) The result is that this value will be
489c9b5090e2 Initial load
duke
parents:
diff changeset
   541
      //   treated as a prev_younger_gen value in a subsequent collection,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   542
      //   which is addressed in Case 1 above. [End Case 2]
489c9b5090e2 Initial load
duke
parents:
diff changeset
   543
      // o Case 3. We here consider the "derivative" case from Case 1b. above
489c9b5090e2 Initial load
duke
parents:
diff changeset
   544
      //   because of which we may find a stale
489c9b5090e2 Initial load
duke
parents:
diff changeset
   545
      //   cur_younger_gen_and_prev_non_clean card value in the table.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   546
      //   Once again, as in Case 1, we consider two subcases, depending
489c9b5090e2 Initial load
duke
parents:
diff changeset
   547
      //   on whether the card lies in the occupied or unoccupied part
489c9b5090e2 Initial load
duke
parents:
diff changeset
   548
      //   of the space at the start of the young collection.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   549
      //   o Case 3a. Let us say the card is in the occupied part of
489c9b5090e2 Initial load
duke
parents:
diff changeset
   550
      //     the old gen at the start of the young collection. In that
489c9b5090e2 Initial load
duke
parents:
diff changeset
   551
      //     case, the card will be scanned by the younger refs scanning
489c9b5090e2 Initial load
duke
parents:
diff changeset
   552
      //     code which will set it to cur_younger_gen. In a subsequent
489c9b5090e2 Initial load
duke
parents:
diff changeset
   553
      //     scan, the card will be considered again and get its final
489c9b5090e2 Initial load
duke
parents:
diff changeset
   554
      //     correct value. [End Case 3a]
489c9b5090e2 Initial load
duke
parents:
diff changeset
   555
      //   o Case 3b. Now consider the case where the card is in the
489c9b5090e2 Initial load
duke
parents:
diff changeset
   556
      //     unoccupied part of the old gen, and is occupied as a result
489c9b5090e2 Initial load
duke
parents:
diff changeset
   557
      //     of promotions during thus young gc. In that case,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   558
      //     the card will not be scanned for younger refs. The presence
489c9b5090e2 Initial load
duke
parents:
diff changeset
   559
      //     of newly promoted objects on the card will then result in
489c9b5090e2 Initial load
duke
parents:
diff changeset
   560
      //     its keeping the value cur_younger_gen_and_prev_non_clean
489c9b5090e2 Initial load
duke
parents:
diff changeset
   561
      //     value, which we have dealt with in Case 3 here. [End Case 3b]
489c9b5090e2 Initial load
duke
parents:
diff changeset
   562
      //   [End Case 3]
489c9b5090e2 Initial load
duke
parents:
diff changeset
   563
      //
489c9b5090e2 Initial load
duke
parents:
diff changeset
   564
      // (Please refer to the code in the helper class
49455
848864ed9b17 8199604: Rename CardTableModRefBS to CardTableBarrierSet
eosterlund
parents: 49164
diff changeset
   565
      // ClearNonCleanCardWrapper and in CardTable for details.)
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   566
      //
489c9b5090e2 Initial load
duke
parents:
diff changeset
   567
      // The informal arguments above can be tightened into a formal
489c9b5090e2 Initial load
duke
parents:
diff changeset
   568
      // correctness proof and it behooves us to write up such a proof,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   569
      // or to use model checking to prove that there are no lingering
489c9b5090e2 Initial load
duke
parents:
diff changeset
   570
      // concerns.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   571
      //
489c9b5090e2 Initial load
duke
parents:
diff changeset
   572
      // Clearly because of Case 3b one cannot bound the time for
489c9b5090e2 Initial load
duke
parents:
diff changeset
   573
      // which a card will retain what we have called a "stale" value.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   574
      // However, one can obtain a Loose upper bound on the redundant
489c9b5090e2 Initial load
duke
parents:
diff changeset
   575
      // work as a result of such stale values. Note first that any
489c9b5090e2 Initial load
duke
parents:
diff changeset
   576
      // time a stale card lies in the occupied part of the space at
489c9b5090e2 Initial load
duke
parents:
diff changeset
   577
      // the start of the collection, it is scanned by younger refs
489c9b5090e2 Initial load
duke
parents:
diff changeset
   578
      // code and we can define a rank function on card values that
489c9b5090e2 Initial load
duke
parents:
diff changeset
   579
      // declines when this is so. Note also that when a card does not
489c9b5090e2 Initial load
duke
parents:
diff changeset
   580
      // lie in the occupied part of the space at the beginning of a
489c9b5090e2 Initial load
duke
parents:
diff changeset
   581
      // young collection, its rank can either decline or stay unchanged.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   582
      // In this case, no extra work is done in terms of redundant
489c9b5090e2 Initial load
duke
parents:
diff changeset
   583
      // younger refs scanning of that card.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   584
      // Then, the case analysis above reveals that, in the worst case,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   585
      // any such stale card will be scanned unnecessarily at most twice.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   586
      //
22551
9bf46d16dcc6 8025856: Fix typos in the GC code
jwilhelm
parents: 19289
diff changeset
   587
      // It is nonetheless advisable to try and get rid of some of this
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   588
      // redundant work in a subsequent (low priority) re-design of
489c9b5090e2 Initial load
duke
parents:
diff changeset
   589
      // the card-scanning code, if only to simplify the underlying
489c9b5090e2 Initial load
duke
parents:
diff changeset
   590
      // state machine analysis/proof. ysr 1/28/2002. XXX
489c9b5090e2 Initial load
duke
parents:
diff changeset
   591
      cur_entry++;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   592
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   593
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   594
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   595
489c9b5090e2 Initial load
duke
parents:
diff changeset
   596
void CardTableRS::verify() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   597
  // At present, we only know how to verify the card table RS for
489c9b5090e2 Initial load
duke
parents:
diff changeset
   598
  // generational heaps.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   599
  VerifyCTGenClosure blk(this);
30173
13cf7580b000 8077413: Avoid use of Universe::heap() inside collectors
pliden
parents: 30165
diff changeset
   600
  GenCollectedHeap::heap()->generation_iterate(&blk, false);
49164
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   601
  CardTable::verify();
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   602
}
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   603
49733
6f0a3ea5ab75 8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents: 49592
diff changeset
   604
CardTableRS::CardTableRS(MemRegion whole_heap, bool scanned_concurrently) :
6f0a3ea5ab75 8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents: 49592
diff changeset
   605
  CardTable(whole_heap, scanned_concurrently),
49164
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   606
  _cur_youngergen_card_val(youngergenP1_card),
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   607
  // LNC functionality
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   608
  _lowest_non_clean(NULL),
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   609
  _lowest_non_clean_chunk_size(NULL),
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   610
  _lowest_non_clean_base_chunk_index(NULL),
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   611
  _last_LNC_resizing_collection(NULL)
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   612
{
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   613
  // max_gens is really GenCollectedHeap::heap()->gen_policy()->number_of_generations()
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   614
  // (which is always 2, young & old), but GenCollectedHeap has not been initialized yet.
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   615
  uint max_gens = 2;
54110
f4f0dce5d0bb 8220301: Remove jbyte use in CardTable
tschatzl
parents: 51959
diff changeset
   616
  _last_cur_val_in_gen = NEW_C_HEAP_ARRAY3(CardValue, max_gens + 1,
49164
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   617
                         mtGC, CURRENT_PC, AllocFailStrategy::RETURN_NULL);
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   618
  if (_last_cur_val_in_gen == NULL) {
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   619
    vm_exit_during_initialization("Could not create last_cur_val_in_gen array.");
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   620
  }
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   621
  for (uint i = 0; i < max_gens + 1; i++) {
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   622
    _last_cur_val_in_gen[i] = clean_card_val();
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   623
  }
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   624
}
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   625
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   626
CardTableRS::~CardTableRS() {
58084
cddef3bde924 8230398: Remove NULL checks before FREE_C_HEAP_ARRAY
lkorinth
parents: 58083
diff changeset
   627
  FREE_C_HEAP_ARRAY(CardValue, _last_cur_val_in_gen);
cddef3bde924 8230398: Remove NULL checks before FREE_C_HEAP_ARRAY
lkorinth
parents: 58083
diff changeset
   628
  FREE_C_HEAP_ARRAY(CardArr, _lowest_non_clean);
cddef3bde924 8230398: Remove NULL checks before FREE_C_HEAP_ARRAY
lkorinth
parents: 58083
diff changeset
   629
  FREE_C_HEAP_ARRAY(size_t, _lowest_non_clean_chunk_size);
cddef3bde924 8230398: Remove NULL checks before FREE_C_HEAP_ARRAY
lkorinth
parents: 58083
diff changeset
   630
  FREE_C_HEAP_ARRAY(uintptr_t, _lowest_non_clean_base_chunk_index);
cddef3bde924 8230398: Remove NULL checks before FREE_C_HEAP_ARRAY
lkorinth
parents: 58083
diff changeset
   631
  FREE_C_HEAP_ARRAY(int, _last_LNC_resizing_collection);
30173
13cf7580b000 8077413: Avoid use of Universe::heap() inside collectors
pliden
parents: 30165
diff changeset
   632
}
49164
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   633
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   634
void CardTableRS::initialize() {
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   635
  CardTable::initialize();
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   636
  _lowest_non_clean =
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   637
    NEW_C_HEAP_ARRAY(CardArr, _max_covered_regions, mtGC);
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   638
  _lowest_non_clean_chunk_size =
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   639
    NEW_C_HEAP_ARRAY(size_t, _max_covered_regions, mtGC);
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   640
  _lowest_non_clean_base_chunk_index =
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   641
    NEW_C_HEAP_ARRAY(uintptr_t, _max_covered_regions, mtGC);
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   642
  _last_LNC_resizing_collection =
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   643
    NEW_C_HEAP_ARRAY(int, _max_covered_regions, mtGC);
58083
9046db64ca39 8227168: Cleanup usage of NEW_C_HEAP_ARRAY
lkorinth
parents: 54110
diff changeset
   644
49164
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   645
  for (int i = 0; i < _max_covered_regions; i++) {
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   646
    _lowest_non_clean[i] = NULL;
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   647
    _lowest_non_clean_chunk_size[i] = 0;
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   648
    _last_LNC_resizing_collection[i] = -1;
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   649
  }
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   650
}
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   651
54110
f4f0dce5d0bb 8220301: Remove jbyte use in CardTable
tschatzl
parents: 51959
diff changeset
   652
bool CardTableRS::card_will_be_scanned(CardValue cv) {
49164
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   653
  return card_is_dirty_wrt_gen_iter(cv) || is_prev_nonclean_card_val(cv);
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   654
}
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   655
54110
f4f0dce5d0bb 8220301: Remove jbyte use in CardTable
tschatzl
parents: 51959
diff changeset
   656
bool CardTableRS::card_may_have_been_dirty(CardValue cv) {
49164
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   657
  return
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   658
    cv != clean_card &&
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   659
    (card_is_dirty_wrt_gen_iter(cv) ||
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   660
     CardTableRS::youngergen_may_have_been_dirty(cv));
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   661
}
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   662
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   663
void CardTableRS::non_clean_card_iterate_possibly_parallel(
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   664
  Space* sp,
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   665
  MemRegion mr,
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   666
  OopsInGenClosure* cl,
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   667
  CardTableRS* ct,
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   668
  uint n_threads)
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   669
{
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   670
  if (!mr.is_empty()) {
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   671
    if (n_threads > 0) {
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   672
      non_clean_card_iterate_parallel_work(sp, mr, cl, ct, n_threads);
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   673
    } else {
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   674
      // clear_cl finds contiguous dirty ranges of cards to process and clear.
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   675
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   676
      // This is the single-threaded version used by DefNew.
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   677
      const bool parallel = false;
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   678
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   679
      DirtyCardToOopClosure* dcto_cl = sp->new_dcto_cl(cl, precision(), cl->gen_boundary(), parallel);
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   680
      ClearNoncleanCardWrapper clear_cl(dcto_cl, ct, parallel);
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   681
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   682
      clear_cl.do_MemRegion(mr);
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   683
    }
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   684
  }
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   685
}
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   686
49733
6f0a3ea5ab75 8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents: 49592
diff changeset
   687
void CardTableRS::non_clean_card_iterate_parallel_work(Space* sp, MemRegion mr,
6f0a3ea5ab75 8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents: 49592
diff changeset
   688
                                                       OopsInGenClosure* cl, CardTableRS* ct,
6f0a3ea5ab75 8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents: 49592
diff changeset
   689
                                                       uint n_threads) {
6f0a3ea5ab75 8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents: 49592
diff changeset
   690
  fatal("Parallel gc not supported here.");
6f0a3ea5ab75 8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents: 49592
diff changeset
   691
}
6f0a3ea5ab75 8201209: Separate out CMS specific functions into CMSCardTable
stefank
parents: 49592
diff changeset
   692
49164
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   693
bool CardTableRS::is_in_young(oop obj) const {
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   694
  return GenCollectedHeap::heap()->is_in_young(obj);
7e958a8ebcd3 8195142: Refactor out card table from CardTableModRefBS to flatten the BarrierSet hierarchy
eosterlund
parents: 47580
diff changeset
   695
}