hotspot/src/share/vm/gc_implementation/g1/heapRegionSet.cpp
author drchase
Fri, 09 May 2014 16:50:54 -0400
changeset 24424 2658d7834c6e
parent 23471 ec9427262f0a
child 26157 70eddb655686
permissions -rw-r--r--
8037816: Fix for 8036122 breaks build with Xcode5/clang Summary: Repaired or selectively disabled offending formats; future-proofed with additional checking Reviewed-by: kvn, jrose, stefank
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
     1
/*
23471
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
     2
 * Copyright (c) 2011, 2014, 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"
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
    26
#include "gc_implementation/g1/heapRegionRemSet.hpp"
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    27
#include "gc_implementation/g1/heapRegionSet.inline.hpp"
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    28
24424
2658d7834c6e 8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents: 23471
diff changeset
    29
PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC
2658d7834c6e 8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents: 23471
diff changeset
    30
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
    31
uint FreeRegionList::_unrealistically_long_length = 0;
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    32
8680
f1c414e16a4c 7014923: G1: code cleanup
tonyp
parents: 7923
diff changeset
    33
void HeapRegionSetBase::fill_in_ext_msg(hrs_ext_msg* msg, const char* message) {
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
    34
  msg->append("[%s] %s ln: %u cy: "SIZE_FORMAT,
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
    35
              name(), message, length(), total_capacity_bytes());
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    36
  fill_in_ext_msg_extra(msg);
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    37
}
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    38
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
    39
#ifndef PRODUCT
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
    40
void HeapRegionSetBase::verify_region(HeapRegion* hr) {
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
    41
  assert(hr->containing_set() == this, err_msg("Inconsistent containing set for %u", hr->hrs_index()));
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
    42
  assert(!hr->is_young(), err_msg("Adding young region %u", hr->hrs_index())); // currently we don't use these sets for young regions
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
    43
  assert(hr->isHumongous() == regions_humongous(), err_msg("Wrong humongous state for region %u and set %s", hr->hrs_index(), name()));
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
    44
  assert(hr->is_empty() == regions_empty(), err_msg("Wrong empty state for region %u and set %s", hr->hrs_index(), name()));
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
    45
  assert(hr->rem_set()->verify_ready_for_par_iteration(), err_msg("Wrong iteration state %u", hr->hrs_index()));
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    46
}
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
    47
#endif
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    48
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    49
void HeapRegionSetBase::verify() {
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    50
  // It's important that we also observe the MT safety protocol even
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    51
  // for the verification calls. If we do verification without the
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    52
  // appropriate locks and the set changes underneath our feet
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    53
  // verification might fail and send us on a wild goose chase.
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
    54
  check_mt_safety();
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    55
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
    56
  guarantee(( is_empty() && length() == 0 && total_capacity_bytes() == 0) ||
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
    57
            (!is_empty() && length() >= 0 && total_capacity_bytes() >= 0),
8680
f1c414e16a4c 7014923: G1: code cleanup
tonyp
parents: 7923
diff changeset
    58
            hrs_ext_msg(this, "invariant"));
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    59
}
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    60
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    61
void HeapRegionSetBase::verify_start() {
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    62
  // See comment in verify() about MT safety and verification.
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
    63
  check_mt_safety();
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    64
  assert(!_verify_in_progress,
8680
f1c414e16a4c 7014923: G1: code cleanup
tonyp
parents: 7923
diff changeset
    65
         hrs_ext_msg(this, "verification should not be in progress"));
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
  // 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
    68
  HeapRegionSetBase::verify();
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    69
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    70
  _verify_in_progress        = true;
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    71
}
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
void HeapRegionSetBase::verify_end() {
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    74
  // See comment in verify() about MT safety and verification.
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
    75
  check_mt_safety();
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    76
  assert(_verify_in_progress,
8680
f1c414e16a4c 7014923: G1: code cleanup
tonyp
parents: 7923
diff changeset
    77
         hrs_ext_msg(this, "verification should be in progress"));
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    78
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    79
  _verify_in_progress = false;
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    80
}
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    81
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    82
void HeapRegionSetBase::print_on(outputStream* out, bool print_contents) {
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    83
  out->cr();
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    84
  out->print_cr("Set: %s ("PTR_FORMAT")", name(), this);
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    85
  out->print_cr("  Region Assumptions");
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    86
  out->print_cr("    humongous         : %s", BOOL_TO_STR(regions_humongous()));
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    87
  out->print_cr("    empty             : %s", BOOL_TO_STR(regions_empty()));
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    88
  out->print_cr("  Attributes");
12381
1438e0fbfa27 7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents: 10996
diff changeset
    89
  out->print_cr("    length            : %14u", length());
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    90
  out->print_cr("    total capacity    : "SIZE_FORMAT_W(14)" bytes",
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    91
                total_capacity_bytes());
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    92
}
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
HeapRegionSetBase::HeapRegionSetBase(const char* name, bool humongous, bool empty, HRSMtSafeChecker* mt_safety_checker)
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    95
  : _name(name), _verify_in_progress(false),
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
    96
    _is_humongous(humongous), _is_empty(empty), _mt_safety_checker(mt_safety_checker),
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
    97
    _count()
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
    98
{ }
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
    99
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   100
void FreeRegionList::set_unrealistically_long_length(uint len) {
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   101
  guarantee(_unrealistically_long_length == 0, "should only be set once");
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   102
  _unrealistically_long_length = len;
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   103
}
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   104
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   105
void FreeRegionList::fill_in_ext_msg_extra(hrs_ext_msg* msg) {
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   106
  msg->append(" hd: "PTR_FORMAT" tl: "PTR_FORMAT, head(), tail());
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   107
}
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   108
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   109
void FreeRegionList::add_as_head_or_tail(FreeRegionList* from_list, bool as_head) {
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   110
  check_mt_safety();
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   111
  from_list->check_mt_safety();
8927
461fa7ee5254 7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents: 8680
diff changeset
   112
461fa7ee5254 7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents: 8680
diff changeset
   113
  verify_optional();
461fa7ee5254 7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents: 8680
diff changeset
   114
  from_list->verify_optional();
461fa7ee5254 7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents: 8680
diff changeset
   115
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   116
  if (from_list->is_empty()) {
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   117
    return;
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   118
  }
8927
461fa7ee5254 7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents: 8680
diff changeset
   119
461fa7ee5254 7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents: 8680
diff changeset
   120
#ifdef ASSERT
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   121
  FreeRegionListIterator iter(from_list);
8927
461fa7ee5254 7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents: 8680
diff changeset
   122
  while (iter.more_available()) {
461fa7ee5254 7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents: 8680
diff changeset
   123
    HeapRegion* hr = iter.get_next();
461fa7ee5254 7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents: 8680
diff changeset
   124
    // In set_containing_set() we check that we either set the value
461fa7ee5254 7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents: 8680
diff changeset
   125
    // from NULL to non-NULL or vice versa to catch bugs. So, we have
461fa7ee5254 7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents: 8680
diff changeset
   126
    // to NULL it first before setting it to the value.
461fa7ee5254 7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents: 8680
diff changeset
   127
    hr->set_containing_set(NULL);
461fa7ee5254 7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents: 8680
diff changeset
   128
    hr->set_containing_set(this);
461fa7ee5254 7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents: 8680
diff changeset
   129
  }
461fa7ee5254 7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents: 8680
diff changeset
   130
#endif // ASSERT
461fa7ee5254 7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents: 8680
diff changeset
   131
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   132
  if (_head == NULL) {
13334
a737bbd385f5 7184772: G1: Incorrect assert in HeapRegionLinkedList::add_as_head()
johnc
parents: 12381
diff changeset
   133
    assert(length() == 0 && _tail == NULL, hrs_ext_msg(this, "invariant"));
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   134
    _head = from_list->_head;
8927
461fa7ee5254 7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents: 8680
diff changeset
   135
    _tail = from_list->_tail;
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   136
  } else {
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   137
    assert(length() > 0 && _tail != NULL, hrs_ext_msg(this, "invariant"));
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   138
    if (as_head) {
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   139
      from_list->_tail->set_next(_head);
23471
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   140
      _head->set_prev(from_list->_tail);
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   141
      _head = from_list->_head;
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   142
    } else {
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   143
      _tail->set_next(from_list->_head);
23471
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   144
      from_list->_head->set_prev(_tail);
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   145
      _tail = from_list->_tail;
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   146
    }
8927
461fa7ee5254 7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents: 8680
diff changeset
   147
  }
461fa7ee5254 7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents: 8680
diff changeset
   148
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   149
  _count.increment(from_list->length(), from_list->total_capacity_bytes());
8927
461fa7ee5254 7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents: 8680
diff changeset
   150
  from_list->clear();
461fa7ee5254 7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents: 8680
diff changeset
   151
461fa7ee5254 7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents: 8680
diff changeset
   152
  verify_optional();
461fa7ee5254 7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents: 8680
diff changeset
   153
  from_list->verify_optional();
461fa7ee5254 7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents: 8680
diff changeset
   154
}
461fa7ee5254 7029458: G1: Add newly-reclaimed regions to the beginning of the region free list, not the end
tonyp
parents: 8680
diff changeset
   155
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   156
void FreeRegionList::add_as_head(FreeRegionList* from_list) {
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   157
  add_as_head_or_tail(from_list, true /* as_head */);
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   158
}
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   159
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   160
void FreeRegionList::add_as_tail(FreeRegionList* from_list) {
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   161
  add_as_head_or_tail(from_list, false /* as_head */);
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   162
}
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   163
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   164
void FreeRegionList::remove_all() {
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   165
  check_mt_safety();
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   166
  verify_optional();
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   167
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   168
  HeapRegion* curr = _head;
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   169
  while (curr != NULL) {
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   170
    verify_region(curr);
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   171
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   172
    HeapRegion* next = curr->next();
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   173
    curr->set_next(NULL);
23471
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   174
    curr->set_prev(NULL);
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   175
    curr->set_containing_set(NULL);
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   176
    curr = next;
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   177
  }
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   178
  clear();
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   179
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   180
  verify_optional();
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   181
}
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   182
23471
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   183
void FreeRegionList::add_ordered(FreeRegionList* from_list) {
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   184
  check_mt_safety();
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   185
  from_list->check_mt_safety();
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   186
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   187
  verify_optional();
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   188
  from_list->verify_optional();
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   189
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   190
  if (from_list->is_empty()) {
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   191
    return;
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   192
  }
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   193
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   194
  if (is_empty()) {
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   195
    add_as_head(from_list);
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   196
    return;
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   197
  }
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   198
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   199
  #ifdef ASSERT
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   200
  FreeRegionListIterator iter(from_list);
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   201
  while (iter.more_available()) {
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   202
    HeapRegion* hr = iter.get_next();
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   203
    // 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
   204
    // 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
   205
    // 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
   206
    hr->set_containing_set(NULL);
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   207
    hr->set_containing_set(this);
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   208
  }
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   209
  #endif // ASSERT
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   210
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   211
  HeapRegion* curr_to = _head;
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   212
  HeapRegion* curr_from = from_list->_head;
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   213
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   214
  while (curr_from != NULL) {
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   215
    while (curr_to != NULL && curr_to->hrs_index() < curr_from->hrs_index()) {
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   216
      curr_to = curr_to->next();
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   217
    }
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   218
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   219
    if (curr_to == NULL) {
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   220
      // The rest of the from list should be added as tail
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   221
      _tail->set_next(curr_from);
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   222
      curr_from->set_prev(_tail);
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   223
      curr_from = NULL;
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   224
    } else {
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   225
      HeapRegion* next_from = curr_from->next();
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   226
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   227
      curr_from->set_next(curr_to);
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   228
      curr_from->set_prev(curr_to->prev());
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   229
      if (curr_to->prev() == NULL) {
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   230
        _head = curr_from;
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   231
      } else {
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   232
        curr_to->prev()->set_next(curr_from);
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   233
      }
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   234
      curr_to->set_prev(curr_from);
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   235
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   236
      curr_from = next_from;
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   237
    }
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   238
  }
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   239
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   240
  if (_tail->hrs_index() < from_list->_tail->hrs_index()) {
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   241
    _tail = from_list->_tail;
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   242
  }
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   243
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   244
  _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
   245
  from_list->clear();
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   246
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   247
  verify_optional();
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   248
  from_list->verify_optional();
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   249
}
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   250
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   251
void FreeRegionList::remove_all_pending(uint target_count) {
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   252
  check_mt_safety();
8680
f1c414e16a4c 7014923: G1: code cleanup
tonyp
parents: 7923
diff changeset
   253
  assert(target_count > 1, hrs_ext_msg(this, "pre-condition"));
f1c414e16a4c 7014923: G1: code cleanup
tonyp
parents: 7923
diff changeset
   254
  assert(!is_empty(), hrs_ext_msg(this, "pre-condition"));
7923
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
  verify_optional();
12381
1438e0fbfa27 7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents: 10996
diff changeset
   257
  DEBUG_ONLY(uint old_length = length();)
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   258
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   259
  HeapRegion* curr = _head;
12381
1438e0fbfa27 7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents: 10996
diff changeset
   260
  uint count = 0;
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   261
  while (curr != NULL) {
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   262
    verify_region(curr);
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   263
    HeapRegion* next = curr->next();
23471
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   264
    HeapRegion* prev = curr->prev();
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   265
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   266
    if (curr->pending_removal()) {
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   267
      assert(count < target_count,
8680
f1c414e16a4c 7014923: G1: code cleanup
tonyp
parents: 7923
diff changeset
   268
             hrs_err_msg("[%s] should not come across more regions "
12381
1438e0fbfa27 7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents: 10996
diff changeset
   269
                         "pending for removal than target_count: %u",
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   270
                         name(), target_count));
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   271
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   272
      if (prev == NULL) {
8680
f1c414e16a4c 7014923: G1: code cleanup
tonyp
parents: 7923
diff changeset
   273
        assert(_head == curr, hrs_ext_msg(this, "invariant"));
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   274
        _head = next;
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   275
      } else {
8680
f1c414e16a4c 7014923: G1: code cleanup
tonyp
parents: 7923
diff changeset
   276
        assert(_head != curr, hrs_ext_msg(this, "invariant"));
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   277
        prev->set_next(next);
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   278
      }
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   279
      if (next == NULL) {
8680
f1c414e16a4c 7014923: G1: code cleanup
tonyp
parents: 7923
diff changeset
   280
        assert(_tail == curr, hrs_ext_msg(this, "invariant"));
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   281
        _tail = prev;
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   282
      } else {
8680
f1c414e16a4c 7014923: G1: code cleanup
tonyp
parents: 7923
diff changeset
   283
        assert(_tail != curr, hrs_ext_msg(this, "invariant"));
23471
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   284
        next->set_prev(prev);
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   285
      }
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   286
      if (_last = curr) {
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   287
        _last = NULL;
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   288
      }
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   289
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   290
      curr->set_next(NULL);
23471
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   291
      curr->set_prev(NULL);
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   292
      remove(curr);
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   293
      curr->set_pending_removal(false);
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   294
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   295
      count += 1;
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   296
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   297
      // If we have come across the target number of regions we can
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   298
      // just bail out. However, for debugging purposes, we can just
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   299
      // carry on iterating to make sure there are not more regions
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   300
      // tagged with pending removal.
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   301
      DEBUG_ONLY(if (count == target_count) break;)
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   302
    }
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   303
    curr = next;
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   304
  }
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   305
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   306
  assert(count == target_count,
12381
1438e0fbfa27 7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents: 10996
diff changeset
   307
         hrs_err_msg("[%s] count: %u should be == target_count: %u",
1438e0fbfa27 7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents: 10996
diff changeset
   308
                     name(), count, target_count));
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   309
  assert(length() + target_count == old_length,
8680
f1c414e16a4c 7014923: G1: code cleanup
tonyp
parents: 7923
diff changeset
   310
         hrs_err_msg("[%s] new length should be consistent "
12381
1438e0fbfa27 7157073: G1: type change size_t -> uint for region counts / indexes
tonyp
parents: 10996
diff changeset
   311
                     "new length: %u old length: %u target_count: %u",
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   312
                     name(), length(), old_length, target_count));
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   313
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   314
  verify_optional();
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   315
}
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   316
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   317
void FreeRegionList::verify() {
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   318
  // See comment in HeapRegionSetBase::verify() about MT safety and
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   319
  // verification.
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   320
  check_mt_safety();
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   321
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   322
  // This will also do the basic verification too.
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   323
  verify_start();
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   324
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   325
  verify_list();
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   326
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   327
  verify_end();
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   328
}
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   329
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   330
void FreeRegionList::clear() {
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   331
  _count = HeapRegionSetCount();
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   332
  _head = NULL;
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   333
  _tail = NULL;
23471
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   334
  _last = NULL;
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   335
}
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   336
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   337
void FreeRegionList::print_on(outputStream* out, bool print_contents) {
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   338
  HeapRegionSetBase::print_on(out, print_contents);
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   339
  out->print_cr("  Linking");
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   340
  out->print_cr("    head              : "PTR_FORMAT, _head);
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   341
  out->print_cr("    tail              : "PTR_FORMAT, _tail);
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   342
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   343
  if (print_contents) {
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   344
    out->print_cr("  Contents");
23450
c7c6202fc7e2 8034079: G1: Refactor the HeapRegionSet hierarchy
brutisso
parents: 13336
diff changeset
   345
    FreeRegionListIterator iter(this);
7923
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   346
    while (iter.more_available()) {
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   347
      HeapRegion* hr = iter.get_next();
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   348
      hr->print_on(out);
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   349
    }
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   350
  }
fc200fcd4e05 6977804: G1: remove the zero-filling thread
tonyp
parents:
diff changeset
   351
}
23459
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   352
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   353
void FreeRegionList::verify_list() {
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   354
  HeapRegion* curr = head();
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   355
  HeapRegion* prev1 = NULL;
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   356
  HeapRegion* prev0 = NULL;
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   357
  uint count = 0;
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   358
  size_t capacity = 0;
23471
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   359
  uint last_index = 0;
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   360
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   361
  guarantee(_head == NULL || _head->prev() == NULL, "_head should not have a prev");
23459
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   362
  while (curr != NULL) {
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   363
    verify_region(curr);
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   364
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   365
    count++;
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   366
    guarantee(count < _unrealistically_long_length,
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   367
        hrs_err_msg("[%s] the calculated length: %u seems very long, is there maybe a cycle? curr: "PTR_FORMAT" prev0: "PTR_FORMAT" " "prev1: "PTR_FORMAT" length: %u", name(), count, curr, prev0, prev1, length()));
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   368
23471
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   369
    if (curr->next() != NULL) {
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   370
      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
   371
    }
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   372
    guarantee(curr->hrs_index() == 0 || curr->hrs_index() > last_index, "List should be sorted");
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   373
    last_index = curr->hrs_index();
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   374
23459
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   375
    capacity += curr->capacity();
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   376
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   377
    prev1 = prev0;
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   378
    prev0 = curr;
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   379
    curr = curr->next();
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   380
  }
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   381
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   382
  guarantee(tail() == prev0, err_msg("Expected %s to end with %u but it ended with %u.", name(), tail()->hrs_index(), prev0->hrs_index()));
23471
ec9427262f0a 8036025: Sort the freelist in order to shrink the heap
jwilhelm
parents: 23459
diff changeset
   383
  guarantee(_tail == NULL || _tail->next() == NULL, "_tail should not have a next");
23459
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   384
  guarantee(length() == count, err_msg("%s count mismatch. Expected %u, actual %u.", name(), length(), count));
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   385
  guarantee(total_capacity_bytes() == capacity, err_msg("%s capacity mismatch. Expected " SIZE_FORMAT ", actual " SIZE_FORMAT,
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   386
      name(), total_capacity_bytes(), capacity));
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   387
}
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   388
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   389
// Note on the check_mt_safety() methods below:
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   390
//
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   391
// Verification of the "master" heap region sets / lists that are
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   392
// maintained by G1CollectedHeap is always done during a STW pause and
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   393
// by the VM thread at the start / end of the pause. The standard
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   394
// verification methods all assert check_mt_safety(). This is
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   395
// important as it ensures that verification is done without
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   396
// concurrent updates taking place at the same time. It follows, that,
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   397
// for the "master" heap region sets / lists, the check_mt_safety()
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   398
// method should include the VM thread / STW case.
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   399
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   400
void MasterFreeRegionListMtSafeChecker::check() {
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   401
  // Master Free List MT safety protocol:
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   402
  // (a) If we're at a safepoint, operations on the master free list
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   403
  // should be invoked by either the VM thread (which will serialize
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   404
  // them) or by the GC workers while holding the
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   405
  // FreeList_lock.
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   406
  // (b) If we're not at a safepoint, operations on the master free
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   407
  // list should be invoked while holding the Heap_lock.
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   408
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   409
  if (SafepointSynchronize::is_at_safepoint()) {
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   410
    guarantee(Thread::current()->is_VM_thread() ||
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   411
              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
   412
  } else {
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   413
    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
   414
  }
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   415
}
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   416
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   417
void SecondaryFreeRegionListMtSafeChecker::check() {
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   418
  // Secondary Free List MT safety protocol:
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   419
  // Operations on the secondary free list should always be invoked
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   420
  // while holding the SecondaryFreeList_lock.
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   421
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   422
  guarantee(SecondaryFreeList_lock->owned_by_self(), "secondary free list MT safety protocol");
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   423
}
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   424
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   425
void OldRegionSetMtSafeChecker::check() {
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   426
  // Master Old Set MT safety protocol:
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   427
  // (a) If we're at a safepoint, operations on the master old set
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   428
  // should be invoked:
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   429
  // - by the VM thread (which will serialize them), or
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   430
  // - by the GC workers while holding the FreeList_lock, if we're
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   431
  //   at a safepoint for an evacuation pause (this lock is taken
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   432
  //   anyway when an GC alloc region is retired so that a new one
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   433
  //   is allocated from the free list), or
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   434
  // - 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
   435
  //   safepoint for a cleanup pause.
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   436
  // (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
   437
  // should be invoked while holding the Heap_lock.
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   438
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   439
  if (SafepointSynchronize::is_at_safepoint()) {
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   440
    guarantee(Thread::current()->is_VM_thread()
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   441
        || FreeList_lock->owned_by_self() || OldSets_lock->owned_by_self(),
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   442
        "master old set MT safety protocol at a safepoint");
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   443
  } else {
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   444
    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
   445
  }
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   446
}
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   447
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   448
void HumongousRegionSetMtSafeChecker::check() {
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   449
  // Humongous Set MT safety protocol:
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   450
  // (a) If we're at a safepoint, operations on the master humongous
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   451
  // set should be invoked by either the VM thread (which will
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   452
  // serialize them) or by the GC workers while holding the
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   453
  // OldSets_lock.
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   454
  // (b) If we're not at a safepoint, operations on the master
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   455
  // humongous set should be invoked while holding the Heap_lock.
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   456
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   457
  if (SafepointSynchronize::is_at_safepoint()) {
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   458
    guarantee(Thread::current()->is_VM_thread() ||
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   459
              OldSets_lock->owned_by_self(),
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   460
              "master humongous set MT safety protocol at a safepoint");
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   461
  } else {
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   462
    guarantee(Heap_lock->owned_by_self(),
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   463
              "master humongous set MT safety protocol outside a safepoint");
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   464
  }
cf51656f7404 8037407: G1: Remove heapRegionSets.cpp
brutisso
parents: 23450
diff changeset
   465
}