Request Review: 6902182: Starting with jdwp agent should not incur performance penalty
Deneau, Tom
tom.deneau at amd.com
Wed Dec 9 13:31:08 PST 2009
Webrev is at http://cr.openjdk.java.net/~tdeneau/6902182/webrev.01/
This webrev changes two places in the compiler where code for exception
throws is being JITted.
* previously these checked jvmti_can_post_exceptions() at compile
time, compiling in a slow path if true
* now they call a new jvmtiExport::must_post_exception_events at
run time, and take a faster path if must_post_exception_events is
false.
must_post_exception_events uses logic similar to that used
by jvmtiExport::post_exception_throw and returns false if
jvmtiExport::post_exception_throw wouldn't have done anything.
A similar change of calling must_post_exception_events instead of
jvmti_can_post_exceptions is made in the exception runtime code in
trace_exception in opto/runtime.cpp
Throughput Numbers from a JIRA-based web workload (JIRA makes fairly heavy use
of exception throws and catches). Bigger is better.
unmodified hotspot, no jdwp agent 69.7
unmodified hotspot, with jdwp agent 11.8
modified hotspot, with jdwp agent 65.7
All changes are platform-independent.
-- Tom Deneau
tom.deneau at amd.com
More information about the hotspot-compiler-dev
mailing list