hotspot/src/share/vm/gc/g1/g1OopClosures.inline.hpp
author tschatzl
Wed, 06 Apr 2016 13:32:48 +0200
changeset 37413 2f71679d06dd
parent 37258 d5fb0ee5e41a
child 46282 a001553763bc
permissions -rw-r--r--
8077144: Concurrent mark initialization takes too long Summary: Remove per-marking thread liveness bitmaps and recreate liveness bitmap concurrently after the cleanup pause. Reviewed-by: mgerdin, ehelin, kbarrett
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
     1
/*
35862
411842d0c882 8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents: 35069
diff changeset
     2
 * Copyright (c) 2001, 2016, Oracle and/or its affiliates. All rights reserved.
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
     4
 *
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
     7
 * published by the Free Software Foundation.
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
     8
 *
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    13
 * accompanied this code).
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    14
 *
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    18
 *
5547
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 4902
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 4902
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: 4902
diff changeset
    21
 * questions.
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    22
 *
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    23
 */
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    24
30764
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30173
diff changeset
    25
#ifndef SHARE_VM_GC_G1_G1OOPCLOSURES_INLINE_HPP
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30173
diff changeset
    26
#define SHARE_VM_GC_G1_G1OOPCLOSURES_INLINE_HPP
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6247
diff changeset
    27
30764
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30173
diff changeset
    28
#include "gc/g1/g1CollectedHeap.hpp"
35943
e726308008c0 8148973: Rename g1/concurrentMark.{hpp,cpp,inline.hpp} to g1/g1ConcurrentMark.{hpp,cpp,inline.hpp}
ehelin
parents: 35862
diff changeset
    29
#include "gc/g1/g1ConcurrentMark.inline.hpp"
30764
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30173
diff changeset
    30
#include "gc/g1/g1OopClosures.hpp"
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30173
diff changeset
    31
#include "gc/g1/g1ParScanThreadState.inline.hpp"
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30173
diff changeset
    32
#include "gc/g1/g1RemSet.hpp"
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30173
diff changeset
    33
#include "gc/g1/g1RemSet.inline.hpp"
32185
49a57ff2c3cb 8073052: Rename and clean up the allocation manager hierarchy in g1Allocator.?pp
tschatzl
parents: 31592
diff changeset
    34
#include "gc/g1/heapRegion.inline.hpp"
30764
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30173
diff changeset
    35
#include "gc/g1/heapRegionRemSet.hpp"
25492
d27050bdfb04 8049421: G1 Class Unloading after completing a concurrent mark cycle
stefank
parents: 25491
diff changeset
    36
#include "memory/iterator.inline.hpp"
24487
71ff0bd674eb 8042737: Introduce umbrella header prefetch.inline.hpp
goetz
parents: 24424
diff changeset
    37
#include "runtime/prefetch.inline.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6247
diff changeset
    38
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    39
/*
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    40
 * This really ought to be an inline function, but apparently the C++
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    41
 * compiler sometimes sees fit to ignore inline declarations.  Sigh.
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    42
 */
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    43
11584
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11202
diff changeset
    44
template <class T>
35068
b0d5016aecc7 8144584: Change FilterIntoCSClosure to inherit OopClosure
sjohanss
parents: 35067
diff changeset
    45
inline void FilterIntoCSClosure::do_oop_work(T* p) {
3262
30d1c247fc25 6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents: 1374
diff changeset
    46
  T heap_oop = oopDesc::load_heap_oop(p);
30d1c247fc25 6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents: 1374
diff changeset
    47
  if (!oopDesc::is_null(heap_oop) &&
25889
221296ac4359 8027959: Early reclamation of large objects in G1
tschatzl
parents: 25492
diff changeset
    48
      _g1->is_in_cset_or_humongous(oopDesc::decode_heap_oop_not_null(heap_oop))) {
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    49
    _oc->do_oop(p);
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    50
  }
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    51
}
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    52
11584
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11202
diff changeset
    53
template <class T>
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11202
diff changeset
    54
inline void FilterOutOfRegionClosure::do_oop_nv(T* p) {
3262
30d1c247fc25 6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents: 1374
diff changeset
    55
  T heap_oop = oopDesc::load_heap_oop(p);
30d1c247fc25 6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents: 1374
diff changeset
    56
  if (!oopDesc::is_null(heap_oop)) {
30d1c247fc25 6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents: 1374
diff changeset
    57
    HeapWord* obj_hw = (HeapWord*)oopDesc::decode_heap_oop_not_null(heap_oop);
30d1c247fc25 6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents: 1374
diff changeset
    58
    if (obj_hw < _r_bottom || obj_hw >= _r_end) {
30d1c247fc25 6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents: 1374
diff changeset
    59
      _oc->do_oop(p);
30d1c247fc25 6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents: 1374
diff changeset
    60
    }
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    61
  }
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    62
}
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    63
3262
30d1c247fc25 6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents: 1374
diff changeset
    64
