Review request for a test fix for 6992968

David Holmes David.Holmes at oracle.com
Mon Oct 18 16:37:36 PDT 2010


Hi Mandy,

It's hard to evaluate this without knowing what the coordination 
protocol is between the different threads. However you have removed some 
very strange looking synchronization code, and that can only be a good 
thing!

David

Mandy Chung said the following on 10/19/10 08:21:
>  6992968: 
> test/java/lang/management/MemoryMXBean/CollectionUsageThresholdConcMarkSweepGC.sh 
> should not hang
> 
> Webrev at:
>    http://cr.openjdk.java.net/~mchung/6992968/webrev.00/
> 
> If the test fails, the main thread is waiting for the checker thread to 
> finish checking the result but the checker thread terminates with an 
> exception.  The test hangs and the main thread is forever waiting for a 
> monitor to be notified.  I update the test to use the new concurrent 
> utility and reset a CyclicBarrier that causes a BrokenBarrierException 
> to be thrown in the main thread.
> 
> I also add @ignore to CollectionUsageThresholdConcMarkSweepGC.sh test 
> until 6982965 is fixed.
> 
> Mandy
> 


More information about the serviceability-dev mailing list