[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