[riscv-port-jdk17u:riscv-port] RFR: 8276799: Implementation of JEP 422: Linux/RISC-V Port
Fei Yang
fyang at openjdk.org
Sun Feb 12 13:29:03 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
> > I think that deserves another seperate backporting PR after this one? I see the robot is trying to link each backporting PR with the original issue. So it might not be a good idea to put serveral seperate fixes together into one backporting PR.
>
> Last time I was backporting JEP ( [openjdk/jdk11u-dev#715](https://github.com/openjdk/jdk11u-dev/pull/715) ) it was ok. Just order scara-bot few commands /issue 8282306 , /issue 8282477.
Hmm, I think that might be a good way when we are doing the final upstreaming. For now, I still prefer to keep a simple/clean relationship between a backporting PR and the corresponding issue unless there is a special reason for this staging repo.
@yadongw : Please link these two issues (8282306 & 8282477) with this backporting PR.
-------------
PR: https://git.openjdk.org/riscv-port-jdk17u/pull/1
More information about the riscv-port-dev
mailing list