Mercurial > hg > graal-jvmci-8
annotate src/share/vm/compiler/oopMap.cpp @ 24234:ea6f94ab283b default tip
Added tag jvmci-0.36 for changeset 8128b98d4736
author | Gilles Duboscq <gilles.m.duboscq@oracle.com> |
---|---|
date | Mon, 18 Sep 2017 18:49:45 +0200 |
parents | 1051fbaca88b |
children |
rev | line source |
---|---|
0 | 1 /* |
17937
78bbf4d43a14
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
12316
diff
changeset
|
2 * Copyright (c) 1998, 2014, 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:
948
diff
changeset
|
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
c18cbe5936b8
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
948
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:
948
diff
changeset
|
21 * questions. |
0 | 22 * |
23 */ | |
24 | |
1972 | 25 #include "precompiled.hpp" |
26 #include "code/codeBlob.hpp" | |
27 #include "code/codeCache.hpp" | |
28 #include "code/nmethod.hpp" | |
29 #include "code/scopeDesc.hpp" | |
30 #include "compiler/oopMap.hpp" | |
31 #include "gc_interface/collectedHeap.hpp" | |
32 #include "memory/allocation.inline.hpp" | |
33 #include "memory/resourceArea.hpp" | |
34 #include "runtime/frame.inline.hpp" | |
35 #include "runtime/signature.hpp" | |
36 #ifdef COMPILER1 | |
37 #include "c1/c1_Defs.hpp" | |
38 #endif | |
22407
708a95bb443c
All SPARC float registers can be addressed in 4 byte portions; hence don't add 4 bytes to pointer pointing to a Narrow Oop
Stefan Anzinger <stefan.anzinger@oracle.com>
parents:
22298
diff
changeset
|
39 #ifdef SPARC |
708a95bb443c
All SPARC float registers can be addressed in 4 byte portions; hence don't add 4 bytes to pointer pointing to a Narrow Oop
Stefan Anzinger <stefan.anzinger@oracle.com>
parents:
22298
diff
changeset
|
40 #include "vmreg_sparc.inline.hpp" |
708a95bb443c
All SPARC float registers can be addressed in 4 byte portions; hence don't add 4 bytes to pointer pointing to a Narrow Oop
Stefan Anzinger <stefan.anzinger@oracle.com>
parents:
22298
diff
changeset
|
41 #endif |
0 | 42 |
43 // OopMapStream | |
44 | |
45 OopMapStream::OopMapStream(OopMap* oop_map) { | |
46 if(oop_map->omv_data() == NULL) { | |
47 _stream = new CompressedReadStream(oop_map->write_stream()->buffer()); | |
48 } else { | |
49 _stream = new CompressedReadStream(oop_map->omv_data()); | |
50 } | |
51 _mask = OopMapValue::type_mask_in_place; | |
52 _size = oop_map->omv_count(); | |
53 _position = 0; | |
54 _valid_omv = false; | |
55 } | |
56 | |
57 | |
58 OopMapStream::OopMapStream(OopMap* oop_map, int oop_types_mask) { | |
59 if(oop_map->omv_data() == NULL) { | |
60 _stream = new CompressedReadStream(oop_map->write_stream()->buffer()); | |
61 } else { | |
62 _stream = new CompressedReadStream(oop_map->omv_data()); | |
63 } | |
64 _mask = oop_types_mask; | |
65 _size = oop_map->omv_count(); | |
66 _position = 0; | |
67 _valid_omv = false; | |
68 } | |
69 | |
70 | |
71 void OopMapStream::find_next() { | |
72 while(_position++ < _size) { | |
73 _omv.read_from(_stream); | |
74 if(((int)_omv.type() & _mask) > 0) { | |
75 _valid_omv = true; | |
76 return; | |
77 } | |
78 } | |
79 _valid_omv = false; | |
80 } | |
81 | |
82 | |
83 // OopMap | |
84 | |
85 // frame_size units are stack-slots (4 bytes) NOT intptr_t; we can name odd | |
86 // slots to hold 4-byte values like ints and floats in the LP64 build. | |
87 OopMap::OopMap(int frame_size, int arg_count) { | |
88 // OopMaps are usually quite so small, so pick a small initial size | |
89 set_write_stream(new CompressedWriteStream(32)); | |
90 set_omv_data(NULL); | |
91 set_omv_count(0); | |
92 | |
93 #ifdef ASSERT | |
94 _locs_length = VMRegImpl::stack2reg(0)->value() + frame_size + arg_count; | |
95 _locs_used = NEW_RESOURCE_ARRAY(OopMapValue::oop_types, _locs_length); | |
96 for(int i = 0; i < _locs_length; i++) _locs_used[i] = OopMapValue::unused_value; | |
97 #endif | |
98 } | |
99 | |
100 | |
101 OopMap::OopMap(OopMap::DeepCopyToken, OopMap* source) { | |
102 // This constructor does a deep copy | |
103 // of the source OopMap. | |
104 set_write_stream(new CompressedWriteStream(source->omv_count() * 2)); | |
105 set_omv_data(NULL); | |
106 set_omv_count(0); | |
107 set_offset(source->offset()); | |
108 | |
109 #ifdef ASSERT | |
110 _locs_length = source->_locs_length; | |
111 _locs_used = NEW_RESOURCE_ARRAY(OopMapValue::oop_types, _locs_length); | |
112 for(int i = 0; i < _locs_length; i++) _locs_used[i] = OopMapValue::unused_value; | |
113 #endif | |
114 | |
115 // We need to copy the entries too. | |
116 for (OopMapStream oms(source); !oms.is_done(); oms.next()) { | |
117 OopMapValue omv = oms.current(); | |
118 omv.write_on(write_stream()); | |
119 increment_count(); | |
120 } | |
121 } | |
122 | |
123 | |
124 OopMap* OopMap::deep_copy() { | |
125 return new OopMap(_deep_copy_token, this); | |
126 } | |
127 | |
128 | |
129 void OopMap::copy_to(address addr) { | |
130 memcpy(addr,this,sizeof(OopMap)); | |
131 memcpy(addr + sizeof(OopMap),write_stream()->buffer(),write_stream()->position()); | |
132 OopMap* new_oop = (OopMap*)addr; | |
133 new_oop->set_omv_data_size(write_stream()->position()); | |
134 new_oop->set_omv_data((unsigned char *)(addr + sizeof(OopMap))); | |
135 new_oop->set_write_stream(NULL); | |
136 } | |
137 | |
138 | |
139 int OopMap::heap_size() const { | |
140 int size = sizeof(OopMap); | |
141 int align = sizeof(void *) - 1; | |
142 if(write_stream() != NULL) { | |
143 size += write_stream()->position(); | |
144 } else { | |
145 size += omv_data_size(); | |
146 } | |
147 // Align to a reasonable ending point | |
148 size = ((size+align) & ~align); | |
149 return size; | |
150 } | |
151 | |
152 // frame_size units are stack-slots (4 bytes) NOT intptr_t; we can name odd | |
153 // slots to hold 4-byte values like ints and floats in the LP64 build. | |
154 void OopMap::set_xxx(VMReg reg, OopMapValue::oop_types x, VMReg optional) { | |
155 | |
156 assert(reg->value() < _locs_length, "too big reg value for stack size"); | |
157 assert( _locs_used[reg->value()] == OopMapValue::unused_value, "cannot insert twice" ); | |
158 debug_only( _locs_used[reg->value()] = x; ) | |
159 | |
160 OopMapValue o(reg, x); | |
161 | |
162 if(x == OopMapValue::callee_saved_value) { | |
163 // This can never be a stack location, so we don't need to transform it. | |
164 assert(optional->is_reg(), "Trying to callee save a stack location"); | |
165 o.set_content_reg(optional); | |
166 } else if(x == OopMapValue::derived_oop_value) { | |
167 o.set_content_reg(optional); | |
168 } | |
169 | |
170 o.write_on(write_stream()); | |
171 increment_count(); | |
172 } | |
173 | |
174 | |
175 void OopMap::set_oop(VMReg reg) { | |
176 set_xxx(reg, OopMapValue::oop_value, VMRegImpl::Bad()); | |
177 } | |
178 | |
179 | |
180 void OopMap::set_value(VMReg reg) { | |
181 // At this time, we only need value entries in our OopMap when ZapDeadCompiledLocals is active. | |
182 } | |
183 | |
184 | |
113
ba764ed4b6f2
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
100
diff
changeset
|
185 void OopMap::set_narrowoop(VMReg reg) { |
ba764ed4b6f2
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
100
diff
changeset
|
186 set_xxx(reg, OopMapValue::narrowoop_value, VMRegImpl::Bad()); |
0 | 187 } |
188 | |
189 | |
190 void OopMap::set_callee_saved(VMReg reg, VMReg caller_machine_register ) { | |
191 set_xxx(reg, OopMapValue::callee_saved_value, caller_machine_register); | |
192 } | |
193 | |
194 | |
195 void OopMap::set_derived_oop(VMReg reg, VMReg derived_from_local_register ) { | |
196 if( reg == derived_from_local_register ) { | |
197 // Actually an oop, derived shares storage with base, | |
198 set_oop(reg); | |
199 } else { | |
200 set_xxx(reg, OopMapValue::derived_oop_value, derived_from_local_register); | |
201 } | |
202 } | |
203 | |
204 // OopMapSet | |
205 | |
206 OopMapSet::OopMapSet() { | |
207 set_om_size(MinOopMapAllocation); | |
208 set_om_count(0); | |
209 OopMap** temp = NEW_RESOURCE_ARRAY(OopMap*, om_size()); | |
210 set_om_data(temp); | |
211 } | |
212 | |
213 | |
214 void OopMapSet::grow_om_data() { | |
215 int new_size = om_size() * 2; | |
216 OopMap** new_data = NEW_RESOURCE_ARRAY(OopMap*, new_size); | |
217 memcpy(new_data,om_data(),om_size() * sizeof(OopMap*)); | |
218 set_om_size(new_size); | |
219 set_om_data(new_data); | |
220 } | |
221 | |
222 | |
223 void OopMapSet::copy_to(address addr) { | |
224 address temp = addr; | |
225 int align = sizeof(void *) - 1; | |
226 // Copy this | |
227 memcpy(addr,this,sizeof(OopMapSet)); | |
228 temp += sizeof(OopMapSet); | |
229 temp = (address)((intptr_t)(temp + align) & ~align); | |
230 // Do the needed fixups to the new OopMapSet | |
231 OopMapSet* new_set = (OopMapSet*)addr; | |
232 new_set->set_om_data((OopMap**)temp); | |
233 // Allow enough space for the OopMap pointers | |
234 temp += (om_count() * sizeof(OopMap*)); | |
235 | |
236 for(int i=0; i < om_count(); i++) { | |
237 OopMap* map = at(i); | |
238 map->copy_to((address)temp); | |
239 new_set->set(i,(OopMap*)temp); | |
240 temp += map->heap_size(); | |
241 } | |
242 // This "locks" the OopMapSet | |
243 new_set->set_om_size(-1); | |
244 } | |
245 | |
246 | |
247 void OopMapSet::add_gc_map(int pc_offset, OopMap *map ) { | |
248 assert(om_size() != -1,"Cannot grow a fixed OopMapSet"); | |
249 | |
250 if(om_count() >= om_size()) { | |
251 grow_om_data(); | |
252 } | |
253 map->set_offset(pc_offset); | |
254 | |
255 #ifdef ASSERT | |
256 if(om_count() > 0) { | |
257 OopMap* last = at(om_count()-1); | |
258 if (last->offset() == map->offset() ) { | |
259 fatal("OopMap inserted twice"); | |
260 } | |
261 if(last->offset() > map->offset()) { | |
262 tty->print_cr( "WARNING, maps not sorted: pc[%d]=%d, pc[%d]=%d", | |
263 om_count(),last->offset(),om_count()+1,map->offset()); | |
264 } | |
265 } | |
266 #endif // ASSERT | |
267 | |
268 set(om_count(),map); | |
269 increment_count(); | |
270 } | |
271 | |
272 | |
273 int OopMapSet::heap_size() const { | |
274 // The space we use | |
275 int size = sizeof(OopMap); | |
276 int align = sizeof(void *) - 1; | |
277 size = ((size+align) & ~align); | |
278 size += om_count() * sizeof(OopMap*); | |
279 | |
280 // Now add in the space needed for the indivdiual OopMaps | |
281 for(int i=0; i < om_count(); i++) { | |
282 size += at(i)->heap_size(); | |
283 } | |
284 // We don't need to align this, it will be naturally pointer aligned | |
285 return size; | |
286 } | |
287 | |
288 | |
289 OopMap* OopMapSet::singular_oop_map() { | |
290 guarantee(om_count() == 1, "Make sure we only have a single gc point"); | |
291 return at(0); | |
292 } | |
293 | |
294 | |
295 OopMap* OopMapSet::find_map_at_offset(int pc_offset) const { | |
296 int i, len = om_count(); | |
297 assert( len > 0, "must have pointer maps" ); | |
298 | |
299 // Scan through oopmaps. Stop when current offset is either equal or greater | |
300 // than the one we are looking for. | |
301 for( i = 0; i < len; i++) { | |
302 if( at(i)->offset() >= pc_offset ) | |
303 break; | |
304 } | |
305 | |
306 assert( i < len, "oopmap not found" ); | |
307 | |
308 OopMap* m = at(i); | |
309 assert( m->offset() == pc_offset, "oopmap not found" ); | |
310 return m; | |
311 } | |
312 | |
313 class DoNothingClosure: public OopClosure { | |
113
ba764ed4b6f2
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
100
diff
changeset
|
314 public: |
ba764ed4b6f2
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
100
diff
changeset
|
315 void do_oop(oop* p) {} |
ba764ed4b6f2
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
100
diff
changeset
|
316 void do_oop(narrowOop* p) {} |
0 | 317 }; |
318 static DoNothingClosure do_nothing; | |
319 | |
320 static void add_derived_oop(oop* base, oop* derived) { | |
23705
1051fbaca88b
the client VM build is broken when INCLUDE_JVMCI is defined (JDK-8159888)
Doug Simon <doug.simon@oracle.com>
parents:
23404
diff
changeset
|
321 #if !defined(TIERED) && !defined(INCLUDE_JVMCI) |
0 | 322 COMPILER1_PRESENT(ShouldNotReachHere();) |
23705
1051fbaca88b
the client VM build is broken when INCLUDE_JVMCI is defined (JDK-8159888)
Doug Simon <doug.simon@oracle.com>
parents:
23404
diff
changeset
|
323 #endif // !defined(TIERED) && !defined(INCLUDE_JVMCI) |
22298 | 324 #if defined(COMPILER2) || INCLUDE_JVMCI |
0 | 325 DerivedPointerTable::add(derived, base); |
22294
d86b226e331a
Enable derived oop processing by the GC with JVMCI
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
21559
diff
changeset
|
326 #endif // COMPILER2 || JVMCI |
0 | 327 } |
328 | |
329 | |
330 #ifndef PRODUCT | |
331 static void trace_codeblob_maps(const frame *fr, const RegisterMap *reg_map) { | |
332 // Print oopmap and regmap | |
333 tty->print_cr("------ "); | |
334 CodeBlob* cb = fr->cb(); | |
335 OopMapSet* maps = cb->oop_maps(); | |
336 OopMap* map = cb->oop_map_for_return_address(fr->pc()); | |
337 map->print(); | |
338 if( cb->is_nmethod() ) { | |
339 nmethod* nm = (nmethod*)cb; | |
340 // native wrappers have no scope data, it is implied | |
341 if (nm->is_native_method()) { | |
342 tty->print("bci: 0 (native)"); | |
343 } else { | |
344 ScopeDesc* scope = nm->scope_desc_at(fr->pc()); | |
345 tty->print("bci: %d ",scope->bci()); | |
346 } | |
347 } | |
348 tty->cr(); | |
349 fr->print_on(tty); | |
350 tty->print(" "); | |
351 cb->print_value_on(tty); tty->cr(); | |
352 reg_map->print(); | |
353 tty->print_cr("------ "); | |
354 | |
355 } | |
356 #endif // PRODUCT | |
357 | |
358 void OopMapSet::oops_do(const frame *fr, const RegisterMap* reg_map, OopClosure* f) { | |
359 // add derived oops to a table | |
113
ba764ed4b6f2
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
100
diff
changeset
|
360 all_do(fr, reg_map, f, add_derived_oop, &do_nothing); |
0 | 361 } |
362 | |
363 | |
364 void OopMapSet::all_do(const frame *fr, const RegisterMap *reg_map, | |
365 OopClosure* oop_fn, void derived_oop_fn(oop*, oop*), | |
113
ba764ed4b6f2
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
100
diff
changeset
|
366 OopClosure* value_fn) { |
0 | 367 CodeBlob* cb = fr->cb(); |
113
ba764ed4b6f2
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
100
diff
changeset
|
368 assert(cb != NULL, "no codeblob"); |
0 | 369 |
370 NOT_PRODUCT(if (TraceCodeBlobStacks) trace_codeblob_maps(fr, reg_map);) | |
371 | |
372 OopMapSet* maps = cb->oop_maps(); | |
113
ba764ed4b6f2
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
100
diff
changeset
|
373 OopMap* map = cb->oop_map_for_return_address(fr->pc()); |
ba764ed4b6f2
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
100
diff
changeset
|
374 assert(map != NULL, "no ptr map found"); |
0 | 375 |
376 // handle derived pointers first (otherwise base pointer may be | |
377 // changed before derived pointer offset has been collected) | |
378 OopMapValue omv; | |
379 { | |
380 OopMapStream oms(map,OopMapValue::derived_oop_value); | |
381 if (!oms.is_done()) { | |
382 #ifndef TIERED | |
383 COMPILER1_PRESENT(ShouldNotReachHere();) | |
384 #endif // !TIERED | |
385 // Protect the operation on the derived pointers. This | |
386 // protects the addition of derived pointers to the shared | |
387 // derived pointer table in DerivedPointerTable::add(). | |
388 MutexLockerEx x(DerivedPointerTableGC_lock, Mutex::_no_safepoint_check_flag); | |
389 do { | |
390 omv = oms.current(); | |
391 oop* loc = fr->oopmapreg_to_location(omv.reg(),reg_map); | |
20969
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
392 guarantee(loc != NULL, "missing saved register"); |
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
393 oop *base_loc = fr->oopmapreg_to_location(omv.content_reg(), reg_map); |
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
394 oop *derived_loc = loc; |
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
395 oop val = *base_loc; |
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
396 if (val == (oop)NULL || Universe::is_narrow_oop_base(val)) { |
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
397 // Ignore NULL oops and decoded NULL narrow oops which |
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
398 // equal to Universe::narrow_oop_base when a narrow oop |
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
399 // implicit null check is used in compiled code. |
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
400 // The narrow_oop_base could be NULL or be the address |
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
401 // of the page below heap depending on compressed oops mode. |
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
402 } else { |
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
403 derived_oop_fn(base_loc, derived_loc); |
0 | 404 } |
405 oms.next(); | |
406 } while (!oms.is_done()); | |
407 } | |
408 } | |
409 | |
20969
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
410 // We want narrowoop and oop oop_types |
14988
82ff6c8c8f6e
Increase maximum allocatable spill area by factor of 2.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
14124
diff
changeset
|
411 int mask = OopMapValue::oop_value | OopMapValue::narrowoop_value; |
0 | 412 { |
413 for (OopMapStream oms(map,mask); !oms.is_done(); oms.next()) { | |
414 omv = oms.current(); | |
415 oop* loc = fr->oopmapreg_to_location(omv.reg(),reg_map); | |
20969
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
416 // It should be an error if no location can be found for a |
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
417 // register mentioned as contained an oop of some kind. Maybe |
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
418 // this was allowed previously because value_value items might |
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
419 // be missing? |
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
420 guarantee(loc != NULL, "missing saved register"); |
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
421 if ( omv.type() == OopMapValue::oop_value ) { |
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
422 oop val = *loc; |
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
423 if (val == (oop)NULL || Universe::is_narrow_oop_base(val)) { |
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
424 // Ignore NULL oops and decoded NULL narrow oops which |
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
425 // equal to Universe::narrow_oop_base when a narrow oop |
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
426 // implicit null check is used in compiled code. |
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
427 // The narrow_oop_base could be NULL or be the address |
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
428 // of the page below heap depending on compressed oops mode. |
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
429 continue; |
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
430 } |
0 | 431 #ifdef ASSERT |
20969
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
432 if ((((uintptr_t)loc & (sizeof(*loc)-1)) != 0) || |
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
433 !Universe::heap()->is_in_or_null(*loc)) { |
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
434 tty->print_cr("# Found non oop pointer. Dumping state at failure"); |
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
435 // try to dump out some helpful debugging information |
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
436 trace_codeblob_maps(fr, reg_map); |
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
437 omv.print(); |
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
438 tty->print_cr("register r"); |
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
439 omv.reg()->print(); |
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
440 tty->print_cr("loc = %p *loc = %p\n", loc, (address)*loc); |
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
441 // do the real assert. |
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
442 assert(Universe::heap()->is_in_or_null(*loc), "found non oop pointer"); |
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
443 } |
0 | 444 #endif // ASSERT |
20969
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
445 oop_fn->do_oop(loc); |
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
446 } else if ( omv.type() == OopMapValue::narrowoop_value ) { |
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
447 narrowOop *nl = (narrowOop*)loc; |
113
ba764ed4b6f2
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
100
diff
changeset
|
448 #ifndef VM_LITTLE_ENDIAN |
22407
708a95bb443c
All SPARC float registers can be addressed in 4 byte portions; hence don't add 4 bytes to pointer pointing to a Narrow Oop
Stefan Anzinger <stefan.anzinger@oracle.com>
parents:
22298
diff
changeset
|
449 VMReg vmReg = omv.reg(); |
708a95bb443c
All SPARC float registers can be addressed in 4 byte portions; hence don't add 4 bytes to pointer pointing to a Narrow Oop
Stefan Anzinger <stefan.anzinger@oracle.com>
parents:
22298
diff
changeset
|
450 // Don't do this on SPARC float registers as they can be individually addressed |
708a95bb443c
All SPARC float registers can be addressed in 4 byte portions; hence don't add 4 bytes to pointer pointing to a Narrow Oop
Stefan Anzinger <stefan.anzinger@oracle.com>
parents:
22298
diff
changeset
|
451 if (!vmReg->is_stack() SPARC_ONLY(&& !vmReg->is_FloatRegister())) { |
20969
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
452 // compressed oops in registers only take up 4 bytes of an |
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
453 // 8 byte register but they are in the wrong part of the |
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
454 // word so adjust loc to point at the right place. |
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
455 nl = (narrowOop*)((address)nl + 4); |
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
456 } |
113
ba764ed4b6f2
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
100
diff
changeset
|
457 #endif |
20969
534f0dde2810
It should be an error for an OopMap to mention a register that can't be found
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
20804
diff
changeset
|
458 oop_fn->do_oop(nl); |
0 | 459 } |
460 } | |
461 } | |
462 } | |
463 | |
464 | |
465 // Update callee-saved register info for the following frame | |
466 void OopMapSet::update_register_map(const frame *fr, RegisterMap *reg_map) { | |
467 ResourceMark rm; | |
468 CodeBlob* cb = fr->cb(); | |
469 assert(cb != NULL, "no codeblob"); | |
470 | |
471 // Any reg might be saved by a safepoint handler (see generate_handler_blob). | |
472 assert( reg_map->_update_for_id == NULL || fr->is_older(reg_map->_update_for_id), | |
473 "already updated this map; do not 'update' it twice!" ); | |
474 debug_only(reg_map->_update_for_id = fr->id()); | |
475 | |
476 // Check if caller must update oop argument | |
477 assert((reg_map->include_argument_oops() || | |
478 !cb->caller_must_gc_arguments(reg_map->thread())), | |
479 "include_argument_oops should already be set"); | |
480 | |
481 // Scan through oopmap and find location of all callee-saved registers | |
482 // (we do not do update in place, since info could be overwritten) | |
483 | |
484 address pc = fr->pc(); | |
485 OopMap* map = cb->oop_map_for_return_address(pc); | |
20226
6ad19ab94176
8050229: Uninitialised memory in hotspot/src/share/vm/compiler/oopMap.cpp
morris
parents:
17937
diff
changeset
|
486 assert(map != NULL, "no ptr map found"); |
6ad19ab94176
8050229: Uninitialised memory in hotspot/src/share/vm/compiler/oopMap.cpp
morris
parents:
17937
diff
changeset
|
487 DEBUG_ONLY(int nof_callee = 0;) |
0 | 488 |
20226
6ad19ab94176
8050229: Uninitialised memory in hotspot/src/share/vm/compiler/oopMap.cpp
morris
parents:
17937
diff
changeset
|
489 for (OopMapStream oms(map, OopMapValue::callee_saved_value); !oms.is_done(); oms.next()) { |
6ad19ab94176
8050229: Uninitialised memory in hotspot/src/share/vm/compiler/oopMap.cpp
morris
parents:
17937
diff
changeset
|
490 OopMapValue omv = oms.current(); |
6ad19ab94176
8050229: Uninitialised memory in hotspot/src/share/vm/compiler/oopMap.cpp
morris
parents:
17937
diff
changeset
|
491 VMReg reg = omv.content_reg(); |
6ad19ab94176
8050229: Uninitialised memory in hotspot/src/share/vm/compiler/oopMap.cpp
morris
parents:
17937
diff
changeset
|
492 oop* loc = fr->oopmapreg_to_location(omv.reg(), reg_map); |
6ad19ab94176
8050229: Uninitialised memory in hotspot/src/share/vm/compiler/oopMap.cpp
morris
parents:
17937
diff
changeset
|
493 reg_map->set_location(reg, (address) loc); |
6ad19ab94176
8050229: Uninitialised memory in hotspot/src/share/vm/compiler/oopMap.cpp
morris
parents:
17937
diff
changeset
|
494 DEBUG_ONLY(nof_callee++;) |
0 | 495 } |
496 | |
497 // Check that runtime stubs save all callee-saved registers | |
498 #ifdef COMPILER2 | |
21559
be896a1983c0
recast all Graal native code as JVMCI code (JBS:GRAAL-53)
Doug Simon <doug.simon@oracle.com>
parents:
20969
diff
changeset
|
499 assert(cb->is_compiled_by_c1() || cb->is_compiled_by_jvmci() || !cb->is_runtime_stub() || |
0 | 500 (nof_callee >= SAVED_ON_ENTRY_REG_COUNT || nof_callee >= C_SAVED_ON_ENTRY_REG_COUNT), |
501 "must save all"); | |
502 #endif // COMPILER2 | |
503 } | |
504 | |
505 //============================================================================= | |
506 // Non-Product code | |
507 | |
508 #ifndef PRODUCT | |
509 | |
510 bool OopMap::has_derived_pointer() const { | |
23705
1051fbaca88b
the client VM build is broken when INCLUDE_JVMCI is defined (JDK-8159888)
Doug Simon <doug.simon@oracle.com>
parents:
23404
diff
changeset
|
511 #if !defined(TIERED) && !defined(INCLUDE_JVMCI) |
0 | 512 COMPILER1_PRESENT(return false); |
513 #endif // !TIERED | |
22298 | 514 #if defined(COMPILER2) || INCLUDE_JVMCI |
0 | 515 OopMapStream oms((OopMap*)this,OopMapValue::derived_oop_value); |
516 return oms.is_done(); | |
517 #else | |
518 return false; | |
22294
d86b226e331a
Enable derived oop processing by the GC with JVMCI
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
21559
diff
changeset
|
519 #endif // COMPILER2 || JVMCI |
0 | 520 } |
521 | |
100
c7c777385a15
6667042: PrintAssembly option does not work without special plugin
jrose
parents:
25
diff
changeset
|
522 #endif //PRODUCT |
0 | 523 |
100
c7c777385a15
6667042: PrintAssembly option does not work without special plugin
jrose
parents:
25
diff
changeset
|
524 // Printing code is present in product build for -XX:+PrintAssembly. |
c7c777385a15
6667042: PrintAssembly option does not work without special plugin
jrose
parents:
25
diff
changeset
|
525 |
c7c777385a15
6667042: PrintAssembly option does not work without special plugin
jrose
parents:
25
diff
changeset
|
526 static |
c7c777385a15
6667042: PrintAssembly option does not work without special plugin
jrose
parents:
25
diff
changeset
|
527 void print_register_type(OopMapValue::oop_types x, VMReg optional, |
c7c777385a15
6667042: PrintAssembly option does not work without special plugin
jrose
parents:
25
diff
changeset
|
528 outputStream* st) { |
0 | 529 switch( x ) { |
530 case OopMapValue::oop_value: | |
25
c5cbd367e4d1
6621094: PrintOptoAssembly is broken for oops information in DebugInfo
kvn
parents:
0
diff
changeset
|
531 st->print("Oop"); |
0 | 532 break; |
113
ba764ed4b6f2
6420645: Create a vm that uses compressed oops for up to 32gb heapsizes
coleenp
parents:
100
diff
changeset
|
533 case OopMapValue::narrowoop_value: |
7636
a7114d3d712e
8005055: pass outputStream to more opto debug routines
kvn
parents:
6842
diff
changeset
|
534 st->print("NarrowOop"); |
0 | 535 break; |
536 case OopMapValue::callee_saved_value: | |
7636
a7114d3d712e
8005055: pass outputStream to more opto debug routines
kvn
parents:
6842
diff
changeset
|
537 st->print("Callers_"); |
25
c5cbd367e4d1
6621094: PrintOptoAssembly is broken for oops information in DebugInfo
kvn
parents:
0
diff
changeset
|
538 optional->print_on(st); |
0 | 539 break; |
540 case OopMapValue::derived_oop_value: | |
7636
a7114d3d712e
8005055: pass outputStream to more opto debug routines
kvn
parents:
6842
diff
changeset
|
541 st->print("Derived_oop_"); |
25
c5cbd367e4d1
6621094: PrintOptoAssembly is broken for oops information in DebugInfo
kvn
parents:
0
diff
changeset
|
542 optional->print_on(st); |
0 | 543 break; |
544 default: | |
545 ShouldNotReachHere(); | |
546 } | |
547 } | |
548 | |
549 | |
25
c5cbd367e4d1
6621094: PrintOptoAssembly is broken for oops information in DebugInfo
kvn
parents:
0
diff
changeset
|
550 void OopMapValue::print_on(outputStream* st) const { |
c5cbd367e4d1
6621094: PrintOptoAssembly is broken for oops information in DebugInfo
kvn
parents:
0
diff
changeset
|
551 reg()->print_on(st); |
c5cbd367e4d1
6621094: PrintOptoAssembly is broken for oops information in DebugInfo
kvn
parents:
0
diff
changeset
|
552 st->print("="); |
c5cbd367e4d1
6621094: PrintOptoAssembly is broken for oops information in DebugInfo
kvn
parents:
0
diff
changeset
|
553 print_register_type(type(),content_reg(),st); |
c5cbd367e4d1
6621094: PrintOptoAssembly is broken for oops information in DebugInfo
kvn
parents:
0
diff
changeset
|
554 st->print(" "); |
0 | 555 } |
556 | |
557 | |
558 void OopMap::print_on(outputStream* st) const { | |
559 OopMapValue omv; | |
100
c7c777385a15
6667042: PrintAssembly option does not work without special plugin
jrose
parents:
25
diff
changeset
|
560 st->print("OopMap{"); |
0 | 561 for(OopMapStream oms((OopMap*)this); !oms.is_done(); oms.next()) { |
562 omv = oms.current(); | |
563 omv.print_on(st); | |
564 } | |
100
c7c777385a15
6667042: PrintAssembly option does not work without special plugin
jrose
parents:
25
diff
changeset
|
565 st->print("off=%d}", (int) offset()); |
0 | 566 } |
567 | |
568 | |
569 void OopMapSet::print_on(outputStream* st) const { | |
570 int i, len = om_count(); | |
571 | |
572 st->print_cr("OopMapSet contains %d OopMaps\n",len); | |
573 | |
574 for( i = 0; i < len; i++) { | |
575 OopMap* m = at(i); | |
100
c7c777385a15
6667042: PrintAssembly option does not work without special plugin
jrose
parents:
25
diff
changeset
|
576 st->print_cr("#%d ",i); |
0 | 577 m->print_on(st); |
100
c7c777385a15
6667042: PrintAssembly option does not work without special plugin
jrose
parents:
25
diff
changeset
|
578 st->cr(); |
0 | 579 } |
580 } | |
100
c7c777385a15
6667042: PrintAssembly option does not work without special plugin
jrose
parents:
25
diff
changeset
|
581 |
0 | 582 |
583 | |
584 //------------------------------DerivedPointerTable--------------------------- | |
585 | |
22298 | 586 #if defined(COMPILER2) || INCLUDE_JVMCI |
0 | 587 |
6197 | 588 class DerivedPointerEntry : public CHeapObj<mtCompiler> { |
0 | 589 private: |
590 oop* _location; // Location of derived pointer (also pointing to the base) | |
591 intptr_t _offset; // Offset from base pointer | |
592 public: | |
593 DerivedPointerEntry(oop* location, intptr_t offset) { _location = location; _offset = offset; } | |
594 oop* location() { return _location; } | |
595 intptr_t offset() { return _offset; } | |
596 }; | |
597 | |
598 | |
599 GrowableArray<DerivedPointerEntry*>* DerivedPointerTable::_list = NULL; | |
600 bool DerivedPointerTable::_active = false; | |
601 | |
602 | |
603 void DerivedPointerTable::clear() { | |
604 // The first time, we create the list. Otherwise it should be | |
605 // empty. If not, then we have probably forgotton to call | |
606 // update_pointers after last GC/Scavenge. | |
607 assert (!_active, "should not be active"); | |
608 assert(_list == NULL || _list->length() == 0, "table not empty"); | |
609 if (_list == NULL) { | |
6197 | 610 _list = new (ResourceObj::C_HEAP, mtCompiler) GrowableArray<DerivedPointerEntry*>(10, true); // Allocated on C heap |
0 | 611 } |
612 _active = true; | |
613 } | |
614 | |
615 | |
616 // Returns value of location as an int | |
12316
190899198332
7195622: CheckUnhandledOops has limited usefulness now
hseigel
parents:
7636
diff
changeset
|
617 intptr_t value_of_loc(oop *pointer) { return cast_from_oop<intptr_t>((*pointer)); } |
0 | 618 |
619 | |
620 void DerivedPointerTable::add(oop *derived_loc, oop *base_loc) { | |
621 assert(Universe::heap()->is_in_or_null(*base_loc), "not an oop"); | |
622 assert(derived_loc != base_loc, "Base and derived in same location"); | |
623 if (_active) { | |
624 assert(*derived_loc != (oop)base_loc, "location already added"); | |
625 assert(_list != NULL, "list must exist"); | |
626 intptr_t offset = value_of_loc(derived_loc) - value_of_loc(base_loc); | |
3442
f7d55ea6ee56
7045514: SPARC assembly code for JSR 292 ricochet frames
never
parents:
1972
diff
changeset
|
627 // This assert is invalid because derived pointers can be |
f7d55ea6ee56
7045514: SPARC assembly code for JSR 292 ricochet frames
never
parents:
1972
diff
changeset
|
628 // arbitrarily far away from their base. |
f7d55ea6ee56
7045514: SPARC assembly code for JSR 292 ricochet frames
never
parents:
1972
diff
changeset
|
629 // assert(offset >= -1000000, "wrong derived pointer info"); |
0 | 630 |
631 if (TraceDerivedPointers) { | |
632 tty->print_cr( | |
633 "Add derived pointer@" INTPTR_FORMAT | |
634 " - Derived: " INTPTR_FORMAT | |
17937
78bbf4d43a14
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
12316
diff
changeset
|
635 " Base: " INTPTR_FORMAT " (@" INTPTR_FORMAT ") (Offset: " INTX_FORMAT ")", |
78bbf4d43a14
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
12316
diff
changeset
|
636 p2i(derived_loc), p2i((address)*derived_loc), p2i((address)*base_loc), p2i(base_loc), offset |
0 | 637 ); |
638 } | |
639 // Set derived oop location to point to base. | |
640 *derived_loc = (oop)base_loc; | |
641 assert_lock_strong(DerivedPointerTableGC_lock); | |
642 DerivedPointerEntry *entry = new DerivedPointerEntry(derived_loc, offset); | |
643 _list->append(entry); | |
644 } | |
645 } | |
646 | |
647 | |
648 void DerivedPointerTable::update_pointers() { | |
649 assert(_list != NULL, "list must exist"); | |
650 for(int i = 0; i < _list->length(); i++) { | |
651 DerivedPointerEntry* entry = _list->at(i); | |
652 oop* derived_loc = entry->location(); | |
653 intptr_t offset = entry->offset(); | |
654 // The derived oop was setup to point to location of base | |
655 oop base = **(oop**)derived_loc; | |
656 assert(Universe::heap()->is_in_or_null(base), "must be an oop"); | |
657 | |
658 *derived_loc = (oop)(((address)base) + offset); | |
659 assert(value_of_loc(derived_loc) - value_of_loc(&base) == offset, "sanity check"); | |
660 | |
661 if (TraceDerivedPointers) { | |
662 tty->print_cr("Updating derived pointer@" INTPTR_FORMAT | |
17937
78bbf4d43a14
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
12316
diff
changeset
|
663 " - Derived: " INTPTR_FORMAT " Base: " INTPTR_FORMAT " (Offset: " INTX_FORMAT ")", |
78bbf4d43a14
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
12316
diff
changeset
|
664 p2i(derived_loc), p2i((address)*derived_loc), p2i((address)base), offset); |
0 | 665 } |
666 | |
667 // Delete entry | |
668 delete entry; | |
669 _list->at_put(i, NULL); | |
670 } | |
671 // Clear list, so it is ready for next traversal (this is an invariant) | |
672 if (TraceDerivedPointers && !_list->is_empty()) { | |
673 tty->print_cr("--------------------------"); | |
674 } | |
675 _list->clear(); | |
676 _active = false; | |
677 } | |
678 | |
22294
d86b226e331a
Enable derived oop processing by the GC with JVMCI
Tom Rodriguez <tom.rodriguez@oracle.com>
parents:
21559
diff
changeset
|
679 #endif // COMPILER2 || JVMCI |