RFR (M) 8155739: [TESTBUG] VarHandles/Unsafe tests for weakCAS should allow spurious failures

Paul Sandoz paul.sandoz at oracle.com
Fri Apr 29 23:42:43 UTC 2016


> On 29 Apr 2016, at 15:12, Aleksey Shipilev <aleksey.shipilev at oracle.com> wrote:
> 
> Hi,
> 
> I would like to fix a simple testbug in our weakCompareAndSet VarHandles
> and Unsafe intrinsics tests. weakCompareAndSet is spec-ed to allow
> spurious failures, but current tests do not allow that. This blocks
> development and testing on non-x86 platforms.
> 
> Bug:
>  https://bugs.openjdk.java.net/browse/JDK-8155739
> 
> Webrevs:
>  http://cr.openjdk.java.net/~shade/8155739/webrev.hs.00/
>  http://cr.openjdk.java.net/~shade/8155739/webrev.jdk.00/
> 

Looks good.

Small tweak if you so wish to do so:

#if[JdkInternalMisc]
    static final int WEAK_ATTEMPTS = Integer.getInteger("weakAttempts", 10);
#end[JdkInternalMisc]

which avoids changes to the SunMiscUnsafe* tests.

Paul.

> The tests are auto-generated, and the substantiative changes are in
> *.template files. I also removed obsolete generate-unsafe-tests.sh. I
> would like to push through hs-comp to expose this to Power and AArch64
> folks early.
> 
> Testing: x86_64, jdk:java/lang/invoke/VarHandle, hotspot:compiler/unsafe
> 
> Thanks,
> -Aleksey
> 




More information about the core-libs-dev mailing list