RFR: 8295427: popframe004: report more details on error [v2]

Chris Plummer cjplummer at openjdk.org
Wed Oct 19 20:08:03 UTC 2022


On Wed, 19 Oct 2022 00:04:43 GMT, Alex Menkov <amenkov at openjdk.org> wrote:

>> Failures of the test are usually caused by unexpected frame in the stack.
>> Also the failures are intermittent, so it makes sense to report more details on error to simplify analysis.
>> The fix adds stack trace printing if the test fails
>
> Alex Menkov has updated the pull request incrementally with one additional commit since the last revision:
> 
>   merged serviceability/jvmti and nsk/share/jvmti shared code

I think the cleanup is disruptive enough that it should be a separate PR. Probably the easiest approach is to file a new CR for the cleanup and repurpose this PR for it, which means also moving the popframe004.cpp changes to a new PR.

test/hotspot/jtreg/vmTestbase/nsk/share/jni/jni_tools.h line 136:

> 134:  * Declared and defined in jvmti_common.h
> 135:  */
> 136: //char *jlong_to_string(jlong value, char *string);

It seems odd to have jlong_to_string and julong_to_string implemented in different places. Maybe more cleanup is needed here.

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

PR: https://git.openjdk.org/jdk/pull/10733


More information about the serviceability-dev mailing list