RFR: 8252141: Rename G1YoungRemSetSamplingThread to better reflect its purpose

stefan.johansson at oracle.com stefan.johansson at oracle.com
Wed Aug 26 07:18:10 UTC 2020


Hi Kim,

Thanks for reviewing.

On 2020-08-25 04:44, Kim Barrett wrote:
>> On Aug 24, 2020, at 5:36 PM, stefan.johansson at oracle.com wrote:
>>
>> Hi,
>>
>> Please review this rename of the G1 Young RemSet Sampling thread.
>>
>> Webrev: http://cr.openjdk.java.net/~sjohanss/8252141/00/
>> JBS: https://bugs.openjdk.java.net/browse/JDK-8252141
>>
>> Summary
>> The G1YoungRemSetSamplingThread does more than just remset sampling, it is also responsible for the scheduling of periodic GCs. Going forward even more taslks will likely be added to this thread and it should be renamed. The proposal in this patch is to rename it to G1ServiceThread.
>>
>> It also includes renaming some members and other related functions.
>>
>> Testing
>> Mach5 tier1-3
>>
>> Thanks,
>> Stefan
> 
> Generally looks good.
> 
> src/hotspot/share/gc/g1/g1ServiceThread.hpp
>    30 // The G1ServiceThread is used to do a number of different task:
> 
> Pluralize "task"
> 
> Also add "periodically", as in "used to periodically do".
> 
Fixed.

> I also noticed that G1ConcRefinementServiceIntervalMillis is kind of
> misnamed and described, though with the change to "ServiceThread"
> naming it's actually less poorly named than previously.
> 
> Changing the name is a bit of a pain, since it's a product option.
> But the description should be fixed.
> 
Nice catch, updated in:
Full: http://cr.openjdk.java.net/~sjohanss/8252141/01/
Inc: http://cr.openjdk.java.net/~sjohanss/8252141/00-01/

I also filed a issue to take care of the name at some point:
https://bugs.openjdk.java.net/browse/JDK-8252358

Thanks,
Stefan


More information about the hotspot-gc-dev mailing list