Request for review: 7012206

David Holmes david.holmes at
Wed Sep 21 02:14:54 UTC 2011

There was an omission in the original webrev regarding the jtreg 
invocation changes for The @run directive 
needs the name of the class to invoke

+  * @run main/othervm -XX:+UsePerfData  PrologSizeSanityCheck

Updated webrev at:

I'll proceed with the push as this was a minor omission.


On 19/09/2011 3:22 PM, David Holmes wrote:
> This a change to a bunch of serviceability tests (shell scripts that
> launch the various j* tools (jps, jstatd, jstack etc)) that I'd like to
> push through the TL JDK repo.
> The changes were done by Carlos Lucasius but I'm acting as his "sponsor"
> for getting these pushed.
> webrev:
> Summary: for correct operation the tools and/or the VM they target must
> be running with UsePerfData enabled. This VM option is enabled in Java
> SE by default, but is disabled in Java SE Embedded by default. To allow
> the tests to be used regardless of the UsePerfdata setting they are
> augmented to explicitly turn it on.
> There has been some prior internal debate around how "best" to deal with
> this issue and the resulting changes, while somewhat repetitive, are the
> simplest approach to take.
> There is one test - jps/ - that can not pass with such a
> fix because it is actually trying to test the jps output when no
> arguments (VM or application) are passed to the target VM. So for that
> test I've just added a comment.
> Thanks,
> David

More information about the core-libs-dev mailing list