// This closure is applied to the fields of the objects that have just been copied.
11584
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11202
diff changeset
    65
template <class T>
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11202
diff changeset
    66
inline void G1ParScanClosure::do_oop_nv(T* p) {
3262
30d1c247fc25 6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents: 1374
diff changeset
    67
  T heap_oop = oopDesc::load_heap_oop(p);
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    68
3262
30d1c247fc25 6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents: 1374
diff changeset
    69
  if (!oopDesc::is_null(heap_oop)) {
30d1c247fc25 6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents: 1374
diff changeset
    70
    oop obj = oopDesc::decode_heap_oop_not_null(heap_oop);
28213
b0bf57cd1e9d 8060025: Object copy time regressions after JDK-8031323 and JDK-8057536
tschatzl
parents: 25889
diff changeset
    71
    const InCSetState state = _g1->in_cset_state(obj);
b0bf57cd1e9d 8060025: Object copy time regressions after JDK-8031323 and JDK-8057536
tschatzl
parents: 25889
diff changeset
    72
    if (state.is_in_cset()) {
3262
30d1c247fc25 6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents: 1374
diff changeset
    73
      // We're not going to even bother checking whether the object is
30d1c247fc25 6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents: 1374
diff changeset
    74
      // already forwarded or not, as this usually causes an immediate
30d1c247fc25 6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents: 1374
diff changeset
    75
      // stall. We'll try to prefetch the object (for write, given that
30d1c247fc25 6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents: 1374
diff changeset
    76
      // we might need to install the forwarding reference) and we'll
30d1c247fc25 6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents: 1374
diff changeset
    77
      // get back to it when pop it from the queue
30d1c247fc25 6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents: 1374
diff changeset
    78
      Prefetch::write(obj->mark_addr(), 0);
30d1c247fc25 6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents: 1374
diff changeset
    79
      Prefetch::read(obj->mark_addr(), (HeapWordSize*2));
30d1c247fc25 6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents: 1374
diff changeset
    80
30d1c247fc25 6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents: 1374
diff changeset
    81
      // slightly paranoid test; I'm trying to catch potential
30d1c247fc25 6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents: 1374
diff changeset
    82
      // problems before we go into push_on_queue to know where the
30d1c247fc25 6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents: 1374
diff changeset
    83
      // problem is coming from
11202
6be05533238b 7114095: G1: assert(obj == oopDesc::load_decode_heap_oop(p)) failed: p should still be pointing to obj
johnc
parents: 9994
diff changeset
    84
      assert((obj == oopDesc::load_decode_heap_oop(p)) ||
6be05533238b 7114095: G1: assert(obj == oopDesc::load_decode_heap_oop(p)) failed: p should still be pointing to obj
johnc
parents: 9994
diff changeset
    85
             (obj->is_forwarded() &&
6be05533238b 7114095: G1: assert(obj == oopDesc::load_decode_heap_oop(p)) failed: p should still be pointing to obj
johnc
parents: 9994
diff changeset
    86
                 obj->forwardee() == oopDesc::load_decode_heap_oop(p)),
6be05533238b 7114095: G1: assert(obj == oopDesc::load_decode_heap_oop(p)) failed: p should still be pointing to obj
johnc
parents: 9994
diff changeset
    87
             "p should still be pointing to obj or to its forwardee");
6be05533238b 7114095: G1: assert(obj == oopDesc::load_decode_heap_oop(p)) failed: p should still be pointing to obj
johnc
parents: 9994
diff changeset
    88
3262
30d1c247fc25 6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents: 1374
diff changeset
    89
      _par_scan_state->push_on_queue(p);
30d1c247fc25 6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents: 1374
diff changeset
    90
    } else {
28213
b0bf57cd1e9d 8060025: Object copy time regressions after JDK-8031323 and JDK-8057536
tschatzl
parents: 25889
diff changeset
    91
      if (state.is_humongous()) {
25889
221296ac4359 8027959: Early reclamation of large objects in G1
tschatzl
parents: 25492
diff changeset
    92
        _g1->set_humongous_is_live(obj);
35051
002d874a4284 8144714: Add extension point to G1 evacuation closures
mgerdin
parents: 34616
diff changeset
    93
      } else if (state.is_ext()) {
002d874a4284 8144714: Add extension point to G1 evacuation closures
mgerdin
parents: 34616
diff changeset
    94
        _par_scan_state->do_oop_ext(p);
25889
221296ac4359 8027959: Early reclamation of large objects in G1
tschatzl
parents: 25492
diff changeset
    95
      }
34616
421a6405ca30 8144067: Pass obj directly to G1ParScanThreadState::update_rs
ehelin
parents: 34245
diff changeset
    96
      _par_scan_state->update_rs(_from, p, obj);
3262
30d1c247fc25 6700789: G1: Enable use of compressed oops with G1 heaps
ysr
parents: 1374
diff changeset
    97
    }
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    98
  }
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    99
}
4902
991aaddb5165 6923991: G1: improve scalability of RSet scanning
iveresov
parents: 3262
diff changeset
   100
