hotspot/src/share/vm/gc/g1/heapRegionSet.cpp
author david
Mon, 02 Nov 2015 14:28:19 +0100
changeset 33753 3add06d0880f
parent 33752 0782a966d781
child 35061 be6025ebffea
permissions -rw-r--r--
8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet Reviewed-by: pliden, mgerdin
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
     1
/*
29580
a67a581cfe11 8073315: Enable gcc -Wtype-limits and fix upcoming issues.
goetz
parents: 27880
diff changeset
     2
 * Copyright (c) 2011, 2015, Oracle and/or its affiliates. All rights reserved.
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
     4
 *
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
     7
 * published by the Free Software Foundation.
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
     8
 *
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    13
 * accompanied this code).
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    14
 *
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    18
 *
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    21
 * questions.
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    22
 *
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    23
 */
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    24
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    25
#include "precompiled.hpp"
30764
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30158
diff changeset
    26
#include "gc/g1/g1CollectedHeap.inline.hpp"
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30158
diff changeset
    27
#include "gc/g1/heapRegionRemSet.hpp"
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30158
diff changeset
    28
#include "gc/g1/heapRegionSet.inline.hpp"
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    29
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
    30
uint FreeRegionList::_unrealistically_long_length = 0;
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    31
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
    32
#ifndef PRODUCT
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
    33
void HeapRegionSetBase::verify_region(HeapRegion* hr) {
33105
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 31592
diff changeset
    34
  assert(hr->containing_set() == this, "Inconsistent containing set for %u", hr->hrm_index());
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 31592
diff changeset
    35
  assert(!hr->is_young(), "Adding young region %u", hr->hrm_index()); // currently we don't use these sets for young regions
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 31592
diff changeset
    36
  assert(hr->is_humongous() == regions_humongous(), "Wrong humongous state for region %u and set %s", hr->hrm_index(), name());
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 31592
diff changeset
    37
  assert(hr->is_free() == regions_free(), "Wrong free state for region %u and set %s", hr->hrm_index(), name());
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 31592
diff changeset
    38
  assert(!hr->is_free() || hr->is_empty(), "Free region %u is not empty for set %s", hr->hrm_index(), name());
31346
a70d45c06136 8042668: GC Support for shared heap ranges in CDS
jiangli
parents: 30764
diff changeset
    39
  assert(!hr->is_empty() || hr->is_free() || hr->is_archive(),
33105
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 31592
diff changeset
    40
         "Empty region %u is not free or archive for set %s", hr->hrm_index(), name());
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 31592
diff changeset
    41
  assert(hr->rem_set()->verify_ready_for_par_iteration(), "Wrong iteration state %u", hr->hrm_index());
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    42
}
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
    43
