RFR: 8154153: PS: Restore preserved marks in parallel
Thomas Schatzl
thomas.schatzl at oracle.com
Thu Apr 21 11:01:29 UTC 2016
Hi Tony,
On Wed, 2016-04-13 at 09:24 -0400, Tony Printezis wrote:
> Follow-up change to do preserved mark restoration in parallel in PS:
>
> http://cr.openjdk.java.net/~tonyp/8154153/webrev.0/
>
> I also changed the “Promotion failed” log message from:
> log_info(gc)("Promotion failed”);
> to:
> log_info(gc, promotion)("Promotion failed");
> to be consistent with the other GCs.
>
- renaming of ParRestoreTask to ParRestoreGangTask: no other
AbstractGangTask child class has "Gang" in its name.
I can kind of see the name clashes with "ParRestoreGCTask", but I do
not think it really helps.
Feel free to ignore this, but maybe thinking about this again yields
some better names.
- preservedMarks.cpp needs a memory/resourceArea.hpp include due to the
use of ResourceMark.
- preservedMarks.cpp: line 118, maybe an extra CR makes the code look
less cramped.
- preservedMarks.cpp: line 128: style: if argument list needs to be
split across multiple lines, we (in the gc team) favor one argument per
line.
- some copyrights need updates
I will run it through jprt/some tests.
Thanks,
Thomas
More information about the hotspot-gc-dev
mailing list