Re: RFR: 8312174: missing JVMTI events from vthreads parked during JVMTI attach [v5]

2023-09-11 Thread Serguei Spitsyn
On Sat, 9 Sep 2023 18:05:51 GMT, Leonid Mesnik wrote: >> It is strange that the the tested vthreads in sleep(timeout) have sate >> WAITING, not TIMED_WAITING. >> It can be a bug in the implementation. >> I've decided to add a short sleep. Checking states looks a little bit over >> complicated.

Re: RFR: 8312174: missing JVMTI events from vthreads parked during JVMTI attach [v5]

2023-09-11 Thread Serguei Spitsyn
On Mon, 11 Sep 2023 22:18:03 GMT, Alex Menkov wrote: >> Serguei Spitsyn has updated the pull request incrementally with one >> additional commit since the last revision: >> >> addressed second round of review comments on VThreadEventTest.java > > test/hotspot/jtreg/serviceability/jvmti/vthrea

Re: RFR: 8312174: missing JVMTI events from vthreads parked during JVMTI attach [v5]

2023-09-11 Thread Serguei Spitsyn
On Mon, 11 Sep 2023 21:22:18 GMT, Serguei Spitsyn wrote: >> This update fixes two important issues: >> - Issue reported by a bug submitter about missing JVMTI events on virtual >> threads after an a JVMTI agent dynamic attach >> - Known scalability/performance issue: a need to lazily create

Re: RFR: 8312174: missing JVMTI events from vthreads parked during JVMTI attach [v5]

2023-09-11 Thread Alex Menkov
On Mon, 11 Sep 2023 21:22:18 GMT, Serguei Spitsyn wrote: >> This update fixes two important issues: >> - Issue reported by a bug submitter about missing JVMTI events on virtual >> threads after an a JVMTI agent dynamic attach >> - Known scalability/performance issue: a need to lazily create

Re: RFR: 8312174: missing JVMTI events from vthreads parked during JVMTI attach [v5]

2023-09-11 Thread Serguei Spitsyn
> This update fixes two important issues: > - Issue reported by a bug submitter about missing JVMTI events on virtual > threads after an a JVMTI agent dynamic attach > - Known scalability/performance issue: a need to lazily create > `JvmtiThreadState's` for virtual threads > > The issue is tr