annotate src/share/vm/gc_implementation/g1/heapRegionSeq.inline.hpp @ 3766:c3f1170908be

7045330: G1: Simplify/fix the HeapRegionSeq class 7042285: G1: native memory leak during humongous object allocation 6804436: G1: heap region indices should be size_t Summary: A series of fixes and improvements to the HeapRegionSeq class: a) replace the _regions growable array with a standard C array, b) avoid de-allocating / re-allocating HeapRegion instances when the heap shrinks / grows (fix for 7042285), c) introduce fast method to map address to HeapRegion via a "biased" array pointer, d) embed the _hrs object in G1CollectedHeap, instead of pointing to it via an indirection, e) assume that all the regions added to the HeapRegionSeq instance are contiguous, f) replace int's with size_t's for indexes (and expand that to HeapRegion as part of 6804436), g) remove unnecessary / unused methods, h) rename a couple of fields (_alloc_search_start and _seq_bottom), i) fix iterate_from() not to always start from index 0 irrespective of the region passed to it, j) add a verification method to check the HeapRegionSeq assumptions, k) always call the wrappers for _hrs.iterate(), _hrs_length(), and _hrs.at() from G1CollectedHeap, not those methods directly, and l) unify the code that expands the sequence (by either re-using or creating a new HeapRegion) and make it robust wrt to a HeapRegion allocation failing. Reviewed-by: stefank, johnc, brutisso
author tonyp
date Fri, 10 Jun 2011 13:16:40 -0400
parents f95d63e2154a
children 720b6a76dd9d
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
342
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
1 /*
3766
c3f1170908be 7045330: G1: Simplify/fix the HeapRegionSeq class
tonyp
parents: 1972
diff changeset
2 * Copyright (c) 2001, 2011, Oracle and/or its affiliates. All rights reserved.
342
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
3 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
4 *
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
5 * This code is free software; you can redistribute it and/or modify it
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
6 * under the terms of the GNU General Public License version 2 only, as
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
7 * published by the Free Software Foundation.
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
8 *
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
9 * This code is distributed in the hope that it will be useful, but WITHOUT
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
10 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
11 * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
12 * version 2 for more details (a copy is included in the LICENSE file that
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
13 * accompanied this code).
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
14 *
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
15 * You should have received a copy of the GNU General Public License version
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
16 * 2 along with this work; if not, write to the Free Software Foundation,
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
17 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
18 *
1552
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 342
diff changeset
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 342
diff changeset
20 * or visit www.oracle.com if you need additional information or have any
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 342
diff changeset
21 * questions.
342
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
22 *
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
23 */
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
24
1972
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1552
diff changeset
25 #ifndef SHARE_VM_GC_IMPLEMENTATION_G1_HEAPREGIONSEQ_INLINE_HPP
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1552
diff changeset
26 #define SHARE_VM_GC_IMPLEMENTATION_G1_HEAPREGIONSEQ_INLINE_HPP
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1552
diff changeset
27
3766
c3f1170908be 7045330: G1: Simplify/fix the HeapRegionSeq class
tonyp
parents: 1972
diff changeset
28 #include "gc_implementation/g1/heapRegion.hpp"
1972
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1552
diff changeset
29 #include "gc_implementation/g1/heapRegionSeq.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1552
diff changeset
30
3766
c3f1170908be 7045330: G1: Simplify/fix the HeapRegionSeq class
tonyp
parents: 1972
diff changeset
31 inline size_t HeapRegionSeq::addr_to_index_biased(HeapWord* addr) const {
c3f1170908be 7045330: G1: Simplify/fix the HeapRegionSeq class
tonyp
parents: 1972
diff changeset
32 assert(_heap_bottom <= addr && addr < _heap_end,
c3f1170908be 7045330: G1: Simplify/fix the HeapRegionSeq class
tonyp
parents: 1972
diff changeset
33 err_msg("addr: "PTR_FORMAT" bottom: "PTR_FORMAT" end: "PTR_FORMAT,
c3f1170908be 7045330: G1: Simplify/fix the HeapRegionSeq class
tonyp
parents: 1972
diff changeset
34 addr, _heap_bottom, _heap_end));
c3f1170908be 7045330: G1: Simplify/fix the HeapRegionSeq class
tonyp
parents: 1972
diff changeset
35 size_t index = (size_t) addr >> _region_shift;
c3f1170908be 7045330: G1: Simplify/fix the HeapRegionSeq class
tonyp
parents: 1972
diff changeset
36 return index;
c3f1170908be 7045330: G1: Simplify/fix the HeapRegionSeq class
tonyp
parents: 1972
diff changeset
37 }
c3f1170908be 7045330: G1: Simplify/fix the HeapRegionSeq class
tonyp
parents: 1972
diff changeset
38
c3f1170908be 7045330: G1: Simplify/fix the HeapRegionSeq class
tonyp
parents: 1972
diff changeset
39 inline HeapRegion* HeapRegionSeq::addr_to_region_unsafe(HeapWord* addr) const {
c3f1170908be 7045330: G1: Simplify/fix the HeapRegionSeq class
tonyp
parents: 1972
diff changeset
40 assert(_heap_bottom <= addr && addr < _heap_end,
c3f1170908be 7045330: G1: Simplify/fix the HeapRegionSeq class
tonyp
parents: 1972
diff changeset
41 err_msg("addr: "PTR_FORMAT" bottom: "PTR_FORMAT" end: "PTR_FORMAT,
c3f1170908be 7045330: G1: Simplify/fix the HeapRegionSeq class
tonyp
parents: 1972
diff changeset
42 addr, _heap_bottom, _heap_end));
c3f1170908be 7045330: G1: Simplify/fix the HeapRegionSeq class
tonyp
parents: 1972
diff changeset
43 size_t index_biased = addr_to_index_biased(addr);
c3f1170908be 7045330: G1: Simplify/fix the HeapRegionSeq class
tonyp
parents: 1972
diff changeset
44 HeapRegion* hr = _regions_biased[index_biased];
c3f1170908be 7045330: G1: Simplify/fix the HeapRegionSeq class
tonyp
parents: 1972
diff changeset
45 assert(hr != NULL, "invariant");
c3f1170908be 7045330: G1: Simplify/fix the HeapRegionSeq class
tonyp
parents: 1972
diff changeset
46 return hr;
c3f1170908be 7045330: G1: Simplify/fix the HeapRegionSeq class
tonyp
parents: 1972
diff changeset
47 }
c3f1170908be 7045330: G1: Simplify/fix the HeapRegionSeq class
tonyp
parents: 1972
diff changeset
48
c3f1170908be 7045330: G1: Simplify/fix the HeapRegionSeq class
tonyp
parents: 1972
diff changeset
49 inline HeapRegion* HeapRegionSeq::addr_to_region(HeapWord* addr) const {
c3f1170908be 7045330: G1: Simplify/fix the HeapRegionSeq class
tonyp
parents: 1972
diff changeset
50 if (addr != NULL && addr < _heap_end) {
c3f1170908be 7045330: G1: Simplify/fix the HeapRegionSeq class
tonyp
parents: 1972
diff changeset
51 assert(addr >= _heap_bottom,
c3f1170908be 7045330: G1: Simplify/fix the HeapRegionSeq class
tonyp
parents: 1972
diff changeset
52 err_msg("addr: "PTR_FORMAT" bottom: "PTR_FORMAT, addr, _heap_bottom));
c3f1170908be 7045330: G1: Simplify/fix the HeapRegionSeq class
tonyp
parents: 1972
diff changeset
53 return addr_to_region_unsafe(addr);
342
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
54 }
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
55 return NULL;
37f87013dfd8 6711316: Open source the Garbage-First garbage collector
ysr
parents:
diff changeset
56 }
1972
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1552
diff changeset
57
3766
c3f1170908be 7045330: G1: Simplify/fix the HeapRegionSeq class
tonyp
parents: 1972
diff changeset
58 inline HeapRegion* HeapRegionSeq::at(size_t index) const {
c3f1170908be 7045330: G1: Simplify/fix the HeapRegionSeq class
tonyp
parents: 1972
diff changeset
59 assert(index < length(), "pre-condition");
c3f1170908be 7045330: G1: Simplify/fix the HeapRegionSeq class
tonyp
parents: 1972
diff changeset
60 HeapRegion* hr = _regions[index];
c3f1170908be 7045330: G1: Simplify/fix the HeapRegionSeq class
tonyp
parents: 1972
diff changeset
61 assert(hr != NULL, "sanity");
c3f1170908be 7045330: G1: Simplify/fix the HeapRegionSeq class
tonyp
parents: 1972
diff changeset
62 assert(hr->hrs_index() == index, "sanity");
c3f1170908be 7045330: G1: Simplify/fix the HeapRegionSeq class
tonyp
parents: 1972
diff changeset
63 return hr;
c3f1170908be 7045330: G1: Simplify/fix the HeapRegionSeq class
tonyp
parents: 1972
diff changeset
64 }
c3f1170908be 7045330: G1: Simplify/fix the HeapRegionSeq class
tonyp
parents: 1972
diff changeset
65
1972
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1552
diff changeset
66 #endif // SHARE_VM_GC_IMPLEMENTATION_G1_HEAPREGIONSEQ_INLINE_HPP