11584
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11202
diff changeset
   101
template <class T>
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11202
diff changeset
   102
inline void G1ParPushHeapRSClosure::do_oop_nv(T* p) {
4902
991aaddb5165 6923991: G1: improve scalability of RSet scanning
iveresov
parents: 3262
diff changeset
   103
  T heap_oop = oopDesc::load_heap_oop(p);
991aaddb5165 6923991: G1: improve scalability of RSet scanning
iveresov
parents: 3262
diff changeset
   104
991aaddb5165 6923991: G1: improve scalability of RSet scanning
iveresov
parents: 3262
diff changeset
   105
  if (!oopDesc::is_null(heap_oop)) {
991aaddb5165 6923991: G1: improve scalability of RSet scanning
iveresov
parents: 3262
diff changeset
   106
    oop obj = oopDesc::decode_heap_oop_not_null(heap_oop);
35051
002d874a4284 8144714: Add extension point to G1 evacuation closures
mgerdin
parents: 34616
diff changeset
   107
    const InCSetState state = _g1->in_cset_state(obj);
002d874a4284 8144714: Add extension point to G1 evacuation closures
mgerdin
parents: 34616
diff changeset
   108
    if (state.is_in_cset_or_humongous()) {
4902
991aaddb5165 6923991: G1: improve scalability of RSet scanning
iveresov
parents: 3262
diff changeset
   109
      Prefetch::write(obj->mark_addr(), 0);
991aaddb5165 6923991: G1: improve scalability of RSet scanning
iveresov
parents: 3262
diff changeset
   110
      Prefetch::read(obj->mark_addr(), (HeapWordSize*2));
6247
00e5cc407d03 6814437: G1: remove the _new_refs array
johnc
parents: 5547
diff changeset
   111
00e5cc407d03 6814437: G1: remove the _new_refs array
johnc
parents: 5547
diff changeset
   112
      // Place on the references queue
4902
991aaddb5165 6923991: G1: improve scalability of RSet scanning
iveresov
parents: 3262
diff changeset
   113
      _par_scan_state->push_on_queue(p);
35051
002d874a4284 8144714: Add extension point to G1 evacuation closures
mgerdin
parents: 34616
diff changeset
   114
    } else if (state.is_ext()) {
002d874a4284 8144714: Add extension point to G1 evacuation closures
mgerdin
parents: 34616
diff changeset
   115
      _par_scan_state->do_oop_ext(p);
25889
221296ac4359 8027959: Early reclamation of large objects in G1
tschatzl
parents: 25492
diff changeset
   116
    } else {
221296ac4359 8027959: Early reclamation of large objects in G1
tschatzl
parents: 25492
diff changeset
   117
      assert(!_g1->obj_in_cs(obj), "checking");
4902
991aaddb5165 6923991: G1: improve scalability of RSet scanning
iveresov
parents: 3262
diff changeset
   118
    }
991aaddb5165 6923991: G1: improve scalability of RSet scanning
iveresov
parents: 3262
diff changeset
   119
  }
991aaddb5165 6923991: G1: improve scalability of RSet scanning
iveresov
parents: 3262
diff changeset
   120
}
6247
00e5cc407d03 6814437: G1: remove the _new_refs array
johnc
parents: 5547
diff changeset
   121
11584
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11202
diff changeset
   122
template <class T>
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11202
diff changeset
   123
inline void G1CMOopClosure::do_oop_nv(T* p) {
9994
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents: 9988
diff changeset
   124
  oop obj = oopDesc::load_decode_heap_oop(p);
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents: 9988
diff changeset
   125
  _task->deal_with_reference(obj);
efb74fdbd46e 7046558: G1: concurrent marking optimizations
tonyp
parents: 9988
diff changeset
   126
}
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6247
diff changeset
   127
11584
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11202
diff changeset
   128
template <class T>
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11202
diff changeset
   129
