hotspot/src/share/vm/gc/shared/referenceProcessor.cpp
author kbarrett
Mon, 28 Dec 2015 13:48:43 -0500
changeset 35225 2b16576599d7
parent 35061 be6025ebffea
child 35475 c5e6cb508475
permissions -rw-r--r--
8071507: (ref) Clear phantom reference as soft and weak references do Summary: GC clears phantom refs on notification; update spec accordingly. Reviewed-by: mchung, jmasa
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     1
/*
29580
a67a581cfe11 8073315: Enable gcc -Wtype-limits and fix upcoming issues.
goetz
parents: 28510
diff changeset
     2
 * Copyright (c) 2001, 2015, 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: 4738
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 4738
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: 4738
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: 6759
diff changeset
    25
#include "precompiled.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6759
diff changeset
    26
#include "classfile/javaClasses.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6759
diff changeset
    27
#include "classfile/systemDictionary.hpp"
30764
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30579
diff changeset
    28
#include "gc/shared/collectedHeap.hpp"
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30579
diff changeset
    29
#include "gc/shared/collectedHeap.inline.hpp"
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30579
diff changeset
    30
#include "gc/shared/gcTimer.hpp"
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
    31
#include "gc/shared/gcTraceTime.inline.hpp"
30764
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30579
diff changeset
    32
#include "gc/shared/referencePolicy.hpp"
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30579
diff changeset
    33
#include "gc/shared/referenceProcessor.hpp"
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
    34
#include "logging/log.hpp"
33103
116b558af514 8136991: [REDO] Additional number of processed references printed with -XX:+PrintReferenceGC after JDK-8047125
brutisso
parents: 32817
diff changeset
    35
#include "memory/allocation.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6759
diff changeset
    36
#include "oops/oop.inline.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6759
diff changeset
    37
#include "runtime/java.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6759
diff changeset
    38
#include "runtime/jniHandles.hpp"
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    39
1606
dcf9714addbe 6684579: SoftReference processing can be made more efficient
ysr
parents: 1605
diff changeset
    40
ReferencePolicy* ReferenceProcessor::_always_clear_soft_ref_policy = NULL;
dcf9714addbe 6684579: SoftReference processing can be made more efficient
ysr
parents: 1605
diff changeset
    41
ReferencePolicy* ReferenceProcessor::_default_soft_ref_policy      = NULL;
10683
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
    42
jlong            ReferenceProcessor::_soft_ref_timestamp_clock = 0;
1606
dcf9714addbe 6684579: SoftReference processing can be made more efficient
ysr
parents: 1605
diff changeset
    43
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    44
void referenceProcessor_init() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
    45
  ReferenceProcessor::init_statics();
489c9b5090e2 Initial load
duke
parents:
diff changeset
    46
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    47
489c9b5090e2 Initial load
duke
parents:
diff changeset
    48
void ReferenceProcessor::init_statics() {
22551
9bf46d16dcc6 8025856: Fix typos in the GC code
jwilhelm
parents: 22496
diff changeset
    49
  // We need a monotonically non-decreasing time in ms but
11251
e29da6b5622b 7117303: VM uses non-monotonic time source and complains that it is non-monotonic
johnc
parents: 10747
diff changeset
    50
  // os::javaTimeMillis() does not guarantee monotonicity.
e29da6b5622b 7117303: VM uses non-monotonic time source and complains that it is non-monotonic
johnc
parents: 10747
diff changeset
    51
  jlong now = os::javaTimeNanos() / NANOSECS_PER_MILLISEC;
10683
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
    52
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
    53
  // Initialize the soft ref timestamp clock.
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
    54
  _soft_ref_timestamp_clock = now;
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
    55
  // Also update the soft ref clock in j.l.r.SoftReference
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
    56
  java_lang_ref_SoftReference::set_clock(_soft_ref_timestamp_clock);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    57
1606
dcf9714addbe 6684579: SoftReference processing can be made more efficient
ysr
parents: 1605
diff changeset
    58
  _always_clear_soft_ref_policy = new AlwaysClearPolicy();
33160
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 32817
diff changeset
    59
#if defined(COMPILER2) || INCLUDE_JVMCI
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 32817
diff changeset
    60
  _default_soft_ref_policy      = new LRUMaxHeapPolicy();
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 32817
diff changeset
    61
#else
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 32817
diff changeset
    62
  _default_soft_ref_policy      = new LRUCurrentHeapPolicy();
c59f1676d27e 8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents: 32817
diff changeset
    63
#endif
1606
dcf9714addbe 6684579: SoftReference processing can be made more efficient
ysr
parents: 1605
diff changeset
    64
  if (_always_clear_soft_ref_policy == NULL || _default_soft_ref_policy == NULL) {
dcf9714addbe 6684579: SoftReference processing can be made more efficient
ysr
parents: 1605
diff changeset
    65
    vm_exit_during_initialization("Could not allocate reference policy object");
dcf9714addbe 6684579: SoftReference processing can be made more efficient
ysr
parents: 1605
diff changeset
    66
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    67
  guarantee(RefDiscoveryPolicy == ReferenceBasedDiscovery ||
489c9b5090e2 Initial load
duke
parents:
diff changeset
    68
            RefDiscoveryPolicy == ReferentBasedDiscovery,
22775
52bc5222f5f1 8026849: Fix typos in the GC code, part 2
jwilhelm
parents: 22768
diff changeset
    69
            "Unrecognized RefDiscoveryPolicy");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    70
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    71
28212
647b7d0efb88 8066827: Remove ReferenceProcessor::clean_up_discovered_references()
kbarrett
parents: 26831
diff changeset
    72
void ReferenceProcessor::enable_discovery(bool check_no_refs) {
10683
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
    73
#ifdef ASSERT
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
    74
  // Verify that we're not currently discovering refs
28212
647b7d0efb88 8066827: Remove ReferenceProcessor::clean_up_discovered_references()
kbarrett
parents: 26831
diff changeset
    75
  assert(!_discovering_refs, "nested call?");
10683
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
    76
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
    77
  if (check_no_refs) {
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
    78
    // Verify that the discovered lists are empty
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
    79
    verify_no_references_recorded();
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
    80
  }
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
    81
#endif // ASSERT
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
    82
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
    83
  // Someone could have modified the value of the static
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
    84
  // field in the j.l.r.SoftReference class that holds the
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
    85
  // soft reference timestamp clock using reflection or
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
    86
  // Unsafe between GCs. Unconditionally update the static
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
    87
  // field in ReferenceProcessor here so that we use the new
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
    88
  // value during reference discovery.
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
    89
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
    90
  _soft_ref_timestamp_clock = java_lang_ref_SoftReference::clock();
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
    91
  _discovering_refs = true;
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
    92
}
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
    93
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    94
ReferenceProcessor::ReferenceProcessor(MemRegion span,
8688
493d12ccc6db 6668573: CMS: reference processing crash if ParallelCMSThreads > ParallelGCThreads
ysr
parents: 7420
diff changeset
    95
                                       bool      mt_processing,
11396
917d8673b5ef 7121618: Change type of number of GC workers to unsigned int.
jmasa
parents: 11251
diff changeset
    96
                                       uint      mt_processing_degree,
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
    97
                                       bool      mt_discovery,
11396
917d8673b5ef 7121618: Change type of number of GC workers to unsigned int.
jmasa
parents: 11251
diff changeset
    98
                                       uint      mt_discovery_degree,
8688
493d12ccc6db 6668573: CMS: reference processing crash if ParallelCMSThreads > ParallelGCThreads
ysr
parents: 7420
diff changeset
    99
                                       bool      atomic_discovery,
24845
e8b8894a77df 8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents: 24424
diff changeset
   100
                                       BoolObjectClosure* is_alive_non_header)  :
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   101
  _discovering_refs(false),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   102
  _enqueuing_is_done(false),
8688
493d12ccc6db 6668573: CMS: reference processing crash if ParallelCMSThreads > ParallelGCThreads
ysr
parents: 7420
diff changeset
   103
  _is_alive_non_header(is_alive_non_header),
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   104
  _processing_is_mt(mt_processing),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   105
  _next_id(0)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   106
{
489c9b5090e2 Initial load
duke
parents:
diff changeset
   107
  _span = span;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   108
  _discovery_is_atomic = atomic_discovery;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   109
  _discovery_is_mt     = mt_discovery;
11396
917d8673b5ef 7121618: Change type of number of GC workers to unsigned int.
jmasa
parents: 11251
diff changeset
   110
  _num_q               = MAX2(1U, mt_processing_degree);
8688
493d12ccc6db 6668573: CMS: reference processing crash if ParallelCMSThreads > ParallelGCThreads
ysr
parents: 7420
diff changeset
   111
  _max_num_q           = MAX2(_num_q, mt_discovery_degree);
10747
dfdb9eb56e49 7095243: Disambiguate ReferenceProcessor::_discoveredSoftRefs
johnc
parents: 10683
diff changeset
   112
  _discovered_refs     = NEW_C_HEAP_ARRAY(DiscoveredList,
13195
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 11396
diff changeset
   113
            _max_num_q * number_of_subclasses_of_ref(), mtGC);
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 11396
diff changeset
   114
10747
dfdb9eb56e49 7095243: Disambiguate ReferenceProcessor::_discoveredSoftRefs
johnc
parents: 10683
diff changeset
   115
  if (_discovered_refs == NULL) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   116
    vm_exit_during_initialization("Could not allocated RefProc Array");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   117
  }
10747
dfdb9eb56e49 7095243: Disambiguate ReferenceProcessor::_discoveredSoftRefs
johnc
parents: 10683
diff changeset
   118
  _discoveredSoftRefs    = &_discovered_refs[0];
6759
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   119
  _discoveredWeakRefs    = &_discoveredSoftRefs[_max_num_q];
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   120
  _discoveredFinalRefs   = &_discoveredWeakRefs[_max_num_q];
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   121
  _discoveredPhantomRefs = &_discoveredFinalRefs[_max_num_q];
28510
4f1d3611e0bc 8047125: (ref) More phantom object references
jmasa
parents: 28212
diff changeset
   122
  _discoveredCleanerRefs = &_discoveredPhantomRefs[_max_num_q];
10747
dfdb9eb56e49 7095243: Disambiguate ReferenceProcessor::_discoveredSoftRefs
johnc
parents: 10683
diff changeset
   123
dfdb9eb56e49 7095243: Disambiguate ReferenceProcessor::_discoveredSoftRefs
johnc
parents: 10683
diff changeset
   124
  // Initialize all entries to NULL
11396
917d8673b5ef 7121618: Change type of number of GC workers to unsigned int.
jmasa
parents: 11251
diff changeset
   125
  for (uint i = 0; i < _max_num_q * number_of_subclasses_of_ref(); i++) {
10747
dfdb9eb56e49 7095243: Disambiguate ReferenceProcessor::_discoveredSoftRefs
johnc
parents: 10683
diff changeset
   126
    _discovered_refs[i].set_head(NULL);
dfdb9eb56e49 7095243: Disambiguate ReferenceProcessor::_discoveredSoftRefs
johnc
parents: 10683
diff changeset
   127
    _discovered_refs[i].set_length(0);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   128
  }
10747
dfdb9eb56e49 7095243: Disambiguate ReferenceProcessor::_discoveredSoftRefs
johnc
parents: 10683
diff changeset
   129
8688
493d12ccc6db 6668573: CMS: reference processing crash if ParallelCMSThreads > ParallelGCThreads
ysr
parents: 7420
diff changeset
   130
  setup_policy(false /* default soft ref policy */);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   131
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   132
489c9b5090e2 Initial load
duke
parents:
diff changeset
   133
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   134
void ReferenceProcessor::verify_no_references_recorded() {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   135
  guarantee(!_discovering_refs, "Discovering refs?");
11396
917d8673b5ef 7121618: Change type of number of GC workers to unsigned int.
jmasa
parents: 11251
diff changeset
   136
  for (uint i = 0; i < _max_num_q * number_of_subclasses_of_ref(); i++) {
10747
dfdb9eb56e49 7095243: Disambiguate ReferenceProcessor::_discoveredSoftRefs
johnc
parents: 10683
diff changeset
   137
    guarantee(_discovered_refs[i].is_empty(),
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   138
              "Found non-empty discovered list");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   139
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   140
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   141
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   142
489c9b5090e2 Initial load
duke
parents:
diff changeset
   143
void ReferenceProcessor::weak_oops_do(OopClosure* f) {
11396
917d8673b5ef 7121618: Change type of number of GC workers to unsigned int.
jmasa
parents: 11251
diff changeset
   144
  for (uint i = 0; i < _max_num_q * number_of_subclasses_of_ref(); i++) {
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   145
    if (UseCompressedOops) {
10747
dfdb9eb56e49 7095243: Disambiguate ReferenceProcessor::_discoveredSoftRefs
johnc
parents: 10683
diff changeset
   146
      f->do_oop((narrowOop*)_discovered_refs[i].adr_head());
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   147
    } else {
10747
dfdb9eb56e49 7095243: Disambiguate ReferenceProcessor::_discoveredSoftRefs
johnc
parents: 10683
diff changeset
   148
      f->do_oop((oop*)_discovered_refs[i].adr_head());
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   149
    }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   150
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   151
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   152
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   153
void ReferenceProcessor::update_soft_ref_master_clock() {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   154
  // Update (advance) the soft ref master clock field. This must be done
489c9b5090e2 Initial load
duke
parents:
diff changeset
   155
  // after processing the soft ref list.
11251
e29da6b5622b 7117303: VM uses non-monotonic time source and complains that it is non-monotonic
johnc
parents: 10747
diff changeset
   156
22551
9bf46d16dcc6 8025856: Fix typos in the GC code
jwilhelm
parents: 22496
diff changeset
   157
  // We need a monotonically non-decreasing time in ms but
11251
e29da6b5622b 7117303: VM uses non-monotonic time source and complains that it is non-monotonic
johnc
parents: 10747
diff changeset
   158
  // os::javaTimeMillis() does not guarantee monotonicity.
e29da6b5622b 7117303: VM uses non-monotonic time source and complains that it is non-monotonic
johnc
parents: 10747
diff changeset
   159
  jlong now = os::javaTimeNanos() / NANOSECS_PER_MILLISEC;
10683
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
   160
  jlong soft_ref_clock = java_lang_ref_SoftReference::clock();
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
   161
  assert(soft_ref_clock == _soft_ref_timestamp_clock, "soft ref clocks out of sync");
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
   162
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   163
  NOT_PRODUCT(
10683
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
   164
  if (now < _soft_ref_timestamp_clock) {
29800
fa5f7a2bf717 8076073: shared: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents: 29580
diff changeset
   165
    warning("time warp: " JLONG_FORMAT " to " JLONG_FORMAT,
10683
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
   166
            _soft_ref_timestamp_clock, now);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   167
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   168
  )
11251
e29da6b5622b 7117303: VM uses non-monotonic time source and complains that it is non-monotonic
johnc
parents: 10747
diff changeset
   169
  // The values of now and _soft_ref_timestamp_clock are set using
e29da6b5622b 7117303: VM uses non-monotonic time source and complains that it is non-monotonic
johnc
parents: 10747
diff changeset
   170
  // javaTimeNanos(), which is guaranteed to be monotonically
e29da6b5622b 7117303: VM uses non-monotonic time source and complains that it is non-monotonic
johnc
parents: 10747
diff changeset
   171
  // non-decreasing provided the underlying platform provides such
e29da6b5622b 7117303: VM uses non-monotonic time source and complains that it is non-monotonic
johnc
parents: 10747
diff changeset
   172
  // a time source (and it is bug free).
22551
9bf46d16dcc6 8025856: Fix typos in the GC code
jwilhelm
parents: 22496
diff changeset
   173
  // In product mode, however, protect ourselves from non-monotonicity.
10683
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
   174
  if (now > _soft_ref_timestamp_clock) {
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
   175
    _soft_ref_timestamp_clock = now;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   176
    java_lang_ref_SoftReference::set_clock(now);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   177
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   178
  // Else leave clock stalled at its old value until time progresses
489c9b5090e2 Initial load
duke
parents:
diff changeset
   179
  // past clock value.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   180
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   181
18025
b7bcf7497f93 8005849: JEP 167: Event-Based JVM Tracing
sla
parents: 17625
diff changeset
   182
size_t ReferenceProcessor::total_count(DiscoveredList lists[]) {
b7bcf7497f93 8005849: JEP 167: Event-Based JVM Tracing
sla
parents: 17625
diff changeset
   183
  size_t total = 0;
b7bcf7497f93 8005849: JEP 167: Event-Based JVM Tracing
sla
parents: 17625
diff changeset
   184
  for (uint i = 0; i < _max_num_q; ++i) {
b7bcf7497f93 8005849: JEP 167: Event-Based JVM Tracing
sla
parents: 17625
diff changeset
   185
    total += lists[i].length();
b7bcf7497f93 8005849: JEP 167: Event-Based JVM Tracing
sla
parents: 17625
diff changeset
   186
  }
b7bcf7497f93 8005849: JEP 167: Event-Based JVM Tracing
sla
parents: 17625
diff changeset
   187
  return total;
b7bcf7497f93 8005849: JEP 167: Event-Based JVM Tracing
sla
parents: 17625
diff changeset
   188
}
b7bcf7497f93 8005849: JEP 167: Event-Based JVM Tracing
sla
parents: 17625
diff changeset
   189
b7bcf7497f93 8005849: JEP 167: Event-Based JVM Tracing
sla
parents: 17625
diff changeset
   190
ReferenceProcessorStats ReferenceProcessor::process_discovered_references(
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   191
  BoolObjectClosure*           is_alive,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   192
  OopClosure*                  keep_alive,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   193
  VoidClosure*                 complete_gc,
18025
b7bcf7497f93 8005849: JEP 167: Event-Based JVM Tracing
sla
parents: 17625
diff changeset
   194
  AbstractRefProcTaskExecutor* task_executor,
33107
77bf0d2069a3 8134953: Make the GC ID available in a central place
brutisso
parents: 33105
diff changeset
   195
  GCTimer*                     gc_timer) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   196
489c9b5090e2 Initial load
duke
parents:
diff changeset
   197
  assert(!enqueuing_is_done(), "If here enqueuing should not be complete");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   198
  // Stop treating discovered references specially.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   199
  disable_discovery();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   200
10683
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
   201
  // If discovery was concurrent, someone could have modified
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
   202
  // the value of the static field in the j.l.r.SoftReference
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
   203
  // class that holds the soft reference timestamp clock using
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
   204
  // reflection or Unsafe between when discovery was enabled and
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
   205
  // now. Unconditionally update the static field in ReferenceProcessor
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
   206
  // here so that we use the new value during processing of the
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
   207
  // discovered soft refs.
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
   208
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
   209
  _soft_ref_timestamp_clock = java_lang_ref_SoftReference::clock();
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
   210
33103
116b558af514 8136991: [REDO] Additional number of processed references printed with -XX:+PrintReferenceGC after JDK-8047125
brutisso
parents: 32817
diff changeset
   211
  // Include cleaners in phantom statistics.  We expect Cleaner
116b558af514 8136991: [REDO] Additional number of processed references printed with -XX:+PrintReferenceGC after JDK-8047125
brutisso
parents: 32817
diff changeset
   212
  // references to be temporary, and don't want to deal with
116b558af514 8136991: [REDO] Additional number of processed references printed with -XX:+PrintReferenceGC after JDK-8047125
brutisso
parents: 32817
diff changeset
   213
  // possible incompatibilities arising from making it more visible.
116b558af514 8136991: [REDO] Additional number of processed references printed with -XX:+PrintReferenceGC after JDK-8047125
brutisso
parents: 32817
diff changeset
   214
  ReferenceProcessorStats stats(
116b558af514 8136991: [REDO] Additional number of processed references printed with -XX:+PrintReferenceGC after JDK-8047125
brutisso
parents: 32817
diff changeset
   215
      total_count(_discoveredSoftRefs),
116b558af514 8136991: [REDO] Additional number of processed references printed with -XX:+PrintReferenceGC after JDK-8047125
brutisso
parents: 32817
diff changeset
   216
      total_count(_discoveredWeakRefs),
116b558af514 8136991: [REDO] Additional number of processed references printed with -XX:+PrintReferenceGC after JDK-8047125
brutisso
parents: 32817
diff changeset
   217
      total_count(_discoveredFinalRefs),
116b558af514 8136991: [REDO] Additional number of processed references printed with -XX:+PrintReferenceGC after JDK-8047125
brutisso
parents: 32817
diff changeset
   218
      total_count(_discoveredPhantomRefs) + total_count(_discoveredCleanerRefs));
116b558af514 8136991: [REDO] Additional number of processed references printed with -XX:+PrintReferenceGC after JDK-8047125
brutisso
parents: 32817
diff changeset
   219
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   220
  // Soft references
489c9b5090e2 Initial load
duke
parents:
diff changeset
   221
  {
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   222
    GCTraceTime(Debug, gc, ref) tt("SoftReference", gc_timer);
33103
116b558af514 8136991: [REDO] Additional number of processed references printed with -XX:+PrintReferenceGC after JDK-8047125
brutisso
parents: 32817
diff changeset
   223
    process_discovered_reflist(_discoveredSoftRefs, _current_soft_ref_policy, true,
116b558af514 8136991: [REDO] Additional number of processed references printed with -XX:+PrintReferenceGC after JDK-8047125
brutisso
parents: 32817
diff changeset
   224
                               is_alive, keep_alive, complete_gc, task_executor);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   225
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   226
489c9b5090e2 Initial load
duke
parents:
diff changeset
   227
  update_soft_ref_master_clock();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   228
489c9b5090e2 Initial load
duke
parents:
diff changeset
   229
  // Weak references
489c9b5090e2 Initial load
duke
parents:
diff changeset
   230
  {
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   231
    GCTraceTime(Debug, gc, ref) tt("WeakReference", gc_timer);
33103
116b558af514 8136991: [REDO] Additional number of processed references printed with -XX:+PrintReferenceGC after JDK-8047125
brutisso
parents: 32817
diff changeset
   232
    process_discovered_reflist(_discoveredWeakRefs, NULL, true,
116b558af514 8136991: [REDO] Additional number of processed references printed with -XX:+PrintReferenceGC after JDK-8047125
brutisso
parents: 32817
diff changeset
   233
                               is_alive, keep_alive, complete_gc, task_executor);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   234
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   235
489c9b5090e2 Initial load
duke
parents:
diff changeset
   236
  // Final references
489c9b5090e2 Initial load
duke
parents:
diff changeset
   237
  {
35225
2b16576599d7 8071507: (ref) Clear phantom reference as soft and weak references do
kbarrett
parents: 35061
diff changeset
   238
    GCTraceTime(Debug, gc, ref) tt("FinalReference", gc_timer);
33103
116b558af514 8136991: [REDO] Additional number of processed references printed with -XX:+PrintReferenceGC after JDK-8047125
brutisso
parents: 32817
diff changeset
   239
    process_discovered_reflist(_discoveredFinalRefs, NULL, false,
116b558af514 8136991: [REDO] Additional number of processed references printed with -XX:+PrintReferenceGC after JDK-8047125
brutisso
parents: 32817
diff changeset
   240
                               is_alive, keep_alive, complete_gc, task_executor);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   241
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   242
489c9b5090e2 Initial load
duke
parents:
diff changeset
   243
  // Phantom references
489c9b5090e2 Initial load
duke
parents:
diff changeset
   244
  {
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   245
    GCTraceTime(Debug, gc, ref) tt("PhantomReference", gc_timer);
35225
2b16576599d7 8071507: (ref) Clear phantom reference as soft and weak references do
kbarrett
parents: 35061
diff changeset
   246
    process_discovered_reflist(_discoveredPhantomRefs, NULL, true,
33103
116b558af514 8136991: [REDO] Additional number of processed references printed with -XX:+PrintReferenceGC after JDK-8047125
brutisso
parents: 32817
diff changeset
   247
                               is_alive, keep_alive, complete_gc, task_executor);
28510
4f1d3611e0bc 8047125: (ref) More phantom object references
jmasa
parents: 28212
diff changeset
   248
33103
116b558af514 8136991: [REDO] Additional number of processed references printed with -XX:+PrintReferenceGC after JDK-8047125
brutisso
parents: 32817
diff changeset
   249
    // Process cleaners, but include them in phantom timing.  We expect
32817
acc2744fd84b 8136627: Backout JDK-8133818 Additional number of processed references printed with -XX:+PrintReferenceGC after JDK-8047125
kbarrett
parents: 32623
diff changeset
   250
    // Cleaner references to be temporary, and don't want to deal with
acc2744fd84b 8136627: Backout JDK-8133818 Additional number of processed references printed with -XX:+PrintReferenceGC after JDK-8047125
kbarrett
parents: 32623
diff changeset
   251
    // possible incompatibilities arising from making it more visible.
33103
116b558af514 8136991: [REDO] Additional number of processed references printed with -XX:+PrintReferenceGC after JDK-8047125
brutisso
parents: 32817
diff changeset
   252
    process_discovered_reflist(_discoveredCleanerRefs, NULL, true,
28510
4f1d3611e0bc 8047125: (ref) More phantom object references
jmasa
parents: 28212
diff changeset
   253
                                 is_alive, keep_alive, complete_gc, task_executor);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   254
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   255
489c9b5090e2 Initial load
duke
parents:
diff changeset
   256
  // Weak global JNI references. It would make more sense (semantically) to
489c9b5090e2 Initial load
duke
parents:
diff changeset
   257
  // traverse these simultaneously with the regular weak references above, but
489c9b5090e2 Initial load
duke
parents:
diff changeset
   258
  // that is not how the JDK1.2 specification is. See #4126360. Native code can
489c9b5090e2 Initial load
duke
parents:
diff changeset
   259
  // thus use JNI weak references to circumvent the phantom references and
489c9b5090e2 Initial load
duke
parents:
diff changeset
   260
  // resurrect a "post-mortem" object.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   261
  {
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   262
    GCTraceTime(Debug, gc, ref) tt("JNI Weak Reference", gc_timer);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   263
    if (task_executor != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   264
      task_executor->set_single_threaded_mode();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   265
    }
32817
acc2744fd84b 8136627: Backout JDK-8133818 Additional number of processed references printed with -XX:+PrintReferenceGC after JDK-8047125
kbarrett
parents: 32623
diff changeset
   266
    process_phaseJNI(is_alive, keep_alive, complete_gc);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   267
  }
18025
b7bcf7497f93 8005849: JEP 167: Event-Based JVM Tracing
sla
parents: 17625
diff changeset
   268
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   269
  log_debug(gc, ref)("Ref Counts: Soft: " SIZE_FORMAT " Weak: " SIZE_FORMAT " Final: " SIZE_FORMAT " Phantom: " SIZE_FORMAT,
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   270
                     stats.soft_count(), stats.weak_count(), stats.final_count(), stats.phantom_count());
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   271
  log_develop_trace(gc, ref)("JNI Weak Reference count: " SIZE_FORMAT, count_jni_refs());
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   272
33103
116b558af514 8136991: [REDO] Additional number of processed references printed with -XX:+PrintReferenceGC after JDK-8047125
brutisso
parents: 32817
diff changeset
   273
  return stats;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   274
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   275
489c9b5090e2 Initial load
duke
parents:
diff changeset
   276
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   277
// Calculate the number of jni handles.
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   278
size_t ReferenceProcessor::count_jni_refs() {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   279
  class AlwaysAliveClosure: public BoolObjectClosure {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   280
  public:
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   281
    virtual bool do_object_b(oop obj) { return true; }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   282
  };
489c9b5090e2 Initial load
duke
parents:
diff changeset
   283
489c9b5090e2 Initial load
duke
parents:
diff changeset
   284
  class CountHandleClosure: public OopClosure {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   285
  private:
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   286
    size_t _count;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   287
  public:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   288
    CountHandleClosure(): _count(0) {}
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   289
    void do_oop(oop* unused)       { _count++; }
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   290
    void do_oop(narrowOop* unused) { ShouldNotReachHere(); }
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   291
    size_t count() { return _count; }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   292
  };
489c9b5090e2 Initial load
duke
parents:
diff changeset
   293
  CountHandleClosure global_handle_count;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   294
  AlwaysAliveClosure always_alive;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   295
  JNIHandles::weak_oops_do(&always_alive, &global_handle_count);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   296
  return global_handle_count.count();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   297
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   298
#endif
489c9b5090e2 Initial load
duke
parents:
diff changeset
   299
32817
acc2744fd84b 8136627: Backout JDK-8133818 Additional number of processed references printed with -XX:+PrintReferenceGC after JDK-8047125
kbarrett
parents: 32623
diff changeset
   300
void ReferenceProcessor::process_phaseJNI(BoolObjectClosure* is_alive,
acc2744fd84b 8136627: Backout JDK-8133818 Additional number of processed references printed with -XX:+PrintReferenceGC after JDK-8047125
kbarrett
parents: 32623
diff changeset
   301
                                          OopClosure*        keep_alive,
acc2744fd84b 8136627: Backout JDK-8133818 Additional number of processed references printed with -XX:+PrintReferenceGC after JDK-8047125
kbarrett
parents: 32623
diff changeset
   302
                                          VoidClosure*       complete_gc) {
acc2744fd84b 8136627: Backout JDK-8133818 Additional number of processed references printed with -XX:+PrintReferenceGC after JDK-8047125
kbarrett
parents: 32623
diff changeset
   303
  JNIHandles::weak_oops_do(is_alive, keep_alive);
32607
c69a7b61ab02 8133818: Additional number of processed references printed with -XX:+PrintReferenceGC after JDK-8047125
ysr
parents: 30766
diff changeset
   304
  complete_gc->do_void();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   305
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   306
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   307
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   308
template <class T>
4493
9204129f065e 6843629: Make current hotspot build part of jdk5 control build
phh
parents: 3690
diff changeset
   309
bool enqueue_discovered_ref_helper(ReferenceProcessor* ref,
9204129f065e 6843629: Make current hotspot build part of jdk5 control build
phh
parents: 3690
diff changeset
   310
                                   AbstractRefProcTaskExecutor* task_executor) {
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   311
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   312
  // Remember old value of pending references list
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   313
  T* pending_list_addr = (T*)java_lang_ref_Reference::pending_list_addr();
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   314
  T old_pending_list_value = *pending_list_addr;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   315
489c9b5090e2 Initial load
duke
parents:
diff changeset
   316
  // Enqueue references that are not made active again, and
489c9b5090e2 Initial load
duke
parents:
diff changeset
   317
  // clear the decks for the next collection (cycle).
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   318
  ref->enqueue_discovered_reflists((HeapWord*)pending_list_addr, task_executor);
22496
383a5bdef99d 8029255: G1: Reference processing should not enqueue references on the shared SATB queue
pliden
parents: 20282
diff changeset
   319
  // Do the post-barrier on pending_list_addr missed in
383a5bdef99d 8029255: G1: Reference processing should not enqueue references on the shared SATB queue
pliden
parents: 20282
diff changeset
   320
  // enqueue_discovered_reflist.
383a5bdef99d 8029255: G1: Reference processing should not enqueue references on the shared SATB queue
pliden
parents: 20282
diff changeset
   321
  oopDesc::bs()->write_ref_field(pending_list_addr, oopDesc::load_decode_heap_oop(pending_list_addr));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   322
489c9b5090e2 Initial load
duke
parents:
diff changeset
   323
  // Stop treating discovered references specially.
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   324
  ref->disable_discovery();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   325
489c9b5090e2 Initial load
duke
parents:
diff changeset
   326
  // Return true if new pending references were added
489c9b5090e2 Initial load
duke
parents:
diff changeset
   327
  return old_pending_list_value != *pending_list_addr;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   328
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   329
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   330
bool ReferenceProcessor::enqueue_discovered_references(AbstractRefProcTaskExecutor* task_executor) {
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   331
  if (UseCompressedOops) {
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   332
    return enqueue_discovered_ref_helper<narrowOop>(this, task_executor);
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   333
  } else {
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   334
    return enqueue_discovered_ref_helper<oop>(this, task_executor);
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   335
  }
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   336
}
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   337
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   338
void ReferenceProcessor::enqueue_discovered_reflist(DiscoveredList& refs_list,
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   339
                                                    HeapWord* pending_list_addr) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   340
  // Given a list of refs linked through the "discovered" field
10526
3e92f211533f 4965777: GC changes to support use of discovered field for pending references
ysr
parents: 10524
diff changeset
   341
  // (java.lang.ref.Reference.discovered), self-loop their "next" field
3e92f211533f 4965777: GC changes to support use of discovered field for pending references
ysr
parents: 10524
diff changeset
   342
  // thus distinguishing them from active References, then
3e92f211533f 4965777: GC changes to support use of discovered field for pending references
ysr
parents: 10524
diff changeset
   343
  // prepend them to the pending list.
24845
e8b8894a77df 8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents: 24424
diff changeset
   344
  //
e8b8894a77df 8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents: 24424
diff changeset
   345
  // The Java threads will see the Reference objects linked together through
e8b8894a77df 8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents: 24424
diff changeset
   346
  // the discovered field. Instead of trying to do the write barrier updates
e8b8894a77df 8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents: 24424
diff changeset
   347
  // in all places in the reference processor where we manipulate the discovered
e8b8894a77df 8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents: 24424
diff changeset
   348
  // field we make sure to do the barrier here where we anyway iterate through
e8b8894a77df 8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents: 24424
diff changeset
   349
  // all linked Reference objects. Note that it is important to not dirty any
e8b8894a77df 8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents: 24424
diff changeset
   350
  // cards during reference processing since this will cause card table
e8b8894a77df 8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents: 24424
diff changeset
   351
  // verification to fail for G1.
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   352
  log_develop_trace(gc, ref)("ReferenceProcessor::enqueue_discovered_reflist list " INTPTR_FORMAT, p2i(refs_list.head()));
10524
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
   353
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
   354
  oop obj = NULL;
10526
3e92f211533f 4965777: GC changes to support use of discovered field for pending references
ysr
parents: 10524
diff changeset
   355
  oop next_d = refs_list.head();
30558
baec90a2699f 8031401: Remove unused code in the reference processor
kbarrett
parents: 29807
diff changeset
   356
  // Walk down the list, self-looping the next field
baec90a2699f 8031401: Remove unused code in the reference processor
kbarrett
parents: 29807
diff changeset
   357
  // so that the References are not considered active.
baec90a2699f 8031401: Remove unused code in the reference processor
kbarrett
parents: 29807
diff changeset
   358
  while (obj != next_d) {
baec90a2699f 8031401: Remove unused code in the reference processor
kbarrett
parents: 29807
diff changeset
   359
    obj = next_d;
33611
9abd65805e19 8139203: Consistent naming for klass type predicates
coleenp
parents: 33198
diff changeset
   360
    assert(obj->is_instance(), "should be an instance object");
9abd65805e19 8139203: Consistent naming for klass type predicates
coleenp
parents: 33198
diff changeset
   361
    assert(InstanceKlass::cast(obj->klass())->is_reference_instance_klass(), "should be reference object");
30558
baec90a2699f 8031401: Remove unused code in the reference processor
kbarrett
parents: 29807
diff changeset
   362
    next_d = java_lang_ref_Reference::discovered(obj);
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   363
    log_develop_trace(gc, ref)("        obj " INTPTR_FORMAT "/next_d " INTPTR_FORMAT, p2i(obj), p2i(next_d));
30558
baec90a2699f 8031401: Remove unused code in the reference processor
kbarrett
parents: 29807
diff changeset
   364
    assert(java_lang_ref_Reference::next(obj) == NULL,
baec90a2699f 8031401: Remove unused code in the reference processor
kbarrett
parents: 29807
diff changeset
   365
           "Reference not active; should not be discovered");
baec90a2699f 8031401: Remove unused code in the reference processor
kbarrett
parents: 29807
diff changeset
   366
    // Self-loop next, so as to make Ref not active.
baec90a2699f 8031401: Remove unused code in the reference processor
kbarrett
parents: 29807
diff changeset
   367
    java_lang_ref_Reference::set_next_raw(obj, obj);
baec90a2699f 8031401: Remove unused code in the reference processor
kbarrett
parents: 29807
diff changeset
   368
    if (next_d != obj) {
baec90a2699f 8031401: Remove unused code in the reference processor
kbarrett
parents: 29807
diff changeset
   369
      oopDesc::bs()->write_ref_field(java_lang_ref_Reference::discovered_addr(obj), next_d);
baec90a2699f 8031401: Remove unused code in the reference processor
kbarrett
parents: 29807
diff changeset
   370
    } else {
baec90a2699f 8031401: Remove unused code in the reference processor
kbarrett
parents: 29807
diff changeset
   371
      // This is the last object.
baec90a2699f 8031401: Remove unused code in the reference processor
kbarrett
parents: 29807
diff changeset
   372
      // Swap refs_list into pending_list_addr and
baec90a2699f 8031401: Remove unused code in the reference processor
kbarrett
parents: 29807
diff changeset
   373
      // set obj's discovered to what we read from pending_list_addr.
baec90a2699f 8031401: Remove unused code in the reference processor
kbarrett
parents: 29807
diff changeset
   374
      oop old = oopDesc::atomic_exchange_oop(refs_list.head(), pending_list_addr);
baec90a2699f 8031401: Remove unused code in the reference processor
kbarrett
parents: 29807
diff changeset
   375
      // Need post-barrier on pending_list_addr. See enqueue_discovered_ref_helper() above.
baec90a2699f 8031401: Remove unused code in the reference processor
kbarrett
parents: 29807
diff changeset
   376
      java_lang_ref_Reference::set_discovered_raw(obj, old); // old may be NULL
baec90a2699f 8031401: Remove unused code in the reference processor
kbarrett
parents: 29807
diff changeset
   377
      oopDesc::bs()->write_ref_field(java_lang_ref_Reference::discovered_addr(obj), old);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   378
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   379
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   380
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   381
489c9b5090e2 Initial load
duke
parents:
diff changeset
   382
// Parallel enqueue task
489c9b5090e2 Initial load
duke
parents:
diff changeset
   383
class RefProcEnqueueTask: public AbstractRefProcTaskExecutor::EnqueueTask {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   384
public:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   385
  RefProcEnqueueTask(ReferenceProcessor& ref_processor,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   386
                     DiscoveredList      discovered_refs[],
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   387
                     HeapWord*           pending_list_addr,
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   388
                     int                 n_queues)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   389
    : EnqueueTask(ref_processor, discovered_refs,
10524
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
   390
                  pending_list_addr, n_queues)
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   391
  { }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   392
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   393
  virtual void work(unsigned int work_id) {
8688
493d12ccc6db 6668573: CMS: reference processing crash if ParallelCMSThreads > ParallelGCThreads
ysr
parents: 7420
diff changeset
   394
    assert(work_id < (unsigned int)_ref_processor.max_num_q(), "Index out-of-bounds");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   395
    // Simplest first cut: static partitioning.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   396
    int index = work_id;
6759
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   397
    // The increment on "index" must correspond to the maximum number of queues
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   398
    // (n_queues) with which that ReferenceProcessor was created.  That
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   399
    // is because of the "clever" way the discovered references lists were
8688
493d12ccc6db 6668573: CMS: reference processing crash if ParallelCMSThreads > ParallelGCThreads
ysr
parents: 7420
diff changeset
   400
    // allocated and are indexed into.
493d12ccc6db 6668573: CMS: reference processing crash if ParallelCMSThreads > ParallelGCThreads
ysr
parents: 7420
diff changeset
   401
    assert(_n_queues == (int) _ref_processor.max_num_q(), "Different number not expected");
6759
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   402
    for (int j = 0;
10670
4ea0e7d2ffbc 6484982: G1: process references during evacuation pauses
johnc
parents: 10526
diff changeset
   403
         j < ReferenceProcessor::number_of_subclasses_of_ref();
6759
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   404
         j++, index += _n_queues) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   405
      _ref_processor.enqueue_discovered_reflist(
489c9b5090e2 Initial load
duke
parents:
diff changeset
   406
        _refs_lists[index], _pending_list_addr);
10524
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
   407
      _refs_lists[index].set_head(NULL);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   408
      _refs_lists[index].set_length(0);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   409
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   410
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   411
};
489c9b5090e2 Initial load
duke
parents:
diff changeset
   412
489c9b5090e2 Initial load
duke
parents:
diff changeset
   413
// Enqueue references that are not made active again
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   414
void ReferenceProcessor::enqueue_discovered_reflists(HeapWord* pending_list_addr,
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   415
  AbstractRefProcTaskExecutor* task_executor) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   416
  if (_processing_is_mt && task_executor != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   417
    // Parallel code
10747
dfdb9eb56e49 7095243: Disambiguate ReferenceProcessor::_discoveredSoftRefs
johnc
parents: 10683
diff changeset
   418
    RefProcEnqueueTask tsk(*this, _discovered_refs,
10524
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
   419
                           pending_list_addr, _max_num_q);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   420
    task_executor->execute(tsk);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   421
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   422
    // Serial code: call the parent class's implementation
11396
917d8673b5ef 7121618: Change type of number of GC workers to unsigned int.
jmasa
parents: 11251
diff changeset
   423
    for (uint i = 0; i < _max_num_q * number_of_subclasses_of_ref(); i++) {
10747
dfdb9eb56e49 7095243: Disambiguate ReferenceProcessor::_discoveredSoftRefs
johnc
parents: 10683
diff changeset
   424
      enqueue_discovered_reflist(_discovered_refs[i], pending_list_addr);
dfdb9eb56e49 7095243: Disambiguate ReferenceProcessor::_discoveredSoftRefs
johnc
parents: 10683
diff changeset
   425
      _discovered_refs[i].set_head(NULL);
dfdb9eb56e49 7095243: Disambiguate ReferenceProcessor::_discoveredSoftRefs
johnc
parents: 10683
diff changeset
   426
      _discovered_refs[i].set_length(0);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   427
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   428
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   429
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   430
10670
4ea0e7d2ffbc 6484982: G1: process references during evacuation pauses
johnc
parents: 10526
diff changeset
   431
void DiscoveredListIterator::load_ptrs(DEBUG_ONLY(bool allow_null_referent)) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   432
  _discovered_addr = java_lang_ref_Reference::discovered_addr(_ref);
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   433
  oop discovered = java_lang_ref_Reference::discovered(_ref);
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   434
  assert(_discovered_addr && discovered->is_oop_or_null(),
33105
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 33103
diff changeset
   435
         "Expected an oop or NULL for discovered field at " PTR_FORMAT, p2i(discovered));
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   436
  _next = discovered;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   437
  _referent_addr = java_lang_ref_Reference::referent_addr(_ref);
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   438
  _referent = java_lang_ref_Reference::referent(_ref);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   439
  assert(Universe::heap()->is_in_reserved_or_null(_referent),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   440
         "Wrong oop found in java.lang.Reference object");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   441
  assert(allow_null_referent ?
489c9b5090e2 Initial load
duke
parents:
diff changeset
   442
             _referent->is_oop_or_null()
489c9b5090e2 Initial load
duke
parents:
diff changeset
   443
           : _referent->is_oop(),
33105
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 33103
diff changeset
   444
         "Expected an oop%s for referent field at " PTR_FORMAT,
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 33103
diff changeset
   445
         (allow_null_referent ? " or NULL" : ""),
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 33103
diff changeset
   446
         p2i(_referent));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   447
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   448
10670
4ea0e7d2ffbc 6484982: G1: process references during evacuation pauses
johnc
parents: 10526
diff changeset
   449
void DiscoveredListIterator::remove() {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   450
  assert(_ref->is_oop(), "Dropping a bad reference");
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   451
  oop_store_raw(_discovered_addr, NULL);
10524
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
   452
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   453
  // First _prev_next ref actually points into DiscoveredList (gross).
10524
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
   454
  oop new_next;
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
   455
  if (_next == _ref) {
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
   456
    // At the end of the list, we should make _prev point to itself.
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
   457
    // If _ref is the first ref, then _prev_next will be in the DiscoveredList,
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
   458
    // and _prev will be NULL.
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
   459
    new_next = _prev;
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
   460
  } else {
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
   461
    new_next = _next;
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
   462
  }
22768
363a44995a71 8031703: Missing post-barrier in ReferenceProcessor
pliden
parents: 22551
diff changeset
   463
  // Remove Reference object from discovered list. Note that G1 does not need a
363a44995a71 8031703: Missing post-barrier in ReferenceProcessor
pliden
parents: 22551
diff changeset
   464
  // pre-barrier here because we know the Reference has already been found/marked,
363a44995a71 8031703: Missing post-barrier in ReferenceProcessor
pliden
parents: 22551
diff changeset
   465
  // that's how it ended up in the discovered list in the first place.
363a44995a71 8031703: Missing post-barrier in ReferenceProcessor
pliden
parents: 22551
diff changeset
   466
  oop_store_raw(_prev_next, new_next);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   467
  NOT_PRODUCT(_removed++);
1605
6b43d186eb8d 6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents: 1388
diff changeset
   468
  _refs_list.dec_length(1);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   469
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   470
10670
4ea0e7d2ffbc 6484982: G1: process references during evacuation pauses
johnc
parents: 10526
diff changeset
   471
void DiscoveredListIterator::clear_referent() {
4ea0e7d2ffbc 6484982: G1: process references during evacuation pauses
johnc
parents: 10526
diff changeset
   472
  oop_store_raw(_referent_addr, NULL);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   473
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   474
489c9b5090e2 Initial load
duke
parents:
diff changeset
   475
// NOTE: process_phase*() are largely similar, and at a high level
489c9b5090e2 Initial load
duke
parents:
diff changeset
   476
// merely iterate over the extant list applying a predicate to
489c9b5090e2 Initial load
duke
parents:
diff changeset
   477
// each of its elements and possibly removing that element from the
489c9b5090e2 Initial load
duke
parents:
diff changeset
   478
// list and applying some further closures to that element.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   479
// We should consider the possibility of replacing these
489c9b5090e2 Initial load
duke
parents:
diff changeset
   480
// process_phase*() methods by abstracting them into
489c9b5090e2 Initial load
duke
parents:
diff changeset
   481
// a single general iterator invocation that receives appropriate
489c9b5090e2 Initial load
duke
parents:
diff changeset
   482
// closures that accomplish this work.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   483
489c9b5090e2 Initial load
duke
parents:
diff changeset
   484
// (SoftReferences only) Traverse the list and remove any SoftReferences whose
489c9b5090e2 Initial load
duke
parents:
diff changeset
   485
// referents are not alive, but that should be kept alive for policy reasons.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   486
// Keep alive the transitive closure of all such referents.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   487
void
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   488
ReferenceProcessor::process_phase1(DiscoveredList&    refs_list,
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   489
                                   ReferencePolicy*   policy,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   490
                                   BoolObjectClosure* is_alive,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   491
                                   OopClosure*        keep_alive,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   492
                                   VoidClosure*       complete_gc) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   493
  assert(policy != NULL, "Must have a non-NULL policy");
24845
e8b8894a77df 8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents: 24424
diff changeset
   494
  DiscoveredListIterator iter(refs_list, keep_alive, is_alive);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   495
  // Decide which softly reachable refs should be kept alive.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   496
  while (iter.has_next()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   497
    iter.load_ptrs(DEBUG_ONLY(!discovery_is_atomic() /* allow_null_referent */));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   498
    bool referent_is_dead = (iter.referent() != NULL) && !iter.is_referent_alive();
10683
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
   499
    if (referent_is_dead &&
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
   500
        !policy->should_clear_reference(iter.obj(), _soft_ref_timestamp_clock)) {
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   501
      log_develop_trace(gc, ref)("Dropping reference (" INTPTR_FORMAT ": %s"  ") by policy",
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   502
                                 p2i(iter.obj()), iter.obj()->klass()->internal_name());
1605
6b43d186eb8d 6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents: 1388
diff changeset
   503
      // Remove Reference object from list
6b43d186eb8d 6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents: 1388
diff changeset
   504
      iter.remove();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   505
      // keep the referent around
489c9b5090e2 Initial load
duke
parents:
diff changeset
   506
      iter.make_referent_alive();
1605
6b43d186eb8d 6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents: 1388
diff changeset
   507
      iter.move_to_next();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   508
    } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   509
      iter.next();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   510
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   511
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   512
  // Close the reachable set
489c9b5090e2 Initial load
duke
parents:
diff changeset
   513
  complete_gc->do_void();
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   514
  log_develop_trace(gc, ref)(" Dropped " SIZE_FORMAT " dead Refs out of " SIZE_FORMAT " discovered Refs by policy, from list " INTPTR_FORMAT,
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   515
                             iter.removed(), iter.processed(), p2i(refs_list.head()));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   516
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   517
489c9b5090e2 Initial load
duke
parents:
diff changeset
   518
// Traverse the list and remove any Refs that are not active, or
489c9b5090e2 Initial load
duke
parents:
diff changeset
   519
// whose referents are either alive or NULL.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   520
void
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   521
ReferenceProcessor::pp2_work(DiscoveredList&    refs_list,
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   522
                             BoolObjectClosure* is_alive,
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   523
                             OopClosure*        keep_alive) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   524
  assert(discovery_is_atomic(), "Error");
24845
e8b8894a77df 8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents: 24424
diff changeset
   525
  DiscoveredListIterator iter(refs_list, keep_alive, is_alive);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   526
  while (iter.has_next()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   527
    iter.load_ptrs(DEBUG_ONLY(false /* allow_null_referent */));
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   528
    DEBUG_ONLY(oop next = java_lang_ref_Reference::next(iter.obj());)
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   529
    assert(next == NULL, "Should not discover inactive Reference");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   530
    if (iter.is_referent_alive()) {
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   531
      log_develop_trace(gc, ref)("Dropping strongly reachable reference (" INTPTR_FORMAT ": %s)",
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   532
                                 p2i(iter.obj()), iter.obj()->klass()->internal_name());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   533
      // The referent is reachable after all.
1605
6b43d186eb8d 6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents: 1388
diff changeset
   534
      // Remove Reference object from list.
6b43d186eb8d 6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents: 1388
diff changeset
   535
      iter.remove();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   536
      // Update the referent pointer as necessary: Note that this
489c9b5090e2 Initial load
duke
parents:
diff changeset
   537
      // should not entail any recursive marking because the
489c9b5090e2 Initial load
duke
parents:
diff changeset
   538
      // referent must already have been traversed.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   539
      iter.make_referent_alive();
1605
6b43d186eb8d 6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents: 1388
diff changeset
   540
      iter.move_to_next();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   541
    } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   542
      iter.next();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   543
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   544
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   545
  NOT_PRODUCT(
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   546
    if (iter.processed() > 0) {
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   547
      log_develop_trace(gc, ref)(" Dropped " SIZE_FORMAT " active Refs out of " SIZE_FORMAT
29800
fa5f7a2bf717 8076073: shared: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents: 29580
diff changeset
   548
        " Refs in discovered list " INTPTR_FORMAT,
fa5f7a2bf717 8076073: shared: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents: 29580
diff changeset
   549
        iter.removed(), iter.processed(), p2i(refs_list.head()));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   550
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   551
  )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   552
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   553
489c9b5090e2 Initial load
duke
parents:
diff changeset
   554
void
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   555
ReferenceProcessor::pp2_work_concurrent_discovery(DiscoveredList&    refs_list,
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   556
                                                  BoolObjectClosure* is_alive,
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   557
                                                  OopClosure*        keep_alive,
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   558
                                                  VoidClosure*       complete_gc) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   559
  assert(!discovery_is_atomic(), "Error");
24845
e8b8894a77df 8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents: 24424
diff changeset
   560
  DiscoveredListIterator iter(refs_list, keep_alive, is_alive);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   561
  while (iter.has_next()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   562
    iter.load_ptrs(DEBUG_ONLY(true /* allow_null_referent */));
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   563
    HeapWord* next_addr = java_lang_ref_Reference::next_addr(iter.obj());
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   564
    oop next = java_lang_ref_Reference::next(iter.obj());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   565
    if ((iter.referent() == NULL || iter.is_referent_alive() ||
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   566
         next != NULL)) {
33105
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 33103
diff changeset
   567
      assert(next->is_oop_or_null(), "Expected an oop or NULL for next field at " PTR_FORMAT, p2i(next));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   568
      // Remove Reference object from list
489c9b5090e2 Initial load
duke
parents:
diff changeset
   569
      iter.remove();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   570
      // Trace the cohorts
489c9b5090e2 Initial load
duke
parents:
diff changeset
   571
      iter.make_referent_alive();
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   572
      if (UseCompressedOops) {
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   573
        keep_alive->do_oop((narrowOop*)next_addr);
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   574
      } else {
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   575
        keep_alive->do_oop((oop*)next_addr);
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   576
      }
1605
6b43d186eb8d 6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents: 1388
diff changeset
   577
      iter.move_to_next();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   578
    } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   579
      iter.next();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   580
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   581
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   582
  // Now close the newly reachable set
489c9b5090e2 Initial load
duke
parents:
diff changeset
   583
  complete_gc->do_void();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   584
  NOT_PRODUCT(
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   585
    if (iter.processed() > 0) {
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   586
      log_develop_trace(gc, ref)(" Dropped " SIZE_FORMAT " active Refs out of " SIZE_FORMAT
29800
fa5f7a2bf717 8076073: shared: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents: 29580
diff changeset
   587
        " Refs in discovered list " INTPTR_FORMAT,
fa5f7a2bf717 8076073: shared: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents: 29580
diff changeset
   588
        iter.removed(), iter.processed(), p2i(refs_list.head()));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   589
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   590
  )
489c9b5090e2 Initial load
duke
parents:
diff changeset
   591
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   592
489c9b5090e2 Initial load
duke
parents:
diff changeset
   593
// Traverse the list and process the referents, by either
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   594
// clearing them or keeping them (and their reachable
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   595
// closure) alive.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   596
void
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   597
ReferenceProcessor::process_phase3(DiscoveredList&    refs_list,
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   598
                                   bool               clear_referent,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   599
                                   BoolObjectClosure* is_alive,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   600
                                   OopClosure*        keep_alive,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   601
                                   VoidClosure*       complete_gc) {
6759
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   602
  ResourceMark rm;
24845
e8b8894a77df 8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents: 24424
diff changeset
   603
  DiscoveredListIterator iter(refs_list, keep_alive, is_alive);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   604
  while (iter.has_next()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   605
    iter.load_ptrs(DEBUG_ONLY(false /* allow_null_referent */));
489c9b5090e2 Initial load
duke
parents:
diff changeset
   606
    if (clear_referent) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   607
      // NULL out referent pointer
489c9b5090e2 Initial load
duke
parents:
diff changeset
   608
      iter.clear_referent();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   609
    } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   610
      // keep the referent around
489c9b5090e2 Initial load
duke
parents:
diff changeset
   611
      iter.make_referent_alive();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   612
    }
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   613
    log_develop_trace(gc, ref)("Adding %sreference (" INTPTR_FORMAT ": %s) as pending",
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   614
                               clear_referent ? "cleared " : "", p2i(iter.obj()), iter.obj()->klass()->internal_name());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   615
    assert(iter.obj()->is_oop(UseConcMarkSweepGC), "Adding a bad reference");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   616
    iter.next();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   617
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   618
  // Close the reachable set
489c9b5090e2 Initial load
duke
parents:
diff changeset
   619
  complete_gc->do_void();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   620
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   621
489c9b5090e2 Initial load
duke
parents:
diff changeset
   622
void
10524
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
   623
ReferenceProcessor::clear_discovered_references(DiscoveredList& refs_list) {
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
   624
  oop obj = NULL;
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
   625
  oop next = refs_list.head();
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
   626
  while (next != obj) {
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
   627
    obj = next;
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
   628
    next = java_lang_ref_Reference::discovered(obj);
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   629
    java_lang_ref_Reference::set_discovered_raw(obj, NULL);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   630
  }
10524
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
   631
  refs_list.set_head(NULL);
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   632
  refs_list.set_length(0);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   633
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   634
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents: 360
diff changeset
   635
void ReferenceProcessor::abandon_partial_discovery() {
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents: 360
diff changeset
   636
  // loop over the lists
11396
917d8673b5ef 7121618: Change type of number of GC workers to unsigned int.
jmasa
parents: 11251
diff changeset
   637
  for (uint i = 0; i < _max_num_q * number_of_subclasses_of_ref(); i++) {
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   638
    if ((i % _max_num_q) == 0) {
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   639
      log_develop_trace(gc, ref)("Abandoning %s discovered list", list_name(i));
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents: 360
diff changeset
   640
    }
30766
8afe4435d599 8058265: No callers of ReferenceProcessor::clear_discovered_references
kbarrett
parents: 30764
diff changeset
   641
    clear_discovered_references(_discovered_refs[i]);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   642
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   643
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   644
489c9b5090e2 Initial load
duke
parents:
diff changeset
   645
class RefProcPhase1Task: public AbstractRefProcTaskExecutor::ProcessTask {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   646
public:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   647
  RefProcPhase1Task(ReferenceProcessor& ref_processor,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   648
                    DiscoveredList      refs_lists[],
489c9b5090e2 Initial load
duke
parents:
diff changeset
   649
                    ReferencePolicy*    policy,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   650
                    bool                marks_oops_alive)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   651
    : ProcessTask(ref_processor, refs_lists, marks_oops_alive),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   652
      _policy(policy)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   653
  { }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   654
  virtual void work(unsigned int i, BoolObjectClosure& is_alive,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   655
                    OopClosure& keep_alive,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   656
                    VoidClosure& complete_gc)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   657
  {
6759
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   658
    Thread* thr = Thread::current();
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   659
    int refs_list_index = ((WorkerThread*)thr)->id();
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   660
    _ref_processor.process_phase1(_refs_lists[refs_list_index], _policy,
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   661
                                  &is_alive, &keep_alive, &complete_gc);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   662
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   663
private:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   664
  ReferencePolicy* _policy;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   665
};
489c9b5090e2 Initial load
duke
parents:
diff changeset
   666
489c9b5090e2 Initial load
duke
parents:
diff changeset
   667
class RefProcPhase2Task: public AbstractRefProcTaskExecutor::ProcessTask {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   668
public:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   669
  RefProcPhase2Task(ReferenceProcessor& ref_processor,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   670
                    DiscoveredList      refs_lists[],
489c9b5090e2 Initial load
duke
parents:
diff changeset
   671
                    bool                marks_oops_alive)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   672
    : ProcessTask(ref_processor, refs_lists, marks_oops_alive)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   673
  { }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   674
  virtual void work(unsigned int i, BoolObjectClosure& is_alive,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   675
                    OopClosure& keep_alive,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   676
                    VoidClosure& complete_gc)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   677
  {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   678
    _ref_processor.process_phase2(_refs_lists[i],
489c9b5090e2 Initial load
duke
parents:
diff changeset
   679
                                  &is_alive, &keep_alive, &complete_gc);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   680
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   681
};
489c9b5090e2 Initial load
duke
parents:
diff changeset
   682
489c9b5090e2 Initial load
duke
parents:
diff changeset
   683
class RefProcPhase3Task: public AbstractRefProcTaskExecutor::ProcessTask {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   684
public:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   685
  RefProcPhase3Task(ReferenceProcessor& ref_processor,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   686
                    DiscoveredList      refs_lists[],
489c9b5090e2 Initial load
duke
parents:
diff changeset
   687
                    bool                clear_referent,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   688
                    bool                marks_oops_alive)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   689
    : ProcessTask(ref_processor, refs_lists, marks_oops_alive),
489c9b5090e2 Initial load
duke
parents:
diff changeset
   690
      _clear_referent(clear_referent)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   691
  { }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   692
  virtual void work(unsigned int i, BoolObjectClosure& is_alive,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   693
                    OopClosure& keep_alive,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   694
                    VoidClosure& complete_gc)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   695
  {
6759
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   696
    // Don't use "refs_list_index" calculated in this way because
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   697
    // balance_queues() has moved the Ref's into the first n queues.
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   698
    // Thread* thr = Thread::current();
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   699
    // int refs_list_index = ((WorkerThread*)thr)->id();
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   700
    // _ref_processor.process_phase3(_refs_lists[refs_list_index], _clear_referent,
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   701
    _ref_processor.process_phase3(_refs_lists[i], _clear_referent,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   702
                                  &is_alive, &keep_alive, &complete_gc);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   703
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   704
private:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   705
  bool _clear_referent;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   706
};
489c9b5090e2 Initial load
duke
parents:
diff changeset
   707
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   708
#ifndef PRODUCT
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   709
void ReferenceProcessor::log_reflist_counts(DiscoveredList ref_lists[], size_t total_refs) {
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   710
  if (!log_is_enabled(Trace, gc, ref)) {
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   711
    return;
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   712
  }
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   713
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   714
  stringStream st;
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   715
  for (uint i = 0; i < _max_num_q; ++i) {
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   716
    st.print(SIZE_FORMAT " ", ref_lists[i].length());
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   717
  }
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   718
  log_develop_trace(gc, ref)("%s= " SIZE_FORMAT, st.as_string(), total_refs);
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   719
}
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   720
#endif
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   721
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   722
// Balances reference queues.
6759
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   723
// Move entries from all queues[0, 1, ..., _max_num_q-1] to
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   724
// queues[0, 1, ..., _num_q-1] because only the first _num_q
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   725
// corresponding to the active workers will be processed.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   726
void ReferenceProcessor::balance_queues(DiscoveredList ref_lists[])
489c9b5090e2 Initial load
duke
parents:
diff changeset
   727
{
489c9b5090e2 Initial load
duke
parents:
diff changeset
   728
  // calculate total length
489c9b5090e2 Initial load
duke
parents:
diff changeset
   729
  size_t total_refs = 0;
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   730
  log_develop_trace(gc, ref)("Balance ref_lists ");
6759
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   731
11396
917d8673b5ef 7121618: Change type of number of GC workers to unsigned int.
jmasa
parents: 11251
diff changeset
   732
  for (uint i = 0; i < _max_num_q; ++i) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   733
    total_refs += ref_lists[i].length();
6759
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   734
    }
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   735
  log_reflist_counts(ref_lists, total_refs);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   736
  size_t avg_refs = total_refs / _num_q + 1;
11396
917d8673b5ef 7121618: Change type of number of GC workers to unsigned int.
jmasa
parents: 11251
diff changeset
   737
  uint to_idx = 0;
917d8673b5ef 7121618: Change type of number of GC workers to unsigned int.
jmasa
parents: 11251
diff changeset
   738
  for (uint from_idx = 0; from_idx < _max_num_q; from_idx++) {
6759
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   739
    bool move_all = false;
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   740
    if (from_idx >= _num_q) {
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   741
      move_all = ref_lists[from_idx].length() > 0;
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   742
    }
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   743
    while ((ref_lists[from_idx].length() > avg_refs) ||
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   744
           move_all) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   745
      assert(to_idx < _num_q, "Sanity Check!");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   746
      if (ref_lists[to_idx].length() < avg_refs) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   747
        // move superfluous refs
6759
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   748
        size_t refs_to_move;
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   749
        // Move all the Ref's if the from queue will not be processed.
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   750
        if (move_all) {
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   751
          refs_to_move = MIN2(ref_lists[from_idx].length(),
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   752
                              avg_refs - ref_lists[to_idx].length());
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   753
        } else {
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   754
          refs_to_move = MIN2(ref_lists[from_idx].length() - avg_refs,
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   755
                              avg_refs - ref_lists[to_idx].length());
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   756
        }
10524
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
   757
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
   758
        assert(refs_to_move > 0, "otherwise the code below will fail");
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
   759
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   760
        oop move_head = ref_lists[from_idx].head();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   761
        oop move_tail = move_head;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   762
        oop new_head  = move_head;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   763
        // find an element to split the list on
489c9b5090e2 Initial load
duke
parents:
diff changeset
   764
        for (size_t j = 0; j < refs_to_move; ++j) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   765
          move_tail = new_head;
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   766
          new_head = java_lang_ref_Reference::discovered(new_head);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   767
        }
10524
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
   768
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
   769
        // Add the chain to the to list.
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
   770
        if (ref_lists[to_idx].head() == NULL) {
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
   771
          // to list is empty. Make a loop at the end.
24845
e8b8894a77df 8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents: 24424
diff changeset
   772
          java_lang_ref_Reference::set_discovered_raw(move_tail, move_tail);
10524
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
   773
        } else {
24845
e8b8894a77df 8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents: 24424
diff changeset
   774
          java_lang_ref_Reference::set_discovered_raw(move_tail, ref_lists[to_idx].head());
10524
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
   775
        }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   776
        ref_lists[to_idx].set_head(move_head);
1605
6b43d186eb8d 6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents: 1388
diff changeset
   777
        ref_lists[to_idx].inc_length(refs_to_move);
10524
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
   778
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
   779
        // Remove the chain from the from list.
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
   780
        if (move_tail == new_head) {
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
   781
          // We found the end of the from list.
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
   782
          ref_lists[from_idx].set_head(NULL);
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
   783
        } else {
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
   784
          ref_lists[from_idx].set_head(new_head);
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
   785
        }
1605
6b43d186eb8d 6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents: 1388
diff changeset
   786
        ref_lists[from_idx].dec_length(refs_to_move);
6759
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   787
        if (ref_lists[from_idx].length() == 0) {
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   788
          break;
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   789
        }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   790
      } else {
6759
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   791
        to_idx = (to_idx + 1) % _num_q;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   792
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   793
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   794
  }
6759
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   795
#ifdef ASSERT
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   796
  size_t balanced_total_refs = 0;
11396
917d8673b5ef 7121618: Change type of number of GC workers to unsigned int.
jmasa
parents: 11251
diff changeset
   797
  for (uint i = 0; i < _max_num_q; ++i) {
6759
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   798
    balanced_total_refs += ref_lists[i].length();
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   799
    }
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   800
  log_reflist_counts(ref_lists, balanced_total_refs);
6759
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   801
  assert(total_refs == balanced_total_refs, "Balancing was incomplete");
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   802
#endif
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   803
}
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   804
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   805
void ReferenceProcessor::balance_all_queues() {
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   806
  balance_queues(_discoveredSoftRefs);
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   807
  balance_queues(_discoveredWeakRefs);
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   808
  balance_queues(_discoveredFinalRefs);
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   809
  balance_queues(_discoveredPhantomRefs);
28510
4f1d3611e0bc 8047125: (ref) More phantom object references
jmasa
parents: 28212
diff changeset
   810
  balance_queues(_discoveredCleanerRefs);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   811
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   812
33103
116b558af514 8136991: [REDO] Additional number of processed references printed with -XX:+PrintReferenceGC after JDK-8047125
brutisso
parents: 32817
diff changeset
   813
void ReferenceProcessor::process_discovered_reflist(
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   814
  DiscoveredList               refs_lists[],
489c9b5090e2 Initial load
duke
parents:
diff changeset
   815
  ReferencePolicy*             policy,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   816
  bool                         clear_referent,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   817
  BoolObjectClosure*           is_alive,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   818
  OopClosure*                  keep_alive,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   819
  VoidClosure*                 complete_gc,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   820
  AbstractRefProcTaskExecutor* task_executor)
489c9b5090e2 Initial load
duke
parents:
diff changeset
   821
{
6759
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   822
  bool mt_processing = task_executor != NULL && _processing_is_mt;
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   823
  // If discovery used MT and a dynamic number of GC threads, then
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   824
  // the queues must be balanced for correctness if fewer than the
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   825
  // maximum number of queues were used.  The number of queue used
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   826
  // during discovery may be different than the number to be used
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   827
  // for processing so don't depend of _num_q < _max_num_q as part
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   828
  // of the test.
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   829
  bool must_balance = _discovery_is_mt;
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   830
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   831
  if ((mt_processing && ParallelRefProcBalancingEnabled) ||
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   832
      must_balance) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   833
    balance_queues(refs_lists);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   834
  }
18025
b7bcf7497f93 8005849: JEP 167: Event-Based JVM Tracing
sla
parents: 17625
diff changeset
   835
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   836
  // Phase 1 (soft refs only):
489c9b5090e2 Initial load
duke
parents:
diff changeset
   837
  // . Traverse the list and remove any SoftReferences whose
489c9b5090e2 Initial load
duke
parents:
diff changeset
   838
  //   referents are not alive, but that should be kept alive for
489c9b5090e2 Initial load
duke
parents:
diff changeset
   839
  //   policy reasons. Keep alive the transitive closure of all
489c9b5090e2 Initial load
duke
parents:
diff changeset
   840
  //   such referents.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   841
  if (policy != NULL) {
6759
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   842
    if (mt_processing) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   843
      RefProcPhase1Task phase1(*this, refs_lists, policy, true /*marks_oops_alive*/);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   844
      task_executor->execute(phase1);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   845
    } else {
11396
917d8673b5ef 7121618: Change type of number of GC workers to unsigned int.
jmasa
parents: 11251
diff changeset
   846
      for (uint i = 0; i < _max_num_q; i++) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   847
        process_phase1(refs_lists[i], policy,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   848
                       is_alive, keep_alive, complete_gc);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   849
      }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   850
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   851
  } else { // policy == NULL
489c9b5090e2 Initial load
duke
parents:
diff changeset
   852
    assert(refs_lists != _discoveredSoftRefs,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   853
           "Policy must be specified for soft references.");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   854
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   855
489c9b5090e2 Initial load
duke
parents:
diff changeset
   856
  // Phase 2:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   857
  // . Traverse the list and remove any refs whose referents are alive.
6759
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   858
  if (mt_processing) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   859
    RefProcPhase2Task phase2(*this, refs_lists, !discovery_is_atomic() /*marks_oops_alive*/);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   860
    task_executor->execute(phase2);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   861
  } else {
11396
917d8673b5ef 7121618: Change type of number of GC workers to unsigned int.
jmasa
parents: 11251
diff changeset
   862
    for (uint i = 0; i < _max_num_q; i++) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   863
      process_phase2(refs_lists[i], is_alive, keep_alive, complete_gc);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   864
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   865
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   866
489c9b5090e2 Initial load
duke
parents:
diff changeset
   867
  // Phase 3:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   868
  // . Traverse the list and process referents as appropriate.
6759
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
   869
  if (mt_processing) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   870
    RefProcPhase3Task phase3(*this, refs_lists, clear_referent, true /*marks_oops_alive*/);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   871
    task_executor->execute(phase3);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   872
  } else {
11396
917d8673b5ef 7121618: Change type of number of GC workers to unsigned int.
jmasa
parents: 11251
diff changeset
   873
    for (uint i = 0; i < _max_num_q; i++) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   874
      process_phase3(refs_lists[i], clear_referent,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   875
                     is_alive, keep_alive, complete_gc);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   876
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   877
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   878
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   879
489c9b5090e2 Initial load
duke
parents:
diff changeset
   880
inline DiscoveredList* ReferenceProcessor::get_discovered_list(ReferenceType rt) {
11396
917d8673b5ef 7121618: Change type of number of GC workers to unsigned int.
jmasa
parents: 11251
diff changeset
   881
  uint id = 0;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   882
  // Determine the queue index to use for this object.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   883
  if (_discovery_is_mt) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   884
    // During a multi-threaded discovery phase,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   885
    // each thread saves to its "own" list.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   886
    Thread* thr = Thread::current();
7399
4ecd771fa2d1 6983204: G1: Nightly test nsk/regression/b4958615 failing with +ExplicitGCInvokesConcurrent
johnc
parents: 7397
diff changeset
   887
    id = thr->as_Worker_thread()->id();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   888
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   889
    // single-threaded discovery, we save in round-robin
489c9b5090e2 Initial load
duke
parents:
diff changeset
   890
    // fashion to each of the lists.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   891
    if (_processing_is_mt) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   892
      id = next_id();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   893
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   894
  }
29580
a67a581cfe11 8073315: Enable gcc -Wtype-limits and fix upcoming issues.
goetz
parents: 28510
diff changeset
   895
  assert(id < _max_num_q, "Id is out-of-bounds (call Freud?)");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   896
489c9b5090e2 Initial load
duke
parents:
diff changeset
   897
  // Get the discovered queue to which we will add
489c9b5090e2 Initial load
duke
parents:
diff changeset
   898
  DiscoveredList* list = NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   899
  switch (rt) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   900
    case REF_OTHER:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   901
      // Unknown reference type, no special treatment
489c9b5090e2 Initial load
duke
parents:
diff changeset
   902
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   903
    case REF_SOFT:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   904
      list = &_discoveredSoftRefs[id];
489c9b5090e2 Initial load
duke
parents:
diff changeset
   905
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   906
    case REF_WEAK:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   907
      list = &_discoveredWeakRefs[id];
489c9b5090e2 Initial load
duke
parents:
diff changeset
   908
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   909
    case REF_FINAL:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   910
      list = &_discoveredFinalRefs[id];
489c9b5090e2 Initial load
duke
parents:
diff changeset
   911
      break;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   912
    case REF_PHANTOM:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   913
      list = &_discoveredPhantomRefs[id];
489c9b5090e2 Initial load
duke
parents:
diff changeset
   914
      break;
28510
4f1d3611e0bc 8047125: (ref) More phantom object references
jmasa
parents: 28212
diff changeset
   915
    case REF_CLEANER:
4f1d3611e0bc 8047125: (ref) More phantom object references
jmasa
parents: 28212
diff changeset
   916
      list = &_discoveredCleanerRefs[id];
4f1d3611e0bc 8047125: (ref) More phantom object references
jmasa
parents: 28212
diff changeset
   917
      break;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   918
    case REF_NONE:
13738
d67be49a5beb 7195833: NPG: Rename instanceClassLoaderKlass, instanceRefKlass and instanceMirrorKlass
coleenp
parents: 13728
diff changeset
   919
      // we should not reach here if we are an InstanceRefKlass
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   920
    default:
32817
acc2744fd84b 8136627: Backout JDK-8133818 Additional number of processed references printed with -XX:+PrintReferenceGC after JDK-8047125
kbarrett
parents: 32623
diff changeset
   921
      ShouldNotReachHere();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   922
  }
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   923
  log_develop_trace(gc, ref)("Thread %d gets list " INTPTR_FORMAT, id, p2i(list));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   924
  return list;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   925
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   926
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   927
inline void
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   928
ReferenceProcessor::add_to_discovered_list_mt(DiscoveredList& refs_list,
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   929
                                              oop             obj,
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   930
                                              HeapWord*       discovered_addr) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   931
  assert(_discovery_is_mt, "!_discovery_is_mt should have been handled by caller");
489c9b5090e2 Initial load
duke
parents:
diff changeset
   932
  // First we must make sure this object is only enqueued once. CAS in a non null
489c9b5090e2 Initial load
duke
parents:
diff changeset
   933
  // discovered_addr.
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents: 360
diff changeset
   934
  oop current_head = refs_list.head();
10524
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
   935
  // The last ref must have its discovered field pointing to itself.
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
   936
  oop next_discovered = (current_head != NULL) ? current_head : obj;
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents: 360
diff changeset
   937
10524
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
   938
  oop retest = oopDesc::atomic_compare_exchange_oop(next_discovered, discovered_addr,
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   939
                                                    NULL);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   940
  if (retest == NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   941
    // This thread just won the right to enqueue the object.
10526
3e92f211533f 4965777: GC changes to support use of discovered field for pending references
ysr
parents: 10524
diff changeset
   942
    // We have separate lists for enqueueing, so no synchronization
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   943
    // is necessary.
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
   944
    refs_list.set_head(obj);
1605
6b43d186eb8d 6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents: 1388
diff changeset
   945
    refs_list.inc_length(1);
7399
4ecd771fa2d1 6983204: G1: Nightly test nsk/regression/b4958615 failing with +ExplicitGCInvokesConcurrent
johnc
parents: 7397
diff changeset
   946
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   947
    log_develop_trace(gc, ref)("Discovered reference (mt) (" INTPTR_FORMAT ": %s)",
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   948
                               p2i(obj), obj->klass()->internal_name());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   949
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   950
    // If retest was non NULL, another thread beat us to it:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   951
    // The reference has already been discovered...
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   952
    log_develop_trace(gc, ref)("Already discovered reference (" INTPTR_FORMAT ": %s)",
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
   953
                               p2i(obj), obj->klass()->internal_name());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   954
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   955
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   956
7420
24071b15dde6 7005259: CMS: BubbleUpRef asserts referent(obj)->is_oop() failed: Enqueued a bad referent
ysr
parents: 7399
diff changeset
   957
#ifndef PRODUCT
24071b15dde6 7005259: CMS: BubbleUpRef asserts referent(obj)->is_oop() failed: Enqueued a bad referent
ysr
parents: 7399
diff changeset
   958
// Non-atomic (i.e. concurrent) discovery might allow us
24071b15dde6 7005259: CMS: BubbleUpRef asserts referent(obj)->is_oop() failed: Enqueued a bad referent
ysr
parents: 7399
diff changeset
   959
// to observe j.l.References with NULL referents, being those
24071b15dde6 7005259: CMS: BubbleUpRef asserts referent(obj)->is_oop() failed: Enqueued a bad referent
ysr
parents: 7399
diff changeset
   960
// cleared concurrently by mutators during (or after) discovery.
24071b15dde6 7005259: CMS: BubbleUpRef asserts referent(obj)->is_oop() failed: Enqueued a bad referent
ysr
parents: 7399
diff changeset
   961
void ReferenceProcessor::verify_referent(oop obj) {
24071b15dde6 7005259: CMS: BubbleUpRef asserts referent(obj)->is_oop() failed: Enqueued a bad referent
ysr
parents: 7399
diff changeset
   962
  bool da = discovery_is_atomic();
24071b15dde6 7005259: CMS: BubbleUpRef asserts referent(obj)->is_oop() failed: Enqueued a bad referent
ysr
parents: 7399
diff changeset
   963
  oop referent = java_lang_ref_Reference::referent(obj);
24071b15dde6 7005259: CMS: BubbleUpRef asserts referent(obj)->is_oop() failed: Enqueued a bad referent
ysr
parents: 7399
diff changeset
   964
  assert(da ? referent->is_oop() : referent->is_oop_or_null(),
33105
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 33103
diff changeset
   965
         "Bad referent " INTPTR_FORMAT " found in Reference "
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 33103
diff changeset
   966
         INTPTR_FORMAT " during %satomic discovery ",
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 33103
diff changeset
   967
         p2i(referent), p2i(obj), da ? "" : "non-");
7420
24071b15dde6 7005259: CMS: BubbleUpRef asserts referent(obj)->is_oop() failed: Enqueued a bad referent
ysr
parents: 7399
diff changeset
   968
}
24071b15dde6 7005259: CMS: BubbleUpRef asserts referent(obj)->is_oop() failed: Enqueued a bad referent
ysr
parents: 7399
diff changeset
   969
#endif
24071b15dde6 7005259: CMS: BubbleUpRef asserts referent(obj)->is_oop() failed: Enqueued a bad referent
ysr
parents: 7399
diff changeset
   970
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   971
// We mention two of several possible choices here:
489c9b5090e2 Initial load
duke
parents:
diff changeset
   972
// #0: if the reference object is not in the "originating generation"
489c9b5090e2 Initial load
duke
parents:
diff changeset
   973
//     (or part of the heap being collected, indicated by our "span"
489c9b5090e2 Initial load
duke
parents:
diff changeset
   974
//     we don't treat it specially (i.e. we scan it as we would
489c9b5090e2 Initial load
duke
parents:
diff changeset
   975
//     a normal oop, treating its references as strong references).
10526
3e92f211533f 4965777: GC changes to support use of discovered field for pending references
ysr
parents: 10524
diff changeset
   976
//     This means that references can't be discovered unless their
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   977
//     referent is also in the same span. This is the simplest,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   978
//     most "local" and most conservative approach, albeit one
489c9b5090e2 Initial load
duke
parents:
diff changeset
   979
//     that may cause weak references to be enqueued least promptly.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   980
//     We call this choice the "ReferenceBasedDiscovery" policy.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   981
// #1: the reference object may be in any generation (span), but if
489c9b5090e2 Initial load
duke
parents:
diff changeset
   982
//     the referent is in the generation (span) being currently collected
489c9b5090e2 Initial load
duke
parents:
diff changeset
   983
//     then we can discover the reference object, provided
489c9b5090e2 Initial load
duke
parents:
diff changeset
   984
//     the object has not already been discovered by
489c9b5090e2 Initial load
duke
parents:
diff changeset
   985
//     a different concurrently running collector (as may be the
489c9b5090e2 Initial load
duke
parents:
diff changeset
   986
//     case, for instance, if the reference object is in CMS and
489c9b5090e2 Initial load
duke
parents:
diff changeset
   987
//     the referent in DefNewGeneration), and provided the processing
489c9b5090e2 Initial load
duke
parents:
diff changeset
   988
//     of this reference object by the current collector will
489c9b5090e2 Initial load
duke
parents:
diff changeset
   989
//     appear atomic to every other collector in the system.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   990
//     (Thus, for instance, a concurrent collector may not
489c9b5090e2 Initial load
duke
parents:
diff changeset
   991
//     discover references in other generations even if the
489c9b5090e2 Initial load
duke
parents:
diff changeset
   992
//     referent is in its own generation). This policy may,
489c9b5090e2 Initial load
duke
parents:
diff changeset
   993
//     in certain cases, enqueue references somewhat sooner than
489c9b5090e2 Initial load
duke
parents:
diff changeset
   994
//     might Policy #0 above, but at marginally increased cost
489c9b5090e2 Initial load
duke
parents:
diff changeset
   995
//     and complexity in processing these references.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   996
//     We call this choice the "RefeferentBasedDiscovery" policy.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   997
bool ReferenceProcessor::discover_reference(oop obj, ReferenceType rt) {
10526
3e92f211533f 4965777: GC changes to support use of discovered field for pending references
ysr
parents: 10524
diff changeset
   998
  // Make sure we are discovering refs (rather than processing discovered refs).
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   999
  if (!_discovering_refs || !RegisterReferences) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1000
    return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1001
  }
10526
3e92f211533f 4965777: GC changes to support use of discovered field for pending references
ysr
parents: 10524
diff changeset
  1002
  // We only discover active references.
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
  1003
  oop next = java_lang_ref_Reference::next(obj);
10526
3e92f211533f 4965777: GC changes to support use of discovered field for pending references
ysr
parents: 10524
diff changeset
  1004
  if (next != NULL) {   // Ref is no longer active
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1005
    return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1006
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1007
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1008
  HeapWord* obj_addr = (HeapWord*)obj;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1009
  if (RefDiscoveryPolicy == ReferenceBasedDiscovery &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1010
      !_span.contains(obj_addr)) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1011
    // Reference is not in the originating generation;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1012
    // don't treat it specially (i.e. we want to scan it as a normal
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1013
    // object with strong references).
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1014
    return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1015
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1016
10526
3e92f211533f 4965777: GC changes to support use of discovered field for pending references
ysr
parents: 10524
diff changeset
  1017
  // We only discover references whose referents are not (yet)
3e92f211533f 4965777: GC changes to support use of discovered field for pending references
ysr
parents: 10524
diff changeset
  1018
  // known to be strongly reachable.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1019
  if (is_alive_non_header() != NULL) {
7420
24071b15dde6 7005259: CMS: BubbleUpRef asserts referent(obj)->is_oop() failed: Enqueued a bad referent
ysr
parents: 7399
diff changeset
  1020
    verify_referent(obj);
24071b15dde6 7005259: CMS: BubbleUpRef asserts referent(obj)->is_oop() failed: Enqueued a bad referent
ysr
parents: 7399
diff changeset
  1021
    if (is_alive_non_header()->do_object_b(java_lang_ref_Reference::referent(obj))) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1022
      return false;  // referent is reachable
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1023
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1024
  }
1606
dcf9714addbe 6684579: SoftReference processing can be made more efficient
ysr
parents: 1605
diff changeset
  1025
  if (rt == REF_SOFT) {
dcf9714addbe 6684579: SoftReference processing can be made more efficient
ysr
parents: 1605
diff changeset
  1026
    // For soft refs we can decide now if these are not
dcf9714addbe 6684579: SoftReference processing can be made more efficient
ysr
parents: 1605
diff changeset
  1027
    // current candidates for clearing, in which case we
dcf9714addbe 6684579: SoftReference processing can be made more efficient
ysr
parents: 1605
diff changeset
  1028
    // can mark through them now, rather than delaying that
dcf9714addbe 6684579: SoftReference processing can be made more efficient
ysr
parents: 1605
diff changeset
  1029
    // to the reference-processing phase. Since all current
dcf9714addbe 6684579: SoftReference processing can be made more efficient
ysr
parents: 1605
diff changeset
  1030
    // time-stamp policies advance the soft-ref clock only
32623
390a27af5657 8134626: Misc cleanups after generation array removal
jwilhelm
parents: 32607
diff changeset
  1031
    // at a full collection cycle, this is always currently
1606
dcf9714addbe 6684579: SoftReference processing can be made more efficient
ysr
parents: 1605
diff changeset
  1032
    // accurate.
10683
4b5a5a507864 7098282: G1: assert(interval >= 0) failed: Sanity check, referencePolicy.cpp: 76
johnc
parents: 10670
diff changeset
  1033
    if (!_current_soft_ref_policy->should_clear_reference(obj, _soft_ref_timestamp_clock)) {
1606
dcf9714addbe 6684579: SoftReference processing can be made more efficient
ysr
parents: 1605
diff changeset
  1034
      return false;
dcf9714addbe 6684579: SoftReference processing can be made more efficient
ysr
parents: 1605
diff changeset
  1035
    }
dcf9714addbe 6684579: SoftReference processing can be made more efficient
ysr
parents: 1605
diff changeset
  1036
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1037
10670
4ea0e7d2ffbc 6484982: G1: process references during evacuation pauses
johnc
parents: 10526
diff changeset
  1038
  ResourceMark rm;      // Needed for tracing.
4ea0e7d2ffbc 6484982: G1: process references during evacuation pauses
johnc
parents: 10526
diff changeset
  1039
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents: 360
diff changeset
  1040
  HeapWord* const discovered_addr = java_lang_ref_Reference::discovered_addr(obj);
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents: 360
diff changeset
  1041
  const oop  discovered = java_lang_ref_Reference::discovered(obj);
33105
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 33103
diff changeset
  1042
  assert(discovered->is_oop_or_null(), "Expected an oop or NULL for discovered field at " PTR_FORMAT, p2i(discovered));
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
  1043
  if (discovered != NULL) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1044
    // The reference has already been discovered...
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
  1045
    log_develop_trace(gc, ref)("Already discovered reference (" INTPTR_FORMAT ": %s)",
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
  1046
                               p2i(obj), obj->klass()->internal_name());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1047
    if (RefDiscoveryPolicy == ReferentBasedDiscovery) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1048
      // assumes that an object is not processed twice;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1049
      // if it's been already discovered it must be on another
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1050
      // generation's discovered list; so we won't discover it.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1051
      return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1052
    } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1053
      assert(RefDiscoveryPolicy == ReferenceBasedDiscovery,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1054
             "Unrecognized policy");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1055
      // Check assumption that an object is not potentially
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1056
      // discovered twice except by concurrent collectors that potentially
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1057
      // trace the same Reference object twice.
7399
4ecd771fa2d1 6983204: G1: Nightly test nsk/regression/b4958615 failing with +ExplicitGCInvokesConcurrent
johnc
parents: 7397
diff changeset
  1058
      assert(UseConcMarkSweepGC || UseG1GC,
4ecd771fa2d1 6983204: G1: Nightly test nsk/regression/b4958615 failing with +ExplicitGCInvokesConcurrent
johnc
parents: 7397
diff changeset
  1059
             "Only possible with a concurrent marking collector");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1060
      return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1061
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1062
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1063
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1064
  if (RefDiscoveryPolicy == ReferentBasedDiscovery) {
7420
24071b15dde6 7005259: CMS: BubbleUpRef asserts referent(obj)->is_oop() failed: Enqueued a bad referent
ysr
parents: 7399
diff changeset
  1065
    verify_referent(obj);
10526
3e92f211533f 4965777: GC changes to support use of discovered field for pending references
ysr
parents: 10524
diff changeset
  1066
    // Discover if and only if EITHER:
3e92f211533f 4965777: GC changes to support use of discovered field for pending references
ysr
parents: 10524
diff changeset
  1067
    // .. reference is in our span, OR
3e92f211533f 4965777: GC changes to support use of discovered field for pending references
ysr
parents: 10524
diff changeset
  1068
    // .. we are an atomic collector and referent is in our span
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1069
    if (_span.contains(obj_addr) ||
7420
24071b15dde6 7005259: CMS: BubbleUpRef asserts referent(obj)->is_oop() failed: Enqueued a bad referent
ysr
parents: 7399
diff changeset
  1070
        (discovery_is_atomic() &&
24071b15dde6 7005259: CMS: BubbleUpRef asserts referent(obj)->is_oop() failed: Enqueued a bad referent
ysr
parents: 7399
diff changeset
  1071
         _span.contains(java_lang_ref_Reference::referent(obj)))) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1072
      // should_enqueue = true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1073
    } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1074
      return false;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1075
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1076
  } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1077
    assert(RefDiscoveryPolicy == ReferenceBasedDiscovery &&
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1078
           _span.contains(obj_addr), "code inconsistency");
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1079
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1080
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1081
  // Get the right type of discovered queue head.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1082
  DiscoveredList* list = get_discovered_list(rt);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1083
  if (list == NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1084
    return false;   // nothing special needs to be done
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1085
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1086
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1087
  if (_discovery_is_mt) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1088
    add_to_discovered_list_mt(*list, obj, discovered_addr);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1089
  } else {
24845
e8b8894a77df 8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents: 24424
diff changeset
  1090
    // We do a raw store here: the field will be visited later when processing
e8b8894a77df 8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents: 24424
diff changeset
  1091
    // the discovered references.
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents: 360
diff changeset
  1092
    oop current_head = list->head();
10524
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
  1093
    // The last ref must have its discovered field pointing to itself.
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
  1094
    oop next_discovered = (current_head != NULL) ? current_head : obj;
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
  1095
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents: 360
diff changeset
  1096
    assert(discovered == NULL, "control point invariant");
10524
6594ca81279a 7085906: Replace the permgen allocated sentinelRef with a self-looped end
stefank
parents: 8688
diff changeset
  1097
    oop_store_raw(discovered_addr, next_discovered);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1098
    list->set_head(obj);
1605
6b43d186eb8d 6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents: 1388
diff changeset
  1099
    list->inc_length(1);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1100
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
  1101
    log_develop_trace(gc, ref)("Discovered reference (" INTPTR_FORMAT ": %s)", p2i(obj), obj->klass()->internal_name());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1102
  }
10526
3e92f211533f 4965777: GC changes to support use of discovered field for pending references
ysr
parents: 10524
diff changeset
  1103
  assert(obj->is_oop(), "Discovered a bad reference");
7420
24071b15dde6 7005259: CMS: BubbleUpRef asserts referent(obj)->is_oop() failed: Enqueued a bad referent
ysr
parents: 7399
diff changeset
  1104
  verify_referent(obj);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1105
  return true;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1106
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1107
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1108
// Preclean the discovered references by removing those
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1109
// whose referents are alive, and by marking from those that
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1110
// are not active. These lists can be handled here
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1111
// in any order and, indeed, concurrently.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1112
void ReferenceProcessor::preclean_discovered_references(
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1113
  BoolObjectClosure* is_alive,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1114
  OopClosure* keep_alive,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1115
  VoidClosure* complete_gc,
18025
b7bcf7497f93 8005849: JEP 167: Event-Based JVM Tracing
sla
parents: 17625
diff changeset
  1116
  YieldClosure* yield,
33107
77bf0d2069a3 8134953: Make the GC ID available in a central place
brutisso
parents: 33105
diff changeset
  1117
  GCTimer* gc_timer) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1118
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1119
  // Soft references
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1120
  {
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
  1121
    GCTraceTime(Debug, gc, ref) tm("Preclean SoftReferences", gc_timer);
11396
917d8673b5ef 7121618: Change type of number of GC workers to unsigned int.
jmasa
parents: 11251
diff changeset
  1122
    for (uint i = 0; i < _max_num_q; i++) {
1605
6b43d186eb8d 6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents: 1388
diff changeset
  1123
      if (yield->should_return()) {
6b43d186eb8d 6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents: 1388
diff changeset
  1124
        return;
6b43d186eb8d 6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents: 1388
diff changeset
  1125
      }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1126
      preclean_discovered_reflist(_discoveredSoftRefs[i], is_alive,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1127
                                  keep_alive, complete_gc, yield);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1128
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1129
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1130
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1131
  // Weak references
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1132
  {
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
  1133
    GCTraceTime(Debug, gc, ref) tm("Preclean WeakReferences", gc_timer);
11396
917d8673b5ef 7121618: Change type of number of GC workers to unsigned int.
jmasa
parents: 11251
diff changeset
  1134
    for (uint i = 0; i < _max_num_q; i++) {
1605
6b43d186eb8d 6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents: 1388
diff changeset
  1135
      if (yield->should_return()) {
6b43d186eb8d 6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents: 1388
diff changeset
  1136
        return;
6b43d186eb8d 6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents: 1388
diff changeset
  1137
      }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1138
      preclean_discovered_reflist(_discoveredWeakRefs[i], is_alive,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1139
                                  keep_alive, complete_gc, yield);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1140
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1141
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1142
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1143
  // Final references
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1144
  {
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
  1145
    GCTraceTime(Debug, gc, ref) tm("Preclean FinalReferences", gc_timer);
11396
917d8673b5ef 7121618: Change type of number of GC workers to unsigned int.
jmasa
parents: 11251
diff changeset
  1146
    for (uint i = 0; i < _max_num_q; i++) {
1605
6b43d186eb8d 6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents: 1388
diff changeset
  1147
      if (yield->should_return()) {
6b43d186eb8d 6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents: 1388
diff changeset
  1148
        return;
6b43d186eb8d 6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents: 1388
diff changeset
  1149
      }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1150
      preclean_discovered_reflist(_discoveredFinalRefs[i], is_alive,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1151
                                  keep_alive, complete_gc, yield);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1152
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1153
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1154
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1155
  // Phantom references
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1156
  {
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
  1157
    GCTraceTime(Debug, gc, ref) tm("Preclean PhantomReferences", gc_timer);
11396
917d8673b5ef 7121618: Change type of number of GC workers to unsigned int.
jmasa
parents: 11251
diff changeset
  1158
    for (uint i = 0; i < _max_num_q; i++) {
1605
6b43d186eb8d 6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents: 1388
diff changeset
  1159
      if (yield->should_return()) {
6b43d186eb8d 6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents: 1388
diff changeset
  1160
        return;
6b43d186eb8d 6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents: 1388
diff changeset
  1161
      }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1162
      preclean_discovered_reflist(_discoveredPhantomRefs[i], is_alive,
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1163
                                  keep_alive, complete_gc, yield);
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1164
    }
28510
4f1d3611e0bc 8047125: (ref) More phantom object references
jmasa
parents: 28212
diff changeset
  1165
4f1d3611e0bc 8047125: (ref) More phantom object references
jmasa
parents: 28212
diff changeset
  1166
    // Cleaner references.  Included in timing for phantom references.  We
4f1d3611e0bc 8047125: (ref) More phantom object references
jmasa
parents: 28212
diff changeset
  1167
    // expect Cleaner references to be temporary, and don't want to deal with
4f1d3611e0bc 8047125: (ref) More phantom object references
jmasa
parents: 28212
diff changeset
  1168
    // possible incompatibilities arising from making it more visible.
4f1d3611e0bc 8047125: (ref) More phantom object references
jmasa
parents: 28212
diff changeset
  1169
    for (uint i = 0; i < _max_num_q; i++) {
4f1d3611e0bc 8047125: (ref) More phantom object references
jmasa
parents: 28212
diff changeset
  1170
      if (yield->should_return()) {
4f1d3611e0bc 8047125: (ref) More phantom object references
jmasa
parents: 28212
diff changeset
  1171
        return;
4f1d3611e0bc 8047125: (ref) More phantom object references
jmasa
parents: 28212
diff changeset
  1172
      }
4f1d3611e0bc 8047125: (ref) More phantom object references
jmasa
parents: 28212
diff changeset
  1173
      preclean_discovered_reflist(_discoveredCleanerRefs[i], is_alive,
4f1d3611e0bc 8047125: (ref) More phantom object references
jmasa
parents: 28212
diff changeset
  1174
                                  keep_alive, complete_gc, yield);
4f1d3611e0bc 8047125: (ref) More phantom object references
jmasa
parents: 28212
diff changeset
  1175
    }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1176
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1177
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1178
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1179
// Walk the given discovered ref list, and remove all reference objects
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1180
// whose referents are still alive, whose referents are NULL or which
1605
6b43d186eb8d 6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents: 1388
diff changeset
  1181
// are not active (have a non-NULL next field). NOTE: When we are
6b43d186eb8d 6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents: 1388
diff changeset
  1182
// thus precleaning the ref lists (which happens single-threaded today),
22551
9bf46d16dcc6 8025856: Fix typos in the GC code
jwilhelm
parents: 22496
diff changeset
  1183
// we do not disable refs discovery to honor the correct semantics of
1605
6b43d186eb8d 6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents: 1388
diff changeset
  1184
// java.lang.Reference. As a result, we need to be careful below
6b43d186eb8d 6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents: 1388
diff changeset
  1185
// that ref removal steps interleave safely with ref discovery steps
6b43d186eb8d 6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents: 1388
diff changeset
  1186
// (in this thread).
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
  1187
void
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
  1188
ReferenceProcessor::preclean_discovered_reflist(DiscoveredList&    refs_list,
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
  1189
                                                BoolObjectClosure* is_alive,
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
  1190
                                                OopClosure*        keep_alive,
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
  1191
                                                VoidClosure*       complete_gc,
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
  1192
                                                YieldClosure*      yield) {
24845
e8b8894a77df 8043239: G1: Missing post barrier in processing of j.l.ref.Reference objects
brutisso
parents: 24424
diff changeset
  1193
  DiscoveredListIterator iter(refs_list, keep_alive, is_alive);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1194
  while (iter.has_next()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1195
    iter.load_ptrs(DEBUG_ONLY(true /* allow_null_referent */));
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
  1196
    oop obj = iter.obj();
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
  1197
    oop next = java_lang_ref_Reference::next(obj);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1198
    if (iter.referent() == NULL || iter.is_referent_alive() ||
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
  1199
        next != NULL) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1200
      // The referent has been cleared, or is alive, or the Reference is not
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1201
      // active; we need to trace and mark its cohort.
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
  1202
      log_develop_trace(gc, ref)("Precleaning Reference (" INTPTR_FORMAT ": %s)",
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
  1203
                                 p2i(iter.obj()), iter.obj()->klass()->internal_name());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1204
      // Remove Reference object from list
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1205
      iter.remove();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1206
      // Keep alive its cohort.
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1207
      iter.make_referent_alive();
360
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
  1208
      if (UseCompressedOops) {
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
  1209
        narrowOop* next_addr = (narrowOop*)java_lang_ref_Reference::next_addr(obj);
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
  1210
        keep_alive->do_oop(next_addr);
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
  1211
      } else {
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
  1212
        oop* next_addr = (oop*)java_lang_ref_Reference::next_addr(obj);
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
  1213
        keep_alive->do_oop(next_addr);
21d113ecbf6a 6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents: 178
diff changeset
  1214
      }
1605
6b43d186eb8d 6722113: CMS: Incorrect overflow handling during precleaning of Reference lists
ysr
parents: 1388
diff changeset
  1215
      iter.move_to_next();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1216
    } else {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1217
      iter.next();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1218
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1219
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1220
  // Close the reachable set
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1221
  complete_gc->do_void();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1222
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1223
  NOT_PRODUCT(
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
  1224
    if (iter.processed() > 0) {
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33611
diff changeset
  1225
      log_develop_trace(gc, ref)(" Dropped " SIZE_FORMAT " Refs out of " SIZE_FORMAT " Refs in discovered list " INTPTR_FORMAT,
29800
fa5f7a2bf717 8076073: shared: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents: 29580
diff changeset
  1226
        iter.removed(), iter.processed(), p2i(refs_list.head()));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1227
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1228
  )
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1229
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1230
11396
917d8673b5ef 7121618: Change type of number of GC workers to unsigned int.
jmasa
parents: 11251
diff changeset
  1231
const char* ReferenceProcessor::list_name(uint i) {
29580
a67a581cfe11 8073315: Enable gcc -Wtype-limits and fix upcoming issues.
goetz
parents: 28510
diff changeset
  1232
   assert(i <= _max_num_q * number_of_subclasses_of_ref(),
10670
4ea0e7d2ffbc 6484982: G1: process references during evacuation pauses
johnc
parents: 10526
diff changeset
  1233
          "Out of bounds index");
4ea0e7d2ffbc 6484982: G1: process references during evacuation pauses
johnc
parents: 10526
diff changeset
  1234
6759
67b1a69ef5aa 6984287: Regularize how GC parallel workers are specified.
jmasa
parents: 5547
diff changeset
  1235
   int j = i / _max_num_q;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1236
   switch (j) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1237
     case 0: return "SoftRef";
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1238
     case 1: return "WeakRef";
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1239
     case 2: return "FinalRef";
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1240
     case 3: return "PhantomRef";
28510
4f1d3611e0bc 8047125: (ref) More phantom object references
jmasa
parents: 28212
diff changeset
  1241
     case 4: return "CleanerRef";
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1242
   }
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1243
   ShouldNotReachHere();
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1244
   return NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1245
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
  1246