Review request for 8022208: Intermittent test failures in java/lang/Thread/ThreadStateTest.java

[Alan Bateman]

On 31/10/2013 19:11, Mandy Chung wrote:
> Updated webrev that has a new 
> test/lib/testlibrary/ThreadStateController.java and also change to use 
> AtomicInteger:
> http://cr.openjdk.java.net/~mchung/jdk8/webrevs/8022208/webrev.01/
>
> Mandy
Is ThreadStateController general enough for the unname package in 
/lib/testlibrary? Just wondering if it should be moved down into a 
package (I like using testlibrary, just wonder how it will scale if we 
make over-use of it).

Also does the refactoring mean that methods can be removed from 
ThreadMXBean/Utils.java? I realize this is expanding your scope a bit 
and maybe changing SynchronizationStatistics (which appears to be the 
user other of Utils).

-Alan.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20131031/d4bbbc4f/attachment.html