inline void G1RootRegionScanClosure::do_oop_nv(T* p) {
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11202
diff changeset
   130
  T heap_oop = oopDesc::load_heap_oop(p);
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11202
diff changeset
   131
  if (!oopDesc::is_null(heap_oop)) {
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11202
diff changeset
   132
    oop obj = oopDesc::decode_heap_oop_not_null(heap_oop);
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11202
diff changeset
   133
    HeapRegion* hr = _g1h->heap_region_containing((HeapWord*) obj);
37413
2f71679d06dd 8077144: Concurrent mark initialization takes too long
tschatzl
parents: 37258
diff changeset
   134
    _cm->grayRoot(obj, hr);
11584
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11202
diff changeset
   135
  }
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11202
diff changeset
   136
}
e1df4d08a1f4 7127706: G1: re-enable survivors during the initial-mark pause
tonyp
parents: 11202
diff changeset
   137
11586
ccc217c177ee 7133038: G1: Some small profile based optimizations
johnc
parents: 11584
diff changeset
   138
template <class T>
35069
24d8a63c9486 8144701: Change three G1 remembererd set closures to be OopClosures
sjohanss
parents: 35068
diff changeset
   139
inline void G1Mux2Closure::do_oop_work(T* p) {
11586
ccc217c177ee 7133038: G1: Some small profile based optimizations
johnc
parents: 11584
diff changeset
   140
  // Apply first closure; then apply the second.
ccc217c177ee 7133038: G1: Some small profile based optimizations
johnc
parents: 11584
diff changeset
   141
  _c1->do_oop(p);
ccc217c177ee 7133038: G1: Some small profile based optimizations
johnc
parents: 11584
diff changeset
   142
  _c2->do_oop(p);
ccc217c177ee 7133038: G1: Some small profile based optimizations
johnc
parents: 11584
diff changeset
   143
}
35862
411842d0c882 8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents: 35069
diff changeset
   144
void G1Mux2Closure::do_oop(oop* p)       { do_oop_work(p); }
411842d0c882 8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents: 35069
diff changeset
   145
void G1Mux2Closure::do_oop(narrowOop* p) { do_oop_work(p); }
11586
ccc217c177ee 7133038: G1: Some small profile based optimizations
johnc
parents: 11584
diff changeset
   146
ccc217c177ee 7133038: G1: Some small profile based optimizations
johnc
parents: 11584
diff changeset
   147
template <class T>
35069
24d8a63c9486 8144701: Change three G1 remembererd set closures to be OopClosures
sjohanss
parents: 35068
diff changeset
   148
inline void G1TriggerClosure::do_oop_work(T* p) {
11586
ccc217c177ee 7133038: G1: Some small profile based optimizations
johnc
parents: 11584
diff changeset
   149
  // Record that this closure was actually applied (triggered).
ccc217c177ee 7133038: G1: Some small profile based optimizations
johnc
parents: 11584
diff changeset
   150
  _triggered = true;
ccc217c177ee 7133038: G1: Some small profile based optimizations
johnc
parents: 11584
diff changeset
   151
}
35862
411842d0c882 8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents: 35069
diff changeset
   152
void G1TriggerClosure::do_oop(oop* p)       { do_oop_work(p); }
411842d0c882 8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents: 35069
diff changeset
   153
void G1TriggerClosure::do_oop(narrowOop* p) { do_oop_work(p); }
11586
ccc217c177ee 7133038: G1: Some small profile based optimizations
johnc
parents: 11584
diff changeset
   154
ccc217c177ee 7133038: G1: Some small profile based optimizations
johnc
parents: 11584
diff changeset
   155
template <class T>
35069
24d8a63c9486 8144701: Change three G1 remembererd set closures to be OopClosures
sjohanss
parents: 35068
diff changeset
   156
inline void G1InvokeIfNotTriggeredClosure::do_oop_work(T* p) {
11586
ccc217c177ee 7133038: G1: Some small profile based optimizations
johnc
parents: 11584
diff changeset
   157
  if (!_trigger_cl->triggered()) {
ccc217c177ee 7133038: G1: Some small profile based optimizations
johnc
parents: 11584
diff changeset
   158
    _oop_cl->do_oop(p);
ccc217c177ee 7133038: G1: Some small profile based optimizations
johnc
parents: 11584
diff changeset
   159
  }
ccc217c177ee 7133038: G1: Some small profile based optimizations
johnc
parents: 11584
diff changeset
   160
}
35862
411842d0c882 8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents: 35069
diff changeset
   161
void G1InvokeIfNotTriggeredClosure::do_oop(oop* p)       { do_oop_work(p); }
411842d0c882 8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents: 35069
diff changeset
   162
void G1InvokeIfNotTriggeredClosure::do_oop(narrowOop* p) { do_oop_work(p); }
11586
ccc217c177ee 7133038: G1: Some small profile based optimizations
johnc
parents: 11584
diff changeset
   163
ccc217c177ee 7133038: G1: Some small profile based optimizations
johnc
parents: 11584
diff changeset
   164
template <class T>
35062
f3e0dee91918 8144486: Change G1UpdateRSOrPushRefOopClosure to inherit OopClosure
sjohanss
parents: 35051
diff changeset
   165
