<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">I do not have a full patch yet. A demo should be fine, but I'll include some relative code below in case it helps:<div><br></div><div>In javaClasses.cpp (all new):</div><div><div>oop java_lang_StackTraceFrame::create(Method* method, int version, int bci, TRAPS) {</div><div>&nbsp; // Allocate java.lang.StackTraceFrame instance</div><div>&nbsp; Klass* k = SystemDictionary::StackTraceFrame_klass();</div><div>&nbsp; assert(k != NULL, "must be loaded in 1.8+");</div><div>&nbsp; instanceKlassHandle ik (THREAD, k);</div><div>&nbsp; if (ik-&gt;should_be_initialized()) {</div><div>&nbsp; &nbsp; ik-&gt;initialize(CHECK_0);</div><div>&nbsp; }</div><div><br></div><div>&nbsp; Handle stFrame = ik-&gt;allocate_instance_handle(CHECK_0);</div><div><br></div><div>&nbsp; // Fill in declaringClass</div><div>&nbsp; ResourceMark rm(THREAD);</div><div>&nbsp; InstanceKlass* holder = InstanceKlass::cast(method-&gt;method_holder());</div><div>&nbsp; java_lang_StackTraceFrame::set_declaringClass(stFrame(), method-&gt;method_holder()-&gt;java_mirror());</div><div><br></div><div>&nbsp; // Fill in methodName</div><div>&nbsp; oop methodname = StringTable::intern(method-&gt;name(), CHECK_0);</div><div>&nbsp; java_lang_StackTraceFrame::set_methodName(stFrame(), methodname);</div><div><br></div><div>&nbsp; if (!version_matches(method, version)) {</div><div>&nbsp; &nbsp; // The method was redefined, accurate source information isn't available</div><div>&nbsp; &nbsp; java_lang_StackTraceFrame::set_fileName(stFrame(), NULL);</div><div>&nbsp; &nbsp; java_lang_StackTraceFrame::set_lineNumber(stFrame(), -1);</div><div>&nbsp; } else {</div><div>&nbsp; &nbsp; // Fill in source fileName and lineNumber.</div><div>&nbsp; &nbsp; Symbol* source = holder-&gt;source_file_name();</div><div>&nbsp; &nbsp; if (ShowHiddenFrames &amp;&amp; source == NULL)</div><div>&nbsp; &nbsp; &nbsp; source = vmSymbols::unknown_class_name();</div><div>&nbsp; &nbsp; oop filename = StringTable::intern(source, CHECK_0);</div><div>&nbsp; &nbsp; java_lang_StackTraceFrame::set_fileName(stFrame(), filename);</div><div><br></div><div>&nbsp; &nbsp; int line_number = get_line_number(method, bci);</div><div>&nbsp; &nbsp; java_lang_StackTraceFrame::set_lineNumber(stFrame(), line_number);</div><div>&nbsp; }</div><div>&nbsp; return stFrame();</div><div>}</div><div><br></div><div>oop java_lang_StackTraceFrame::create(vframeStream* vfst, TRAPS) {</div><div>&nbsp; return create(vfst-&gt;method(), vfst-&gt;method()-&gt;constants()-&gt;version(), vfst-&gt;bci(), THREAD);</div><div>}</div></div><div><br></div><div>In library_call.cpp:</div><div><div>bool LibraryCallKit::inline_native_ReflectionOrStackTraceFrame_getCallerClassOrFrame(bool getFrame) {</div><div>#ifndef PRODUCT</div><div>&nbsp; if ((PrintIntrinsics || PrintInlining || PrintOptoInlining) &amp;&amp; Verbose) {</div><div>&nbsp; &nbsp; if (getFrame) {</div><div>&nbsp; &nbsp; &nbsp; tty-&gt;print_cr("Attempting to inline java.lang.StackTraceFrame#getCallerFrame()");</div><div>&nbsp; &nbsp; } else {</div><div>&nbsp; &nbsp; &nbsp; tty-&gt;print_cr("Attempting to inline getCallerClass() in sun.reflect.Reflection or java.lang.StackTraceFrame");</div><div>&nbsp; &nbsp; }</div><div>&nbsp; }</div><div>#endif</div><div><br></div><div>&nbsp; if (!jvms()-&gt;has_method()) {</div><div>#ifndef PRODUCT</div><div>&nbsp; &nbsp; if ((PrintIntrinsics || PrintInlining || PrintOptoInlining) &amp;&amp; Verbose) {</div><div>&nbsp; &nbsp; &nbsp; tty-&gt;print_cr(" &nbsp;Bailing out because intrinsic was inlined at top level");</div><div>&nbsp; &nbsp; }</div><div>#endif</div><div>&nbsp; &nbsp; return false;</div><div>&nbsp; }</div><div><br></div><div>&nbsp; // Walk back up the JVM state to find the caller at the required</div><div>&nbsp; // depth.</div><div>&nbsp; JVMState* caller_jvms = jvms();</div><div><br></div><div>&nbsp; // Cf. JVM_GetCallerClass</div><div>&nbsp; // NOTE: Start the loop at depth 1 because the current JVM state does</div><div>&nbsp; // not include the Reflection/StackTraceFrame.getCallerClass() frame.</div><div>&nbsp; for (int n = 1; caller_jvms != NULL; caller_jvms = caller_jvms-&gt;caller(), n++) {</div><div>&nbsp; &nbsp; ciMethod* m = caller_jvms-&gt;method();</div><div>&nbsp; &nbsp; switch (n) {</div><div>&nbsp; &nbsp; case 0:</div><div>&nbsp; &nbsp; &nbsp; fatal("current JVM state does not include the Reflection/StackTraceFrame.getCallerClass frame");</div><div>&nbsp; &nbsp; &nbsp; break;</div><div>&nbsp; &nbsp; case 1:</div><div>&nbsp; &nbsp; &nbsp; // Frame 0 and 1 must be caller sensitive (see JVM_GetCallerClass).</div><div>&nbsp; &nbsp; &nbsp; if (!m-&gt;caller_sensitive()) {</div><div>#ifndef PRODUCT</div><div>&nbsp; &nbsp; &nbsp; &nbsp; if ((PrintIntrinsics || PrintInlining || PrintOptoInlining) &amp;&amp; Verbose) {</div><div>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; tty-&gt;print_cr(" &nbsp;Bailing out: CallerSensitive annotation expected at frame %d", n);</div><div>&nbsp; &nbsp; &nbsp; &nbsp; }</div><div>#endif</div><div>&nbsp; &nbsp; &nbsp; &nbsp; return false; &nbsp;// bail-out; let JVM_GetCallerClass do the work</div><div>&nbsp; &nbsp; &nbsp; }</div><div>&nbsp; &nbsp; &nbsp; break;</div><div>&nbsp; &nbsp; default:</div><div>&nbsp; &nbsp; &nbsp; if (!m-&gt;is_ignored_by_security_stack_walk()) {</div><div>&nbsp; &nbsp; &nbsp; &nbsp; // We have reached the desired frame; return the caller.</div><div>&nbsp; &nbsp; &nbsp; &nbsp; if (getFrame) {</div><div>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; fatal("Got into getFrame anyway, which is weird, yo");</div><div>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; return false; // TODO fix this</div><div>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; // Acquire java.lang.StackTraceFrame and push as constant</div><div>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; //Method* method = (Method*)m-&gt;constant_encoding();</div><div>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; //oop stFrame = java_lang_StackTraceFrame::create(method, method-&gt;constants()-&gt;version(), caller_jvms-&gt;bci(), NULL);</div><div>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; // TODO fix this ciObject* stfObject = ciEnv::current()-&gt;get_object(stFrame);</div><div>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; // TODO fix this set_result(makecon(TypeInstPtr::make(stfObject)));</div><div>&nbsp; &nbsp; &nbsp; &nbsp; } else {</div><div>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; // Acquire method holder as java.lang.Class and push as constant.</div><div>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; ciInstanceKlass* caller_klass = m-&gt;holder();</div><div>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; ciInstance* caller_mirror = caller_klass-&gt;java_mirror();</div><div>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; set_result(makecon(TypeInstPtr::make(caller_mirror)));</div><div>&nbsp; &nbsp; &nbsp; &nbsp; }</div><div><br></div><div>#ifndef PRODUCT</div><div>&nbsp; &nbsp; &nbsp; &nbsp; if ((PrintIntrinsics || PrintInlining || PrintOptoInlining) &amp;&amp; Verbose) {</div><div>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; tty-&gt;print_cr(" &nbsp;Succeeded: caller = %d) %s.%s, JVMS depth = %d", n, caller_klass-&gt;name()-&gt;as_utf8(), caller_jvms-&gt;method()-&gt;name()-&gt;as_utf8(), jvms()-&gt;depth());</div><div>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; tty-&gt;print_cr(" &nbsp;JVM state at this point:");</div><div>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; for (int i = jvms()-&gt;depth(), n = 1; i &gt;= 1; i--, n++) {</div><div>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; ciMethod* m = jvms()-&gt;of_depth(i)-&gt;method();</div><div>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; tty-&gt;print_cr(" &nbsp; %d) %s.%s", n, m-&gt;holder()-&gt;name()-&gt;as_utf8(), m-&gt;name()-&gt;as_utf8());</div><div>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; }</div><div>&nbsp; &nbsp; &nbsp; &nbsp; }</div><div>#endif</div><div>&nbsp; &nbsp; &nbsp; &nbsp; return true;</div><div>&nbsp; &nbsp; &nbsp; }</div><div>&nbsp; &nbsp; &nbsp; break;</div><div>&nbsp; &nbsp; }</div><div>&nbsp; }</div><div><br></div><div>#ifndef PRODUCT</div><div>&nbsp; if ((PrintIntrinsics || PrintInlining || PrintOptoInlining) &amp;&amp; Verbose) {</div><div>&nbsp; &nbsp; tty-&gt;print_cr(" &nbsp;Bailing out because caller depth exceeded inlining depth = %d", jvms()-&gt;depth());</div><div>&nbsp; &nbsp; tty-&gt;print_cr(" &nbsp;JVM state at this point:");</div><div>&nbsp; &nbsp; for (int i = jvms()-&gt;depth(), n = 1; i &gt;= 1; i--, n++) {</div><div>&nbsp; &nbsp; &nbsp; ciMethod* m = jvms()-&gt;of_depth(i)-&gt;method();</div><div>&nbsp; &nbsp; &nbsp; tty-&gt;print_cr(" &nbsp; %d) %s.%s", n, m-&gt;holder()-&gt;name()-&gt;as_utf8(), m-&gt;name()-&gt;as_utf8());</div><div>&nbsp; &nbsp; }</div><div>&nbsp; }</div><div>#endif</div><div><br></div><div>&nbsp; return false; &nbsp;// bail-out; let JVM_GetCallerClass do the work</div><div>}</div></div><div><br></div><div><br><div><div>On Aug 1, 2013, at 9:19 PM, Krystal Mok wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div dir="ltr">On Fri, Aug 2, 2013 at 10:15 AM, Nick Williams <span dir="ltr">&lt;<a href="mailto:nicholas+openjdk@nicholaswilliams.net" target="_blank">nicholas+openjdk@nicholaswilliams.net</a>&gt;</span> wrote:<br><div class="gmail_extra">
<div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><br><div><div class="im"><div>On Aug 1, 2013, at 9:07 PM, Krystal Mok wrote:</div>
<br></div><blockquote type="cite"><div dir="ltr">Hi Nick,<div><br></div><div class="im"><div>All right, that explains everything. My take on this:</div><div><br></div><div>In HotSpot VM, the compiler threads shouldn't manipulate the Java object graph; in other words, although the compilers have access to certain Java objects through the Compiler Interface, it should only be "reading", but not "writing" or creating new Java objects.</div>

