Mercurial > hg > truffle
annotate src/share/vm/gc_implementation/shared/generationCounters.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 | b52782ae3880 |
children | 8229bd737950 |
rev | line source |
---|---|
0 | 1 /* |
3289
b52782ae3880
6946417: G1: Java VisualVM does not support G1 properly.
jmasa
parents:
1972
diff
changeset
|
2 * Copyright (c) 2002, 2011, Oracle and/or its affiliates. All rights reserved. |
0 | 3 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
4 * | |
5 * This code is free software; you can redistribute it and/or modify it | |
6 * under the terms of the GNU General Public License version 2 only, as | |
7 * published by the Free Software Foundation. | |
8 * | |
9 * This code is distributed in the hope that it will be useful, but WITHOUT | |
10 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or | |
11 * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License | |
12 * version 2 for more details (a copy is included in the LICENSE file that | |
13 * accompanied this code). | |
14 * | |
15 * You should have received a copy of the GNU General Public License version | |
16 * 2 along with this work; if not, write to the Free Software Foundation, | |
17 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. | |
18 * | |
1552
c18cbe5936b8
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
0
diff
changeset
|
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
c18cbe5936b8
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
0
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:
0
diff
changeset
|
21 * questions. |
0 | 22 * |
23 */ | |
24 | |
1972 | 25 #ifndef SHARE_VM_GC_IMPLEMENTATION_SHARED_GENERATIONCOUNTERS_HPP |
26 #define SHARE_VM_GC_IMPLEMENTATION_SHARED_GENERATIONCOUNTERS_HPP | |
27 | |
28 #include "runtime/perfData.hpp" | |
29 #include "runtime/virtualspace.hpp" | |
30 | |
0 | 31 // A GenerationCounter is a holder class for performance counters |
32 // that track a generation | |
33 | |
34 class GenerationCounters: public CHeapObj { | |
35 friend class VMStructs; | |
36 | |
37 protected: | |
38 PerfVariable* _current_size; | |
39 VirtualSpace* _virtual_space; | |
40 | |
41 // Constant PerfData types don't need to retain a reference. | |
42 // However, it's a good idea to document them here. | |
43 // PerfStringConstant* _name; | |
44 // PerfConstant* _min_size; | |
45 // PerfConstant* _max_size; | |
46 // PerfConstant* _spaces; | |
47 | |
48 char* _name_space; | |
49 | |
50 // This constructor is only meant for use with the PSGenerationCounters | |
51 // constructor. The need for such an constructor should be eliminated | |
52 // when VirtualSpace and PSVirtualSpace are unified. | |
53 GenerationCounters() : _name_space(NULL), _current_size(NULL), _virtual_space(NULL) {} | |
54 public: | |
55 | |
56 GenerationCounters(const char* name, int ordinal, int spaces, | |
57 VirtualSpace* v); | |
58 | |
59 ~GenerationCounters() { | |
60 if (_name_space != NULL) FREE_C_HEAP_ARRAY(char, _name_space); | |
61 } | |
62 | |
63 virtual void update_all() { | |
3289
b52782ae3880
6946417: G1: Java VisualVM does not support G1 properly.
jmasa
parents:
1972
diff
changeset
|
64 _current_size->set_value(_virtual_space == NULL ? 0 : |
b52782ae3880
6946417: G1: Java VisualVM does not support G1 properly.
jmasa
parents:
1972
diff
changeset
|
65 _virtual_space->committed_size()); |
0 | 66 } |
67 | |
68 const char* name_space() const { return _name_space; } | |
3289
b52782ae3880
6946417: G1: Java VisualVM does not support G1 properly.
jmasa
parents:
1972
diff
changeset
|
69 |
0 | 70 }; |
1972 | 71 #endif // SHARE_VM_GC_IMPLEMENTATION_SHARED_GENERATIONCOUNTERS_HPP |