inline void G1UpdateRSOrPushRefOopClosure::do_oop_work(T* p) {
11586
ccc217c177ee 7133038: G1: Some small profile based optimizations
johnc
parents: 11584
diff changeset
   166
  oop obj = oopDesc::load_decode_heap_oop(p);
24106
dae9277bdf2a 8040722: G1: Clean up usages of heap_region_containing
brutisso
parents: 22905
diff changeset
   167
  if (obj == NULL) {
dae9277bdf2a 8040722: G1: Clean up usages of heap_region_containing
brutisso
parents: 22905
diff changeset
   168
    return;
dae9277bdf2a 8040722: G1: Clean up usages of heap_region_containing
brutisso
parents: 22905
diff changeset
   169
  }
dae9277bdf2a 8040722: G1: Clean up usages of heap_region_containing
brutisso
parents: 22905
diff changeset
   170
11586
ccc217c177ee 7133038: G1: Some small profile based optimizations
johnc
parents: 11584
diff changeset
   171
#ifdef ASSERT
ccc217c177ee 7133038: G1: Some small profile based optimizations
johnc
parents: 11584
diff changeset
   172
  // can't do because of races
ccc217c177ee 7133038: G1: Some small profile based optimizations
johnc
parents: 11584
diff changeset
   173
  // assert(obj == NULL || obj->is_oop(), "expected an oop");
36572
bdbf53032b6a 8149973: Optimize object alignment check in debug builds.
ddmitriev
parents: 35943
diff changeset
   174
  assert(check_obj_alignment(obj), "not oop aligned");
30173
13cf7580b000 8077413: Avoid use of Universe::heap() inside collectors
pliden
parents: 28213
diff changeset
   175
  assert(_g1->is_in_reserved(obj), "must be in heap");
11586
ccc217c177ee 7133038: G1: Some small profile based optimizations
johnc
parents: 11584
diff changeset
   176
#endif // ASSERT
ccc217c177ee 7133038: G1: Some small profile based optimizations
johnc
parents: 11584
diff changeset
   177
ccc217c177ee 7133038: G1: Some small profile based optimizations
johnc
parents: 11584
diff changeset
   178
  assert(_from != NULL, "from region must be non-NULL");
37258
d5fb0ee5e41a 8151499: gc/g1/g1OopClosures.inline.hpp: assert(_from->is_in_reserved(p)) failed: p is not in from
tschatzl
parents: 36572
diff changeset
   179
  assert(_from->is_in_reserved(p) ||
d5fb0ee5e41a 8151499: gc/g1/g1OopClosures.inline.hpp: assert(_from->is_in_reserved(p)) failed: p is not in from
tschatzl
parents: 36572
diff changeset
   180
         (_from->is_humongous() &&
d5fb0ee5e41a 8151499: gc/g1/g1OopClosures.inline.hpp: assert(_from->is_in_reserved(p)) failed: p is not in from
tschatzl
parents: 36572
diff changeset
   181
          _g1->heap_region_containing(p)->is_humongous() &&
d5fb0ee5e41a 8151499: gc/g1/g1OopClosures.inline.hpp: assert(_from->is_in_reserved(p)) failed: p is not in from
tschatzl
parents: 36572
diff changeset
   182
          _from->humongous_start_region() == _g1->heap_region_containing(p)->humongous_start_region()),
d5fb0ee5e41a 8151499: gc/g1/g1OopClosures.inline.hpp: assert(_from->is_in_reserved(p)) failed: p is not in from
tschatzl
parents: 36572
diff changeset
   183
         "p " PTR_FORMAT " is not in the same region %u or part of the correct humongous object starting at region %u.",
d5fb0ee5e41a 8151499: gc/g1/g1OopClosures.inline.hpp: assert(_from->is_in_reserved(p)) failed: p is not in from
tschatzl
parents: 36572
diff changeset
   184
         p2i(p), _from->hrm_index(), _from->humongous_start_region()->hrm_index());
11586
ccc217c177ee 7133038: G1: Some small profile based optimizations
johnc
parents: 11584
diff changeset
   185
ccc217c177ee 7133038: G1: Some small profile based optimizations
johnc
parents: 11584
diff changeset
   186
  HeapRegion* to = _g1->heap_region_containing(obj);
24106
dae9277bdf2a 8040722: G1: Clean up usages of heap_region_containing
brutisso
parents: 22905
diff changeset
   187
  if (_from == to) {
dae9277bdf2a 8040722: G1: Clean up usages of heap_region_containing
brutisso
parents: 22905
diff changeset
   188
    // Normally this closure should only be called with cross-region references.
dae9277bdf2a 8040722: G1: Clean up usages of heap_region_containing
brutisso
parents: 22905
diff changeset
   189
    // But since Java threads are manipulating the references concurrently and we
dae9277bdf2a 8040722: G1: Clean up usages of heap_region_containing
brutisso
parents: 22905
diff changeset
   190
    // reload the values things may have changed.
37258
d5fb0ee5e41a 8151499: gc/g1/g1OopClosures.inline.hpp: assert(_from->is_in_reserved(p)) failed: p is not in from
tschatzl
parents: 36572
diff changeset
   191
    // Also this check lets slip through references from a humongous continues region
d5fb0ee5e41a 8151499: gc/g1/g1OopClosures.inline.hpp: assert(_from->is_in_reserved(p)) failed: p is not in from
tschatzl
parents: 36572
diff changeset
   192
    // to its humongous start region, as they are in different regions, and adds a
d5fb0ee5e41a 8151499: gc/g1/g1OopClosures.inline.hpp: assert(_from->is_in_reserved(p)) failed: p is not in from
tschatzl
parents: 36572
diff changeset
   193
    // remembered set entry. This is benign (apart from memory usage), as we never
d5fb0ee5e41a 8151499: gc/g1/g1OopClosures.inline.hpp: assert(_from->is_in_reserved(p)) failed: p is not in from
tschatzl
parents: 36572
diff changeset
   194
    // try to either evacuate or eager reclaim these kind of regions.
24106
dae9277bdf2a 8040722: G1: Clean up usages of heap_region_containing
brutisso
parents: 22905
diff changeset
   195
    return;
dae9277bdf2a 8040722: G1: Clean up usages of heap_region_containing
brutisso
parents: 22905
diff changeset
   196
  }
11586
ccc217c177ee 7133038: G1: Some small profile based optimizations
johnc
parents: 11584
diff changeset
   197
24106
dae9277bdf2a 8040722: G1: Clean up usages of heap_region_containing
brutisso
parents: 22905
diff changeset
   198
  // The _record_refs_into_cset flag is true during the RSet
dae9277bdf2a 8040722: G1: Clean up usages of heap_region_containing
brutisso
parents: 22905
diff changeset
   199
  // updating part of an evacuation pause. It is false at all
dae9277bdf2a 8040722: G1: Clean up usages of heap_region_containing
brutisso
parents: 22905
diff changeset
   200
  // other times:
dae9277bdf2a 8040722: G1: Clean up usages of heap_region_containing
brutisso
parents: 22905
diff changeset
   201
  //  * rebuilding the remembered sets after a full GC
dae9277bdf2a 8040722: G1: Clean up usages of heap_region_containing
brutisso
parents: 22905
diff changeset
   202
  //  * during concurrent refinement.
dae9277bdf2a 8040722: G1: Clean up usages of heap_region_containing
brutisso
parents: 22905
diff changeset
   203
  //  * updating the remembered sets of regions in the collection
dae9277bdf2a 8040722: G1: Clean up usages of heap_region_containing
brutisso
parents: 22905
diff changeset
   204
  //    set in the event of an evacuation failure (when deferred
dae9277bdf2a 8040722: G1: Clean up usages of heap_region_containing
brutisso
parents: 22905
diff changeset
   205
  //    updates are enabled).
11586
ccc217c177ee 7133038: G1: Some small profile based optimizations
johnc
parents: 11584
diff changeset
   206
24106
dae9277bdf2a 8040722: G1: Clean up usages of heap_region_containing
brutisso
parents: 22905
diff changeset
   207
  if (_record_refs_into_cset && to->in_collection_set()) {
dae9277bdf2a 8040722: G1: Clean up usages of heap_region_containing
brutisso
parents: 22905
diff changeset
   208
    // We are recording references that point into the collection
dae9277bdf2a 8040722: G1: Clean up usages of heap_region_containing
brutisso
parents: 22905
diff changeset
   209
    // set and this particular reference does exactly that...
dae9277bdf2a 8040722: G1: Clean up usages of heap_region_containing
brutisso
parents: 22905
diff changeset
   210
    // If the referenced object has already been forwarded
dae9277bdf2a 8040722: G1: Clean up usages of heap_region_containing
brutisso
parents: 22905
diff changeset
   211
    // to itself, we are handling an evacuation failure and
dae9277bdf2a 8040722: G1: Clean up usages of heap_region_containing
brutisso
parents: 22905
diff changeset
   212
    // we have already visited/tried to copy this object
dae9277bdf2a 8040722: G1: Clean up usages of heap_region_containing
brutisso
parents: 22905
diff changeset
   213
    // there is no need to retry.
dae9277bdf2a 8040722: G1: Clean up usages of heap_region_containing
brutisso
parents: 22905
diff changeset
   214
    if (!self_forwarded(obj)) {
dae9277bdf2a 8040722: G1: Clean up usages of heap_region_containing
brutisso
parents: 22905
diff changeset
   215
      assert(_push_ref_cl != NULL, "should not be null");
dae9277bdf2a 8040722: G1: Clean up usages of heap_region_containing
brutisso
parents: 22905
diff changeset
   216
      // Push the reference in the refs queue of the G1ParScanThreadState
dae9277bdf2a 8040722: G1: Clean up usages of heap_region_containing
brutisso
parents: 22905
diff changeset
   217
      // instance for this worker thread.
dae9277bdf2a 8040722: G1: Clean up usages of heap_region_containing
brutisso
parents: 22905
diff changeset
   218
      _push_ref_cl->do_oop(p);
11586
ccc217c177ee 7133038: G1: Some small profile based optimizations
johnc
parents: 11584
diff changeset
   219
    }
13333
f51af0093d29 7173712: G1: Duplicated code in G1UpdateRSOrPushRefOopClosure::do_oop_nv()
johnc
parents: 11586
diff changeset
   220
24106
dae9277bdf2a 8040722: G1: Clean up usages of heap_region_containing
brutisso
parents: 22905
diff changeset
   221
    // Deferred updates to the CSet are either discarded (in the normal case),
dae9277bdf2a 8040722: G1: Clean up usages of heap_region_containing
brutisso
parents: 22905
diff changeset
   222
    // or processed (if an evacuation failure occurs) at the end
dae9277bdf2a 8040722: G1: Clean up usages of heap_region_containing
brutisso
parents: 22905
diff changeset
   223
    // of the collection.
dae9277bdf2a 8040722: G1: Clean up usages of heap_region_containing
brutisso
parents: 22905
diff changeset
   224
    // See G1RemSet::cleanup_after_oops_into_collection_set_do().
dae9277bdf2a 8040722: G1: Clean up usages of heap_region_containing
brutisso
parents: 22905
diff changeset
   225
  } else {
13333
f51af0093d29 7173712: G1: Duplicated code in G1UpdateRSOrPushRefOopClosure::do_oop_nv()
johnc
parents: 11586
diff changeset
   226
    // We either don't care about pushing references that point into the
f51af0093d29 7173712: G1: Duplicated code in G1UpdateRSOrPushRefOopClosure::do_oop_nv()
johnc
parents: 11586
diff changeset
   227
    // collection set (i.e. we're not during an evacuation pause) _or_
f51af0093d29 7173712: G1: Duplicated code in G1UpdateRSOrPushRefOopClosure::do_oop_nv()
johnc
parents: 11586
diff changeset
   228
    // the reference doesn't point into the collection set. Either way
f51af0093d29 7173712: G1: Duplicated code in G1UpdateRSOrPushRefOopClosure::do_oop_nv()
johnc
parents: 11586
diff changeset
   229
    // we add the reference directly to the RSet of the region containing
f51af0093d29 7173712: G1: Duplicated code in G1UpdateRSOrPushRefOopClosure::do_oop_nv()
johnc
parents: 11586
diff changeset
   230
    // the referenced object.
f51af0093d29 7173712: G1: Duplicated code in G1UpdateRSOrPushRefOopClosure::do_oop_nv()
johnc
parents: 11586
diff changeset
   231
    assert(to->rem_set() != NULL, "Need per-region 'into' remsets.");
f51af0093d29 7173712: G1: Duplicated code in G1UpdateRSOrPushRefOopClosure::do_oop_nv()
johnc
parents: 11586
diff changeset
   232
    to->rem_set()->add_reference(p, _worker_i);
11586
ccc217c177ee 7133038: G1: Some small profile based optimizations
johnc
parents: 11584
diff changeset
   233
  }
ccc217c177ee 7133038: G1: Some small profile based optimizations
johnc
parents: 11584
diff changeset
   234
}
35862
411842d0c882 8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents: 35069
diff changeset
   235