#endif
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    44
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    45
void HeapRegionSetBase::verify() {
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    46
  // It's important that we also observe the MT safety protocol even
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    47
  // for the verification calls. If we do verification without the
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    48
  // appropriate locks and the set changes underneath our feet
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    49
  // verification might fail and send us on a wild goose chase.
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
    50
  check_mt_safety();
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    51
33753
3add06d0880f 8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet
david
parents: 33752
diff changeset
    52
  guarantee_heap_region_set(( is_empty() && length() == 0 && total_capacity_bytes() == 0) ||
3add06d0880f 8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet
david
parents: 33752
diff changeset
    53
                            (!is_empty() && length() > 0  && total_capacity_bytes() > 0) ,
3add06d0880f 8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet
david
parents: 33752
diff changeset
    54
                            "invariant");
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    55
}
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    56
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    57
void HeapRegionSetBase::verify_start() {
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    58
  // See comment in verify() about MT safety and verification.
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
    59
  check_mt_safety();
33753
3add06d0880f 8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet
david
parents: 33752
diff changeset
    60
  assert_heap_region_set(!_verify_in_progress, "verification should not be in progress");
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    61
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    62
  // Do the basic verification first before we do the checks over the regions.
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    63
  HeapRegionSetBase::verify();
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    64
26157
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
    65
  _verify_in_progress = true;
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    66
}
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    67
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    68
void HeapRegionSetBase::verify_end() {
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    69
  // See comment in verify() about MT safety and verification.
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
    70
  check_mt_safety();
33753
3add06d0880f 8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet
david
parents: 33752
diff changeset
    71
  assert_heap_region_set(_verify_in_progress, "verification should be in progress");
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    72
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    73
  _verify_in_progress = false;
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    74
}
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    75
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    76
void HeapRegionSetBase::print_on(outputStream* out, bool print_contents) {
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    77
  out->cr();
31592
43f48e165466 8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents: 31346
diff changeset
    78
  out->print_cr("Set: %s (" PTR_FORMAT ")", name(), p2i(this));
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    79
  out->print_cr("  Region Assumptions");
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    80
  out->print_cr("    humongous         : %s", BOOL_TO_STR(regions_humongous()));
26696
623a25e6c686 8057768: Make heap region region type in G1 HeapRegion explicit
tonyp
parents: 26316
diff changeset
    81
  out->print_cr("    free              : %s", BOOL_TO_STR(regions_free()));
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    82
  out->print_cr("  Attributes");
12381
1438e0fbfa27 7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents: 10996
diff changeset
    83
  out->print_cr("    length            : %14u", length());
31592
43f48e165466 8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents: 31346
diff changeset
    84
  out->print_cr("    total capacity    : " SIZE_FORMAT_W(14) " bytes",
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    85
                total_capacity_bytes());
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    86
}
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    87
26696
623a25e6c686 8057768: Make heap region region type in G1 HeapRegion explicit
tonyp
parents: 26316
diff changeset
    88
HeapRegionSetBase::HeapRegionSetBase(const char* name, bool humongous, bool free, HRSMtSafeChecker* mt_safety_checker)
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    89
  : _name(name), _verify_in_progress(false),
26696
623a25e6c686 8057768: Make heap region region type in G1 HeapRegion explicit
tonyp
parents: 26316
diff changeset
    90
    _is_humongous(humongous), _is_free(free), _mt_safety_checker(mt_safety_checker),
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
    91
    _count()
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
    92
{ }
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    93
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
    94
void FreeRegionList::set_unrealistically_long_length(uint len) {
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
    95
  guarantee(_unrealistically_long_length == 0, "should only be set once");
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
    96
  _unrealistically_long_length = len;
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    97
}
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    98
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
    99
