8226706: (se) Reduce the number of outer loop iterations on Windows in java/nio/channels/Selector/RacyDeregister.java
Brian Burkhalter
brian.burkhalter at oracle.com
Mon Jun 24 23:03:38 UTC 2019
https://bugs.openjdk.java.net/browse/JDK-8226706
The number of outer loop iterations of this test was increased, for Windows only, by the patch that resolved [1]. A comment was also added to the test reminding to reset the number of iterations back down to its previous smaller value once [2] was resolved. Subsequently [2] was resolved as irreproducible but the number of iterations in the test was not in fact reduced. As the higher number of iterations might be the reason for the failure in [3], it is proposed here to change the number of outer loop resolutions from 150 on Windows back to 15, the same as on the other platforms.
Thanks,
Brian
--- a/test/jdk/java/nio/channels/Selector/RacyDeregister.java
+++ b/test/jdk/java/nio/channels/Selector/RacyDeregister.java
@@ -41,11 +41,6 @@
*/
public class RacyDeregister {
- // FIXME: NUM_OUTER_LOOP_ITERATIONS should be reverted to the hard-coded
- // value 15 when JDK-8161083 is resolved as either a bug or a non-issue.
- static final int NUM_OUTER_LOOP_ITERATIONS =
- System.getProperty("os.name").startsWith("Windows") ? 150 : 15;
-
// 90% of 1200 second timeout as milliseconds
static final int TIMEOUT_THRESHOLD_MILLIS = 1200*900;
@@ -90,7 +85,7 @@
public void run() {
try {
- for (int k = 0; k < NUM_OUTER_LOOP_ITERATIONS; k++) {
+ for (int k = 0; k < 15; k++) {
System.out.format("outer loop %3d at %7d ms%n", k,
System.currentTimeMillis() - t0);
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.java.net/pipermail/nio-dev/attachments/20190624/ec0fb03a/attachment.html>
More information about the nio-dev
mailing list