Re: RFR: 8296089: Remove debug agent code for special handling of Thread.resume() [v2]

2022-11-10 Thread Chris Plummer
On Thu, 10 Nov 2022 11:03:25 GMT, Serguei Spitsyn wrote: >> Chris Plummer has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Cleanup some suspect suspendOnStart and toBeResumed checks. > > src/jdk.jdwp.agent/share/native/libjdwp/threadContr

Re: RFR: 8296089: Remove debug agent code for special handling of Thread.resume() [v2]

2022-11-10 Thread Serguei Spitsyn
On Wed, 9 Nov 2022 05:57:32 GMT, Chris Plummer wrote: >> The debug agent sets a breakpoint in Thread.resume() so it can prevent the >> debugger from suspending threads while in the resume call: >> >> /* >> * Track the resuming thread by marking it as being within >>

Re: RFR: 8296089: Remove debug agent code for special handling of Thread.resume() [v2]

2022-11-10 Thread Serguei Spitsyn
On Wed, 9 Nov 2022 05:57:32 GMT, Chris Plummer wrote: >> The debug agent sets a breakpoint in Thread.resume() so it can prevent the >> debugger from suspending threads while in the resume call: >> >> /* >> * Track the resuming thread by marking it as being within >>

Re: RFR: 8296089: Remove debug agent code for special handling of Thread.resume() [v2]

2022-11-09 Thread Chris Plummer
On Wed, 9 Nov 2022 09:50:21 GMT, Serguei Spitsyn wrote: > I've started looking at this fix. > Wow, it is very big and nice cleanup! Fortunately it is pretty much entirely code deletion. The main exception is some cleanup of suspect suspendOnStart and toBeResumed checks. I'm guessing the were d

Re: RFR: 8296089: Remove debug agent code for special handling of Thread.resume() [v2]

2022-11-09 Thread Serguei Spitsyn
On Wed, 9 Nov 2022 05:57:32 GMT, Chris Plummer wrote: >> The debug agent sets a breakpoint in Thread.resume() so it can prevent the >> debugger from suspending threads while in the resume call: >> >> /* >> * Track the resuming thread by marking it as being within >>

Re: RFR: 8296089: Remove debug agent code for special handling of Thread.resume() [v2]

2022-11-08 Thread Chris Plummer
On Wed, 9 Nov 2022 05:57:32 GMT, Chris Plummer wrote: >> The debug agent sets a breakpoint in Thread.resume() so it can prevent the >> debugger from suspending threads while in the resume call: >> >> /* >> * Track the resuming thread by marking it as being within >>

Re: RFR: 8296089: Remove debug agent code for special handling of Thread.resume() [v2]

2022-11-08 Thread Chris Plummer
> The debug agent sets a breakpoint in Thread.resume() so it can prevent the > debugger from suspending threads while in the resume call: > > /* > * Track the resuming thread by marking it as being within > * a resume and by setting up for notification on