void FreeRegionList::remove_all() {
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   100
  check_mt_safety();
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   101
  verify_optional();
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   102
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   103
  HeapRegion* curr = _head;
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   104
  while (curr != NULL) {
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   105
    verify_region(curr);
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   106
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   107
    HeapRegion* next = curr->next();
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   108
    curr->set_next(NULL);
23471
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   109
    curr->set_prev(NULL);
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   110
    curr->set_containing_set(NULL);
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   111
    curr = next;
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   112
  }
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   113
  clear();
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   114
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   115
  verify_optional();
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   116
}
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   117
23471
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   118
void FreeRegionList::add_ordered(FreeRegionList* from_list) {
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   119
  check_mt_safety();
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   120
  from_list->check_mt_safety();
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   121
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   122
  verify_optional();
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   123
  from_list->verify_optional();
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   124
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   125
  if (from_list->is_empty()) {
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   126
    return;
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   127
  }
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   128
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   129
  #ifdef ASSERT
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   130
  FreeRegionListIterator iter(from_list);
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   131
  while (iter.more_available()) {
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   132
    HeapRegion* hr = iter.get_next();
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   133
    // In set_containing_set() we check that we either set the value
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   134
    // from NULL to non-NULL or vice versa to catch bugs. So, we have
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   135
    // to NULL it first before setting it to the value.
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   136
    hr->set_containing_set(NULL);
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   137
    hr->set_containing_set(this);
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   138
  }
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   139
  #endif // ASSERT
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   140
26157
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   141
  if (is_empty()) {
33753
3add06d0880f 8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet
david
parents: 33752
diff changeset
   142
    assert_free_region_list(length() == 0 && _tail == NULL, "invariant");
26157
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   143
    _head = from_list->_head;
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   144
    _tail = from_list->_tail;
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   145
  } else {
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   146
    HeapRegion* curr_to = _head;
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   147
    HeapRegion* curr_from = from_list->_head;
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   148
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   149
    while (curr_from != NULL) {
26316
93f6b40c038b 8054819: Rename HeapRegionSeq to HeapRegionManager
tschatzl
parents: 26160
diff changeset
   150
      while (curr_to != NULL && curr_to->hrm_index() < curr_from->hrm_index()) {
26157
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   151
        curr_to = curr_to->next();
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   152
      }
23471
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   153
26157
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   154
      if (curr_to == NULL) {
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   155
        // The rest of the from list should be added as tail
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   156
        _tail->set_next(curr_from);
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   157
        curr_from->set_prev(_tail);
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   158
        curr_from = NULL;
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   159
      } else {
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   160
        HeapRegion* next_from = curr_from->next();
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   161
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   162
        curr_from->set_next(curr_to);
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   163
        curr_from->set_prev(curr_to->prev());
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   164
        if (curr_to->prev() == NULL) {
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   165
          _head = curr_from;
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   166
        } else {
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   167
          curr_to->prev()->set_next(curr_from);
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   168
        }
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   169
        curr_to->set_prev(curr_from);
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   170
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   171
        curr_from = next_from;
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   172
      }
23471
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   173
    }
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   174
26316
93f6b40c038b 8054819: Rename HeapRegionSeq to HeapRegionManager
tschatzl
parents: 26160
diff changeset
   175
    if (_tail->hrm_index() < from_list->_tail->hrm_index()) {
26157
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   176
      _tail = from_list->_tail;
23471
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   177
    }
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   178
  }
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   179
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   180
  _count.increment(from_list->length(), from_list->total_capacity_bytes());
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   181
  from_list->clear();
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   182
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   183
  verify_optional();
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   184
  from_list->verify_optional();
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   185
}
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   186
26157
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   187
void FreeRegionList::remove_starting_at(HeapRegion* first, uint num_regions) {
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   188
  check_mt_safety();
33753
3add06d0880f 8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet
david
parents: 33752
diff changeset
   189
  assert_free_region_list(num_regions >= 1, "pre-condition");
3add06d0880f 8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet
david
parents: 33752
diff changeset
   190
  assert_free_region_list(!is_empty(), "pre-condition");
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   191
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   192
  verify_optional();
12381
1438e0fbfa27 7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents: 10996
diff changeset
   193
  DEBUG_ONLY(uint old_length = length();)
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   194
26157
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   195
  HeapRegion* curr = first;
12381
1438e0fbfa27 7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents: 10996
diff changeset
   196
  uint count = 0;
26157
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   197
  while (count < num_regions) {
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   198
    verify_region(curr);
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   199
    HeapRegion* next = curr->next();
23471
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   200
    HeapRegion* prev = curr->prev();
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   201
26157
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   202
    assert(count < num_regions,
33753
3add06d0880f 8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet
david
parents: 33752
diff changeset
   203
           "[%s] should not come across more regions "
3add06d0880f 8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet
david
parents: 33752
diff changeset
   204
           "pending for removal than num_regions: %u",
3add06d0880f 8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet
david
parents: 33752
diff changeset
   205
           name(), num_regions);
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   206
26157
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   207
    if (prev == NULL) {
33753
3add06d0880f 8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet
david
parents: 33752
diff changeset
   208
      assert_free_region_list(_head == curr, "invariant");
26157
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   209
      _head = next;
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   210
    } else {
33753
3add06d0880f 8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet
david
parents: 33752
diff changeset
   211
      assert_free_region_list(_head != curr, "invariant");
26157
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   212
      prev->set_next(next);
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   213
    }
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   214
    if (next == NULL) {
33753
3add06d0880f 8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet
david
parents: 33752
diff changeset
   215
      assert_free_region_list(_tail == curr, "invariant");
26157
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   216
      _tail = prev;
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   217
    } else {
33753
3add06d0880f 8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet
david
parents: 33752
diff changeset
   218
      assert_free_region_list(_tail != curr, "invariant");
26157
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   219
      next->set_prev(prev);
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   220
    }
33752
0782a966d781 8141056: Erroneous assignment in HeapRegionSet.cpp
david
parents: 33105
diff changeset
   221
    if (_last == curr) {
26157
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   222
      _last = NULL;
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   223
    }
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   224
26157
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   225
    curr->set_next(NULL);
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   226
    curr->set_prev(NULL);
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   227
    remove(curr);
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   228
26157
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   229
    count++;
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   230
    curr = next;
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   231
  }
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   232
26157
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   233
  assert(count == num_regions,
33753
3add06d0880f 8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet
david
parents: 33752
diff changeset
   234
         "[%s] count: %u should be == num_regions: %u",
3add06d0880f 8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet
david
parents: 33752
diff changeset
   235
         name(), count, num_regions);
26157
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   236
  assert(length() + num_regions == old_length,
33753
3add06d0880f 8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet
david
parents: 33752
diff changeset
   237
         "[%s] new length should be consistent "
3add06d0880f 8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet
david
parents: 33752
diff changeset
   238
         "new length: %u old length: %u num_regions: %u",
3add06d0880f 8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet
david
parents: 33752
diff changeset
   239
         name(), length(), old_length, num_regions);
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   240
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   241
  verify_optional();
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   242
}
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   243
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   244
void FreeRegionList::verify() {
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   245
  // See comment in HeapRegionSetBase::verify() about MT safety and
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   246
  // verification.
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   247
  check_mt_safety();
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   248
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   249
  // This will also do the basic verification too.
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   250
  verify_start();
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   251
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   252
  verify_list();
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   253
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   254
  verify_end();
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   255
}
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   256
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   257
void FreeRegionList::clear() {
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   258
  _count = HeapRegionSetCount();
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   259
  _head = NULL;
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   260
  _tail = NULL;
23471
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   261
  _last = NULL;
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   262
}
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   263
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   264
void FreeRegionList::print_on(outputStream* out, bool print_contents) {
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   265
  HeapRegionSetBase::print_on(out, print_contents);
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   266
  out->print_cr("  Linking");
31592
43f48e165466 8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents: 31346
diff changeset
   267
  out->print_cr("    head              : " PTR_FORMAT, p2i(_head));
43f48e165466 8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents: 31346
diff changeset
   268
  out->print_cr("    tail              : " PTR_FORMAT, p2i(_tail));
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   269
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   270
  if (print_contents) {
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   271
    out->print_cr("  Contents");
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   272
    FreeRegionListIterator iter(this);
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   273
    while (iter.more_available()) {
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   274
      HeapRegion* hr = iter.get_next();
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   275
      hr->print_on(out);
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   276
    }
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   277
  }
26157
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   278
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   279
  out->cr();
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   280
}
23459
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   281
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   282
void FreeRegionList::verify_list() {
26157
70eddb655686 8054818: Refactor HeapRegionSeq to manage heap region and auxiliary data
tschatzl
parents: 24424
diff changeset
   283
  HeapRegion* curr = _head;
23459
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   284
  HeapRegion* prev1 = NULL;
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   285
  HeapRegion* prev0 = NULL;
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   286
  uint count = 0;
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   287
  size_t capacity = 0;
23471
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   288
  uint last_index = 0;
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   289
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   290
  guarantee(_head == NULL || _head->prev() == NULL, "_head should not have a prev");
23459
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   291
  while (curr != NULL) {
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   292
    verify_region(curr);
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   293
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   294
    count++;
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   295
    guarantee(count < _unrealistically_long_length,
33753
3add06d0880f 8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet
david
parents: 33752
diff changeset
   296
              "[%s] the calculated length: %u seems very long, is there maybe a cycle? curr: " PTR_FORMAT " prev0: " PTR_FORMAT " " "prev1: " PTR_FORMAT " length: %u",
3add06d0880f 8137756: Remove hrs_err_msg and hrs_ext_msg from heapRegionSet
david
parents: 33752
diff changeset
   297
              name(), count, p2i(curr), p2i(prev0), p2i(prev1), length());
23459
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   298
23471
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   299
    if (curr->next() != NULL) {
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   300
      guarantee(curr->next()->prev() == curr, "Next or prev pointers messed up");
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   301
    }
26316
93f6b40c038b 8054819: Rename HeapRegionSeq to HeapRegionManager
tschatzl
parents: 26160
diff changeset
   302
    guarantee(curr->hrm_index() == 0 || curr->hrm_index() > last_index, "List should be sorted");
93f6b40c038b 8054819: Rename HeapRegionSeq to HeapRegionManager
tschatzl
parents: 26160
diff changeset
   303
    last_index = curr->hrm_index();
23471
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   304
23459
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   305
    capacity += curr->capacity();
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   306
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   307
    prev1 = prev0;
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   308
    prev0 = curr;
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   309
    curr = curr->next();
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   310
  }
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   311
33105
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 31592
diff changeset
   312
  guarantee(_tail == prev0, "Expected %s to end with %u but it ended with %u.", name(), _tail->hrm_index(), prev0->hrm_index());
23471
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   313
  guarantee(_tail == NULL || _tail->next() == NULL, "_tail should not have a next");
33105
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 31592
diff changeset
   314
  guarantee(length() == count, "%s count mismatch. Expected %u, actual %u.", name(), length(), count);
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 31592
diff changeset
   315
  guarantee(total_capacity_bytes() == capacity, "%s capacity mismatch. Expected " SIZE_FORMAT ", actual " SIZE_FORMAT,
294e48b4f704 8080775: Better argument formatting for assert() and friends
david
parents: 31592
diff changeset
   316
            name(), total_capacity_bytes(), capacity);
23459
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   317
}
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   318
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   319
// Note on the check_mt_safety() methods below:
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   320
//
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   321
// Verification of the "master" heap region sets / lists that are
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   322
// maintained by G1CollectedHeap is always done during a STW pause and
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   323
// by the VM thread at the start / end of the pause. The standard
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   324
// verification methods all assert check_mt_safety(). This is
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   325
// important as it ensures that verification is done without
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   326
// concurrent updates taking place at the same time. It follows, that,
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   327
// for the "master" heap region sets / lists, the check_mt_safety()
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   328
// method should include the VM thread / STW case.
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   329
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   330
void MasterFreeRegionListMtSafeChecker::check() {
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   331
  // Master Free List MT safety protocol:
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   332
  // (a) If we're at a safepoint, operations on the master free list
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   333
  // should be invoked by either the VM thread (which will serialize
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   334
  // them) or by the GC workers while holding the
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   335
  // FreeList_lock.
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   336
  // (b) If we're not at a safepoint, operations on the master free
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   337
  // list should be invoked while holding the Heap_lock.
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   338
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   339
  if (SafepointSynchronize::is_at_safepoint()) {
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   340
    guarantee(Thread::current()->is_VM_thread() ||
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   341
              FreeList_lock->owned_by_self(), "master free list MT safety protocol at a safepoint");
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   342
  } else {
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   343
    guarantee(Heap_lock->owned_by_self(), "master free list MT safety protocol outside a safepoint");
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   344
  }
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   345
}
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   346
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   347
void SecondaryFreeRegionListMtSafeChecker::check() {
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   348
  // Secondary Free List MT safety protocol:
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   349
  // Operations on the secondary free list should always be invoked
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   350
  // while holding the SecondaryFreeList_lock.
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   351
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   352
  guarantee(SecondaryFreeList_lock->owned_by_self(), "secondary free list MT safety protocol");
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   353
}
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   354
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   355
void OldRegionSetMtSafeChecker::check() {
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   356
  // Master Old Set MT safety protocol:
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   357
  // (a) If we're at a safepoint, operations on the master old set
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   358
  // should be invoked:
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   359
  // - by the VM thread (which will serialize them), or
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   360
  // - by the GC workers while holding the FreeList_lock, if we're
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   361
  //   at a safepoint for an evacuation pause (this lock is taken
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   362
  //   anyway when an GC alloc region is retired so that a new one
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   363
  //   is allocated from the free list), or
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   364
  // - by the GC workers while holding the OldSets_lock, if we're at a
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   365
  //   safepoint for a cleanup pause.
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   366
  // (b) If we're not at a safepoint, operations on the master old set
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   367
  // should be invoked while holding the Heap_lock.
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   368
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   369
  if (SafepointSynchronize::is_at_safepoint()) {
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   370
    guarantee(Thread::current()->is_VM_thread()
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   371
        || FreeList_lock->owned_by_self() || OldSets_lock->owned_by_self(),
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   372
        "master old set MT safety protocol at a safepoint");
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   373
  } else {
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   374
    guarantee(Heap_lock->owned_by_self(), "master old set MT safety protocol outside a safepoint");
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   375
  }
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   376
}
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   377
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   378
void HumongousRegionSetMtSafeChecker::check() {
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   379
  // Humongous Set MT safety protocol:
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   380
  // (a) If we're at a safepoint, operations on the master humongous
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   381
  // set should be invoked by either the VM thread (which will
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   382
  // serialize them) or by the GC workers while holding the
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   383
  // OldSets_lock.
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   384
  // (b) If we're not at a safepoint, operations on the master
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   385
  // humongous set should be invoked while holding the Heap_lock.
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   386
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   387
  if (SafepointSynchronize::is_at_safepoint()) {
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   388
    guarantee(Thread::current()->is_VM_thread() ||
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   389
              OldSets_lock->owned_by_self(),
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   390
              "master humongous set MT safety protocol at a safepoint");
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   391
  } else {
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   392
    guarantee(Heap_lock->owned_by_self(),
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   393
              "master humongous set MT safety protocol outside a safepoint");
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   394
  }
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   395
}
26160
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 26157
diff changeset
   396
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 26157
diff changeset
   397
