On Tue, 11 Aug 2026 07:07:39 GMT, sendaoYan <[email protected]> wrote:

>> Shiv Shah has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Restore the shared lock and problem list the test for virtual threads
>
> test/hotspot/jtreg/vmTestbase/nsk/jdb/suspend/suspend001/suspend001a.java 
> line 64:
> 
>> 62:         synchronized (lockSuspended) {
>> 63:             synchronized (lockMyThread) {
>> 64:             synchronized (waitnotify) {
> 
> should we consistently indent the third synchronized to make the lock 
> ordering visually clear

gone with the latest commit, that nesting was part of the dropped two lock 
change

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

PR Review Comment: https://git.openjdk.org/jdk/pull/32267#discussion_r3788376315

Reply via email to