[jdk11u-dev] RFR: 8303861: Error handling step timeouts should never be blocked by OnError and others

[Goetz Lindenmaier]

On Fri, 5 May 2023 10:24:07 GMT, Alexey Pavlyutkin <duke at openjdk.org> wrote:

> HI!
> 
> Here is backport of JDK-8303861. The patch fixes error reporting timeout processing for the case when a user specifies OnError hook. Before the patch in this case JVM just ignored timeout and therefore WatcherThread could not break malloc() deadlock in error reporting routine.
> 
> The patch from 17 applied with the only change: in `void WatcherThread::run()` is located in `thread.cpp` (`nonJavaThread.cpp` in 17).
> 
> Verification/regression (amd64/LTS20.04): test/hotspot/jtreg/runtime/ErrorHandling (includes updated test)

[11u]
[~apavlyutkin] @apavlyutkin are you sure you ran the test? I see compile errors in our CI tonight:
test/hotspot/jtreg/runtime/ErrorHandling/TimeoutInErrorHandlingTest.java:99: error: cannot find symbol
                "-XX:ErrorLogTimeout=" + ERROR_LOG_TIMEOUT,
                                         ^
  symbol: variable ERROR_LOG_TIMEOUT
  location: class TimeoutInErrorHandlingTest
1 error 

Please backout or fix the test  !!!

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

PR Comment: https://git.openjdk.org/jdk11u-dev/pull/1856#issuecomment-1541415196