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


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.


More information about the hotspot-gc-dev mailing list