<div dir="ltr"><div>Hello again,<br><br>I'm struggling to understanding how the value can be something other than zero:<br><br></div>In the JBS ticket the error happened for (bsd-amd64) so I'm checking the code in :<br><br><div><a href="https://github.com/openjdk/jdk/blob/38a578d547f39c3637d97f5e0242f4a69f3bbb31/src/hotspot/cpu/x86/sharedRuntime_x86_64.cpp#L1624">https://github.com/openjdk/jdk/blob/38a578d547f39c3637d97f5e0242f4a69f3bbb31/src/hotspot/cpu/x86/sharedRuntime_x86_64.cpp#L1624</a></div><div><br></div><div>In that code, I see the call to thaw being done almost immediately after the call to `fill_continuation_entry` that sets the value of `held_monitor_count` to zero.<br><br></div><div>I'm guessing that it can be some kind of race condition, but I can't think of which other thread could be having an impact on the `held_monitor_count` for our JavaThread.<br><br>>> What command are you using to run the test?<br><br></div><div>Sorry Chris, I forgot to answer your question. I'm running the following command:<br><br>make run-test TEST=test/hotspot/jtreg/serviceability/jvmti/GetOwnedMonitorInfo/GetOwnedMonitorInfoTest.java JTREG_TEST_THREAD_FACTORY=Virtual JTREG_REPEAT_COUNT=100<br><br></div><div>Cheers</div><div><br></div><div>Íñigo<br></div><div><font color="#888888"></font></div><div><br><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Jul 16, 2024 at 11:01 AM Iñigo Mediavilla <<a href="mailto:imediava@gmail.com">imediava@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>Thanks Alan,<br><br></div>>> <span style="white-space:pre-wrap">Note that this code has changed significantly in the loom repo and the assert that is triggered in main line doesn't exist in the new code.
</span><div><br></div><div>My assumption is that:<br><br></div><div>- In the main line the assertion should hold because we don't unmount a virtual thread during blocking operations (we pin it to the carrier and park the thread)<br></div><div>- In loom the assertion has been removed because there are changes like commit (756743da) that will make it possible for a thread to be unmounted while holding a lock.<br></div><div><br></div><div>Can you confirm if this is correct ?<br></div><div><br>Íñigo<br><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Jul 13, 2024 at 10:27 AM Alan Bateman <<a href="mailto:Alan.Bateman@oracle.com" target="_blank">Alan.Bateman@oracle.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><u></u>

  
  <div>
    On 13/07/2024 08:07, Iñigo Mediavilla wrote:<br>
    <blockquote type="cite">
      
      <div dir="auto">
        <div>I see, in that case Serguei would you want to still own
          this JBS or would you be OK if I try to have a look at it ?</div>
        <br>
      </div>
    </blockquote>
    Debug build and <span style="white-space:pre-wrap">JTREG_TEST_THREAD_FACTORY=Virtual. Note that this code has changed significantly in the loom repo and the assert that is triggered in main line doesn't exist in the new code.

-Alan.
</span>
  </div>

</blockquote></div>
</blockquote></div>