RFR: 8300708: Some nsk jvmti tests fail with virtual thread wrapper due to jvmti missing some virtual thread support [v3]
Serguei Spitsyn
sspitsyn at openjdk.org
Tue Jan 28 14:08:26 UTC 2025
On Tue, 28 Jan 2025 12:54:31 GMT, Serguei Spitsyn <sspitsyn at openjdk.org> wrote:
>> test/hotspot/jtreg/vmTestbase/nsk/jvmti/SetLocalVariable/setlocal001/setlocal001.cpp line 78:
>>
>>> 76: err = jvmti_env->SetLocalLong(thr, 1,
>>> 77: table[i].slot, longVal);
>>> 78: if (err != JVMTI_ERROR_NONE && !(is_virtual && err == JVMTI_ERROR_OPAQUE_FRAME)) {
>>
>> Maybe add a new API called something `invalidError(err)`. Not necessary, but might be a little better.
>
> Thanks. I was also thinking about this but was not sure it is worth it.
> Will update now as there are two of us now. :)
Refactored now.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23314#discussion_r1932229188
More information about the serviceability-dev
mailing list