<div dir="ltr"><div dir="ltr"><div>Hi,</div><div><br></div><div>This is the webrev for ReduceSpinningInGCThreadTermination, our local feature for the ISMM paper:</div><div dir="ltr"><a href="https://cr.openjdk.java.net/~manc/ReduceSpinningInGCThreadTermination/webrev.00/">https://cr.openjdk.java.net/~manc/ReduceSpinningInGCThreadTermination/webrev.00/</a></div><div dir="ltr">Note that I have to create this webrev against revision d2f118d3f8e7, the revision immediately before the Shenandoah terminator's revision, in order to avoid conflicts.</div><div dir="ltr"><br></div><div dir="ltr"><div><div dir="ltr" class="gmail_signature"><div dir="ltr">-Man</div></div></div><br></div></div></div><br><div class="gmail_quote"><div dir="ltr">On Sat, Dec 8, 2018 at 3:35 AM Wessam Hassanein <<a href="mailto:wessam@google.com">wessam@google.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">Hi All,<div><br></div><div>We are glade to see this work and can provide our implementation which is used in the paper for your reference. The work was initially created for the CMS GC and then ported to G1. We have recently forward ported our work to JDK11. We have not compared codewise to this work but feel free to do so and use any/all you see fit.  I have asked Man to generate a webrev and post it here.</div><div><br></div><div>Thanks,</div><div><br></div><div>Wessam Hassanein</div></div><br><div class="gmail_quote"><div dir="ltr">On Fri, Dec 7, 2018 at 6:02 AM <<a href="mailto:zgu@redhat.com" target="_blank">zgu@redhat.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">Thanks, Thomas and Roman.<br>
<br>
-Zhengyu<br>
<br>
On Fri, 2018-12-07 at 14:58 +0100, Roman Kennke wrote:<br>
> > > On Thu, 2018-12-06 at 08:56 -0500, <a href="mailto:zgu@redhat.com" target="_blank">zgu@redhat.com</a> wrote:<br>
> > > > > Looks good otherwise. I do not need a re-review for above<br>
> > > > > change.<br>
> > > > > I<br>
> > > > > will have this final change run through our testing one more<br>
> > > > > time<br>
> > > > > and will report back (tomorrow).<br>
> > > > > <br>
> > > > <br>
> > > > Awesome! In the meanwhile, I will run the latest through<br>
> > > > submit.<br>
> > > <br>
> > > Hs-tier1-7,jdk-tier1-3 of the final patch look good.<br>
> > > <br>
> > > One final note: could you update the reference to Google's paper<br>
> > > to<br>
> > > also show authors and other publication info?<br>
> > > <br>
> > > Something like the ACM ref on the ACM page so that people<br>
> > > preferring<br>
> > > paper can more easily find it, e.g.<br>
> > > <br>
> > > "Wessam Hassanein. 2016. Understanding and improving JVM GC work<br>
> > > stealing at the data center scale. In Proceedings of the 2016 ACM<br>
> > > SIGPLAN International Symposium on Memory Management (ISMM 2016).<br>
> > > ACM,<br>
> > > New York, NY, USA, 46-54. DOI:<br>
> > > <a href="https://doi.org/10.1145/2926697.2926706" rel="noreferrer" target="_blank">https://doi.org/10.1145/2926697.2926706</a>"<br>
> > > <br>
> > > No need for a re-review for me if you change it.<br>
> > <br>
> > Certainly, I will make the change as you suggested.<br>
> > <br>
> > <br>
> > Also, the latest submit passed. Now, I need a second reviewer and<br>
> > CSR<br>
> > approval to push, right?<br>
> <br>
> Patch looks good. Please wait for CSR approval.<br>
> <br>
> Thanks,<br>
> Roman<br>
> <br>
</blockquote></div>
</blockquote></div>