<div><br></div><div>In your case, calling&nbsp;java_lang_StackTraceFrame::create() from compiler code creates a new object, and that's probably not the right way.</div><div>You're trying to embed a pointer to a StackTraceFrame object in the generated code, as a constant. You don't have to do that. It might work if you embed the Method* (as a TypeMetadataPtr), and generate a runtime call to java_lang_StackTraceFrame::create() with that pointer, instead of calling it at compile time.</div>
</div></div></blockquote><div><br></div><div>Sounds good, but I'm not exactly sure how to do it. Do you mind assisting a bit? Remember that "ciMethod* m" is the executing method on the stack frame we're interested in. "Method* method" is the same thing, just in a different form. It sounds like I need to actually embed a runtime call to java_lang_StackTraceFrame::create()? I may be missing something. :-)</div>
<span class="HOEnZb"><font color="#888888"><div><br></div></font></span></div></div></blockquote><div style="">Sure. I can give it a shot and see if it works.</div><div style="">Do you have a full patch that I can use as a base, or would you just like to see a demo of doing a runtime call passing an embedded pointer constant?</div>
<div style=""><br></div><div style="">- Kris</div><div>&nbsp;</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div><span class="HOEnZb"><font color="#888888"><div>
</div><div>Nick</div></font></span><div><div class="h5"><br><blockquote type="cite"><div dir="ltr">
<div><br></div><div>HTH,</div><div>Kris</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Aug 2, 2013 at 9:28 AM, Nick Williams <span dir="ltr">&lt;<a href="mailto:nicholas+openjdk@nicholaswilliams.net" target="_blank">nicholas+openjdk@nicholaswilliams.net</a>&gt;</span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">Well let me explain what I'm trying to do, and maybe someone can point me in the right direction.<div>

