Looks good!

Since we want to implement this event for G1 as well, maybe we should remove the "CMS" from the event name and path?

Thanks,
/Jesper

On 1/3/13 6:34 PM, Kevin Walls wrote:
Hi,

I'd like some reviews on this CMS Concurrent Mode Failure event:

http://cr.openjdk.java.net/~kevinw/8008917/hotspot/

The event doesn't actually carry any new information, but it is a warning we
need to capture.

This is against hsx24, I'll prepare the same, or reviewed, changes against very
latest hotspot also.

Thanks
Kevin

Reply via email to