changeset 64:e29395e8f369

Relax some volatile field in constructors tests to match the discussion on c-i@.
author shade
date Thu, 28 Nov 2013 11:30:09 +0400
parents 6dc6598907d6
children 08da99c27fa4
files tests-custom/src/main/resources/org/openjdk/jcstress/desc/atomic-integer.xml tests-custom/src/main/resources/org/openjdk/jcstress/desc/atomic-long.xml tests-custom/src/main/resources/org/openjdk/jcstress/desc/initialization.xml
diffstat 3 files changed, 28 insertions(+), 8 deletions(-) [+]
line wrap: on
line diff
--- a/tests-custom/src/main/resources/org/openjdk/jcstress/desc/atomic-integer.xml	Tue Nov 19 01:09:10 2013 +0400
+++ b/tests-custom/src/main/resources/org/openjdk/jcstress/desc/atomic-integer.xml	Thu Nov 28 11:30:09 2013 +0400
@@ -32,6 +32,16 @@
             Tests the visibility of AtomicInteger initial value.
         </description>
         <case>
+            <match>[0]</match>
+            <expect>ACCEPTABLE</expect>
+            <description>
+                Possible to see the default value, since the instance is published via race.
+            </description>
+            <urls>
+                <url>http://cs.oswego.edu/pipermail/concurrency-interest/2013-November/011951.html</url>
+            </urls>
+        </case>
+        <case>
             <match>[1]</match>
             <expect>ACCEPTABLE</expect>
             <description>
--- a/tests-custom/src/main/resources/org/openjdk/jcstress/desc/atomic-long.xml	Tue Nov 19 01:09:10 2013 +0400
+++ b/tests-custom/src/main/resources/org/openjdk/jcstress/desc/atomic-long.xml	Thu Nov 28 11:30:09 2013 +0400
@@ -32,6 +32,16 @@
             Tests the visibility of AtomicLong initial value.
         </description>
         <case>
+            <match>[0]</match>
+            <expect>ACCEPTABLE</expect>
+            <description>
+                Possible to see the default value, since the instance is published via race.
+            </description>
+            <urls>
+                <url>http://cs.oswego.edu/pipermail/concurrency-interest/2013-November/011951.html</url>
+            </urls>
+        </case>
+        <case>
             <match>[1]</match>
             <expect>ACCEPTABLE</expect>
             <description>
--- a/tests-custom/src/main/resources/org/openjdk/jcstress/desc/initialization.xml	Tue Nov 19 01:09:10 2013 +0400
+++ b/tests-custom/src/main/resources/org/openjdk/jcstress/desc/initialization.xml	Thu Nov 28 11:30:09 2013 +0400
@@ -407,14 +407,14 @@
     <test name="org.openjdk.jcstress.tests.init.primitives.instance.DoubleInstanceInitializerTest"    template="float_canSeeMost" />
     <test name="org.openjdk.jcstress.tests.init.primitives.instance.FloatInstanceInitializerTest"     template="double_canSeeMost" />
 
-    <test name="org.openjdk.jcstress.tests.init.primitives.volatiles.BooleanVolatileTest"   template="int_shouldSeeFull" />
-    <test name="org.openjdk.jcstress.tests.init.primitives.volatiles.ByteVolatileTest"      template="int_shouldSeeFull" />
-    <test name="org.openjdk.jcstress.tests.init.primitives.volatiles.ShortVolatileTest"     template="int_shouldSeeFull" />
-    <test name="org.openjdk.jcstress.tests.init.primitives.volatiles.IntVolatileTest"       template="int_shouldSeeFull" />
-    <test name="org.openjdk.jcstress.tests.init.primitives.volatiles.LongVolatileTest"      template="int_shouldSeeFull" />
-    <test name="org.openjdk.jcstress.tests.init.primitives.volatiles.CharVolatileTest"      template="char_shouldSeeFull" />
-    <test name="org.openjdk.jcstress.tests.init.primitives.volatiles.DoubleVolatileTest"    template="float_shouldSeeFull" />
-    <test name="org.openjdk.jcstress.tests.init.primitives.volatiles.FloatVolatileTest"     template="float_shouldSeeFull" />
+    <test name="org.openjdk.jcstress.tests.init.primitives.volatiles.BooleanVolatileTest"   template="int_canSeeDefault" />
+    <test name="org.openjdk.jcstress.tests.init.primitives.volatiles.ByteVolatileTest"      template="int_canSeeDefault" />
+    <test name="org.openjdk.jcstress.tests.init.primitives.volatiles.ShortVolatileTest"     template="int_canSeeDefault" />
+    <test name="org.openjdk.jcstress.tests.init.primitives.volatiles.IntVolatileTest"       template="int_canSeeDefault" />
+    <test name="org.openjdk.jcstress.tests.init.primitives.volatiles.LongVolatileTest"      template="int_canSeeDefault" />
+    <test name="org.openjdk.jcstress.tests.init.primitives.volatiles.CharVolatileTest"      template="char_canSeeDefault" />
+    <test name="org.openjdk.jcstress.tests.init.primitives.volatiles.DoubleVolatileTest"    template="float_canSeeDefault" />
+    <test name="org.openjdk.jcstress.tests.init.primitives.volatiles.FloatVolatileTest"     template="float_canSeeDefault" />
 
     <test name="org.openjdk.jcstress.tests.init.primitives.finals.BooleanFinalTest"   template="int_shouldSeeFull" />
     <test name="org.openjdk.jcstress.tests.init.primitives.finals.ByteFinalTest"      template="int_shouldSeeFull" />