<br></div><div>On the core-libs-dev mailing list, I proposed a public API replacement for the sun.reflect.Reflection#getCallerClass(int) of old and the&nbsp;sun.reflect.Reflection#getCallerClass() of now. The new class, java.lang.StackTraceFrame, has the following four methods that are relevant to this discussion:</div>

<div><br></div><div>@CallerSensitive</div><div>public native Class&lt;?&gt; getCallerClass();</div><div>public native Class&lt;?&gt; getCallerClass(int);</div><div>@CallerSensitive</div><div>public native StackTraceFrame getCallerFrame();</div>

<div>public native StackTraceFrame getCallerFrame(int);</div><div><br></div><div>There is code in library_call.cpp that inlines Reflection.getCallerClass(), and I took advantage of that code to also inline StackTraceFrame.getCallerClass(). I also recreated (with changes necessary due to new way of doing things in library_call.cpp) the inline code for Reflection.getCallerClass(int) from the jdk7 source code. All of that works perfectly. Code is inlined as expected, and all is well.</div>

<div><br></div><div>Now I want to inline getCallerFrame() and getCallerFrame(int) as well. If the Class&lt;?&gt; versions can be inlined, I see no reason that the StackTraceFrame versions can't be inlined, but that may be my first wrong assumption. If these methods aren't eligible for inlining for some reason, please let me know and I can move on.</div>

