RFR(L): Remove global safepoint code
Erik Österlund
erik.osterlund at oracle.com
Fri Mar 20 11:13:21 UTC 2020
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
>>
More information about the hotspot-dev
mailing list