RFR: 8151556: Use the PreservedMarks* classes for the G1 preserved mark stacks

Thomas Schatzl thomas.schatzl at oracle.com
Tue Apr 19 15:29:11 UTC 2016


Hi,

On Wed, 2016-04-13 at 09:06 -0400, Tony Printezis wrote:
> Hi all,
> 
> First, thanks to Thomas for pushing the patch for 8152312. Here’s a
> new webrev with the refactored G1 changes:
> 
> http://cr.openjdk.java.net/~tonyp/8151556/webrev.1/
> 

  looks good to me. Testing (jprt, vm.gc testlist) looks fine.

> Related question: Does G1 report on the GC log that an evacuation
> failure has occurred, the same way all the other GCs report promotion
> failures?
> 
> [6.319s][info][gc,promotion   ] Promotion failed
> 
> Should I add an extra message as part of this change?

G1 reports promotion failure by indicating this in the GC kind message
(like "Pause Young (to-space-exhausted)").

It would be nice to add this message to get consistent output for all
gcs though.

Thanks,
  Thomas



More information about the hotspot-gc-dev mailing list