<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;">Thomas,</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;">I’ll fix the initializer list, give me a few mins and I’ll post a new webrev.</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 April 22, 2016 at 9:01:16 AM, 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>

<br>On Thu, 2016-04-21 at 08:35 -0400, Tony Printezis wrote:

<br>> Thomas,

<br>>  

<br>> New webrev:

<br>>  

<br>> http://cr.openjdk.java.net/~tonyp/8154153/webrev.1/

<br>>  

<br>> On April 21, 2016 at 8:08:28 AM, Tony Printezis (tprintezis@twitter.c

<br>> om) wrote:

<br>> > Thomas,

<br>> >  

> > Thanks for looking at it. Inline.

<br>> >  

<br>> > On April 21, 2016 at 7:01:39 AM, Thomas Schatzl (thomas.schatzl@ora

<br>> > cle.com) wrote:

<br>> > > Hi Tony, 

<br>> > >  

<br>> > > On Wed, 2016-04-13 at 09:24 -0400, Tony Printezis wrote: 

<br>> > > > Follow-up change to do preserved mark restoration in parallel

<br>> > > in PS: 

<br>> > > > 

<br>> > > > http://cr.openjdk.java.net/~tonyp/8154153/webrev.0/ 

<br>> > > > 

<br>> > > > I also changed the “Promotion failed” log message from: 

<br>> > > > log_info(gc)("Promotion failed”); 

<br>> > > > to: 

<br>> > > > log_info(gc, promotion)("Promotion failed"); 

<br>> > > > to be consistent with the other GCs. 

<br>> > > > 

<br>> > >  

<br>> > > - renaming of ParRestoreTask to ParRestoreGangTask: no other 

<br>> > > AbstractGangTask child class has "Gang" in its name. 

<br>> > >  

<br>> > > I can kind of see the name clashes with "ParRestoreGCTask", but I

<br>> > > do 

<br>> > > not think it really helps. 

> > Yeah, I renamed it to differentiate it a bit from ParRestoreGCTask.

<br>> > No strong opinion here. If you’re happy with ParRestoreTask /

> > ParRestoreGCTask we can go with that.

<br>

Either way is fine with me too. Let's keep your suggestion.

<br>

<br>> > >  

<br>> > > - preservedMarks.cpp: line 118, maybe an extra CR makes the code

<br>> > > look 

<br>> > > less cramped. 

> > Between the name() and do_it() methods?

<br>> >  

<br>

Yes.

<br>

<br>> > >  

<br>> > > - preservedMarks.cpp: line 128: style: if argument list needs to

<br>> > > be 

<br>> > > split across multiple lines, we (in the gc team) favor one

<br>> > > argument per 

<br>> > > line. 

<br>> > Sure, sounds good and I’ll keep that in mind in the future. Both at

<br>> > the method declaration and definition?  

<br>

Both.

<br>

> > Does that also apply to the initializer list?

<br>

Yes.

<br>

<br>> > > - some copyrights need updates 

> > Will do. I’ll post a new version shortly.

<br>

<br>The change above looks good. I can fix the initializer list if you

<br>don't just update the webrev in place :)

<br>

<br>Thanks,

<br>  Thomas

<br></div></div></span></blockquote> <div id="bloop_sign_1461333109666994176" 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>