[riscv-port-jdk17u:riscv-port] RFR: 8276799: Implementation of JEP 422: Linux/RISC-V Port
Fei Yang
fyang at openjdk.org
Sun Feb 12 12:57:04 UTC 2023
On Tue, 7 Feb 2023 11:20:37 GMT, Yadong Wang <yadongwang at openjdk.org> wrote:
> This PR backports the implementation of RISC-V port into jdk17u.
>
> It based on the initial implementation of JEP-422 with some modifications for jdk17u:
> 1. The initial patch
> 8276799: Implementation of JEP 422: Linux/RISC-V Port
> 2. remove features that do not belong to jdk17u
> revert 8277417: C1 LIR instruction for load-klass
> revert 8278387: Implement UseHeavyMonitors consistently
> revert 8227369: pd_disjoint_words_atomic() needs to be atomic
> revert 8258192: Obsolete the CriticalNatives flag
> revert 8281632: riscv: Improve interpreter stack banging
> revert 8283364: Intrinsify countPositives
> 3. add features that belongs to jdk17u
> enable the biased locking
> 4. Wrap modifications to shared code under RISCV and revert changes to other platforms
> isolate modification to other platforms
> 5. Patches for making compilation pass
> 8282306: os::is_first_C_frame(frame*) crashes on invalid link access
> 8282477: vmassert(_last_Java_pc == NULL, "already walkable"); fails with async profiler
> Hello could you include https://bugs.openjdk.org/browse/JDK-8293100 into initial backport ?
>
> it was really critical for risc-v and some jni stress tests from vmtestBase/hotspot:tier4 was failing
>
> JDK17 ready backport can be used from [syntacore/syntaj17 at 2053a4b](https://github.com/syntacore/syntaj17/commit/2053a4ba85078bcfc956676256264f6d7b68061c)
I think that deserves another seperate PR after this one? I see the robot is trying to link backporting PRs with the original issue. So it might not be a good idea to put serveral seperate fixes together into one backporting PR.
But the following two changes included in this PR are kind of special as they are needed for a successfull build/bootcycle:
8282306: os::is_first_C_frame(frame*) crashes on invalid link access
8282477: vmassert(_last_Java_pc == NULL, "already walkable"); fails with async profiler
-------------
PR: https://git.openjdk.org/riscv-port-jdk17u/pull/1
More information about the riscv-port-dev
mailing list