src/hotspot/share/gc/g1/collectionSetChooser.cpp
author tschatzl
Wed, 22 Aug 2018 20:37:07 +0200
changeset 51494 1906adbef2dc
parent 50441 523c2a73a3dc
child 53703 24341625d8f2
permissions -rw-r--r--
8208498: Put archive regions into a first-class HeapRegionSet Summary: Maintain archive regions in a HeapRegionSet like other region types. Reviewed-by: phh, sangheki
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
     1
/*
49605
784f3f2dea14 8197570: Make rules for choosing collection set candidates more explicit
tschatzl
parents: 48969
diff changeset
     2
 * Copyright (c) 2001, 2018, Oracle and/or its affiliates. All rights reserved.
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
     4
 *
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
     7
 * published by the Free Software Foundation.
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
     8
 *
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    13
 * accompanied this code).
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    14
 *
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    18
 *
5547
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 4473
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 4473
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: 4473
diff changeset
    21
 * questions.
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    22
 *
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    23
 */
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    24
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 6183
diff changeset
    25
#include "precompiled.hpp"
30764
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 26846
diff changeset
    26
#include "gc/g1/collectionSetChooser.hpp"
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 26846
diff changeset
    27
#include "gc/g1/g1CollectedHeap.inline.hpp"
49607
acffe6ff3ae7 8180415: Rebuild remembered sets during the concurrent cycle
tschatzl
parents: 49605
diff changeset
    28
#include "gc/g1/heapRegionRemSet.hpp"
30764
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 26846
diff changeset
    29
#include "gc/shared/space.inline.hpp"
40655
9f644073d3a0 8157907: Incorrect inclusion of atomic.hpp instead of atomic.inline.hpp
dholmes
parents: 37985
diff changeset
    30
#include "runtime/atomic.hpp"
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    31
11756
28b6fe22e43d 7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents: 11396
diff changeset
    32
// Even though we don't use the GC efficiency in our heuristics as
28b6fe22e43d 7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents: 11396
diff changeset
    33
// much as we used to, we still order according to GC efficiency. This
28b6fe22e43d 7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents: 11396
diff changeset
    34
// will cause regions with a lot of live objects and large RSets to
28b6fe22e43d 7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents: 11396
diff changeset
    35
// end up at the end of the array. Given that we might skip collecting
28b6fe22e43d 7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents: 11396
diff changeset
    36
// the last few old regions, if after a few mixed GCs the remaining
28b6fe22e43d 7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents: 11396
diff changeset
    37
// have reclaimable bytes under a certain threshold, the hope is that
28b6fe22e43d 7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents: 11396
diff changeset
    38
// the ones we'll skip are ones with both large RSets and a lot of
28b6fe22e43d 7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents: 11396
diff changeset
    39
// live objects, not the ones with just a lot of live objects if we
28b6fe22e43d 7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents: 11396
diff changeset
    40
// ordered according to the amount of reclaimable bytes per region.
12382
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
    41
static int order_regions(HeapRegion* hr1, HeapRegion* hr2) {
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    42
  if (hr1 == NULL) {
11756
28b6fe22e43d 7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents: 11396
diff changeset
    43
    if (hr2 == NULL) {
28b6fe22e43d 7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents: 11396
diff changeset
    44
      return 0;
28b6fe22e43d 7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents: 11396
diff changeset
    45
    } else {
28b6fe22e43d 7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents: 11396
diff changeset
    46
      return 1;
28b6fe22e43d 7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents: 11396
diff changeset
    47
    }
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    48
  } else if (hr2 == NULL) {
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    49
    return -1;
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    50
  }
11756
28b6fe22e43d 7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents: 11396
diff changeset
    51
28b6fe22e43d 7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents: 11396
diff changeset
    52
  double gc_eff1 = hr1->gc_efficiency();
28b6fe22e43d 7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents: 11396
diff changeset
    53
  double gc_eff2 = hr2->gc_efficiency();
28b6fe22e43d 7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents: 11396
diff changeset
    54
  if (gc_eff1 > gc_eff2) {
28b6fe22e43d 7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents: 11396
diff changeset
    55
    return -1;
28b6fe22e43d 7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents: 11396
diff changeset
    56
  } if (gc_eff1 < gc_eff2) {
28b6fe22e43d 7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents: 11396
diff changeset
    57
    return 1;
28b6fe22e43d 7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents: 11396
diff changeset
    58
  } else {
28b6fe22e43d 7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents: 11396
diff changeset
    59
    return 0;
28b6fe22e43d 7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents: 11396
diff changeset
    60
  }
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    61
}
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    62
12382
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
    63
