commit 13ede9cc6e51aee7984e4a53b1b53d0fba712c8a
author Hans Boehm <hboehm@google.com> Wed May 25 16:57:31 2022 -0700
committer Treehugger Robot <treehugger-gerrit@google.com> Wed Jun 08 08:05:24 2022 +0000
tree 7a9f1e6303714568ff067e59c5624237036afbce
parent aed0183b0d9903bf3fd34c318931648479289705

Disable 2043-reference_pauses for CMS

And fix a minor issue in the test that should have no impact on its
execution.

Bug: 232459100
Test: Treehugger

Change-Id: I1c0abaa23a45d5f162beb8e467f67cd6ea66fbbc

test/2043-reference-pauses/src/Main.java

diff --git a/test/2043-reference-pauses/src/Main.java b/test/2043-reference-pauses/src/Main.java
index dc64d9a..e390155 100644
--- a/test/2043-reference-pauses/src/Main.java
+++ b/test/2043-reference-pauses/src/Main.java
@@ -187,7 +187,7 @@
         // which should thus not get cleared.
         Integer[] strongRefs = new Integer[NREFS];
         for (int i = 0; i < NBATCHES; ++i) {
-            for (int j = i * REFS_PER_BATCH; j < (i + 1) * NREFS / NBATCHES; ++j) {
+            for (int j = i * REFS_PER_BATCH; j < (i + 1) * REFS_PER_BATCH; ++j) {
                 Integer newObj = new Integer(j);
                 strongRefs[j] = newObj;
                 weakRefs.set(j, new WeakReference(newObj));

test/knownfailures.json

diff --git a/test/knownfailures.json b/test/knownfailures.json
index 013272f..b057e69 100644
--- a/test/knownfailures.json
+++ b/test/knownfailures.json
@@ -1368,6 +1368,13 @@
         "description": ["Slow test. Prone to timeouts."]
     },
     {
+        "tests": ["2043-reference-pauses"],
+        "env_vars": {"ART_USE_READ_BARRIER": "false"},
+        "variant": "host",
+        "bug": "b/232459100",
+        "description": ["Fails intermittently for CMS."]
+    },
+    {
         "tests": ["096-array-copy-concurrent-gc"],
         "variant": "gcstress & debuggable & debug & host",
         "bug": "b/149708943",