RFR(s): 8235410: Enable handshakes on Linux x86 (32-bit)

Robbin Ehn robbin.ehn at oracle.com
Tue Dec 10 07:44:28 UTC 2019


Hi Andrew,

Could you have a look please?

Thanks, Robbin

On 2019-12-05 15:01, Robbin Ehn wrote:
> Hi all, please review.
> 
> The flag ThreadLocalHandshakes is going to be obsolete in JDK 14.
> So we change the (unchangeable) default for Linux x86 to on/true.
> 
> There is a follow-up which removes ThreadLocalHandshakes completely.
> If the platform defines THREAD_LOCAL_POLL it will use handshakes.
> When arm32 have implemented local polls, the plan is to remove the global poll
> code paths.
> 
> Last time I checked with some of the affected parties there was no objections.
> 
> Built and sanity test.
> 
> Issue:
> https://bugs.openjdk.java.net/browse/JDK-8235410
> 
> Code below.
> 
> Thanks, Robbin
> 
> diff -r 636d71e53732 src/hotspot/cpu/x86/globals_x86.hpp
> --- a/src/hotspot/cpu/x86/globals_x86.hpp    Wed Dec 04 10:26:32 2019 +0100
> +++ b/src/hotspot/cpu/x86/globals_x86.hpp    Thu Dec 05 14:13:57 2019 +0100
> @@ -89,12 +89,7 @@
> 
>   define_pd_global(intx, InitArrayShortSize, 8*BytesPerLong);
> 
> -#if defined(_LP64) || defined(_WINDOWS)
>   define_pd_global(bool, ThreadLocalHandshakes, true);
> -#else
> -// get_thread() is slow on linux 32 bit, therefore off by default
> -define_pd_global(bool, ThreadLocalHandshakes, false);
> -#endif
> 
>   #define ARCH_FLAGS(develop, \
>                      product, \


More information about the hotspot-dev mailing list