On Wed, 16 Oct 2024 00:30:49 GMT, Alex Menkov <[email protected]> wrote:
>> Serguei Spitsyn has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> minor comment tweak
>
> test/hotspot/jtreg/serviceability/jvmti/events/NotifyFramePopStressTest/NotifyFramePopStressTest.java
> line 37:
>
>> 35: public class NotifyFramePopStressTest {
>> 36: static volatile boolean done = false;
>> 37: static volatile int notifyCount = 0;
>
> don't need to be volatile (actually it can be local in `control` method)
Good suggestion. Fixed.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/21468#discussion_r1802227945