annotate src/share/vm/utilities/stack.inline.hpp @ 1836:894b1d7c7e01

6423256: GC stacks should use a better data structure 6942771: SEGV in ParScanThreadState::take_from_overflow_stack Reviewed-by: apetrusenko, ysr, pbk
author jcoomes
date Tue, 28 Sep 2010 15:56:15 -0700
parents
children f95d63e2154a
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
1836
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
1 /*
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
2 * Copyright 2009 Sun Microsystems, Inc. All Rights Reserved.
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
3 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
4 *
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
5 * This code is free software; you can redistribute it and/or modify it
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
6 * under the terms of the GNU General Public License version 2 only, as
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
7 * published by the Free Software Foundation.
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
8 *
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
9 * This code is distributed in the hope that it will be useful, but WITHOUT
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
10 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
11 * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
12 * version 2 for more details (a copy is included in the LICENSE file that
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
13 * accompanied this code).
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
14 *
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
15 * You should have received a copy of the GNU General Public License version
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
16 * 2 along with this work; if not, write to the Free Software Foundation,
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
17 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
18 *
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
19 * Please contact Sun Microsystems, Inc., 4150 Network Circle, Santa Clara,
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
20 * CA 95054 USA or visit www.sun.com if you need additional information or
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
21 * have any questions.
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
22 *
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
23 */
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
24
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
25 StackBase::StackBase(size_t segment_size, size_t max_cache_size,
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
26 size_t max_size):
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
27 _seg_size(segment_size),
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
28 _max_cache_size(max_cache_size),
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
29 _max_size(adjust_max_size(max_size, segment_size))
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
30 {
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
31 assert(_max_size % _seg_size == 0, "not a multiple");
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
32 }
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
33
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
34 size_t StackBase::adjust_max_size(size_t max_size, size_t seg_size)
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
35 {
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
36 assert(seg_size > 0, "cannot be 0");
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
37 assert(max_size >= seg_size || max_size == 0, "max_size too small");
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
38 const size_t limit = max_uintx - (seg_size - 1);
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
39 if (max_size == 0 || max_size > limit) {
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
40 max_size = limit;
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
41 }
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
42 return (max_size + seg_size - 1) / seg_size * seg_size;
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
43 }
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
44
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
45 template <class E>
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
46 Stack<E>::Stack(size_t segment_size, size_t max_cache_size, size_t max_size):
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
47 StackBase(adjust_segment_size(segment_size), max_cache_size, max_size)
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
48 {
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
49 reset(true);
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
50 }
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
51
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
52 template <class E>
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
53 void Stack<E>::push(E item)
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
54 {
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
55 assert(!is_full(), "pushing onto a full stack");
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
56 if (_cur_seg_size == _seg_size) {
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
57 push_segment();
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
58 }
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
59 _cur_seg[_cur_seg_size] = item;
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
60 ++_cur_seg_size;
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
61 }
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
62
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
63 template <class E>
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
64 E Stack<E>::pop()
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
65 {
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
66 assert(!is_empty(), "popping from an empty stack");
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
67 if (_cur_seg_size == 1) {
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
68 E tmp = _cur_seg[--_cur_seg_size];
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
69 pop_segment();
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
70 return tmp;
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
71 }
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
72 return _cur_seg[--_cur_seg_size];
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
73 }
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
74
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
75 template <class E>
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
76 void Stack<E>::clear(bool clear_cache)
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
77 {
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
78 free_segments(_cur_seg);
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
79 if (clear_cache) free_segments(_cache);
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
80 reset(clear_cache);
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
81 }
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
82
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
83 template <class E>
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
84 size_t Stack<E>::default_segment_size()
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
85 {
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
86 // Number of elements that fit in 4K bytes minus the size of two pointers
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
87 // (link field and malloc header).
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
88 return (4096 - 2 * sizeof(E*)) / sizeof(E);
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
89 }
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
90
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
91 template <class E>
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
92 size_t Stack<E>::adjust_segment_size(size_t seg_size)
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
93 {
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
94 const size_t elem_sz = sizeof(E);
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
95 const size_t ptr_sz = sizeof(E*);
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
96 assert(elem_sz % ptr_sz == 0 || ptr_sz % elem_sz == 0, "bad element size");
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
97 if (elem_sz < ptr_sz) {
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
98 return align_size_up(seg_size * elem_sz, ptr_sz) / elem_sz;
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
99 }
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
100 return seg_size;
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
101 }
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
102
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
103 template <class E>
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
104 size_t Stack<E>::link_offset() const
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
105 {
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
106 return align_size_up(_seg_size * sizeof(E), sizeof(E*));
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
107 }
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
108
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
109 template <class E>
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
110 size_t Stack<E>::segment_bytes() const
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
111 {
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
112 return link_offset() + sizeof(E*);
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
113 }
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
114
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
115 template <class E>
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
116 E** Stack<E>::link_addr(E* seg) const
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
117 {
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
118 return (E**) ((char*)seg + link_offset());
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
119 }
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
120
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
121 template <class E>
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
122 E* Stack<E>::get_link(E* seg) const
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
123 {
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
124 return *link_addr(seg);
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
125 }
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
126
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
127 template <class E>
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
128 E* Stack<E>::set_link(E* new_seg, E* old_seg)
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
129 {
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
130 *link_addr(new_seg) = old_seg;
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
131 return new_seg;
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
132 }
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
133
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
134 template <class E>
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
135 E* Stack<E>::alloc(size_t bytes)
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
136 {
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
137 return (E*) NEW_C_HEAP_ARRAY(char, bytes);
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
138 }
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
139
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
140 template <class E>
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
141 void Stack<E>::free(E* addr, size_t bytes)
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
142 {
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
143 FREE_C_HEAP_ARRAY(char, (char*) addr);
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
144 }
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
145
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
146 template <class E>
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
147 void Stack<E>::push_segment()
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
148 {
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
149 assert(_cur_seg_size == _seg_size, "current segment is not full");
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
150 E* next;
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
151 if (_cache_size > 0) {
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
152 // Use a cached segment.
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
153 next = _cache;
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
154 _cache = get_link(_cache);
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
155 --_cache_size;
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
156 } else {
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
157 next = alloc(segment_bytes());
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
158 DEBUG_ONLY(zap_segment(next, true);)
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
159 }
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
160 const bool at_empty_transition = is_empty();
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
161 _cur_seg = set_link(next, _cur_seg);
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
162 _cur_seg_size = 0;
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
163 _full_seg_size += at_empty_transition ? 0 : _seg_size;
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
164 DEBUG_ONLY(verify(at_empty_transition);)
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
165 }
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
166
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
167 template <class E>
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
168 void Stack<E>::pop_segment()
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
169 {
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
170 assert(_cur_seg_size == 0, "current segment is not empty");
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
171 E* const prev = get_link(_cur_seg);
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
172 if (_cache_size < _max_cache_size) {
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
173 // Add the current segment to the cache.
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
174 DEBUG_ONLY(zap_segment(_cur_seg, false);)
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
175 _cache = set_link(_cur_seg, _cache);
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
176 ++_cache_size;
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
177 } else {
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
178 DEBUG_ONLY(zap_segment(_cur_seg, true);)
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
179 free(_cur_seg, segment_bytes());
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
180 }
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
181 const bool at_empty_transition = prev == NULL;
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
182 _cur_seg = prev;
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
183 _cur_seg_size = _seg_size;
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
184 _full_seg_size -= at_empty_transition ? 0 : _seg_size;
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
185 DEBUG_ONLY(verify(at_empty_transition);)
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
186 }
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
187
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
188 template <class E>
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
189 void Stack<E>::free_segments(E* seg)
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
190 {
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
191 const size_t bytes = segment_bytes();
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
192 while (seg != NULL) {
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
193 E* const prev = get_link(seg);
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
194 free(seg, bytes);
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
195 seg = prev;
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
196 }
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
197 }
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
198
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
199 template <class E>
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
200 void Stack<E>::reset(bool reset_cache)
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
201 {
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
202 _cur_seg_size = _seg_size; // So push() will alloc a new segment.
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
203 _full_seg_size = 0;
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
204 _cur_seg = NULL;
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
205 if (reset_cache) {
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
206 _cache_size = 0;
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
207 _cache = NULL;
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
208 }
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
209 }
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
210
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
211 #ifdef ASSERT
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
212 template <class E>
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
213 void Stack<E>::verify(bool at_empty_transition) const
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
214 {
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
215 assert(size() <= max_size(), "stack exceeded bounds");
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
216 assert(cache_size() <= max_cache_size(), "cache exceeded bounds");
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
217 assert(_cur_seg_size <= segment_size(), "segment index exceeded bounds");
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
218
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
219 assert(_full_seg_size % _seg_size == 0, "not a multiple");
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
220 assert(at_empty_transition || is_empty() == (size() == 0), "mismatch");
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
221 assert((_cache == NULL) == (cache_size() == 0), "mismatch");
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
222
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
223 if (is_empty()) {
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
224 assert(_cur_seg_size == segment_size(), "sanity");
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
225 }
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
226 }
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
227
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
228 template <class E>
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
229 void Stack<E>::zap_segment(E* seg, bool zap_link_field) const
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
230 {
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
231 if (!ZapStackSegments) return;
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
232 const size_t zap_bytes = segment_bytes() - (zap_link_field ? 0 : sizeof(E*));
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
233 uint32_t* cur = (uint32_t*)seg;
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
234 const uint32_t* end = cur + zap_bytes / sizeof(uint32_t);
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
235 while (cur < end) {
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
236 *cur++ = 0xfadfaded;
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
237 }
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
238 }
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
239 #endif
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
240
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
241 template <class E>
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
242 E* ResourceStack<E>::alloc(size_t bytes)
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
243 {
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
244 return (E*) resource_allocate_bytes(bytes);
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
245 }
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
246
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
247 template <class E>
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
248 void ResourceStack<E>::free(E* addr, size_t bytes)
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
249 {
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
250 resource_free_bytes((char*) addr, bytes);
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
251 }
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
252
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
253 template <class E>
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
254 void StackIterator<E>::sync()
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
255 {
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
256 _full_seg_size = _stack._full_seg_size;
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
257 _cur_seg_size = _stack._cur_seg_size;
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
258 _cur_seg = _stack._cur_seg;
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
259 }
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
260
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
261 template <class E>
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
262 E* StackIterator<E>::next_addr()
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
263 {
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
264 assert(!is_empty(), "no items left");
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
265 if (_cur_seg_size == 1) {
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
266 E* addr = _cur_seg;
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
267 _cur_seg = _stack.get_link(_cur_seg);
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
268 _cur_seg_size = _stack.segment_size();
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
269 _full_seg_size -= _stack.segment_size();
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
270 return addr;
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
271 }
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
272 return _cur_seg + --_cur_seg_size;
894b1d7c7e01 6423256: GC stacks should use a better data structure
jcoomes
parents:
diff changeset
273 }