RFR: 8364004: Add an option for delayed crash tests
David Holmes
dholmes at openjdk.org
Thu Jul 24 23:59:54 UTC 2025
On Thu, 24 Jul 2025 08:02:00 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:
> Pretty simple small addition to make `-XX:ErrorHandlerTest` work with an optional delay. This is a preparatory patch for some future planned refinements/additions to the error handler tests. It makes it possible to let the JVM warm up for a second or three, then to crash, in order to then observe an error log from a warmed-up JVM.
Not sure I see the point of this being in the VM. First you are now going to trigger crashes in the WatcherThread instead of the main thread - not sure if that changes what you want to see. Second seems easier to just have a WhiteBox test that triggers different kinds of crashes after initialization.
If you want to trigger crashes at arbitrary times during initialization then I have no doubt you will find issues with crash reporting and it will be totally timing related and would make the test quite unstable.
-------------
PR Review: https://git.openjdk.org/jdk/pull/26453#pullrequestreview-3053672357
More information about the hotspot-dev
mailing list