Greetings,
The jvmticmlr.h stuff is in:
- OpenJDK6 - JDK and HotSpot sides
- JDK7 - JDK side
- HSX17 - HotSpot side
When HSX17-B08 pushes to JDK7, then the HotSpot side will
be in JDK7 also.
We'll be looking at the JDK6-Update train next...
Dan
Daniel D. Daugherty wrote:
Greetings,
The folks at AMD Labs have been kind enough to provide an
extension to the JVM/TI CompileMethodLoad event in order
to provide additional information about in-lining. This
extension uses the existing (but previously unused)
compile_info paramter:
http://java.sun.com/javase/6/docs/platform/jvmti/jvmti.html#CompiledMethodLoad
Vasanth and company provided the HotSpot code changes and
the original demo program. I just did the Makefile changes
to export the new jvmticmlr.h file in the HotSpot repo and
the integration of the demo program into JAVA_HOME/demo/jvmti
in the JDK repo.
Here is the webrev for the OpenJDK6 version of the changes:
http://cr.openjdk.java.net/~dcubed/6580131-webrev/0/
The OpenJDK7 version of these changes are not expected to be
very different from this version.
For the Sun folks, the CCC request for adding the jvmticmlr.h
is almost final. I'm waiting for the VM-SQE team to agree that
the latest version addresses their review concerns, but I
think the St Petersburg team is on holiday at the moment.
Any reviews are appreciated.
Dan
--- Begin Message ---
Changeset: 7fbf850d87b7
Author: dcubed
Date: 2010-01-13 09:39 -0700
URL: http://hg.openjdk.java.net/jdk6/jdk6/hotspot/rev/7fbf850d87b7
6580131: 3/4 CompiledMethodLoad events don't produce the expected extra
notifications to describe inlining
Summary: Add support for additional implementation specific info to the JVM/TI
CompiledMethodLoad event via the compile_info parameter.
Reviewed-by: never, ohair, tbell, tdeneau
Contributed-by: Vasanth Venkatachalam <[email protected]>
! make/Makefile
! make/defs.make
+ src/share/vm/code/jvmticmlr.h
! src/share/vm/includeDB_core
! src/share/vm/prims/jvmtiExport.cpp
Changeset: 7dbe24cb959c
Author: dcubed
Date: 2010-01-20 10:35 -0700
URL: http://hg.openjdk.java.net/jdk6/jdk6/hotspot/rev/7dbe24cb959c
Merge
! make/Makefile
! make/defs.make
! src/share/vm/includeDB_core
! src/share/vm/prims/jvmtiExport.cpp
--- End Message ---
--- Begin Message ---
Changeset: 7fbf850d87b7
Author: dcubed
Date: 2010-01-13 09:39 -0700
URL: http://hg.openjdk.java.net/jdk7/hotspot-rt/hotspot/rev/7fbf850d87b7
6580131: 3/4 CompiledMethodLoad events don't produce the expected extra
notifications to describe inlining
Summary: Add support for additional implementation specific info to the JVM/TI
CompiledMethodLoad event via the compile_info parameter.
Reviewed-by: never, ohair, tbell, tdeneau
Contributed-by: Vasanth Venkatachalam <[email protected]>
! make/Makefile
! make/defs.make
+ src/share/vm/code/jvmticmlr.h
! src/share/vm/includeDB_core
! src/share/vm/prims/jvmtiExport.cpp
Changeset: 3908ad124838
Author: dcubed
Date: 2010-01-20 11:32 -0700
URL: http://hg.openjdk.java.net/jdk7/hotspot-rt/hotspot/rev/3908ad124838
Merge
! make/Makefile
! make/defs.make
! src/share/vm/includeDB_core
! src/share/vm/prims/jvmtiExport.cpp
--- End Message ---
--- Begin Message ---
Changeset: 6073840fbd22
Author: dcubed
Date: 2010-01-13 09:42 -0700
URL: http://hg.openjdk.java.net/jdk6/jdk6/jdk/rev/6073840fbd22
6580131: 3/4 CompiledMethodLoad events don't produce the expected extra
notifications to describe inlining
Summary: Add support for additional implementation specific info to the JVM/TI
CompiledMethodLoad event via the compile_info parameter.
Reviewed-by: never, ohair, tbell, tdeneau
Contributed-by: Vasanth Venkatachalam <[email protected]>
! make/common/shared/Sanity.gmk
! make/java/jvm/Makefile
! make/mkdemo/jvmti/Makefile
! make/mkdemo/jvmti/README.txt
+ make/mkdemo/jvmti/compiledMethodLoad/Makefile
+ src/share/demo/jvmti/compiledMethodLoad/README.txt
+ src/share/demo/jvmti/compiledMethodLoad/compiledMethodLoad.c
+ src/share/demo/jvmti/compiledMethodLoad/sample.makefile.txt
! src/share/demo/jvmti/index.html
+ src/share/javavm/export/jvmticmlr.h
+ test/demo/jvmti/compiledMethodLoad/CompiledMethodLoadTest.java
! test/demo/jvmti/heapTracker/HeapTrackerTest.java
! test/demo/jvmti/hprof/CpuTimesDefineClassTest.java
! test/demo/jvmti/hprof/CpuTimesTest.java
! test/demo/jvmti/minst/MinstTest.java
! test/demo/jvmti/mtrace/TraceJFrame.java
Changeset: b2510c4f0228
Author: dcubed
Date: 2010-01-20 11:47 -0700
URL: http://hg.openjdk.java.net/jdk6/jdk6/jdk/rev/b2510c4f0228
Merge
--- End Message ---
--- Begin Message ---
Changeset: b19cd193245e
Author: dcubed
Date: 2010-01-20 12:09 -0700
URL: http://hg.openjdk.java.net/jdk7/tl/jdk/rev/b19cd193245e
6580131: 3/4 CompiledMethodLoad events don't produce the expected extra
notifications to describe inlining
Summary: Add support for additional implementation specific info to the JVM/TI
CompiledMethodLoad event via the compile_info parameter.
Reviewed-by: never, ohair, tbell, tdeneau
Contributed-by: Vasanth Venkatachalam <[email protected]>
! make/common/shared/Sanity.gmk
! make/java/jvm/Makefile
! make/mkdemo/jvmti/Makefile
! make/mkdemo/jvmti/README.txt
+ make/mkdemo/jvmti/compiledMethodLoad/Makefile
+ src/share/demo/jvmti/compiledMethodLoad/README.txt
+ src/share/demo/jvmti/compiledMethodLoad/compiledMethodLoad.c
+ src/share/demo/jvmti/compiledMethodLoad/sample.makefile.txt
! src/share/demo/jvmti/index.html
+ src/share/javavm/export/jvmticmlr.h
+ test/demo/jvmti/compiledMethodLoad/CompiledMethodLoadTest.java
! test/demo/jvmti/heapTracker/HeapTrackerTest.java
! test/demo/jvmti/hprof/CpuTimesDefineClassTest.java
! test/demo/jvmti/hprof/CpuTimesTest.java
! test/demo/jvmti/minst/MinstTest.java
! test/demo/jvmti/mtrace/TraceJFrame.java
--- End Message ---
