Path: blob/aarch64-shenandoah-jdk8u272-b10/hotspot/src/cpu/x86/vm/frame_x86.cpp
32285 views
/*1* Copyright (c) 1997, 2014, Oracle and/or its affiliates. All rights reserved.2* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.3*4* This code is free software; you can redistribute it and/or modify it5* under the terms of the GNU General Public License version 2 only, as6* published by the Free Software Foundation.7*8* This code is distributed in the hope that it will be useful, but WITHOUT9* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or10* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License11* version 2 for more details (a copy is included in the LICENSE file that12* accompanied this code).13*14* You should have received a copy of the GNU General Public License version15* 2 along with this work; if not, write to the Free Software Foundation,16* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.17*18* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA19* or visit www.oracle.com if you need additional information or have any20* questions.21*22*/2324#include "precompiled.hpp"25#include "interpreter/interpreter.hpp"26#include "memory/resourceArea.hpp"27#include "oops/markOop.hpp"28#include "oops/method.hpp"29#include "oops/oop.inline.hpp"30#include "prims/methodHandles.hpp"31#include "runtime/frame.inline.hpp"32#include "runtime/handles.inline.hpp"33#include "runtime/javaCalls.hpp"34#include "runtime/monitorChunk.hpp"35#include "runtime/os.hpp"36#include "runtime/signature.hpp"37#include "runtime/stubCodeGenerator.hpp"38#include "runtime/stubRoutines.hpp"39#include "vmreg_x86.inline.hpp"40#ifdef COMPILER141#include "c1/c1_Runtime1.hpp"42#include "runtime/vframeArray.hpp"43#endif4445#ifdef ASSERT46void RegisterMap::check_location_valid() {47}48#endif4950PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC5152// Profiling/safepoint support5354bool frame::safe_for_sender(JavaThread *thread) {55address sp = (address)_sp;56address fp = (address)_fp;57address unextended_sp = (address)_unextended_sp;5859// consider stack guards when trying to determine "safe" stack pointers60static size_t stack_guard_size = os::uses_stack_guard_pages() ? (StackYellowPages + StackRedPages) * os::vm_page_size() : 0;61size_t usable_stack_size = thread->stack_size() - stack_guard_size;6263// sp must be within the usable part of the stack (not in guards)64bool sp_safe = (sp < thread->stack_base()) &&65(sp >= thread->stack_base() - usable_stack_size);666768if (!sp_safe) {69return false;70}7172// unextended sp must be within the stack and above or equal sp73bool unextended_sp_safe = (unextended_sp < thread->stack_base()) &&74(unextended_sp >= sp);7576if (!unextended_sp_safe) {77return false;78}7980// an fp must be within the stack and above (but not equal) sp81// second evaluation on fp+ is added to handle situation where fp is -182bool fp_safe = (fp < thread->stack_base() && (fp > sp) && (((fp + (return_addr_offset * sizeof(void*))) < thread->stack_base())));8384// We know sp/unextended_sp are safe only fp is questionable here8586// If the current frame is known to the code cache then we can attempt to87// to construct the sender and do some validation of it. This goes a long way88// toward eliminating issues when we get in frame construction code8990if (_cb != NULL ) {9192// First check if frame is complete and tester is reliable93// Unfortunately we can only check frame complete for runtime stubs and nmethod94// other generic buffer blobs are more problematic so we just assume they are95// ok. adapter blobs never have a frame complete and are never ok.9697if (!_cb->is_frame_complete_at(_pc)) {98if (_cb->is_nmethod() || _cb->is_adapter_blob() || _cb->is_runtime_stub()) {99return false;100}101}102103// Could just be some random pointer within the codeBlob104if (!_cb->code_contains(_pc)) {105return false;106}107108// Entry frame checks109if (is_entry_frame()) {110// an entry frame must have a valid fp.111112if (!fp_safe) return false;113114// Validate the JavaCallWrapper an entry frame must have115116address jcw = (address)entry_frame_call_wrapper();117118bool jcw_safe = (jcw < thread->stack_base()) && ( jcw > fp);119120return jcw_safe;121122}123124intptr_t* sender_sp = NULL;125address sender_pc = NULL;126127if (is_interpreted_frame()) {128// fp must be safe129if (!fp_safe) {130return false;131}132133sender_pc = (address) this->fp()[return_addr_offset];134sender_sp = (intptr_t*) addr_at(sender_sp_offset);135136} else {137// must be some sort of compiled/runtime frame138// fp does not have to be safe (although it could be check for c1?)139140// check for a valid frame_size, otherwise we are unlikely to get a valid sender_pc141if (_cb->frame_size() <= 0) {142return false;143}144145sender_sp = _unextended_sp + _cb->frame_size();146// On Intel the return_address is always the word on the stack147sender_pc = (address) *(sender_sp-1);148}149150151// If the potential sender is the interpreter then we can do some more checking152if (Interpreter::contains(sender_pc)) {153154// ebp is always saved in a recognizable place in any code we generate. However155// only if the sender is interpreted/call_stub (c1 too?) are we certain that the saved ebp156// is really a frame pointer.157158intptr_t *saved_fp = (intptr_t*)*(sender_sp - frame::sender_sp_offset);159bool saved_fp_safe = ((address)saved_fp < thread->stack_base()) && (saved_fp > sender_sp);160161if (!saved_fp_safe) {162return false;163}164165// construct the potential sender166167frame sender(sender_sp, saved_fp, sender_pc);168169return sender.is_interpreted_frame_valid(thread);170171}172173// We must always be able to find a recognizable pc174CodeBlob* sender_blob = CodeCache::find_blob_unsafe(sender_pc);175if (sender_pc == NULL || sender_blob == NULL) {176return false;177}178179// Could be a zombie method180if (sender_blob->is_zombie() || sender_blob->is_unloaded()) {181return false;182}183184// Could just be some random pointer within the codeBlob185if (!sender_blob->code_contains(sender_pc)) {186return false;187}188189// We should never be able to see an adapter if the current frame is something from code cache190if (sender_blob->is_adapter_blob()) {191return false;192}193194// Could be the call_stub195if (StubRoutines::returns_to_call_stub(sender_pc)) {196intptr_t *saved_fp = (intptr_t*)*(sender_sp - frame::sender_sp_offset);197bool saved_fp_safe = ((address)saved_fp < thread->stack_base()) && (saved_fp > sender_sp);198199if (!saved_fp_safe) {200return false;201}202203// construct the potential sender204205frame sender(sender_sp, saved_fp, sender_pc);206207// Validate the JavaCallWrapper an entry frame must have208address jcw = (address)sender.entry_frame_call_wrapper();209210bool jcw_safe = (jcw < thread->stack_base()) && ( jcw > (address)sender.fp());211212return jcw_safe;213}214215if (sender_blob->is_nmethod()) {216nmethod* nm = sender_blob->as_nmethod_or_null();217if (nm != NULL) {218if (nm->is_deopt_mh_entry(sender_pc) || nm->is_deopt_entry(sender_pc) ||219nm->method()->is_method_handle_intrinsic()) {220return false;221}222}223}224225// If the frame size is 0 something (or less) is bad because every nmethod has a non-zero frame size226// because the return address counts against the callee's frame.227228if (sender_blob->frame_size() <= 0) {229assert(!sender_blob->is_nmethod(), "should count return address at least");230return false;231}232233// We should never be able to see anything here except an nmethod. If something in the234// code cache (current frame) is called by an entity within the code cache that entity235// should not be anything but the call stub (already covered), the interpreter (already covered)236// or an nmethod.237238if (!sender_blob->is_nmethod()) {239return false;240}241242// Could put some more validation for the potential non-interpreted sender243// frame we'd create by calling sender if I could think of any. Wait for next crash in forte...244245// One idea is seeing if the sender_pc we have is one that we'd expect to call to current cb246247// We've validated the potential sender that would be created248return true;249}250251// Must be native-compiled frame. Since sender will try and use fp to find252// linkages it must be safe253254if (!fp_safe) {255return false;256}257258// Will the pc we fetch be non-zero (which we'll find at the oldest frame)259260if ( (address) this->fp()[return_addr_offset] == NULL) return false;261262263// could try and do some more potential verification of native frame if we could think of some...264265return true;266267}268269270void frame::patch_pc(Thread* thread, address pc) {271address* pc_addr = &(((address*) sp())[-1]);272if (TracePcPatching) {273tty->print_cr("patch_pc at address " INTPTR_FORMAT " [" INTPTR_FORMAT " -> " INTPTR_FORMAT "]",274pc_addr, *pc_addr, pc);275}276// Either the return address is the original one or we are going to277// patch in the same address that's already there.278assert(_pc == *pc_addr || pc == *pc_addr, "must be");279*pc_addr = pc;280_cb = CodeCache::find_blob(pc);281address original_pc = nmethod::get_deopt_original_pc(this);282if (original_pc != NULL) {283assert(original_pc == _pc, "expected original PC to be stored before patching");284_deopt_state = is_deoptimized;285// leave _pc as is286} else {287_deopt_state = not_deoptimized;288_pc = pc;289}290}291292bool frame::is_interpreted_frame() const {293return Interpreter::contains(pc());294}295296int frame::frame_size(RegisterMap* map) const {297frame sender = this->sender(map);298return sender.sp() - sp();299}300301intptr_t* frame::entry_frame_argument_at(int offset) const {302// convert offset to index to deal with tsi303int index = (Interpreter::expr_offset_in_bytes(offset)/wordSize);304// Entry frame's arguments are always in relation to unextended_sp()305return &unextended_sp()[index];306}307308// sender_sp309#ifdef CC_INTERP310intptr_t* frame::interpreter_frame_sender_sp() const {311assert(is_interpreted_frame(), "interpreted frame expected");312// QQQ why does this specialize method exist if frame::sender_sp() does same thing?313// seems odd and if we always know interpreted vs. non then sender_sp() is really314// doing too much work.315return get_interpreterState()->sender_sp();316}317318// monitor elements319320BasicObjectLock* frame::interpreter_frame_monitor_begin() const {321return get_interpreterState()->monitor_base();322}323324BasicObjectLock* frame::interpreter_frame_monitor_end() const {325return (BasicObjectLock*) get_interpreterState()->stack_base();326}327328#else // CC_INTERP329330intptr_t* frame::interpreter_frame_sender_sp() const {331assert(is_interpreted_frame(), "interpreted frame expected");332return (intptr_t*) at(interpreter_frame_sender_sp_offset);333}334335void frame::set_interpreter_frame_sender_sp(intptr_t* sender_sp) {336assert(is_interpreted_frame(), "interpreted frame expected");337ptr_at_put(interpreter_frame_sender_sp_offset, (intptr_t) sender_sp);338}339340341// monitor elements342343BasicObjectLock* frame::interpreter_frame_monitor_begin() const {344return (BasicObjectLock*) addr_at(interpreter_frame_monitor_block_bottom_offset);345}346347BasicObjectLock* frame::interpreter_frame_monitor_end() const {348BasicObjectLock* result = (BasicObjectLock*) *addr_at(interpreter_frame_monitor_block_top_offset);349// make sure the pointer points inside the frame350assert(sp() <= (intptr_t*) result, "monitor end should be above the stack pointer");351assert((intptr_t*) result < fp(), "monitor end should be strictly below the frame pointer");352return result;353}354355void frame::interpreter_frame_set_monitor_end(BasicObjectLock* value) {356*((BasicObjectLock**)addr_at(interpreter_frame_monitor_block_top_offset)) = value;357}358359// Used by template based interpreter deoptimization360void frame::interpreter_frame_set_last_sp(intptr_t* sp) {361*((intptr_t**)addr_at(interpreter_frame_last_sp_offset)) = sp;362}363#endif // CC_INTERP364365frame frame::sender_for_entry_frame(RegisterMap* map) const {366assert(map != NULL, "map must be set");367// Java frame called from C; skip all C frames and return top C368// frame of that chunk as the sender369JavaFrameAnchor* jfa = entry_frame_call_wrapper()->anchor();370assert(!entry_frame_is_first(), "next Java fp must be non zero");371assert(jfa->last_Java_sp() > sp(), "must be above this frame on stack");372// Since we are walking the stack now this nested anchor is obviously walkable373// even if it wasn't when it was stacked.374if (!jfa->walkable()) {375// Capture _last_Java_pc (if needed) and mark anchor walkable.376jfa->capture_last_Java_pc();377}378map->clear();379assert(map->include_argument_oops(), "should be set by clear");380assert(jfa->last_Java_pc() != NULL, "not walkable");381frame fr(jfa->last_Java_sp(), jfa->last_Java_fp(), jfa->last_Java_pc());382return fr;383}384385//------------------------------------------------------------------------------386// frame::verify_deopt_original_pc387//388// Verifies the calculated original PC of a deoptimization PC for the389// given unextended SP.390#ifdef ASSERT391void frame::verify_deopt_original_pc(nmethod* nm, intptr_t* unextended_sp) {392frame fr;393394// This is ugly but it's better than to change {get,set}_original_pc395// to take an SP value as argument. And it's only a debugging396// method anyway.397fr._unextended_sp = unextended_sp;398399address original_pc = nm->get_original_pc(&fr);400assert(nm->insts_contains(original_pc), "original PC must be in nmethod");401}402#endif403404//------------------------------------------------------------------------------405// frame::adjust_unextended_sp406void frame::adjust_unextended_sp() {407// On x86, sites calling method handle intrinsics and lambda forms are treated408// as any other call site. Therefore, no special action is needed when we are409// returning to any of these call sites.410411nmethod* sender_nm = (_cb == NULL) ? NULL : _cb->as_nmethod_or_null();412if (sender_nm != NULL) {413// If the sender PC is a deoptimization point, get the original PC.414if (sender_nm->is_deopt_entry(_pc) ||415sender_nm->is_deopt_mh_entry(_pc)) {416DEBUG_ONLY(verify_deopt_original_pc(sender_nm, _unextended_sp));417}418}419}420421//------------------------------------------------------------------------------422// frame::update_map_with_saved_link423void frame::update_map_with_saved_link(RegisterMap* map, intptr_t** link_addr) {424// The interpreter and compiler(s) always save EBP/RBP in a known425// location on entry. We must record where that location is426// so this if EBP/RBP was live on callout from c2 we can find427// the saved copy no matter what it called.428429// Since the interpreter always saves EBP/RBP if we record where it is then430// we don't have to always save EBP/RBP on entry and exit to c2 compiled431// code, on entry will be enough.432map->set_location(rbp->as_VMReg(), (address) link_addr);433#ifdef AMD64434// this is weird "H" ought to be at a higher address however the435// oopMaps seems to have the "H" regs at the same address and the436// vanilla register.437// XXXX make this go away438if (true) {439map->set_location(rbp->as_VMReg()->next(), (address) link_addr);440}441#endif // AMD64442}443444445//------------------------------------------------------------------------------446// frame::sender_for_interpreter_frame447frame frame::sender_for_interpreter_frame(RegisterMap* map) const {448// SP is the raw SP from the sender after adapter or interpreter449// extension.450intptr_t* sender_sp = this->sender_sp();451452// This is the sp before any possible extension (adapter/locals).453intptr_t* unextended_sp = interpreter_frame_sender_sp();454455#ifdef COMPILER2456if (map->update_map()) {457update_map_with_saved_link(map, (intptr_t**) addr_at(link_offset));458}459#endif // COMPILER2460461return frame(sender_sp, unextended_sp, link(), sender_pc());462}463464465//------------------------------------------------------------------------------466// frame::sender_for_compiled_frame467frame frame::sender_for_compiled_frame(RegisterMap* map) const {468assert(map != NULL, "map must be set");469470// frame owned by optimizing compiler471assert(_cb->frame_size() >= 0, "must have non-zero frame size");472intptr_t* sender_sp = unextended_sp() + _cb->frame_size();473intptr_t* unextended_sp = sender_sp;474475// On Intel the return_address is always the word on the stack476address sender_pc = (address) *(sender_sp-1);477478// This is the saved value of EBP which may or may not really be an FP.479// It is only an FP if the sender is an interpreter frame (or C1?).480intptr_t** saved_fp_addr = (intptr_t**) (sender_sp - frame::sender_sp_offset);481482if (map->update_map()) {483// Tell GC to use argument oopmaps for some runtime stubs that need it.484// For C1, the runtime stub might not have oop maps, so set this flag485// outside of update_register_map.486map->set_include_argument_oops(_cb->caller_must_gc_arguments(map->thread()));487if (_cb->oop_maps() != NULL) {488OopMapSet::update_register_map(this, map);489}490491// Since the prolog does the save and restore of EBP there is no oopmap492// for it so we must fill in its location as if there was an oopmap entry493// since if our caller was compiled code there could be live jvm state in it.494update_map_with_saved_link(map, saved_fp_addr);495}496497assert(sender_sp != sp(), "must have changed");498return frame(sender_sp, unextended_sp, *saved_fp_addr, sender_pc);499}500501502//------------------------------------------------------------------------------503// frame::sender504frame frame::sender(RegisterMap* map) const {505// Default is we done have to follow them. The sender_for_xxx will506// update it accordingly507map->set_include_argument_oops(false);508509if (is_entry_frame()) return sender_for_entry_frame(map);510if (is_interpreted_frame()) return sender_for_interpreter_frame(map);511assert(_cb == CodeCache::find_blob(pc()),"Must be the same");512513if (_cb != NULL) {514return sender_for_compiled_frame(map);515}516// Must be native-compiled frame, i.e. the marshaling code for native517// methods that exists in the core system.518return frame(sender_sp(), link(), sender_pc());519}520521522bool frame::interpreter_frame_equals_unpacked_fp(intptr_t* fp) {523assert(is_interpreted_frame(), "must be interpreter frame");524Method* method = interpreter_frame_method();525// When unpacking an optimized frame the frame pointer is526// adjusted with:527int diff = (method->max_locals() - method->size_of_parameters()) *528Interpreter::stackElementWords;529return _fp == (fp - diff);530}531532void frame::pd_gc_epilog() {533// nothing done here now534}535536bool frame::is_interpreted_frame_valid(JavaThread* thread) const {537// QQQ538#ifdef CC_INTERP539#else540assert(is_interpreted_frame(), "Not an interpreted frame");541// These are reasonable sanity checks542if (fp() == 0 || (intptr_t(fp()) & (wordSize-1)) != 0) {543return false;544}545if (sp() == 0 || (intptr_t(sp()) & (wordSize-1)) != 0) {546return false;547}548if (fp() + interpreter_frame_initial_sp_offset < sp()) {549return false;550}551// These are hacks to keep us out of trouble.552// The problem with these is that they mask other problems553if (fp() <= sp()) { // this attempts to deal with unsigned comparison above554return false;555}556557// do some validation of frame elements558559// first the method560561Method* m = *interpreter_frame_method_addr();562563// validate the method we'd find in this potential sender564if (!m->is_valid_method()) return false;565566// stack frames shouldn't be much larger than max_stack elements567568if (fp() - sp() > 1024 + m->max_stack()*Interpreter::stackElementSize) {569return false;570}571572// validate bci/bcx573574intptr_t bcx = interpreter_frame_bcx();575if (m->validate_bci_from_bcx(bcx) < 0) {576return false;577}578579// validate ConstantPoolCache*580ConstantPoolCache* cp = *interpreter_frame_cache_addr();581if (cp == NULL || !cp->is_metaspace_object()) return false;582583// validate locals584585address locals = (address) *interpreter_frame_locals_addr();586587if (locals > thread->stack_base() || locals < (address) fp()) return false;588589// We'd have to be pretty unlucky to be mislead at this point590591#endif // CC_INTERP592return true;593}594595BasicType frame::interpreter_frame_result(oop* oop_result, jvalue* value_result) {596#ifdef CC_INTERP597// Needed for JVMTI. The result should always be in the598// interpreterState object599interpreterState istate = get_interpreterState();600#endif // CC_INTERP601assert(is_interpreted_frame(), "interpreted frame expected");602Method* method = interpreter_frame_method();603BasicType type = method->result_type();604605intptr_t* tos_addr;606if (method->is_native()) {607// Prior to calling into the runtime to report the method_exit the possible608// return value is pushed to the native stack. If the result is a jfloat/jdouble609// then ST0 is saved before EAX/EDX. See the note in generate_native_result610tos_addr = (intptr_t*)sp();611if (type == T_FLOAT || type == T_DOUBLE) {612// QQQ seems like this code is equivalent on the two platforms613#ifdef AMD64614// This is times two because we do a push(ltos) after pushing XMM0615// and that takes two interpreter stack slots.616tos_addr += 2 * Interpreter::stackElementWords;617#else618tos_addr += 2;619#endif // AMD64620}621} else {622tos_addr = (intptr_t*)interpreter_frame_tos_address();623}624625switch (type) {626case T_OBJECT :627case T_ARRAY : {628oop obj;629if (method->is_native()) {630#ifdef CC_INTERP631obj = istate->_oop_temp;632#else633obj = cast_to_oop(at(interpreter_frame_oop_temp_offset));634#endif // CC_INTERP635} else {636oop* obj_p = (oop*)tos_addr;637obj = (obj_p == NULL) ? (oop)NULL : *obj_p;638}639assert(obj == NULL || Universe::heap()->is_in(obj), "sanity check");640*oop_result = obj;641break;642}643case T_BOOLEAN : value_result->z = *(jboolean*)tos_addr; break;644case T_BYTE : value_result->b = *(jbyte*)tos_addr; break;645case T_CHAR : value_result->c = *(jchar*)tos_addr; break;646case T_SHORT : value_result->s = *(jshort*)tos_addr; break;647case T_INT : value_result->i = *(jint*)tos_addr; break;648case T_LONG : value_result->j = *(jlong*)tos_addr; break;649case T_FLOAT : {650#ifdef AMD64651value_result->f = *(jfloat*)tos_addr;652#else653if (method->is_native()) {654jdouble d = *(jdouble*)tos_addr; // Result was in ST0 so need to convert to jfloat655value_result->f = (jfloat)d;656} else {657value_result->f = *(jfloat*)tos_addr;658}659#endif // AMD64660break;661}662case T_DOUBLE : value_result->d = *(jdouble*)tos_addr; break;663case T_VOID : /* Nothing to do */ break;664default : ShouldNotReachHere();665}666667return type;668}669670671intptr_t* frame::interpreter_frame_tos_at(jint offset) const {672int index = (Interpreter::expr_offset_in_bytes(offset)/wordSize);673return &interpreter_frame_tos_address()[index];674}675676#ifndef PRODUCT677678#define DESCRIBE_FP_OFFSET(name) \679values.describe(frame_no, fp() + frame::name##_offset, #name)680681void frame::describe_pd(FrameValues& values, int frame_no) {682if (is_interpreted_frame()) {683DESCRIBE_FP_OFFSET(interpreter_frame_sender_sp);684DESCRIBE_FP_OFFSET(interpreter_frame_last_sp);685DESCRIBE_FP_OFFSET(interpreter_frame_method);686DESCRIBE_FP_OFFSET(interpreter_frame_mdx);687DESCRIBE_FP_OFFSET(interpreter_frame_cache);688DESCRIBE_FP_OFFSET(interpreter_frame_locals);689DESCRIBE_FP_OFFSET(interpreter_frame_bcx);690DESCRIBE_FP_OFFSET(interpreter_frame_initial_sp);691}692}693#endif694695intptr_t *frame::initial_deoptimization_info() {696// used to reset the saved FP697return fp();698}699700intptr_t* frame::real_fp() const {701if (_cb != NULL) {702// use the frame size if valid703int size = _cb->frame_size();704if (size > 0) {705return unextended_sp() + size;706}707}708// else rely on fp()709assert(! is_compiled_frame(), "unknown compiled frame size");710return fp();711}712713#ifndef PRODUCT714// This is a generic constructor which is only used by pns() in debug.cpp.715frame::frame(void* sp, void* fp, void* pc) {716init((intptr_t*)sp, (intptr_t*)fp, (address)pc);717}718#endif719720void JavaFrameAnchor::make_walkable(JavaThread* thread) {721// last frame set?722if (last_Java_sp() == NULL) return;723// already walkable?724if (walkable()) return;725assert(Thread::current() == (Thread*)thread, "not current thread");726assert(last_Java_sp() != NULL, "not called from Java code?");727assert(last_Java_pc() == NULL, "already walkable");728capture_last_Java_pc();729assert(walkable(), "something went wrong");730}731732void JavaFrameAnchor::capture_last_Java_pc() {733assert(_last_Java_sp != NULL, "no last frame set");734assert(_last_Java_pc == NULL, "already walkable");735_last_Java_pc = (address)_last_Java_sp[-1];736}737738739