RFR(L): Remove global safepoint code

[Robbin Ehn]

Thanks Coleen!

/Robbin

On 2020-03-20 12:35, coleen.phillimore at oracle.com wrote:
> I reviewed this code and had the same comment :)
> Everything else looked great.
> 
> Coleen
> 
> On 3/20/20 7:13 AM, Erik Österlund wrote:
>> Hi Robbin,
>>
>> I see. Maybe we can call it fetch_and_read_polling_page() instead.
>> Don't need another webrev for that if you agree; ship it!
>>
>> Thanks,
>> /Erik
>>
>> On 2020-03-20 12:10, Robbin Ehn wrote:
>>> Hi Erik,
>>>
>>> There was two read_polling_page:
>>>
>>> address read_polling_page(Register r, address page, 
>>> relocInfo::relocType rtype);
>>> address read_polling_page(Register r, relocInfo::relocType rtype);
>>>
>>> The first one took the global polling page as argument.
>>> Since we no longer have that argument, to separate the methods I add 
>>> get_ to the one that both gets and reads the thread polling page.
>>> While the other one just reads the polling page from the address in 
>>> the register is still called just read_.
>>>
>>> Understandable?
>>>
>>> On 2020-03-20 11:58, Erik Österlund wrote:
>>>> Hi Robbin,
>>>>
>>>> Was there any particular reason for changing the name of 
>>>> read_polling_page() on AArch64 to get_read_polling_page()?
>>>> It now sounds like tautology.
>>>>
>>>> Otherwise this looks good, and I really like all the red lines. Dead 
>>>> code is the best code.
>>>
>>> Great, thanks!
>>>
>>> /Robbin
>>>
>>>>
>>>> Thanks,
>>>> /Erik
>>>>
>>>> On 2020-03-19 13:44, Robbin Ehn wrote:
>>>>> Hi all, please build, test and review.
>>>>>
>>>>> We removed (obsoleted) the flag ThreadLocalHandshakes in JDK 14.
>>>>> Any platform defining THREAD_LOCAL_POLL can only use per thread poll.
>>>>> Since all platforms now define THREAD_LOCAL_POLL the global poll 
>>>>> code is dead.
>>>>> Let's remove it!
>>>>>
>>>>> Issue: https://bugs.openjdk.java.net/browse/JDK-8220051
>>>>> Changeset: 
>>>>> http://cr.openjdk.java.net/~rehn/8220051/v1/webrev/index.html
>>>>>
>>>>> I have built fastdebug:
>>>>> linux-aarch64
>>>>> linux-arm32
>>>>> linux-ppc64le
>>>>> linux-s390x
>>>>> linux-x86_64-zero
>>>>> linux-x86_64 (no PCH)
>>>>> linux-x86_64 (slowdebug) (no PCH)
>>>>> linux-x86
>>>>> osx-x86_64
>>>>> windows-x86_64
>>>>> solaris-sparcv9
>>>>>
>>>>> Tested t1-5:
>>>>> linux-x86_64
>>>>> osx-x86_64
>>>>> windows-x86_64
>>>>> solaris-sparcv9
>>>>>
>>>>> Manually:
>>>>> linux-aarch64
>>>>> linux-x86_64-zero
>>>>>
>>>>> Any extra help building and testing is appreciated, especially on 
>>>>> arm32, ppc64le, s390x.
>>>>>
>>>>> Thanks, Robbin
>>>>
>>
>