<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <br>
    <br>
    <div class="moz-cite-prefix">On 10/18/18 12:27 AM, David Holmes
      wrote:<br>
    </div>
    <blockquote type="cite"
      cite="mid:ac6fdd39-d2ec-7578-bccc-bb32a22ae4ec@oracle.com">Hi
      Dean,
      <br>
      <br>
      On 18/10/2018 2:06 PM, <a class="moz-txt-link-abbreviated" href="mailto:dean.long@oracle.com">dean.long@oracle.com</a> wrote:
      <br>
      <blockquote type="cite">
        <br>
        You're right, I missed that.  I think the right thing to do is
        call current_thread_exiting while holding the Threads_lock.
        <br>
        Then we can get rid of the parallel atomic counters.  So, here's
        one more try:
        <br>
        <br>
             <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~dlong/8021335/webrev.7/">http://cr.openjdk.java.net/~dlong/8021335/webrev.7/</a>
        <br>
      </blockquote>
      <br>
      Okay that is the simple and obvious solution that doesn't require
      split counts. So I have to ask Mandy if she recalls why this
      approach wasn't taken 15 years ago when the exit counts were added
      as part of:
      <br>
      <br>
    </blockquote>
    <br>
    It has been so long.  I think it's likely an oversight. <br>
    <blockquote type="cite"
      cite="mid:ac6fdd39-d2ec-7578-bccc-bb32a22ae4ec@oracle.com"><a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-4530538">https://bugs.openjdk.java.net/browse/JDK-4530538</a>
      ?
      <br>
      <br>
      Does taking the Threads_lock here cost too much and cause a thread
      termination bottleneck?
      <br>
    </blockquote>
    <br>
    If the contention on Threads_lock is not high (that seems to me), it
    should be okay.   I'm not close to the VM implementation (lot of
    changes since then) and I don't have a definitive answer unless I
    study the code closely.   You and others have a better judgement on
    this.<br>
    <br>
    AFAICT the change is okay.<br>
    <br>
    Mandy<br>
    <br>
    <br>
  </body>
</html>