RFR: 8048269: Add flag to turn off class unloading after G1 concurrent mark

[Bengt Rutisson]

On 2014-08-04 16:07, Stefan Karlsson wrote:
> Hi all,
>
> On 2014-07-18 00:14, Jon Masamitsu wrote:
>>
>> On 07/17/2014 12:25 AM, Thomas Schatzl wrote:
>>> Hi,
>>>
>>> On Thu, 2014-07-17 at 08:38 +0200, Bengt Rutisson wrote:
>>>> On 2014-07-17 07:02, Kirk Pepperdine wrote:
>>>>>> I don't care for ClassUnloadingAfterConcurrentMark because 
>>>>>> turning it
>>>>>> off makes me think that class unloading is going to occur "Before"
>>>>>> concurrent mark.  How about
>>>>>>
>>>>>> ClassUnloadingWithConcurrentMark
>>>>> +1
>>>> I'm fine with this one too.
>>> I would suggest ClassUnloadingAtConcurrentMark, because it seems more
>>> concise, but either is okay.
>>>
>>> That would be similar to PrintHeapAtGC and so on. :)
>>
>> The flag sounds more concise but is it that correct?   With CMS
>> at least (which we hope to make use this same flag) some
>> of the reduction in work would be during marking but the
>> freeing of the metaspace and the cleaning of the system
>> dictionary come later (after the sweep).  I still like
>> ClassUnloadingWithConcurrentMark - not as precise but
>> maybe more correct?
>
> I'm fine with this flag name.
>
> Does anyone strongly oppose the flag name: 
> ClassUnloadingWithConcurrentMark?

No objections from me.

Bengt

>
> thanks,
> StefanK
>
>>
>> Jon
>>
>>>
>>> Thomas
>>>
>>>
>>
>