[riscv-port-jdk11u:riscv-port] RFR: 8276799: Implementation of JEP 422: Linux/RISC-V Port [v3]

Fei Yang fyang at openjdk.org
Mon Aug 28 07:20:25 UTC 2023


On Tue, 22 Aug 2023 09:17:24 GMT, kuaiwei <duke at openjdk.org> wrote:

>> It's the initial patch based on https://github.com/openjdk/jdk17u-dev/pull/1427 , and revert patches not required in JDK11u. The testing is running on a linux riscv board. There are some failed cases and we are working on fix them , the progress will updated.
>> 
>> Thanks the contribution of OpenJDK community and hard working of @zhengxiaolinX .
>> 
>> - [x]  slowdebug/fastdebug/release build
>> - [x] Benchmark: SPECJbb2015, Renaissance
>> - [ ]  Tier1 tests
>> - [ ] Tier2 tests
>> - [ ] Tier3 tests
>> - [ ] Tier4 tests
>
> kuaiwei has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 129 commits:
> 
>  - Build with gcc 13
>  - Save all call-clobbered registers for spark tests may crash
>  - Misc adaptations to jdk11u
>  - Revert JDK-8213436: Obsolete UseMembar && JDK-8188764: Obsolete AssumeMP and then remove all support for non-MP builds, always enabled
>  - Revert:
>    JDK-8243208: Clean up JVMFlag implementation
>    JDK-8236625: Remove writeable macro from JVM flags declaration
>  - Revert JDK-8220049: Obsolete ThreadLocalHandshakes
>  - Revert JDK-8272873: C2: Inlining should not depend on absolute call site counts
>  - Revert JDK-8220051: Remove global safepoint code
>  - Revert JDK-8222637: Obsolete NeedsDeoptSuspend
>  - Revert:
>    JDK-8246689: Enable independent compressed oops/class ptrs on Aarch64
>    JDK-8241825: Make compressed oops and compressed class pointers independent (x86_64, PPC, S390)
>  - ... and 119 more: https://git.openjdk.org/riscv-port-jdk11u/compare/b9c9c2c2...1b8778b0

Changes requested by fyang (Lead).

make/autoconf/build-aux/config.guess line 3:

> 1: #!/bin/sh
> 2: #
> 3: # Copyright (c) 2012, 2022, Oracle and/or its affiliates. All rights reserved.

Simillar here.

test/hotspot/jtreg/serviceability/jvmti/HeapMonitor/MyPackage/HeapMonitorEventsForTwoThreadsTest.java line 2:

> 1: /*
> 2:  * Copyright (c) 2018, Google and/or its affiliates. All rights reserved.

I didn't see any changes made in this file. So why change the copyright header?

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

PR Review: https://git.openjdk.org/riscv-port-jdk11u/pull/3#pullrequestreview-1597618840
PR Review Comment: https://git.openjdk.org/riscv-port-jdk11u/pull/3#discussion_r1307004710
PR Review Comment: https://git.openjdk.org/riscv-port-jdk11u/pull/3#discussion_r1307004393


More information about the riscv-port-dev mailing list