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

Thomas Stuefe stuefe at openjdk.org
Wed Jul 12 16:19:19 UTC 2023


On Wed, 12 Jul 2023 15:10:03 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 with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:
> 
>  - RiscV part
>  - Fix builds without precompiled headers
>  - Backport bdf8a2a2050393e91800786f8d5a5d6805f936eb

riscv parts are still missing.

Would be good to get a test from @RealFYang on linux riscv (gtest should be enough).

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

Changes requested by stuefe (Reviewer).

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


More information about the jdk-updates-dev mailing list