<html><head><style>body{font-family:Helvetica,Arial;font-size:13px}</style></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;">Thanks Thomas! Did it go through JPRT OK?</div><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;"><br></div><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;">Tony</div> <br><p class="airmail_on">On February 17, 2016 at 1:27:52 PM, Thomas Schatzl (<a href="mailto:thomas.schatzl@oracle.com">thomas.schatzl@oracle.com</a>) wrote:</p> <blockquote type="cite" class="clean_bq"><span><div><div></div><div>Hi Tony,

<br>

sorry for the late reply.

<br>

<br>Change looks good to me. I will sponsor it after waiting a few more days

to push for other persons to have a look.

<br>

<br>Thanks,

<br>  Thomas

<br>

<br>On Mo, 2016-02-08 at 16:16 -0500, Tony Printezis wrote:

<br>> Thomas,

<br>>  

<br>>  

<br>> Latest changes:

<br>>  

<br>>  

<br>> http://cr.openjdk.java.net/~tonyp/8146989/webrev.4/

<br>>  

<br>>  

<br>> BTW, I moved do_object() to the .cpp file instead of the .inline.hpp

> 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>> (tprintezis@twitter.com) wrote:

<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>> > (thomas.schatzl@oracle.com) 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>> > >  

> > > 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

> > > 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>> > tprintezis@twitter.com

<br>> >  

<br>> >  

<br>> -----

<br>>  

<br>>  

<br>> Tony Printezis | JVM/GC Engineer / VM Team | Twitter

<br>>  

<br>>  

<br>> @TonyPrintezis

<br>> tprintezis@twitter.com

<br>>  

<br>>  

<br>

<br>

<br></div></div></span></blockquote> <div id="bloop_sign_1455733851590982912" class="bloop_sign"><div style="font-family:helvetica,arial;font-size:13px"><div>-----</div><div><br></div><div>Tony Printezis | JVM/GC Engineer / VM Team | Twitter</div><div><br></div><div>@TonyPrintezis</div><div><a href="mailto:tprintezis@twitter.com">tprintezis@twitter.com</a></div><div><br></div></div></div></body></html>