[jdk17u-dev] RFR: 8283326: Implement SafeFetch statically [v3]

Thomas Stuefe stuefe at openjdk.org
Thu Jul 13 05:11:21 UTC 2023


On Wed, 12 Jul 2023 19:54:04 GMT, Sergey Nazarkin <snazarki at openjdk.org> wrote:

>> Hi! 
>> I'd like to backport this changeset to prevent a crash when AsyncProfiler is used on Mac M1. 
>> 
>> The backport is not clean:
>> 1. RISCV is not part of jdk17
>> 2. mallocTracker.cpp is not updated since 8281023 was not backported to jdk17
>> 3. vmError_posix.cpp requires manual update due to  8289350 is missed at jdk19+
>> 4. Need to add globals.hpp to fix builds without precompiled headers
>> 
>> There is follow up for this case: 
>> 1. 8284997: arm32 build crashes since JDK-8283326
>> 2. 8288719: [arm32] SafeFetch32 thumb interleaving causes random crashes
>
> Sergey Nazarkin has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Missed riscv assembler

Looks good.

When you do the fix request, please mention the two addon changes needed for arm32. I don't remember the procedure for downporting such change chains, but @GoeLin will know.

-------------

Marked as reviewed by stuefe (Reviewer).

PR Review: https://git.openjdk.org/jdk17u-dev/pull/1557#pullrequestreview-1527578565


More information about the jdk-updates-dev mailing list