On Sat, 19 Apr 2025 02:25:33 GMT, Leonid Mesnik <lmes...@openjdk.org> wrote:

>> The
>> CheckUnhandledOops
>> cause failure if JvmtiExport::post_resource_exhausted(...)
>> is called in
>> MemAllocator::Allocation::check_out_of_memory()
>> The obj is null so it is not a real bug. 
>> 
>> I am fixing it to reduce noise for CheckUnhandledOops mode for jvmti tests 
>> execution.
>> The 
>> vmTestbase/nsk/jvmti/ResourceExhausted/resexhausted002/TestDescription.java 
>> failed with -XX:+CheckUnhandledOops
>> 
>> If define are unwelcome here, the
>> ``` PreserveObj obj_h(_thread, _obj_ptr);```
>> might be added instead with comment why it is needed for null obj.
>
> Leonid Mesnik has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   typo fixes

Looks okay to me.

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

Marked as reviewed by sspitsyn (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/24766#pullrequestreview-2785885049

Reply via email to