RFR(s): 8211403: Rename SafepointMechanism::poll(...)

Robbin Ehn robbin.ehn at oracle.com
Wed Nov 7 10:04:29 UTC 2018


Hi David,

On 2018-11-07 10:32, David Holmes wrote:
> Hi Robbin,
> 
> On 7/11/2018 6:32 PM, Robbin Ehn wrote:
>> Hi all, please review.
>>
>> We rename SafepointMechanism::poll to SafepointMechanism::should_block to make 
>> it more clear what the method do.
> 
> Fair enough, but doesn't that just imply that global_poll and local_poll are 
> also mis-named? How far should this rename go?

I think they are fine, because of the different context. As I see it
should_block is implemented by actively checking (polling) the threads page or
the global state.

Thanks, Robbin

> 
> Thanks,
> David
> 
>>
>> CR: https://bugs.openjdk.java.net/browse/JDK-8211403
>> Webrev: http://cr.openjdk.java.net/~rehn/8211403/webrev/
>>
>> Sanity on tier 1.
>>
>> Thanks, Robbin


More information about the hotspot-dev mailing list