<div><br></div><div>So the code that gets the Class&lt;?&gt; and sets it as an inline constant replacing the method call works like this (where m is a ciMethod*):</div><div><br></div><div>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; // Acquire method holder as java.lang.Class and push as constant.</div>

<div><div>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; ciInstanceKlass* caller_klass = m-&gt;holder();</div><div>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; ciInstance* caller_mirror = caller_klass-&gt;java_mirror();</div><div>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; set_result(makecon(TypeInstPtr::make(caller_mirror)));</div>

</div><div><br></div><div>In javaClasses.cpp, I already have a method that can create a StackTraceFrame from a Method* (not ciMethod*), method version (int), and bci (int):</div><div><br></div><div>oop java_lang_StackTraceFrame::create(Method* method, int version, int bci, TRAPS);</div>

<div><br></div><div>I tried calling that to create the StackTraceFrame, but it appears I can't actually set it to the result. First I tried this:</div><div><br></div><div><div>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; // Acquire java.lang.StackTraceFrame and push as constant</div>

<div>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; Method* method = (Method*)m-&gt;constant_encoding();</div><div>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; oop stFrame = java_lang_StackTraceFrame::create(method, method-&gt;constants()-&gt;version(), caller_jvms-&gt;bci(), NULL);</div><div>

&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; set_result(makecon(TypeInstPtr::make(oop)));</div></div><div><br></div><div>But this obviously didn't work because there is no TypeInstPtr::make() method that takes an oopDesc&amp;. So I looked around trying to figure out how to convert an oop (or jobject, which I can get from an oop) &nbsp;to a ciObject. I discovered that ciObject actually holds a jobject internally, so I figured surely this is possible. I found a few methods that appear to do it. ciObjectFactory has a ciObject* get(oop) method, but I can't figure out how to get the ciObjectFactory. ciEnv has a ciObject* get_object(oop) method which I tried to use, only to discover that it was private:</div>