static int order_regions(HeapRegion** hr1p, HeapRegion** hr2p) {
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
    64
  return order_regions(*hr1p, *hr2p);
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    65
}
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    66
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    67
CollectionSetChooser::CollectionSetChooser() :
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    68
  // The line below is the worst bit of C++ hackery I've ever written
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    69
  // (Detlefs, 11/23).  You should think of it as equivalent to
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    70
  // "_regions(100, true)": initialize the growable array and inform it
6183
4c74cfe14f20 6975078: assert(allocated_on_res_area() || allocated_on_C_heap() || allocated_on_arena()
kvn
parents: 5547
diff changeset
    71
  // that it should allocate its elem array(s) on the C heap.
4c74cfe14f20 6975078: assert(allocated_on_res_area() || allocated_on_C_heap() || allocated_on_arena()
kvn
parents: 5547
diff changeset
    72
  //
4c74cfe14f20 6975078: assert(allocated_on_res_area() || allocated_on_C_heap() || allocated_on_arena()
kvn
parents: 5547
diff changeset
    73
  // The first argument, however, is actually a comma expression
4c74cfe14f20 6975078: assert(allocated_on_res_area() || allocated_on_C_heap() || allocated_on_arena()
kvn
parents: 5547
diff changeset
    74
  // (set_allocation_type(this, C_HEAP), 100). The purpose of the
4c74cfe14f20 6975078: assert(allocated_on_res_area() || allocated_on_C_heap() || allocated_on_arena()
kvn
parents: 5547
diff changeset
    75
  // set_allocation_type() call is to replace the default allocation
4c74cfe14f20 6975078: assert(allocated_on_res_area() || allocated_on_C_heap() || allocated_on_arena()
kvn
parents: 5547
diff changeset
    76
  // type for embedded objects STACK_OR_EMBEDDED with C_HEAP. It will
4c74cfe14f20 6975078: assert(allocated_on_res_area() || allocated_on_C_heap() || allocated_on_arena()
kvn
parents: 5547
diff changeset
    77
  // allow to pass the assert in GenericGrowableArray() which checks
4c74cfe14f20 6975078: assert(allocated_on_res_area() || allocated_on_C_heap() || allocated_on_arena()
kvn
parents: 5547
diff changeset
    78
  // that a growable array object must be on C heap if elements are.
4c74cfe14f20 6975078: assert(allocated_on_res_area() || allocated_on_C_heap() || allocated_on_arena()
kvn
parents: 5547
diff changeset
    79
  //
4c74cfe14f20 6975078: assert(allocated_on_res_area() || allocated_on_C_heap() || allocated_on_arena()
kvn
parents: 5547
diff changeset
    80
  // Note: containing object is allocated on C heap since it is CHeapObj.
4c74cfe14f20 6975078: assert(allocated_on_res_area() || allocated_on_C_heap() || allocated_on_arena()
kvn
parents: 5547
diff changeset
    81
  //
12382
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
    82
  _regions((ResourceObj::set_allocation_type((address) &_regions,
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    83
                                             ResourceObj::C_HEAP),
11756
28b6fe22e43d 7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents: 11396
diff changeset
    84
                  100), true /* C_Heap */),
32745
d238cd8170fc 8135253: Add push method to CollectionSetChooser
ehelin
parents: 31592
diff changeset
    85
    _front(0), _end(0), _first_par_unreserved_idx(0),
12382
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
    86
    _region_live_threshold_bytes(0), _remaining_reclaimable_bytes(0) {
49605
784f3f2dea14 8197570: Make rules for choosing collection set candidates more explicit
tschatzl
parents: 48969
diff changeset
    87
  _region_live_threshold_bytes = mixed_gc_live_threshold_bytes();
11756
28b6fe22e43d 7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents: 11396
diff changeset
    88
}
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    89
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
    90
#ifndef PRODUCT
12382
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
    91
void CollectionSetChooser::verify() {
33105
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 32745
diff changeset
    92
  guarantee(_end <= regions_length(), "_end: %u regions length: %u", _end, regions_length());
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 32745
diff changeset
    93
  guarantee(_front <= _end, "_front: %u _end: %u", _front, _end);
12382
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
    94
  uint index = 0;
11756
28b6fe22e43d 7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents: 11396
diff changeset
    95
  size_t sum_of_reclaimable_bytes = 0;
32745
d238cd8170fc 8135253: Add push method to CollectionSetChooser
ehelin
parents: 31592
diff changeset
    96
  while (index < _front) {
12382
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
    97
    guarantee(regions_at(index) == NULL,
32745
d238cd8170fc 8135253: Add push method to CollectionSetChooser
ehelin
parents: 31592
diff changeset
    98
              "all entries before _front should be NULL");
11756
28b6fe22e43d 7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents: 11396
diff changeset
    99
    index += 1;
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   100
  }
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   101
  HeapRegion *prev = NULL;
32745
d238cd8170fc 8135253: Add push method to CollectionSetChooser
ehelin
parents: 31592
diff changeset
   102
  while (index < _end) {
12382
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
   103
    HeapRegion *curr = regions_at(index++);
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
   104
    guarantee(curr != NULL, "Regions in _regions array cannot be NULL");
10680
f7bdba11999b 7095236: G1: _markedRegions never contains NULL regions
ysr
parents: 10523
diff changeset
   105
    guarantee(!curr->is_young(), "should not be young!");
31346
a70d45c06136 8042668: GC Support for shared heap ranges in CDS
jiangli
parents: 30874
diff changeset
   106
    guarantee(!curr->is_pinned(),
33105
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 32745
diff changeset
   107
              "Pinned region should not be in collection set (index %u)", curr->hrm_index());
10680
f7bdba11999b 7095236: G1: _markedRegions never contains NULL regions
ysr
parents: 10523
diff changeset
   108
    if (prev != NULL) {
12382
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
   109
      guarantee(order_regions(prev, curr) != 1,
33105
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 32745
diff changeset
   110
                "GC eff prev: %1.4f GC eff curr: %1.4f",
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 32745
diff changeset
   111
                prev->gc_efficiency(), curr->gc_efficiency());
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   112
    }
11756
28b6fe22e43d 7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents: 11396
diff changeset
   113
    sum_of_reclaimable_bytes += curr->reclaimable_bytes();
10680
f7bdba11999b 7095236: G1: _markedRegions never contains NULL regions
ysr
parents: 10523
diff changeset
   114
    prev = curr;
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   115
  }
12382
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
   116
  guarantee(sum_of_reclaimable_bytes == _remaining_reclaimable_bytes,
33105
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 32745
diff changeset
   117
            "reclaimable bytes inconsistent, "
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 32745
diff changeset
   118
            "remaining: " SIZE_FORMAT " sum: " SIZE_FORMAT,
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 32745
diff changeset
   119
            _remaining_reclaimable_bytes, sum_of_reclaimable_bytes);
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   120
}
12382
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
   121
#endif // !PRODUCT
11756
28b6fe22e43d 7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents: 11396
diff changeset
   122
12382
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
   123
void CollectionSetChooser::sort_regions() {
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   124
  // First trim any unused portion of the top in the parallel case.
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   125
  if (_first_par_unreserved_idx > 0) {
12382
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
   126
    assert(_first_par_unreserved_idx <= regions_length(),
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   127
           "Or we didn't reserved enough length");
12382
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
   128
    regions_trunc_to(_first_par_unreserved_idx);
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   129
  }
12382
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
   130
  _regions.sort(order_regions);
32745
d238cd8170fc 8135253: Add push method to CollectionSetChooser
ehelin
parents: 31592
diff changeset
   131
  assert(_end <= regions_length(), "Requirement");
12382
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
   132
#ifdef ASSERT
32745
d238cd8170fc 8135253: Add push method to CollectionSetChooser
ehelin
parents: 31592
diff changeset
   133
  for (uint i = 0; i < _end; i++) {
12382
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
   134
    assert(regions_at(i) != NULL, "Should be true by sorting!");
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   135
  }
12382
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
   136
#endif // ASSERT
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33105
diff changeset
   137
  if (log_is_enabled(Trace, gc, liveness)) {
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33105
diff changeset
   138
    G1PrintRegionLivenessInfoClosure cl("Post-Sorting");
32745
d238cd8170fc 8135253: Add push method to CollectionSetChooser
ehelin
parents: 31592
diff changeset
   139
    for (uint i = 0; i < _end; ++i) {
12382
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
   140
      HeapRegion* r = regions_at(i);
48969
jwilhelm
parents: 48890
diff changeset
   141
      cl.do_heap_region(r);
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   142
    }
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   143
  }
12382
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
   144
  verify();
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   145
}
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   146
12382
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
   147
void CollectionSetChooser::add_region(HeapRegion* hr) {
31346
a70d45c06136 8042668: GC Support for shared heap ranges in CDS
jiangli
parents: 30874
diff changeset
   148
  assert(!hr->is_pinned(),
33105
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 32745
diff changeset
   149
         "Pinned region shouldn't be added to the collection set (index %u)", hr->hrm_index());
50441
523c2a73a3dc 8204081: Mismatch in rebuild policy and collection set chooser causes remembered sets to be kept errorneously
tschatzl
parents: 49806
diff changeset
   150
  assert(hr->is_old(), "should be old but is %s", hr->get_type_str());
49607
acffe6ff3ae7 8180415: Rebuild remembered sets during the concurrent cycle
tschatzl
parents: 49605
diff changeset
   151
  assert(hr->rem_set()->is_complete(),
acffe6ff3ae7 8180415: Rebuild remembered sets during the concurrent cycle
tschatzl
parents: 49605
diff changeset
   152
         "Trying to add region %u to the collection set with incomplete remembered set", hr->hrm_index());
12382
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
   153
  _regions.append(hr);
32745
d238cd8170fc 8135253: Add push method to CollectionSetChooser
ehelin
parents: 31592
diff changeset
   154
  _end++;
12382
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
   155
  _remaining_reclaimable_bytes += hr->reclaimable_bytes();
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   156
  hr->calc_gc_efficiency();
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   157
}
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   158
32745
d238cd8170fc 8135253: Add push method to CollectionSetChooser
ehelin
parents: 31592
diff changeset
   159
void CollectionSetChooser::push(HeapRegion* hr) {
d238cd8170fc 8135253: Add push method to CollectionSetChooser
ehelin
parents: 31592
diff changeset
   160
  assert(hr != NULL, "Can't put back a NULL region");
d238cd8170fc 8135253: Add push method to CollectionSetChooser
ehelin
parents: 31592
diff changeset
   161
  assert(_front >= 1, "Too many regions have been put back");
d238cd8170fc 8135253: Add push method to CollectionSetChooser
ehelin
parents: 31592
diff changeset
   162
  _front--;
d238cd8170fc 8135253: Add push method to CollectionSetChooser
ehelin
parents: 31592
diff changeset
   163
  regions_at_put(_front, hr);
d238cd8170fc 8135253: Add push method to CollectionSetChooser
ehelin
parents: 31592
diff changeset
   164
  _remaining_reclaimable_bytes += hr->reclaimable_bytes();
d238cd8170fc 8135253: Add push method to CollectionSetChooser
ehelin
parents: 31592
diff changeset
   165
}
d238cd8170fc 8135253: Add push method to CollectionSetChooser
ehelin
parents: 31592
diff changeset
   166
30874
18714bae50db 8080837: Move number of workers calculation out of CollectionSetChooser::prepare_for_par_region_addition
stefank
parents: 30764
diff changeset
   167
void CollectionSetChooser::prepare_for_par_region_addition(uint n_threads,
18714bae50db 8080837: Move number of workers calculation out of CollectionSetChooser::prepare_for_par_region_addition
stefank
parents: 30764
diff changeset
   168
                                                           uint n_regions,
12382
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
   169
                                                           uint chunk_size) {
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   170
  _first_par_unreserved_idx = 0;
12382
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
   171
  uint max_waste = n_threads * chunk_size;
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
   172
  // it should be aligned with respect to chunk_size
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
   173
  uint aligned_n_regions = (n_regions + chunk_size - 1) / chunk_size * chunk_size;
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
   174
  assert(aligned_n_regions % chunk_size == 0, "should be aligned");
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
   175
  regions_at_put_grow(aligned_n_regions + max_waste - 1, NULL);
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   176
}
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   177
12382
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
   178
uint CollectionSetChooser::claim_array_chunk(uint chunk_size) {
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
   179
  uint res = (uint) Atomic::add((jint) chunk_size,
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
   180
                                (volatile jint*) &_first_par_unreserved_idx);
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
   181
  assert(regions_length() > res + chunk_size - 1,
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   182
         "Should already have been expanded");
12382
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
   183
  return res - chunk_size;
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   184
}
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   185
12382
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
   186
void CollectionSetChooser::set_region(uint index, HeapRegion* hr) {
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
   187
  assert(regions_at(index) == NULL, "precondition");
50441
523c2a73a3dc 8204081: Mismatch in rebuild policy and collection set chooser causes remembered sets to be kept errorneously
tschatzl
parents: 49806
diff changeset
   188
  assert(hr->is_old(), "should be old but is %s", hr->get_type_str());
12382
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
   189
  regions_at_put(index, hr);
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   190
  hr->calc_gc_efficiency();
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   191
}
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   192
12382
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
   193
void CollectionSetChooser::update_totals(uint region_num,
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
   194
                                         size_t reclaimable_bytes) {
11756
28b6fe22e43d 7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents: 11396
diff changeset
   195
  // Only take the lock if we actually need to update the totals.
28b6fe22e43d 7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents: 11396
diff changeset
   196
  if (region_num > 0) {
28b6fe22e43d 7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents: 11396
diff changeset
   197
    assert(reclaimable_bytes > 0, "invariant");
28b6fe22e43d 7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents: 11396
diff changeset
   198
    // We could have just used atomics instead of taking the
28b6fe22e43d 7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents: 11396
diff changeset
   199
    // lock. However, we currently don't have an atomic add for size_t.
28b6fe22e43d 7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents: 11396
diff changeset
   200
    MutexLockerEx x(ParGCRareEvent_lock, Mutex::_no_safepoint_check_flag);
32745
d238cd8170fc 8135253: Add push method to CollectionSetChooser
ehelin
parents: 31592
diff changeset
   201
    _end += region_num;
12382
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
   202
    _remaining_reclaimable_bytes += reclaimable_bytes;
11756
28b6fe22e43d 7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents: 11396
diff changeset
   203
  } else {
28b6fe22e43d 7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents: 11396
diff changeset
   204
    assert(reclaimable_bytes == 0, "invariant");
1374
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   205
  }
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   206
}
4c24294029a9 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
   207
49605
784f3f2dea14 8197570: Make rules for choosing collection set candidates more explicit
tschatzl
parents: 48969
diff changeset
   208
void CollectionSetChooser::iterate(HeapRegionClosure* cl) {
784f3f2dea14 8197570: Make rules for choosing collection set candidates more explicit
tschatzl
parents: 48969
diff changeset
   209
  for (uint i = _front; i < _end; i++) {
784f3f2dea14 8197570: Make rules for choosing collection set candidates more explicit
tschatzl
parents: 48969
diff changeset
   210
    HeapRegion* r = regions_at(i);
784f3f2dea14 8197570: Make rules for choosing collection set candidates more explicit
tschatzl
parents: 48969
diff changeset
   211
    if (cl->do_heap_region(r)) {
784f3f2dea14 8197570: Make rules for choosing collection set candidates more explicit
tschatzl
parents: 48969
diff changeset
   212
      cl->set_incomplete();
784f3f2dea14 8197570: Make rules for choosing collection set candidates more explicit
tschatzl
parents: 48969
diff changeset
   213
      break;
784f3f2dea14 8197570: Make rules for choosing collection set candidates more explicit
tschatzl
parents: 48969
diff changeset
   214
    }
784f3f2dea14 8197570: Make rules for choosing collection set candidates more explicit
tschatzl
parents: 48969
diff changeset
   215
  }
784f3f2dea14 8197570: Make rules for choosing collection set candidates more explicit
tschatzl
parents: 48969
diff changeset
   216
}
784f3f2dea14 8197570: Make rules for choosing collection set candidates more explicit
tschatzl
parents: 48969
diff changeset
   217
12382
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
   218
void CollectionSetChooser::clear() {
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
   219
  _regions.clear();
32745
d238cd8170fc 8135253: Add push method to CollectionSetChooser
ehelin
parents: 31592
diff changeset
   220
  _front = 0;
d238cd8170fc 8135253: Add push method to CollectionSetChooser
ehelin
parents: 31592
diff changeset
   221
  _end = 0;
12382
6aaecb1cbfe1 7145441: G1: collection set chooser-related cleanup
tonyp
parents: 12381
diff changeset
   222
  _remaining_reclaimable_bytes = 0;
37143
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   223
}
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   224
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   225
class ParKnownGarbageHRClosure: public HeapRegionClosure {
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   226
  G1CollectedHeap* _g1h;
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   227
  CSetChooserParUpdater _cset_updater;
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   228
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   229
public:
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   230
  ParKnownGarbageHRClosure(CollectionSetChooser* hrSorted,
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   231
                           uint chunk_size) :
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   232
    _g1h(G1CollectedHeap::heap()),
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   233
    _cset_updater(hrSorted, true /* parallel */, chunk_size) { }
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   234
48969
jwilhelm
parents: 48890
diff changeset
   235
  bool do_heap_region(HeapRegion* r) {
50441
523c2a73a3dc 8204081: Mismatch in rebuild policy and collection set chooser causes remembered sets to be kept errorneously
tschatzl
parents: 49806
diff changeset
   236
    // We will skip any region that's currently used as an old GC
523c2a73a3dc 8204081: Mismatch in rebuild policy and collection set chooser causes remembered sets to be kept errorneously
tschatzl
parents: 49806
diff changeset
   237
    // alloc region (we should not consider those for collection
523c2a73a3dc 8204081: Mismatch in rebuild policy and collection set chooser causes remembered sets to be kept errorneously
tschatzl
parents: 49806
diff changeset
   238
    // before we fill them up).
523c2a73a3dc 8204081: Mismatch in rebuild policy and collection set chooser causes remembered sets to be kept errorneously
tschatzl
parents: 49806
diff changeset
   239
    if (_cset_updater.should_add(r) && !_g1h->is_old_gc_alloc_region(r)) {
523c2a73a3dc 8204081: Mismatch in rebuild policy and collection set chooser causes remembered sets to be kept errorneously
tschatzl
parents: 49806
diff changeset
   240
      _cset_updater.add_region(r);
523c2a73a3dc 8204081: Mismatch in rebuild policy and collection set chooser causes remembered sets to be kept errorneously
tschatzl
parents: 49806
diff changeset
   241
    } else if (r->is_old()) {
523c2a73a3dc 8204081: Mismatch in rebuild policy and collection set chooser causes remembered sets to be kept errorneously
tschatzl
parents: 49806
diff changeset
   242
      // Keep remembered sets for humongous regions, otherwise clean out remembered
523c2a73a3dc 8204081: Mismatch in rebuild policy and collection set chooser causes remembered sets to be kept errorneously
tschatzl
parents: 49806
diff changeset
   243
      // sets for old regions.
523c2a73a3dc 8204081: Mismatch in rebuild policy and collection set chooser causes remembered sets to be kept errorneously
tschatzl
parents: 49806
diff changeset
   244
      r->rem_set()->clear(true /* only_cardset */);
523c2a73a3dc 8204081: Mismatch in rebuild policy and collection set chooser causes remembered sets to be kept errorneously
tschatzl
parents: 49806
diff changeset
   245
    } else {
51494
1906adbef2dc 8208498: Put archive regions into a first-class HeapRegionSet
tschatzl
parents: 50441
diff changeset
   246
      assert(r->is_archive() || !r->is_old() || !r->rem_set()->is_tracked(),
50441
523c2a73a3dc 8204081: Mismatch in rebuild policy and collection set chooser causes remembered sets to be kept errorneously
tschatzl
parents: 49806
diff changeset
   247
             "Missed to clear unused remembered set of region %u (%s) that is %s",
523c2a73a3dc 8204081: Mismatch in rebuild policy and collection set chooser causes remembered sets to be kept errorneously
tschatzl
parents: 49806
diff changeset
   248
             r->hrm_index(), r->get_type_str(), r->rem_set()->get_state_str());
37143
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   249
    }
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   250
    return false;
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   251
  }
11756
28b6fe22e43d 7132029: G1: mixed GC phase lasts for longer than it should
tonyp
parents: 11396
diff changeset
   252
};
37143
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   253
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   254
class ParKnownGarbageTask: public AbstractGangTask {
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   255
  CollectionSetChooser* _hrSorted;
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   256
  uint _chunk_size;
49806
2d62570a615c 8200426: Make G1 code use _g1h members
tschatzl
parents: 49607
diff changeset
   257
  G1CollectedHeap* _g1h;
37143
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   258
  HeapRegionClaimer _hrclaimer;
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   259
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   260
public:
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   261
  ParKnownGarbageTask(CollectionSetChooser* hrSorted, uint chunk_size, uint n_workers) :
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   262
      AbstractGangTask("ParKnownGarbageTask"),
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   263
      _hrSorted(hrSorted), _chunk_size(chunk_size),
49806
2d62570a615c 8200426: Make G1 code use _g1h members
tschatzl
parents: 49607
diff changeset
   264
      _g1h(G1CollectedHeap::heap()), _hrclaimer(n_workers) {}
37143
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   265
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   266
  void work(uint worker_id) {
47885
5caa1d5f74c1 8186571: Implementation: JEP 307: Parallel Full GC for G1
sjohanss
parents: 47216
diff changeset
   267
    ParKnownGarbageHRClosure par_known_garbage_cl(_hrSorted, _chunk_size);
49806
2d62570a615c 8200426: Make G1 code use _g1h members
tschatzl
parents: 49607
diff changeset
   268
    _g1h->heap_region_par_iterate_from_worker_offset(&par_known_garbage_cl, &_hrclaimer, worker_id);
37143
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   269
  }
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   270
};
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   271
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   272
uint CollectionSetChooser::calculate_parallel_work_chunk_size(uint n_workers, uint n_regions) const {
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   273
  assert(n_workers > 0, "Active gc workers should be greater than 0");
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   274
  const uint overpartition_factor = 4;
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   275
  const uint min_chunk_size = MAX2(n_regions / n_workers, 1U);
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   276
  return MAX2(n_regions / (n_workers * overpartition_factor), min_chunk_size);
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   277
}
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   278
49605
784f3f2dea14 8197570: Make rules for choosing collection set candidates more explicit
tschatzl
parents: 48969
diff changeset
   279
bool CollectionSetChooser::region_occupancy_low_enough_for_evac(size_t live_bytes) {
784f3f2dea14 8197570: Make rules for choosing collection set candidates more explicit
tschatzl
parents: 48969
diff changeset
   280
  return live_bytes < mixed_gc_live_threshold_bytes();
784f3f2dea14 8197570: Make rules for choosing collection set candidates more explicit
tschatzl
parents: 48969
diff changeset
   281
}
784f3f2dea14 8197570: Make rules for choosing collection set candidates more explicit
tschatzl
parents: 48969
diff changeset
   282
784f3f2dea14 8197570: Make rules for choosing collection set candidates more explicit
tschatzl
parents: 48969
diff changeset
   283
bool CollectionSetChooser::should_add(HeapRegion* hr) const {
50441
523c2a73a3dc 8204081: Mismatch in rebuild policy and collection set chooser causes remembered sets to be kept errorneously
tschatzl
parents: 49806
diff changeset
   284
  return !hr->is_young() &&
523c2a73a3dc 8204081: Mismatch in rebuild policy and collection set chooser causes remembered sets to be kept errorneously
tschatzl
parents: 49806
diff changeset
   285
         !hr->is_pinned() &&
523c2a73a3dc 8204081: Mismatch in rebuild policy and collection set chooser causes remembered sets to be kept errorneously
tschatzl
parents: 49806
diff changeset
   286
         region_occupancy_low_enough_for_evac(hr->live_bytes()) &&
523c2a73a3dc 8204081: Mismatch in rebuild policy and collection set chooser causes remembered sets to be kept errorneously
tschatzl
parents: 49806
diff changeset
   287
         hr->rem_set()->is_complete();
49605
784f3f2dea14 8197570: Make rules for choosing collection set candidates more explicit
tschatzl
parents: 48969
diff changeset
   288
}
784f3f2dea14 8197570: Make rules for choosing collection set candidates more explicit
tschatzl
parents: 48969
diff changeset
   289
37143
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   290
void CollectionSetChooser::rebuild(WorkGang* workers, uint n_regions) {
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   291
  clear();
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   292
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   293
  uint n_workers = workers->active_workers();
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   294
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   295
  uint chunk_size = calculate_parallel_work_chunk_size(n_workers, n_regions);
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   296
  prepare_for_par_region_addition(n_workers, n_regions, chunk_size);
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   297
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   298
  ParKnownGarbageTask par_known_garbage_task(this, chunk_size, n_workers);
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   299
  workers->run_task(&par_known_garbage_task);
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   300
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   301
  sort_regions();
345ad6728be3 8151637: Move CollectionSetChooser rebuild code into CollectionSetChooser
mgerdin
parents: 35061
diff changeset
   302
}