<div dir="ltr">(as Thomas may have remarked earlier, some of the modified files need copyright date updated to 2016. Not sure if that is done nowadays at the time of making a modification to a file, or in a blanket sweep by release engineering at some future time -- as used to be the case in the hoary past at least. :)</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Feb 17, 2016 at 11:18 AM, Srinivas Ramakrishna <span dir="ltr"><<a href="mailto:ysr1729@gmail.com" target="_blank">ysr1729@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div>Reviewed! The changes look good to me. Looking forward to the related changes for the other collectors soon.</div><div><br></div><div>thanks,</div><div>-- ramki (openjdk: ysr)</div><div><br></div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Feb 17, 2016 at 10:27 AM, Thomas Schatzl <span dir="ltr"><<a href="mailto:thomas.schatzl@oracle.com" target="_blank">thomas.schatzl@oracle.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Tony,<br>
<br>
  sorry for the late reply.<br>
<br>
Change looks good to me. I will sponsor it after waiting a few more days<br>
to push for other persons to have a look.<br>
<br>
Thanks,<br>
  Thomas<br>
<div><div><br>
On Mo, 2016-02-08 at 16:16 -0500, Tony Printezis wrote:<br>
> Thomas,<br>
><br>
><br>
> Latest changes:<br>
><br>
><br>
> <a href="http://cr.openjdk.java.net/~tonyp/8146989/webrev.4/" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~tonyp/8146989/webrev.4/</a><br>
><br>
><br>
> BTW, I moved do_object() to the .cpp file instead of the .inline.hpp<br>
> file (it’s virtual, so it won’t be inlined anyway).<br>
><br>
><br>
> Tony<br>
><br>
> On February 8, 2016 at 2:56:56 PM, Tony Printezis<br>
> (<a href="mailto:tprintezis@twitter.com" target="_blank">tprintezis@twitter.com</a>) wrote:<br>
><br>
> ><br>
> > Thanks Thomas, I’ll fix those and post a new webrev shortly...<br>
> ><br>
> ><br>
> > Tony<br>
> ><br>
> > On February 8, 2016 at 10:56:56 AM, Thomas Schatzl<br>
> > (<a href="mailto:thomas.schatzl@oracle.com" target="_blank">thomas.schatzl@oracle.com</a>) wrote:<br>
> ><br>
> > > Hi,<br>
> > ><br>
> > > On Mon, 2016-02-08 at 09:49 -0500, Tony Printezis wrote:<br>
> > > > Thanks Thomas. Also, if you could do a JPRT run I’d very much<br>
> > > > appreciate it.<br>
> > ><br>
> > > that is of course part of it.<br>
> > ><br>
> > > It found two issues:<br>
> > > - RemoveForwardedPointerClosure::do_object() needs to be moved to<br>
> > > the<br>
> > > preservedMarks.inline.hpp as it uses oopDesc::is_forwarded() and<br>
> > > init(). Then also oop.inline.hpp needs to be included there.<br>
> > ><br>
> > > - PreservedMarksSet::reclaim() needs to call<br>
> > > ~Padded<Preservedmarks>(), not just ~PreservedMarks.<br>
> > ><br>
> > > ><br>
> > > > FWIW, I tested it with doing many runs of all the usual test<br>
> > > suites<br>
> > > > with -XX:+PromotionFailureALot -XX:PromotionFailureALotCount=150<br>
> > > > -XX:PromotionFailureALotInterval=2 and this cause a bunch of<br>
> > > > promotion failures. I didn’t see any issues.<br>
> > > ><br>
> > > > I also have the ParallelScavenge changes done (using the<br>
> > > abstractions<br>
> > > > introduced here). I’ll open those for code review when this is<br>
> > > done.<br>
> > ><br>
> > > Thanks,<br>
> > > Thomas<br>
> > ><br>
> > ><br>
> > ><br>
> > -----<br>
> ><br>
> ><br>
> > Tony Printezis | JVM/GC Engineer / VM Team | Twitter<br>
> ><br>
> ><br>
> > @TonyPrintezis<br>
> > <a href="mailto:tprintezis@twitter.com" target="_blank">tprintezis@twitter.com</a><br>
> ><br>
> ><br>
> -----<br>
><br>
><br>
> Tony Printezis | JVM/GC Engineer / VM Team | Twitter<br>
><br>
><br>
> @TonyPrintezis<br>
> <a href="mailto:tprintezis@twitter.com" target="_blank">tprintezis@twitter.com</a><br>
><br>
><br>
<br>
<br>
</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br></div>