RFR: 8303276: Secondary assertion failure in AdapterHandlerLibrary::contains during crash reporting

David Holmes dholmes at openjdk.org
Tue Apr 18 12:50:43 UTC 2023


On Mon, 17 Apr 2023 18:57:50 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

> Don't check lock ownership if an error has been reported.
> Ran tier1-4 tests which include some intentionally crashing tests.

More specifically I think we need to evaluate on a case by case basis - so the check for `is_error_reported()` would be in the function with the `assert_locked_or_safepoint`. This is a bit messy syntactically.

#ifdef ASSERT
if (!VM_Error::is_error_reported()) assert_locked_or_safepoint(...);
#endif

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

PR Comment: https://git.openjdk.org/jdk/pull/13500#issuecomment-1513060796


More information about the hotspot-runtime-dev mailing list