void G1UpdateRSOrPushRefOopClosure::do_oop(oop* p)       { do_oop_work(p); }
411842d0c882 8146395: Add inline qualifier in oop.hpp and fix inlining in gc files
goetz
parents: 35069
diff changeset
   236
void G1UpdateRSOrPushRefOopClosure::do_oop(narrowOop* p) { do_oop_work(p); }
11586
ccc217c177ee 7133038: G1: Some small profile based optimizations
johnc
parents: 11584
diff changeset
   237
33213
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   238
template <class T>
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   239
void G1ParCopyHelper::do_klass_barrier(T* p, oop new_obj) {
33786
ac8da6513351 8139867: Change how startsHumongous and continuesHumongous regions work in G1.
david
parents: 33740
diff changeset
   240
  if (_g1->heap_region_containing(new_obj)->is_young()) {
33213
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   241
    _scanned_klass->record_modified_oops();
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   242
  }
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   243
}
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   244
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   245
void G1ParCopyHelper::mark_object(oop obj) {
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   246
  assert(!_g1->heap_region_containing(obj)->in_collection_set(), "should not mark objects in the CSet");
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   247
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   248
  // We know that the object is not moving so it's safe to read its size.
37413
2f71679d06dd 8077144: Concurrent mark initialization takes too long
tschatzl
parents: 37258
diff changeset
   249
  _cm->grayRoot(obj);
33213
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   250
}
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   251
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   252
void G1ParCopyHelper::mark_forwarded_object(oop from_obj, oop to_obj) {
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   253
  assert(from_obj->is_forwarded(), "from obj should be forwarded");
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   254
  assert(from_obj->forwardee() == to_obj, "to obj should be the forwardee");
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   255
  assert(from_obj != to_obj, "should not be self-forwarded");
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   256
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   257
  assert(_g1->heap_region_containing(from_obj)->in_collection_set(), "from obj should be in the CSet");
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   258
  assert(!_g1->heap_region_containing(to_obj)->in_collection_set(), "should not mark objects in the CSet");
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   259
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   260
  // The object might be in the process of being copied by another
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   261
  // worker so we cannot trust that its to-space image is
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   262
  // well-formed. So we have to read its size from its from-space
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   263
  // image which we know should not be changing.
37413
2f71679d06dd 8077144: Concurrent mark initialization takes too long
tschatzl
parents: 37258
diff changeset
   264
  _cm->grayRoot(to_obj);
33213
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   265
}
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   266
35051
002d874a4284 8144714: Add extension point to G1 evacuation closures
mgerdin
parents: 34616
diff changeset
   267
