Request for review: 7012206

[Alan Bateman]

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:  http://cr.openjdk.java.net/~dholmes/7012206/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/jps-Vvml_2.sh - 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.
This one reminds me that we need to go over all our shell tests so that 
they pass $TESTVMOPTS through to all VMs that they create. Otherwise we 
aren't always testing what we think we are testing.

The changes in the webrev look fine to me but I would have expected to 
see tests other than the jvmstat stats, for example the tests in 
com/sun/tools/attach and sun/management/jmxremote/bootstrap.

-Alan.