[jdk11u-dev] RFR: 8307811: [TEST] compilation of TimeoutInErrorHandlingTest fails after backport of JDK-8303861

[Paul Hohensee]

On Wed, 10 May 2023 11:42:25 GMT, Alexey Pavlyutkin <duke at openjdk.org> wrote:

> Hi!
> 
> This one fixes compilation of 
> 
> runtime/ErrorHandling/ThreadsListHandleInErrorHandlingTest.java
> 
> broken by backport of JDK-8303861. The problem is about backported delta refers to ERROR_LOG_TIMEOUT member that was introduced by different changeset and so was not backported and it stayed uncovered because the test is selected only for debug VM build
> 
> Verification/regression (amd64/20.04LTS): 
> 
> alex at alex-VirtualBox:~/jdk11u-dev$ /opt/jtreg/bin/jtreg -v -jdk:./build/linux-x86_64-normal-server-fastdebug/images/jdk test/hotspot/jtreg/runtime/ErrorHandling
> runner starting test: runtime/ErrorHandling/BadNativeStackInErrorHandlingTest.java
> runner finished test: runtime/ErrorHandling/BadNativeStackInErrorHandlingTest.java
> Passed. Execution successful
> runner starting test: runtime/ErrorHandling/CreateCoredumpOnCrash.java
> runner finished test: runtime/ErrorHandling/CreateCoredumpOnCrash.java
> Passed. Execution successful
> runner starting test: runtime/ErrorHandling/ErrorFileRedirectTest.java
> runner finished test: runtime/ErrorHandling/ErrorFileRedirectTest.java
> Passed. Execution successful
> runner starting test: runtime/ErrorHandling/ErrorHandler.java
> runner finished test: runtime/ErrorHandling/ErrorHandler.java
> Passed. Execution successful
> runner starting test: runtime/ErrorHandling/NestedThreadsListHandleInErrorHandlingTest.java
> runner finished test: runtime/ErrorHandling/NestedThreadsListHandleInErrorHandlingTest.java
> Passed. Execution successful
> runner starting test: runtime/ErrorHandling/ProblematicFrameTest.java
> runner finished test: runtime/ErrorHandling/ProblematicFrameTest.java
> Passed. Execution successful
> runner starting test: runtime/ErrorHandling/SafeFetchInErrorHandlingTest.java
> runner finished test: runtime/ErrorHandling/SafeFetchInErrorHandlingTest.java
> Passed. Execution successful
> runner starting test: runtime/ErrorHandling/SecondaryErrorTest.java
> runner finished test: runtime/ErrorHandling/SecondaryErrorTest.java
> Passed. Execution successful
> runner starting test: runtime/ErrorHandling/ShowRegistersOnAssertTest.java
> runner finished test: runtime/ErrorHandling/ShowRegistersOnAssertTest.java
> Passed. Execution successful
> runner starting test: runtime/ErrorHandling/TestCrashOnOutOfMemoryError.java
> runner finished test: runtime/ErrorHandling/TestCrashOnOutOfMemoryError.java
> Passed. Execution successful
> runner starting test: runtime/ErrorHandling/TestExitOnOutOfMemoryError.java
> runner finished test: runtime/ErrorHandling/TestExitOnOutOfMemoryError.java
> Passed. Execution successful
> runner starting test: runtime/ErrorHandling/TestHeapDumpOnOutOfMemoryError.java
> runner finished test: runtime/ErrorHandling/TestHeapDumpOnOutOfMemoryError.java
> Passed. Execution successful
> runner starting test: runtime/ErrorHandling/TestHeapDumpOnOutOfMemoryErrorInMetaspace.java
> runner finished test: runtime/ErrorHandling/TestHeapDumpOnOutOfMemoryErrorInMetaspace.java
> Passed. Execution successful
> runner starting test: runtime/ErrorHandling/TestHeapDumpPath.java
> runner finished test: runtime/ErrorHandling/TestHeapDumpPath.java
> Passed. Execution successful
> runner starting test: runtime/ErrorHandling/TestOnError.java
> runner finished test: runtime/ErrorHandling/TestOnError.java
> Passed. Execution successful
> runner starting test: runtime/ErrorHandling/TestOnOutOfMemoryError.java
> runner finished test: runtime/ErrorHandling/TestOnOutOfMemoryError.java
> Passed. Execution successful
> runner starting test: runtime/ErrorHandling/ThreadsListHandleInErrorHandlingTest.java
> runner finished test: runtime/ErrorHandling/ThreadsListHandleInErrorHandlingTest.java
> Passed. Execution successful
> runner starting test: runtime/ErrorHandling/TimeoutInErrorHandlingTest.java#default
> runner finished test: runtime/ErrorHandling/TimeoutInErrorHandlingTest.java#default
> Passed. Execution successful
> runner starting test: runtime/ErrorHandling/TimeoutInErrorHandlingTest.java#with-on-error
> runner finished test: runtime/ErrorHandling/TimeoutInErrorHandlingTest.java#with-on-error
> Passed. Execution successful
> runner starting test: runtime/ErrorHandling/VeryEarlyAssertTest.java
> runner finished test: runtime/ErrorHandling/VeryEarlyAssertTest.java
> Passed. Execution successful
> Test results: passed: 20
> Report written to /home/alex/jdk11u-dev/JTreport/html/report.html
> Results written to /home/alex/jdk11u-dev/JTwork
> alex at alex-VirtualBox:~/jdk11u-dev$

Is the different changeset worth backporting on its own, rather than doing this patch?

-------------

PR Review: https://git.openjdk.org/jdk11u-dev/pull/1882#pullrequestreview-1420764031