[jdk17u-dev] RFR: 8276799: Implementation of JEP 422: Linux/RISC-V Port [v7]

[Fei Yang]

On Wed, 5 Jul 2023 13:59:58 GMT, Goetz Lindenmaier <goetz at openjdk.org> wrote:

>> Fei Yang 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 12 additional commits since the last revision:
>> 
>>  - Merge branch 'master' into jep422-17u-backport
>>  - Aleksey's review comments
>>  - Aleksey's review comments
>>  - Merge branch 'master' into jep422-17u-backport
>>  - 8285630: Fix a configure error in RISC-V cross build
>>  - Adds back commit ef86ea2842b1a204834291d9d6665bfcd7b75fbc
>>  - Merge branch 'master' into jep422-17u-backport
>>  - Merge branch 'master' into jep422-17u-backport
>>  - 8305236: Some LoadLoad barriers in the interpreter are unnecessary after JDK-8220051
>>  - Merge remote-tracking branch 'upstream/master' into jep422-17u-backport
>>  - ... and 2 more: https://git.openjdk.org/jdk17u-dev/compare/c159bbad...073f56f3
>
> Isn't there anybody in the Risc-V project that can review this? It needs not necessarily be a jdk-updates reviewer. But there should be someone (maybe besides Aleksey) who will maintain this and takes the responsibility to mark this change reviewed.  It would be a bad idea to admit a port that has no people that can and will review it.

I see push approval is here. Thanks @GoeLin !
And I have added back riscv-specific code for issue [8277417](https://bugs.openjdk.org/browse/JDK-8277417) which has just been integrated into jdk17u-dev.
I will perform some testing over the weekend and integrate next Monday if everything is OK.
Thanks everyone for looking at this :-)

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

PR Comment: https://git.openjdk.org/jdk17u-dev/pull/1427#issuecomment-1625466647