8277350: runtime/jni/checked/TestPrimitiveArrayCriticalWithBadParam.java times out

[dcubed-ojdk]

A (trivial?) fix to keep runtime/jni/checked/TestPrimitiveArrayCriticalWithBadParam.java
from timing out on macosx-X64 Mach5 test machines. Also removes the test from the
ProblemList.txt file.

Tested locally on my MBP-13 to verify that the test no longer creates core files.
Will also test with Mach5 Tier[135] to test for "timeouts in execution" task failures.

---

Progress

• Change must not contain extraneous whitespace
• Commit message must refer to an issue
• Change must be properly reviewed

Issue

• JDK-8277350: runtime/jni/checked/TestPrimitiveArrayCriticalWithBadParam.java times out

Reviewers

• Harold Seigel (@hseigel - Reviewer)
• David Holmes (@dholmes-ora - Reviewer)
• Leonid Mesnik (@lmesnik - Reviewer)

Reviewing

Using git

Checkout this PR locally:
$ git fetch https://git.openjdk.java.net/jdk pull/6510/head:pull/6510
$ git checkout pull/6510

Update a local copy of the PR:
$ git checkout pull/6510
$ git pull https://git.openjdk.java.net/jdk pull/6510/head

Using Skara CLI tools

Checkout this PR locally:
$ git pr checkout 6510

View PR using the GUI difftool:
$ git pr show -t 6510

Using diff file

Download this PR as a diff file:
https://git.openjdk.java.net/jdk/pull/6510.diff

[dcubed-ojdk]

/label add hotspot-runtime

[bridgekeeper]

👋 Welcome back dcubed! A progress list of the required criteria for merging this PR into master will be added to the body of your pull request. There are additional pull request commands available for use with this pull request.

[openjdk]

@dcubed-ojdk
The hotspot-runtime label was successfully added.

[mlbridge]

Webrevs

• 00: Full (eaf87d89)

[hseigel]

Changes look good and trivial.
Thanks, Harold

[openjdk]

@dcubed-ojdk This change now passes all automated pre-integration checks.

ℹ️ This project also has non-automated pre-integration requirements. Please see the file CONTRIBUTING.md for details.

After integration, the commit message for the final commit will be:

8277350: runtime/jni/checked/TestPrimitiveArrayCriticalWithBadParam.java times out

Reviewed-by: hseigel, dholmes, lmesnik

You can use pull request commands such as /summary, /contributor and /issue to adjust it as needed.

At the time when this comment was updated there had been 3 new commits pushed to the master branch:

• bb11c55: 8277542: G1: Move G1CardSetFreePool and related classes to separate files
• f62b81c: 8273095: vmTestbase/vm/mlvm/anonloader/stress/oome/heap/Test.java fails with "wrong OOME"
• 05a9a51: 8277423: ciReplay: hidden class with comment expected error

Please see this link for an up-to-date comparison between the source branch of this pull request and the master branch.
As there are no conflicts, your changes will automatically be rebased on top of these commits when integrating. If you prefer to avoid this automatic rebasing, please check the documentation for the /integrate command for further details.

➡️ To integrate this PR with the above commit message to the master branch, type /integrate in a new comment.

[dcubed-ojdk]

@hseigel - Thanks for the fast review! I'm going to wait until I have Mach5 Tier[1-3]
test results just to have some level of assurance that I don't reintroduce a "timeouts
in execution" test task failure mode.

[dholmes-ora]

LGTM2 :)

[dcubed-ojdk]

@dholmes-ora and @lmesnik - Thanks for the reviews!

[dcubed-ojdk]

Mach5 Tier[13] testing had no failures. Mach5 Tier5 testing had
19 unrelated client test failures and one unrelated zero build failure.

So no sign of the "timeouts in execution" test task failure mode, but that's
just one set of jobs. Only more test runs will really tell the tale...

/integrate

[openjdk]

Going to push as commit e8acac2.
Since your change was applied there have been 16 commits pushed to the master branch:

• 8a44e09: 8268725: jshell does not support the --enable-native-access option
• 7b67a49: 8261847: performance of java.lang.Record::toString should be improved
• 38802ad: 8254108: ciReplay: Support incremental inlining
• 64bdc84: 8277649: [BACKOUT] JDK-8277507 Add jlink.debug system property while launching jpackage tests to help diagonize recent intermittent failures
• 5e7e2da: 8272042: java.util.ImmutableCollections$Map1 and MapN should not be @valuebased
• f4dc03e: 8276696: ParallelObjectIterator freed at the wrong time in VM_HeapDumper
• 90f96fb: 8277441: CompileQueue::add fails with assert(_last->next() == __null) failed: not last
• 66eaf65: 8277413: Remove unused local variables in jdk.hotspot.agent
• 36b887a: 8277560: Remove WorkerDataArray::_is_serial
• 017df14: 8277556: Call ReferenceProcessorPhaseTimes::set_processing_is_mt once
• ... and 6 more: https://git.openjdk.java.net/jdk/compare/1049aba1fb65fd70bd723c80a84250512a68d653...master

Your commit was automatically rebased without conflicts.

[openjdk]

@dcubed-ojdk Pushed as commit e8acac2.

💡 You may see a message that your pull request was closed with unmerged commits. This can be safely ignored.