RFR (S): 8006954: GC Cause equals No GC for CMS background collection in the trace GC event

[Erik Helin]

Thanks!

Erik

On 02/13/2013 10:14 AM, Stefan Karlsson wrote:
> On 02/07/2013 07:11 PM, Erik Helin wrote:
>> Based on some internal feedback, I've updated the change. I've added a
>> new GCCause::Cause called _cms_concurrent_mark to be used as cause for
>> collect_in_background.
>
> We need to verify that the GC cause for collect_in_foreground is
> sensible, but this looks good for now.
>
> thanks,
> StefanK
>
>>
>> The new webrev can be found here:
>> http://cr.openjdk.java.net/~ehelin/8006954/webrev.01/
>>
>> Thanks,
>> Erik
>>
>> On 02/06/2013 06:08 PM, Erik Helin wrote:
>>> Hi all,
>>>
>>> this change makes sure that the "cause" field in the
>>> /vm/gc/collection/garbage_collection event is set for a CMS background
>>> collection. This is done by ensuring that the _cause field in
>>> GenCollectedHeap is set before registering the start of a GC event.
>>>
>>> Webrev:
>>> http://cr.openjdk.java.net/~ehelin/8006954/webrev.00/
>>>
>>> Bug:
>>> http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8006954
>>>
>>> Thanks,
>>> Erik
>>
>