Re: [jdk22] RFR: 8321685: Missing ResourceMark in code called from JvmtiEnvBase::get_vthread_jvf

2024-01-12 Thread Serguei Spitsyn
On Fri, 12 Jan 2024 00:47:20 GMT, Serguei Spitsyn wrote: > Hi all, > > This pull request contains a clean backport of commit > [2806adee](https://github.com/openjdk/jdk/commit/2806adee2d8cca6bc215f285888631799bd02eac) > from the [openjdk/jdk](https://git.openjdk.org/jdk) repository. > > The c

Re: [jdk22] RFR: 8321685: Missing ResourceMark in code called from JvmtiEnvBase::get_vthread_jvf

2024-01-11 Thread Alex Menkov
On Fri, 12 Jan 2024 00:47:20 GMT, Serguei Spitsyn wrote: > Hi all, > > This pull request contains a clean backport of commit > [2806adee](https://github.com/openjdk/jdk/commit/2806adee2d8cca6bc215f285888631799bd02eac) > from the [openjdk/jdk](https://git.openjdk.org/jdk) repository. > > The c

[jdk22] RFR: 8321685: Missing ResourceMark in code called from JvmtiEnvBase::get_vthread_jvf

2024-01-11 Thread Serguei Spitsyn
Hi all, This pull request contains a clean backport of commit [2806adee](https://github.com/openjdk/jdk/commit/2806adee2d8cca6bc215f285888631799bd02eac) from the [openjdk/jdk](https://git.openjdk.org/jdk) repository. The commit being backported was authored by Serguei Spitsyn on 10 Jan

Re: RFR: 8321685: Missing ResourceMark in code called from JvmtiEnvBase::get_vthread_jvf [v2]

2024-01-10 Thread Serguei Spitsyn
> This fix adds a ResourceMark missing in the `SetFramePopClosure::do_thread` > and `SetFramePopClosure::do_vthread`. > > Testing: > - TBD to submit tiers: 1-5 Serguei Spitsyn has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the un

Re: RFR: 8321685: Missing ResourceMark in code called from JvmtiEnvBase::get_vthread_jvf

2024-01-10 Thread Serguei Spitsyn
On Tue, 9 Jan 2024 23:48:35 GMT, Serguei Spitsyn wrote: > This fix adds a ResourceMark missing in the `SetFramePopClosure::do_thread` > and `SetFramePopClosure::do_vthread`. > > Testing: > - TBD to submit tiers: 1-5 Alex and Chris, thank you for review! Chris, I'll add similar comments as you

Re: RFR: 8321685: Missing ResourceMark in code called from JvmtiEnvBase::get_vthread_jvf

2024-01-09 Thread Chris Plummer
On Tue, 9 Jan 2024 23:48:35 GMT, Serguei Spitsyn wrote: > This fix adds a ResourceMark missing in the `SetFramePopClosure::do_thread` > and `SetFramePopClosure::do_vthread`. > > Testing: > - TBD to submit tiers: 1-5 Looks good. Note, it would be good if the jvmti code added comments explainin

Re: RFR: 8321685: Missing ResourceMark in code called from JvmtiEnvBase::get_vthread_jvf

2024-01-09 Thread Alex Menkov
On Tue, 9 Jan 2024 23:48:35 GMT, Serguei Spitsyn wrote: > This fix adds a ResourceMark missing in the `SetFramePopClosure::do_thread` > and `SetFramePopClosure::do_vthread`. > > Testing: > - TBD to submit tiers: 1-5 Marked as reviewed by amenkov (Reviewer). - PR Review: https://

RFR: 8321685: Missing ResourceMark in code called from JvmtiEnvBase::get_vthread_jvf

2024-01-09 Thread Serguei Spitsyn
This fix adds a ResourceMark missing in the `SetFramePopClosure::do_thread` and `SetFramePopClosure::do_vthread`. Testing: - TBD to submit tiers: 1-5 - Commit messages: - 8321685: Missing ResourceMark in code called from JvmtiEnvBase::get_vthread_jvf Changes: https://git.openjdk