void FreeRegionList_test() {
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 26157
diff changeset
   398
  FreeRegionList l("test");
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 26157
diff changeset
   399
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 26157
diff changeset
   400
  const uint num_regions_in_test = 5;
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 26157
diff changeset
   401
  // Create a fake heap. It does not need to be valid, as the HeapRegion constructor
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 26157
diff changeset
   402
  // does not access it.
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 26157
diff changeset
   403
  MemRegion heap(NULL, num_regions_in_test * HeapRegion::GrainWords);
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 26157
diff changeset
   404
  // Allocate a fake BOT because the HeapRegion constructor initializes
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 26157
diff changeset
   405
  // the BOT.
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 26157
diff changeset
   406
  size_t bot_size = G1BlockOffsetSharedArray::compute_size(heap.word_size());
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 26157
diff changeset
   407
  HeapWord* bot_data = NEW_C_HEAP_ARRAY(HeapWord, bot_size, mtGC);
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 26157
diff changeset
   408
  ReservedSpace bot_rs(G1BlockOffsetSharedArray::compute_size(heap.word_size()));
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 26157
diff changeset
   409
  G1RegionToSpaceMapper* bot_storage =
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 26157
diff changeset
   410
    G1RegionToSpaceMapper::create_mapper(bot_rs,
30158
bd6094906ef8 8058354: SPECjvm2008-Derby -2.7% performance regression on Solaris-X64 starting with 9-b29
tschatzl
parents: 29796
diff changeset
   411
                                         bot_rs.size(),
26160
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 26157
diff changeset
   412
                                         os::vm_page_size(),
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 26157
diff changeset
   413
                                         HeapRegion::GrainBytes,
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 26157
diff changeset
   414
                                         G1BlockOffsetSharedArray::N_bytes,
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 26157
diff changeset
   415
                                         mtGC);
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 26157
diff changeset
   416
  G1BlockOffsetSharedArray oa(heap, bot_storage);
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 26157
diff changeset
   417
  bot_storage->commit_regions(0, num_regions_in_test);
27629
81ea316b7244 8061449: G1: FreeRegionList_test() fails with G1 after the JDK-8058534 fix to HeapRegion::orig_end()
brutisso
parents: 26846
diff changeset
   418
81ea316b7244 8061449: G1: FreeRegionList_test() fails with G1 after the JDK-8058534 fix to HeapRegion::orig_end()
brutisso
parents: 26846
diff changeset
   419
  // Set up memory regions for the heap regions.
81ea316b7244 8061449: G1: FreeRegionList_test() fails with G1 after the JDK-8058534 fix to HeapRegion::orig_end()
brutisso
parents: 26846
diff changeset
   420
  MemRegion mr0(heap.start(), HeapRegion::GrainWords);
81ea316b7244 8061449: G1: FreeRegionList_test() fails with G1 after the JDK-8058534 fix to HeapRegion::orig_end()
brutisso
parents: 26846
diff changeset
   421
  MemRegion mr1(mr0.end(), HeapRegion::GrainWords);
81ea316b7244 8061449: G1: FreeRegionList_test() fails with G1 after the JDK-8058534 fix to HeapRegion::orig_end()
brutisso
parents: 26846
diff changeset
   422
  MemRegion mr2(mr1.end(), HeapRegion::GrainWords);
81ea316b7244 8061449: G1: FreeRegionList_test() fails with G1 after the JDK-8058534 fix to HeapRegion::orig_end()
brutisso
parents: 26846
diff changeset
   423
  MemRegion mr3(mr2.end(), HeapRegion::GrainWords);
81ea316b7244 8061449: G1: FreeRegionList_test() fails with G1 after the JDK-8058534 fix to HeapRegion::orig_end()
brutisso
parents: 26846
diff changeset
   424
  MemRegion mr4(mr3.end(), HeapRegion::GrainWords);
81ea316b7244 8061449: G1: FreeRegionList_test() fails with G1 after the JDK-8058534 fix to HeapRegion::orig_end()
brutisso
parents: 26846
diff changeset
   425
81ea316b7244 8061449: G1: FreeRegionList_test() fails with G1 after the JDK-8058534 fix to HeapRegion::orig_end()
brutisso
parents: 26846
diff changeset
   426
  HeapRegion hr0(0, &oa, mr0);
81ea316b7244 8061449: G1: FreeRegionList_test() fails with G1 after the JDK-8058534 fix to HeapRegion::orig_end()
brutisso
parents: 26846
diff changeset
   427
  HeapRegion hr1(1, &oa, mr1);
81ea316b7244 8061449: G1: FreeRegionList_test() fails with G1 after the JDK-8058534 fix to HeapRegion::orig_end()
brutisso
parents: 26846
diff changeset
   428
  HeapRegion hr2(2, &oa, mr2);
81ea316b7244 8061449: G1: FreeRegionList_test() fails with G1 after the JDK-8058534 fix to HeapRegion::orig_end()
brutisso
parents: 26846
diff changeset
   429
  HeapRegion hr3(3, &oa, mr3);
81ea316b7244 8061449: G1: FreeRegionList_test() fails with G1 after the JDK-8058534 fix to HeapRegion::orig_end()
brutisso
parents: 26846
diff changeset
   430
  HeapRegion hr4(4, &oa, mr4);
26160
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 26157
diff changeset
   431
  l.add_ordered(&hr1);
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 26157
diff changeset
   432
  l.add_ordered(&hr0);
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 26157
diff changeset
   433
  l.add_ordered(&hr3);
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 26157
diff changeset
   434
  l.add_ordered(&hr4);
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 26157
diff changeset
   435
  l.add_ordered(&hr2);
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 26157
diff changeset
   436
  assert(l.length() == num_regions_in_test, "wrong length");
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 26157
diff changeset
   437
  l.verify_list();
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 26157
diff changeset
   438
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 26157
diff changeset
   439
  bot_storage->uncommit_regions(0, num_regions_in_test);
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 26157
diff changeset
   440
  delete bot_storage;
27880
afb974a04396 8060074: os::free() takes MemoryTrackingLevel but doesn't need it
coleenp
parents: 27629
diff changeset
   441
  FREE_C_HEAP_ARRAY(HeapWord, bot_data);
26160
aba6b01cb988 8038423: G1: Decommit memory within heap
tschatzl
parents: 26157
diff changeset
   442
}