truffle: src/share/vm/gc_implementation/g1/heapRegionSet.cpp annotate

annotate src/share/vm/gc_implementation/g1/heapRegionSet.cpp @ 20297:ca8b8e21e2ca

8050973: CMS/G1 GC: add missing Resource and Handle mark Summary: Add Resource/HandleMark in the work() method of some AbstractGangTask to reclaim these resources earlier. Reviewed-by: tschatzl, goetz

author	mdoerr
date	Tue, 29 Jul 2014 10:26:09 +0200
parents	78bbf4d43a14
children	52b4284cb496 6701abbc4441

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

Mercurial > hg > truffle

annotate src/share/vm/gc_implementation/g1/heapRegionSet.cpp @ 20297:ca8b8e21e2ca