template <G1Barrier barrier, G1Mark do_mark_object, bool use_ext>
33213
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   268
template <class T>
35067
0fa08ea22241 8144505: Change G1ParCopyHelper to inherit OopClosure
sjohanss
parents: 35062
diff changeset
   269
void G1ParCopyClosure<barrier, do_mark_object, use_ext>::do_oop_work(T* p) {
33213
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   270
  T heap_oop = oopDesc::load_heap_oop(p);
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   271
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   272
  if (oopDesc::is_null(heap_oop)) {
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   273
    return;
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   274
  }
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   275
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   276
  oop obj = oopDesc::decode_heap_oop_not_null(heap_oop);
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   277
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   278
  assert(_worker_id == _par_scan_state->worker_id(), "sanity");
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   279
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   280
  const InCSetState state = _g1->in_cset_state(obj);
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   281
  if (state.is_in_cset()) {
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   282
    oop forwardee;
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   283
    markOop m = obj->mark();
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   284
    if (m->is_marked()) {
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   285
      forwardee = (oop) m->decode_pointer();
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   286
    } else {
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   287
      forwardee = _par_scan_state->copy_to_survivor_space(state, obj, m);
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   288
    }
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   289
    assert(forwardee != NULL, "forwardee should not be NULL");
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   290
    oopDesc::encode_store_heap_oop(p, forwardee);
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   291
    if (do_mark_object != G1MarkNone && forwardee != obj) {
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   292
      // If the object is self-forwarded we don't need to explicitly
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   293
      // mark it, the evacuation failure protocol will do so.
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   294
      mark_forwarded_object(obj, forwardee);
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   295
    }
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   296
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   297
    if (barrier == G1BarrierKlass) {
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   298
      do_klass_barrier(p, forwardee);
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   299
    }
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   300
  } else {
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   301
    if (state.is_humongous()) {
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   302
      _g1->set_humongous_is_live(obj);
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   303
    }
35051
002d874a4284 8144714: Add extension point to G1 evacuation closures
mgerdin
parents: 34616
diff changeset
   304
002d874a4284 8144714: Add extension point to G1 evacuation closures
mgerdin
parents: 34616
diff changeset
   305
    if (use_ext && state.is_ext()) {
002d874a4284 8144714: Add extension point to G1 evacuation closures
mgerdin
parents: 34616
diff changeset
   306
      _par_scan_state->do_oop_ext(p);
002d874a4284 8144714: Add extension point to G1 evacuation closures
mgerdin
parents: 34616
diff changeset
   307
    }
33213
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   308
    // The object is not in collection set. If we're a root scanning
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   309
    // closure during an initial mark pause then attempt to mark the object.
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   310
    if (do_mark_object == G1MarkFromRoot) {
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   311
      mark_object(obj);
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   312
    }
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   313
  }
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   314
}
b937f634f56e 8138762: Refactor setup of evacuation closures in G1
mgerdin
parents: 32185
diff changeset
   315
30764
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30173
diff changeset
   316
#endif // SHARE_VM_GC_G1_G1OOPCLOSURES_INLINE_HPP