<div><br></div><div><div>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; // Acquire java.lang.StackTraceFrame and push as constant</div><div>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; Method* method = (Method*)m-&gt;constant_encoding();</div><div>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; oop stFrame = java_lang_StackTraceFrame::create(method, method-&gt;constants()-&gt;version(), caller_jvms-&gt;bci(), NULL);</div>

<div>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; ciObject* stfObject = ciEnv::current()-&gt;get_object(stFrame);</div><div>&nbsp; &nbsp; &nbsp; &nbsp; &nbsp; set_result(makecon(TypeInstPtr::make(stfObject)));</div></div><div><br></div><div>So that obviously doesn't work either. I'll admit, I'm a little out of my league here. I was pretty confident working in javaClasses.hpp/cpp and jvm.h/cpp, but this inlining/compiler stuff is pretty over my head. Hopefully one of the knowledgeable people on this list can help me out with this. I want my code to perform as well as possible and also be stable--more likely to get accepted that way. :-)</div>

<div><br></div><div>Thanks,</div><div><br></div><div>Nick</div><div><div><div><br><div><div>On Aug 1, 2013, at 8:16 AM, Krystal Mok wrote:</div><br><blockquote type="cite"><div dir="ltr">Hi Nick,<div><br></div>
<div>This topic is related to HotSpot Server Compiler instead of the Java core library, so I'm cc'ing this email to hotspot-compiler-dev and dropping core-libs-dev.</div>
<div><br></div><div>As you already know, HotSpot compilers are shielded from VM runtime implementation details via the Compiler Interface (CI). That's why you shouldn't be getting raw oops in C2 code.</div>
<div><br></div><div>Where are you getting the oopInstance from? If it can be found from some known "roots", e.g. "well-known classes", fields of "well-known classes", or the holder class of the method to be compiled, etc., then it's already available through CI.</div>


<div><br></div><div>- Kris</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Aug 1, 2013 at 3:17 AM, Nick Williams <span dir="ltr">&lt;<a href="mailto:nicholas+openjdk@nicholaswilliams.net" target="_blank">nicholas+openjdk@nicholaswilliams.net</a>&gt;</span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">In native code (library_call.cpp), if I have an oop (which I can convert to a jobject if need be), how do I get a ciObject? I see that ciEnv has a ciObject* get_object(oop) method, but it's private. And ciObjectFactory has a ciObject* get(oop) method, but I can't figure out how to get the ciObjectFactory instance.<br>



<br>
I know that ciObject keeps a jobject internally, and I know that ciObject has a ciObject(oop) constructor, but it's protected (for good reason).<br>
<br>
If it helps, I'm trying to inline a method and need to set_result(makecon(TypeInstPtr::make(oopInstance))). I may be going down the wrong path.<br>
<br>
Thanks in advance for any help,<br>
<br>
Nick</blockquote></div><br></div>
</blockquote></div><br></div></div></div></div></blockquote></div><br></div>
</blockquote></div></div></div><br></div></blockquote></div><br></div></div>
</blockquote></div><br></div></body></html>