December 2021 - riscv-port-dev

git: openjdk/riscv-port: riscv-port: 8278547: riscv: Refactor moving assembler functions for vector instructions into separate file
by Fei Yang 16 Dec '21

16 Dec '21

Changeset: ec881ea7 Author: Yanhong Zhu <yzhu(a)openjdk.org> Committer: Fei Yang <fyang(a)openjdk.org> Date: 2021-12-16 01:30:30 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/ec881ea794451181a124a0f57fcf… 8278547: riscv: Refactor moving assembler functions for vector instructions into separate file Reviewed-by: yadongwang, fyang ! src/hotspot/cpu/riscv/assembler_riscv.hpp + src/hotspot/cpu/riscv/assembler_riscv_v.hpp ! src/hotspot/cpu/riscv/riscv.ad

1 0

[riscv-port] RFR: 8278832: riscv: Inconsistency of Java frame offset definition in jdk.hotspot.agent
by Feilong Jiang 15 Dec '21

15 Dec '21

The Java frame definition was refactored in https://github.com/openjdk/jdk-sandbox/commit/db2415748747a0912749bb8fc160a… to make Java frame offset the same as C frame. In jdk.hotspot.agent, there is another Java frame definition based on the old Java frame offset, which causes jstack stack walking error. These definitions should also be updated. Two tests passed after this fixing: - test/hotspot/jtreg/serviceability/sa/ClhsdbCDSCore.java - test/hotspot/jtreg/serviceability/sa/ClhsdbFindPC.java ------------- Commit messages: - 8278832: riscv: Inconsistency of Java frame offset definition in jdk.hotspot.agent Changes: https://git.openjdk.java.net/riscv-port/pull/30/files Webrev: https://webrevs.openjdk.java.net/?repo=riscv-port&pr=30&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8278832 Stats: 5 lines in 1 file changed: 0 ins; 0 del; 5 mod Patch: https://git.openjdk.java.net/riscv-port/pull/30.diff Fetch: git fetch https://git.openjdk.java.net/riscv-port pull/30/head:pull/30 PR: https://git.openjdk.java.net/riscv-port/pull/30

4 4

[riscv-port] RFR: 8278041: riscv: Use t2 as the dedicated machine flags register in C2
by Yadong Wang 14 Dec '21

14 Dec '21

There are non-allocatable registers t0 and t1 in C2, frequently used as temporary registers for common code snipptes. Meanwhile, C2 use t1 as the flags register, so that many instructs using t1 as a temporary register must declare the data flow effect of t1 which is not part of a match rule. So we can reserve t2 and use it as the machine flags register in C2 instead, to reduce the probability of misuse of t1 without declaring the data flow effect. ------------- Commit messages: - 8278041: riscv: Use t2 as the dedicated machine flags register in C2 Changes: https://git.openjdk.java.net/riscv-port/pull/21/files Webrev: https://webrevs.openjdk.java.net/?repo=riscv-port&pr=21&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8278041 Stats: 144 lines in 4 files changed: 3 ins; 24 del; 117 mod Patch: https://git.openjdk.java.net/riscv-port/pull/21.diff Fetch: git fetch https://git.openjdk.java.net/riscv-port pull/21/head:pull/21 PR: https://git.openjdk.java.net/riscv-port/pull/21

3 11

[riscv-port] RFR: 8278743: riscv: Remove the x4 register saving logic in Java frames
by Xiaolin Zheng 14 Dec '21

14 Dec '21

Hi team, x4 is the thread pointer register and would be modified by the kernel. With this respect, we shall not modify or change its value. In the riscv-port repo, we have no other modifications for the x4 register so I would recommend removing the x4 saving logic in java frames. I have verified this change along with other patches under jdk/hotspot full tiers on different boards and qemu. BTW, due to the logic that can automatically adjust the sp to 16-byte aligned[1], it is safe to change the `sp_after_call_off` to an odd number. [1] https://github.com/openjdk/riscv-port/blob/riscv-port/src/hotspot/cpu/riscv… Thanks, Xiaolin ------------- Commit messages: - Remove x4 in java frames Changes: https://git.openjdk.java.net/riscv-port/pull/29/files Webrev: https://webrevs.openjdk.java.net/?repo=riscv-port&pr=29&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8278743 Stats: 32 lines in 2 files changed: 0 ins; 6 del; 26 mod Patch: https://git.openjdk.java.net/riscv-port/pull/29.diff Fetch: git fetch https://git.openjdk.java.net/riscv-port pull/29/head:pull/29 PR: https://git.openjdk.java.net/riscv-port/pull/29

3 4

CFV: New RISC-V Port Committer: Yadong Wang
by Yangfei (Felix) 10 Dec '21

10 Dec '21

I hereby nominate Yadong Wang (yadongwang) to RISC-V Port Committer. Yadong is the main developer of OpenJDK riscv-port since the beginning of this project. He has contributed 10 patches to JDK and RISC-V Port. These references in JDK and RISC-V Port projects justify the call to make him a RISC-V Port Committer. Votes are due by 9:00 UTC on Friday 24, December 2021. Only current RISC-V Port Committer[1] are eligible to vote on this nomination. Votes must be cast in the open replying to this mailing list. For Lazy Consensus voting instructions, see[2]. [1] https://openjdk.java.net/census [2] https://openjdk.java.net/projects/#committer-vote Thanks, Felix ========= 8247979: aarch64: missing side effect of killing flags for clearArray_reg_reg 8264096: slowdebug jvm crashes when StrInflatedCopy match rule is not supported 8276792: RISC-V Port: Initial support for RV64GV 8277430: riscv: Refactor interpreter native wrappers 8277431: riscv: Intrinsify recursive ObjectMonitor locking for C2 8277508: need to check has_predicated_vectors before calling scalable_predicate_reg_slots 8277877: C2 fast_unlock intrinsic on riscv has unnecessary ownership check 8277890: riscv: fix the infinite LR/SC loop in BarrierSetAssembler::eden_allocate 8278192: riscv: remove unnecessary instruct of DecodeNKlass in C2 8278387: riscv: Implement UseHeavyMonitors consistently

3 2

Test mail service for this list
by Yangfei (Felix) 10 Dec '21

10 Dec '21

Please ignore.

1 0

RE: CFV: New RISC-V Port Committer: Yadong Wang
by Yangfei (Felix) 10 Dec '21

10 Dec '21

Vote: yes > -----Original Message----- > From: Yangfei (Felix) > Sent: Friday, December 10, 2021 10:53 AM > To: riscv-port-dev(a)openjdk.java.net > Subject: CFV: New RISC-V Port Committer: Yadong Wang > > I hereby nominate Yadong Wang (yadongwang) to RISC-V Port Committer. > > Yadong is the main developer of OpenJDK riscv-port since the beginning of > this project. He has contributed 10 patches to JDK and RISC-V Port. > These references in JDK and RISC-V Port projects justify the call to make him > a RISC-V Port Committer. > > Votes are due by 9:00 UTC on Friday 24, December 2021. > > Only current RISC-V Port Committer[1] are eligible to vote on this nomination. > Votes must be cast in the open replying to this mailing list. > > For Lazy Consensus voting instructions, see[2]. > > [1] https://openjdk.java.net/census > [2] https://openjdk.java.net/projects/#committer-vote > > Thanks, > Felix > > ========= > > 8247979: aarch64: missing side effect of killing flags for clearArray_reg_reg > 8264096: slowdebug jvm crashes when StrInflatedCopy match rule is not > supported > 8276792: RISC-V Port: Initial support for RV64GV > 8277430: riscv: Refactor interpreter native wrappers > 8277431: riscv: Intrinsify recursive ObjectMonitor locking for C2 > 8277508: need to check has_predicated_vectors before calling > scalable_predicate_reg_slots > 8277877: C2 fast_unlock intrinsic on riscv has unnecessary ownership check > 8277890: riscv: fix the infinite LR/SC loop in > BarrierSetAssembler::eden_allocate > 8278192: riscv: remove unnecessary instruct of DecodeNKlass in C2 > 8278387: riscv: Implement UseHeavyMonitors consistently

1 0

[riscv-port] RFR: 8278531: riscv: Remove duplicate code bitset_to_fregs
by Feilong Jiang 10 Dec '21

10 Dec '21

After JDK-8278337, the definition of `bitset_to_fregs` is just same as `bitset_to_regs`. Hotspot and jdk tier1 passed on the unmatched board. All tests were tested without new failures. ------------- Commit messages: - 8278531: riscv: Remove duplicate code bitset_to_fregs Changes: https://git.openjdk.java.net/riscv-port/pull/28/files Webrev: https://webrevs.openjdk.java.net/?repo=riscv-port&pr=28&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8278531 Stats: 17 lines in 2 files changed: 0 ins; 14 del; 3 mod Patch: https://git.openjdk.java.net/riscv-port/pull/28.diff Fetch: git fetch https://git.openjdk.java.net/riscv-port pull/28/head:pull/28 PR: https://git.openjdk.java.net/riscv-port/pull/28

2 2

git: openjdk/riscv-port: riscv-port: 46 new changesets
by duke 10 Dec '21

10 Dec '21

Changeset: 10e0318f Author: Julia Boes <jboes(a)openjdk.org> Date: 2021-12-08 09:16:44 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/10e0318ff7289b248289a70a56ad… 8278158: jwebserver should set request timeout Reviewed-by: dfuchs, michaelm ! src/jdk.httpserver/share/classes/sun/net/httpserver/simpleserver/JWebServer.java ! src/jdk.httpserver/share/classes/sun/net/httpserver/simpleserver/Main.java Changeset: 54993b13 Author: Thomas Stuefe <stuefe(a)openjdk.org> Date: 2021-12-08 09:18:58 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/54993b13078ceb05736bb0e1d7ed… 8278309: [windows] use of uninitialized OSThread::_state Reviewed-by: dholmes, shade ! src/hotspot/os/windows/os_windows.cpp Changeset: e5cb84e9 Author: Markus Grönlund <mgronlun(a)openjdk.org> Date: 2021-12-08 11:18:24 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/e5cb84e9a1bca321f22d6e3ce13a… 8278336: Use int64_t to represent byte quantities consistently in JfrObjectAllocationSample Reviewed-by: egahlin ! src/hotspot/share/jfr/support/jfrObjectAllocationSample.cpp Changeset: fd8cb2da Author: Matthias Baesken <mbaesken(a)openjdk.org> Date: 2021-12-08 14:03:56 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/fd8cb2dae87e5726e89a1789e5a1… 8278346: java/nio/file/Files/probeContentType/Basic.java fails on Linux SLES15 machine Reviewed-by: shade ! test/jdk/java/nio/file/Files/probeContentType/Basic.java Changeset: 37921e30 Author: Daniel Fuchs <dfuchs(a)openjdk.org> Date: 2021-12-08 15:31:49 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/37921e30803449c06b4d542fdfee… 8269258: java/net/httpclient/ManyRequestsLegacy.java failed with connection timeout Reviewed-by: michaelm ! test/jdk/java/net/httpclient/ManyRequests.java ! test/jdk/java/net/httpclient/ManyRequests2.java ! test/jdk/java/net/httpclient/ManyRequestsLegacy.java Changeset: e4852c6f Author: Calvin Cheung <ccheung(a)openjdk.org> Date: 2021-12-08 15:59:37 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/e4852c6f0aa25e7d40c577d507ae… 8277998: runtime/cds/appcds/loaderConstraints/DynamicLoaderConstraintsTest.java#custom-cl-zgc failed "assert(ZAddress::is_marked(addr)) failed: Should be marked" Reviewed-by: iklam, minqi ! src/hotspot/share/cds/dynamicArchive.cpp ! src/hotspot/share/cds/lambdaFormInvokers.cpp ! src/hotspot/share/cds/lambdaFormInvokers.hpp ! test/hotspot/jtreg/runtime/cds/appcds/DumpClassListWithLF.java Changeset: 40d726b8 Author: Calvin Cheung <ccheung(a)openjdk.org> Date: 2021-12-08 16:07:49 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/40d726b8aae7dff4316115b3a250… 8278310: Improve logging in CDS DynamicLoaderConstraintsTest.java Reviewed-by: iklam ! src/hotspot/share/cds/archiveBuilder.hpp ! test/hotspot/jtreg/runtime/cds/appcds/loaderConstraints/DynamicLoaderConstraintsTest.java Changeset: 79165b73 Author: Calvin Cheung <ccheung(a)openjdk.org> Date: 2021-12-08 16:10:06 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/79165b738d6d1336740631c85596… 8278324: Update the --generate-cds-archive jlink plugin usage message Reviewed-by: alanb ! src/jdk.jlink/share/classes/jdk/tools/jlink/resources/plugins.properties Changeset: 3e93e0b8 Author: Yumin Qi <minqi(a)openjdk.org> Date: 2021-12-08 16:20:39 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/3e93e0b809f2c6cb4a466af92ec6… 8276769: -Xshare:auto should tolerate problems in the CDS archive Reviewed-by: iklam, ccheung ! src/hotspot/share/cds/filemap.cpp ! src/hotspot/share/include/cds.h ! src/hotspot/share/runtime/arguments.cpp ! src/hotspot/share/runtime/arguments.hpp ! test/hotspot/jtreg/runtime/cds/appcds/SharedArchiveConsistency.java ! test/hotspot/jtreg/runtime/cds/appcds/dynamicArchive/ArchiveConsistency.java ! test/hotspot/jtreg/runtime/cds/appcds/dynamicArchive/DynamicArchiveTestBase.java Changeset: 8af3b27c Author: Roland Westrelin <roland(a)openjdk.org> Date: 2021-12-08 16:39:24 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/8af3b27ce98bcb9cf0c155c98d6b… 8277850: C2: optimize mask checks in counted loops Reviewed-by: kvn, thartmann ! src/hotspot/share/opto/mulnode.cpp ! src/hotspot/share/opto/mulnode.hpp + test/hotspot/jtreg/compiler/c2/irTests/TestShiftAndMask.java ! test/hotspot/jtreg/compiler/lib/ir_framework/IRNode.java Changeset: 24781588 Author: Brian Burkhalter <bpb(a)openjdk.org> Date: 2021-12-08 17:11:14 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/2478158815a9ae0466542e27562b… 8277361: java/nio/channels/Channels/ReadXBytes.java fails with OOM error Reviewed-by: alanb, lancea ! test/jdk/java/nio/channels/Channels/ReadXBytes.java Changeset: 9b747491 Author: Valerie Peng <valeriep(a)openjdk.org> Date: 2021-12-08 17:44:10 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/9b747491de01fd011b09668a6711… 8276660: Scalability bottleneck in java.security.Provider.getService() Reviewed-by: weijun ! src/java.base/share/classes/java/security/Provider.java Changeset: 83454530 Author: Alexey Ushakov <avu(a)openjdk.org> Date: 2021-12-08 17:48:53 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/83454530ea538c80e2f1205cd8d2… 8272392: Lanai: SwingSet2. Black background on expanding tree node Reviewed-by: aghaisas, jdv ! src/java.desktop/macosx/native/libawt_lwawt/java2d/metal/EncoderManager.m ! src/java.desktop/macosx/native/libawt_lwawt/java2d/metal/MTLBlitLoops.m ! src/java.desktop/macosx/native/libawt_lwawt/java2d/metal/MTLClip.m ! src/java.desktop/macosx/native/libawt_lwawt/java2d/metal/MTLPaints.m ! src/java.desktop/macosx/native/libawt_lwawt/java2d/metal/MTLPipelineStatesStorage.m ! src/java.desktop/macosx/native/libawt_lwawt/java2d/metal/RenderOptions.h ! src/java.desktop/macosx/native/libawt_lwawt/java2d/metal/common.h Changeset: 92aa75b7 Author: Aleksey Shipilev <shade(a)openjdk.org> Date: 2021-12-08 17:52:03 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/92aa75b72185c681757e6520d6fd… 8274903: Zero: Support AsyncGetCallTrace Reviewed-by: sspitsyn, aph ! src/hotspot/cpu/zero/frame_zero.cpp ! src/hotspot/os_cpu/linux_zero/thread_linux_zero.cpp ! src/hotspot/os_cpu/linux_zero/thread_linux_zero.hpp ! src/hotspot/share/prims/forte.cpp Changeset: ba86dd4a Author: Daniel D. Daugherty <dcubed(a)openjdk.org> Date: 2021-12-08 18:02:34 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/ba86dd4aaf597a0f0832085b50fa… 8278445: ProblemList tools/jpackage/share/IconTest.java on macosx-x64 Reviewed-by: bpb ! test/jdk/ProblemList.txt Changeset: 42d9b1ba Author: Jan Lahoda <jlahoda(a)openjdk.org> Date: 2021-12-08 18:50:38 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/42d9b1baac25df649a8d575f5ffa… 8277106: Cannot compile certain sources with --release Reviewed-by: darcy, mcimadamore ! make/data/symbols/java.base-C.sym.txt ! make/data/symbols/java.base-E.sym.txt ! make/data/symbols/java.base-G.sym.txt ! make/data/symbols/java.base-H.sym.txt ! make/data/symbols/java.xml-B.sym.txt ! make/data/symbols/java.xml-C.sym.txt ! make/data/symbols/java.xml-D.sym.txt ! make/data/symbols/java.xml-E.sym.txt ! make/data/symbols/jdk.incubator.foreign-F.sym.txt ! make/data/symbols/jdk.incubator.foreign-G.sym.txt ! make/data/symbols/jdk.incubator.foreign-H.sym.txt ! make/data/symbols/jdk.incubator.vector-G.sym.txt ! make/data/symbols/jdk.incubator.vector-H.sym.txt ! make/data/symbols/jdk.javadoc-H.sym.txt ! make/data/symbols/jdk.jfr-B.sym.txt ! make/data/symbols/jdk.jfr-G.sym.txt ! make/langtools/src/classes/build/tools/symbolgenerator/CreateSymbols.java + test/langtools/tools/javac/platform/NonExportedSuperTypes.java ! test/langtools/tools/javac/platform/createsymbols/CreateSymbolsTest.java ! test/langtools/tools/javac/platform/createsymbols/CreateSymbolsTestImpl.java Changeset: fb11d8fa Author: Jonathan Gibbons <jjg(a)openjdk.org> Date: 2021-12-08 19:48:05 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/fb11d8faf21539a8dc08d371658d… 8272945: Use snippets in java.compiler documentation Reviewed-by: erikj, alanb ! make/CompileInterimLangtools.gmk ! make/modules/java.compiler/Java.gmk ! src/java.compiler/share/classes/javax/tools/JavaCompiler.java ! src/java.compiler/share/classes/javax/tools/JavaFileManager.java ! src/java.compiler/share/classes/javax/tools/StandardJavaFileManager.java ! src/java.compiler/share/classes/javax/tools/package-info.java + src/java.compiler/share/classes/javax/tools/snippet-files/JavaSourceFromString.java Changeset: 5a80abf7 Author: Jonathan Gibbons <jjg(a)openjdk.org> Date: 2021-12-08 19:50:14 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/5a80abf706dfde9342be53e78b94… 8272944: Use snippets in jdk.javadoc documentation Reviewed-by: hannesw ! src/jdk.compiler/share/classes/com/sun/tools/javac/model/JavacElements.java ! src/jdk.javadoc/share/classes/jdk/javadoc/doclet/package-info.java Changeset: 8e8fadf2 Author: Roger Riggs <rriggs(a)openjdk.org> Date: 2021-12-08 20:37:41 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/8e8fadf2d4e45f53286dd8424d95… 8278428: ObjectInputStream.readFully range check incorrect Reviewed-by: alanb ! src/java.base/share/classes/java/io/ObjectInputStream.java ! test/jdk/java/io/Serializable/oldTests/WritePrimitive.java Changeset: c83b781b Author: Daniel D. Daugherty <dcubed(a)openjdk.org> Date: 2021-12-08 20:42:52 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/c83b781b6002ca0f067a35e678ab… 8278459: ProblemList javax/swing/JTree/4908142/bug4908142.java on macosx-aarch64 Reviewed-by: lmesnik, bpb, prr ! test/jdk/ProblemList.txt Changeset: 6e7b7f35 Author: Sergey Bylokhov <serb(a)openjdk.org> Date: 2021-12-08 21:15:55 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/6e7b7f352e66c767b16e1e98a162… 8278251: Enable "missing-explicit-ctor" check in the jdk.unsupported.desktop module Reviewed-by: ihse, prr - make/modules/jdk.unsupported.desktop/Java.gmk Changeset: fe2ae8e3 Author: Eamonn McManus <emcmanus(a)openjdk.org> Date: 2021-12-08 21:24:43 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/fe2ae8e38bd8660cc637f13f425c… 8276904: Optional.toString() is unnecessarily expensive Reviewed-by: rriggs, smarks ! src/java.base/share/classes/java/util/Optional.java ! src/java.base/share/classes/java/util/OptionalDouble.java ! src/java.base/share/classes/java/util/OptionalInt.java ! src/java.base/share/classes/java/util/OptionalLong.java Changeset: b3faecf7 Author: Roland Westrelin <roland(a)openjdk.org> Date: 2021-12-08 21:33:33 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/b3faecf73966ce0435644a89f8f4… 8276116: C2: optimize long range checks in int counted loops Reviewed-by: kvn ! src/hotspot/cpu/x86/x86_32.ad ! src/hotspot/share/opto/compile.cpp ! src/hotspot/share/opto/loopTransform.cpp ! src/hotspot/share/opto/loopnode.cpp ! src/hotspot/share/opto/loopnode.hpp ! src/hotspot/share/opto/loopopts.cpp ! test/hotspot/jtreg/compiler/c2/irTests/TestLongRangeChecks.java ! test/hotspot/jtreg/compiler/rangechecks/TestLongRangeCheck.java Changeset: 3893f4f7 Author: Serguei Spitsyn <sspitsyn(a)openjdk.org> Date: 2021-12-08 22:32:00 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/3893f4f70b5828ee8e673ce7b142… 8278330: dump stack trace if the jvmti test nsk/jvmti/GetThreadState/thrstat002 is failed with wrong thread state Reviewed-by: amenkov, lmesnik ! test/hotspot/jtreg/vmTestbase/nsk/jvmti/GetThreadState/thrstat002/thrstat002.cpp Changeset: 03bf55f6 Author: Vicente Romero <vromero(a)openjdk.org> Date: 2021-12-08 22:49:43 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/03bf55f62065f8f9c8ade1eff8ca… 8277980: ObjectMethods::bootstrap throws NPE when lookup is null Reviewed-by: jjg ! src/java.base/share/classes/java/lang/runtime/ObjectMethods.java ! test/jdk/java/lang/runtime/ObjectMethodsTest.java Changeset: 3cec700a Author: Roger Riggs <rriggs(a)openjdk.org> Date: 2021-12-08 22:59:38 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/3cec700a5caa2243772e4b01c406… 8278463: [test] Serialization WritePrimitive test revised for readFully test fails Reviewed-by: darcy ! test/jdk/java/io/Serializable/oldTests/WritePrimitive.java Changeset: 83e6a4c0 Author: Valerie Peng <valeriep(a)openjdk.org> Date: 2021-12-08 23:04:15 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/83e6a4c0e9e4a9474ae0c1252378… 8255409: Support the new C_GetInterfaceList, C_GetInterface, and C_SessionCancel APIs in PKCS#11 v3.0 Reviewed-by: ascarpino, weijun ! src/jdk.crypto.cryptoki/share/classes/sun/security/pkcs11/Config.java ! src/jdk.crypto.cryptoki/share/classes/sun/security/pkcs11/P11AEADCipher.java ! src/jdk.crypto.cryptoki/share/classes/sun/security/pkcs11/P11Cipher.java ! src/jdk.crypto.cryptoki/share/classes/sun/security/pkcs11/P11KeyWrapCipher.java ! src/jdk.crypto.cryptoki/share/classes/sun/security/pkcs11/P11Mac.java ! src/jdk.crypto.cryptoki/share/classes/sun/security/pkcs11/P11PSSSignature.java ! src/jdk.crypto.cryptoki/share/classes/sun/security/pkcs11/P11RSACipher.java ! src/jdk.crypto.cryptoki/share/classes/sun/security/pkcs11/P11Signature.java ! src/jdk.crypto.cryptoki/share/classes/sun/security/pkcs11/P11Util.java ! src/jdk.crypto.cryptoki/share/classes/sun/security/pkcs11/SunPKCS11.java ! src/jdk.crypto.cryptoki/share/classes/sun/security/pkcs11/wrapper/PKCS11.java ! src/jdk.crypto.cryptoki/share/native/libj2pkcs11/p11_sessmgmt.c ! src/jdk.crypto.cryptoki/share/native/libj2pkcs11/p11_util.c ! src/jdk.crypto.cryptoki/share/native/libj2pkcs11/pkcs11wrapper.h ! src/jdk.crypto.cryptoki/unix/native/libj2pkcs11/p11_md.c ! src/jdk.crypto.cryptoki/unix/native/libj2pkcs11/p11_md.h ! src/jdk.crypto.cryptoki/windows/native/libj2pkcs11/p11_md.c ! src/jdk.crypto.cryptoki/windows/native/libj2pkcs11/p11_md.h Changeset: 35243558 Author: Zhiqiang Zang <capapoc(a)gmail.com> Committer: Igor Veresov <iveresov(a)openjdk.org> Date: 2021-12-09 00:18:59 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/352435581e4354741684d66df8a3… 8277882: New subnode ideal optimization: converting "c0 - (x + c1)" into "(c0 - c1) - x" Reviewed-by: dlong, iveresov ! src/hotspot/share/opto/subnode.cpp + test/hotspot/jtreg/compiler/c2/TestSubIdealC0Minus_YPlusC1_.java + test/micro/org/openjdk/bench/vm/compiler/SubIdealC0Minus_YPlusC1_.java Changeset: 8ef1a232 Author: Alexey Semenyuk <asemenyuk(a)openjdk.org> Date: 2021-12-09 02:31:29 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/8ef1a232b100dbac72356ca4844d… 8278311: Debian packaging doesn't work Reviewed-by: almatvee ! src/jdk.jpackage/share/classes/jdk/jpackage/internal/OverridableResource.java ! test/jdk/tools/jpackage/junit/jdk/jpackage/internal/OverridableResourceTest.java Changeset: d1706828 Author: Jonathan Gibbons <jjg(a)openjdk.org> Date: 2021-12-09 02:53:52 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/d1706828fb6a95ecad2bab4112b2… 8278466: "spurious markup" warnings in snippets when building `docs-reference` Reviewed-by: darcy ! src/java.compiler/share/classes/javax/tools/JavaFileManager.java ! src/java.compiler/share/classes/javax/tools/StandardJavaFileManager.java Changeset: 62a7f5d3 Author: Aleksey Shipilev <shade(a)openjdk.org> Date: 2021-12-09 06:50:56 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/62a7f5d3236ab2248518a475b1d8… 8254318: Remove .hgtags Reviewed-by: erikj, xliu - .hgtags Changeset: 1ea766db Author: Markus Grönlund <mgronlun(a)openjdk.org> Date: 2021-12-09 09:23:39 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/1ea766db5519f03e36fc5b0c446c… 8278419: JFR jcmd option contract "This value cannot be changed once JFR has been initialized" is not enforced Reviewed-by: egahlin ! src/hotspot/share/jfr/dcmd/jfrDcmds.cpp + test/jdk/jdk/jfr/jcmd/TestJcmdConfigureReadOnly.java Changeset: 84baea75 Author: Julia Boes <jboes(a)openjdk.org> Date: 2021-12-09 09:29:08 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/84baea753a01428d9dc5befa5f23… 8278433: Use snippets in jdk.incubator.foreign documentation Reviewed-by: mcimadamore ! src/jdk.incubator.foreign/share/classes/jdk/incubator/foreign/CLinker.java ! src/jdk.incubator.foreign/share/classes/jdk/incubator/foreign/MemoryAddress.java ! src/jdk.incubator.foreign/share/classes/jdk/incubator/foreign/MemoryHandles.java ! src/jdk.incubator.foreign/share/classes/jdk/incubator/foreign/MemoryLayout.java ! src/jdk.incubator.foreign/share/classes/jdk/incubator/foreign/MemorySegment.java ! src/jdk.incubator.foreign/share/classes/jdk/incubator/foreign/ResourceScope.java ! src/jdk.incubator.foreign/share/classes/jdk/incubator/foreign/SegmentAllocator.java ! src/jdk.incubator.foreign/share/classes/jdk/incubator/foreign/SequenceLayout.java ! src/jdk.incubator.foreign/share/classes/jdk/incubator/foreign/ValueLayout.java ! src/jdk.incubator.foreign/share/classes/jdk/incubator/foreign/package-info.java Changeset: 965ea8d9 Author: Markus Grönlund <mgronlun(a)openjdk.org> Date: 2021-12-09 09:29:59 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/965ea8d9cd29aee41ba2b1b0b0c6… 8274298: JFR Thread Sampler thread must not acquire malloc lock after suspending a thread because of possible deadlock Reviewed-by: egahlin ! src/hotspot/share/jfr/periodic/sampling/jfrThreadSampler.cpp ! src/hotspot/share/jfr/periodic/sampling/jfrThreadSampler.hpp ! src/hotspot/share/jfr/recorder/checkpoint/types/traceid/jfrTraceIdKlassQueue.cpp ! src/hotspot/share/jfr/recorder/checkpoint/types/traceid/jfrTraceIdKlassQueue.hpp ! src/hotspot/share/jfr/recorder/checkpoint/types/traceid/jfrTraceIdLoadBarrier.cpp ! src/hotspot/share/jfr/recorder/checkpoint/types/traceid/jfrTraceIdLoadBarrier.hpp ! src/hotspot/share/jfr/recorder/stacktrace/jfrStackTrace.cpp ! src/hotspot/share/jfr/utilities/jfrEpochQueue.hpp ! src/hotspot/share/jfr/utilities/jfrEpochQueue.inline.hpp Changeset: 08aad850 Author: Erik Gahlin <egahlin(a)openjdk.org> Date: 2021-12-09 14:13:53 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/08aad8506e60157fc5ee1cc0e3e9… 8211230: JFR: internal events Reviewed-by: mgronlun ! make/src/classes/build/tools/jfr/GenerateJfrFiles.java ! src/hotspot/share/jfr/jni/jfrUpcalls.cpp ! src/hotspot/share/jfr/jni/jfrUpcalls.hpp ! src/hotspot/share/jfr/metadata/metadata.xml ! src/hotspot/share/jfr/metadata/metadata.xsd ! src/hotspot/share/jfr/recorder/checkpoint/jfrMetadataEvent.cpp ! src/hotspot/share/jfr/recorder/jfrEventSetting.cpp ! src/hotspot/share/jfr/recorder/jfrEventSetting.hpp ! src/jdk.jfr/share/classes/jdk/jfr/EventType.java ! src/jdk.jfr/share/classes/jdk/jfr/FlightRecorderPermission.java ! src/jdk.jfr/share/classes/jdk/jfr/internal/EventControl.java ! src/jdk.jfr/share/classes/jdk/jfr/internal/JVMUpcalls.java ! src/jdk.jfr/share/classes/jdk/jfr/internal/MetadataLoader.java ! src/jdk.jfr/share/classes/jdk/jfr/internal/MetadataRepository.java ! src/jdk.jfr/share/classes/jdk/jfr/internal/PrivateAccess.java ! src/jdk.jfr/share/classes/jdk/jfr/internal/Type.java ! src/jdk.jfr/share/classes/jdk/jfr/internal/TypeLibrary.java ! src/jdk.jfr/share/classes/jdk/jfr/internal/tool/Metadata.java ! src/jdk.jfr/share/conf/jfr/default.jfc ! src/jdk.jfr/share/conf/jfr/profile.jfc Changeset: 6dae52f8 Author: Eric Liu <eliu(a)openjdk.org> Committer: Ningsheng Jian <njian(a)openjdk.org> Date: 2021-12-09 14:16:38 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/6dae52f8e3993d529033147de8e3… 8276985: AArch64: [vectorapi] Backend support of VectorMaskToLongNode Reviewed-by: njian, aph ! src/hotspot/cpu/aarch64/aarch64_neon.ad ! src/hotspot/cpu/aarch64/aarch64_neon_ad.m4 ! src/hotspot/cpu/aarch64/aarch64_sve.ad ! src/hotspot/cpu/aarch64/aarch64_sve_ad.m4 ! src/hotspot/cpu/aarch64/c2_MacroAssembler_aarch64.cpp ! src/hotspot/cpu/aarch64/c2_MacroAssembler_aarch64.hpp Changeset: 09831e7a Author: Joe Darcy <darcy(a)openjdk.org> Committer: Jesper Wilhelmsson <jwilhelm(a)openjdk.org> Date: 2021-12-09 17:01:59 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/09831e7aa47ebe41eab2f3014ebb… 8273146: Start of release updates for JDK 19 8277511: Add SourceVersion.RELEASE_19 8277513: Add source 19 and target 19 to javac Reviewed-by: dholmes, alanb, erikj, iris, mikael, ihse ! make/conf/version-numbers.conf + make/data/symbols/java.base-I.sym.txt + make/data/symbols/java.compiler-I.sym.txt + make/data/symbols/java.datatransfer-I.sym.txt + make/data/symbols/java.desktop-I.sym.txt = make/data/symbols/java.instrument-I.sym.txt + make/data/symbols/java.logging-I.sym.txt + make/data/symbols/java.management-I.sym.txt = make/data/symbols/java.management.rmi-I.sym.txt + make/data/symbols/java.naming-I.sym.txt + make/data/symbols/java.net.http-I.sym.txt + make/data/symbols/java.rmi-I.sym.txt = make/data/symbols/java.scripting-I.sym.txt = make/data/symbols/java.security.jgss-I.sym.txt = make/data/symbols/java.security.sasl-I.sym.txt = make/data/symbols/java.smartcardio-I.sym.txt = make/data/symbols/java.sql-I.sym.txt = make/data/symbols/java.sql.rowset-I.sym.txt + make/data/symbols/java.xml-I.sym.txt + make/data/symbols/java.xml.crypto-I.sym.txt + make/data/symbols/jdk.accessibility-I.sym.txt = make/data/symbols/jdk.attach-I.sym.txt + make/data/symbols/jdk.compiler-I.sym.txt = make/data/symbols/jdk.dynalink-I.sym.txt + make/data/symbols/jdk.httpserver-I.sym.txt + make/data/symbols/jdk.incubator.foreign-I.sym.txt + make/data/symbols/jdk.incubator.vector-I.sym.txt = make/data/symbols/jdk.jartool-I.sym.txt + make/data/symbols/jdk.javadoc-I.sym.txt = make/data/symbols/jdk.jconsole-I.sym.txt = make/data/symbols/jdk.jdi-I.sym.txt + make/data/symbols/jdk.jlink-I.sym.txt + make/data/symbols/jdk.jshell-I.sym.txt = make/data/symbols/jdk.jsobject-I.sym.txt = make/data/symbols/jdk.management-I.sym.txt + make/data/symbols/jdk.management.jfr-I.sym.txt = make/data/symbols/jdk.net-I.sym.txt = make/data/symbols/jdk.sctp-I.sym.txt = make/data/symbols/jdk.security.auth-I.sym.txt + make/data/symbols/jdk.security.jgss-I.sym.txt + make/data/symbols/jdk.unsupported-I.sym.txt = make/data/symbols/jdk.xml.dom-I.sym.txt ! make/data/symbols/symbols ! src/hotspot/share/classfile/classFileParser.cpp ! src/java.base/share/classes/jdk/internal/org/objectweb/asm/ClassReader.java ! src/java.base/share/classes/jdk/internal/org/objectweb/asm/Opcodes.java ! src/java.compiler/share/classes/javax/lang/model/SourceVersion.java ! src/java.compiler/share/classes/javax/lang/model/util/AbstractAnnotationValueVisitor14.java ! src/java.compiler/share/classes/javax/lang/model/util/AbstractElementVisitor14.java ! src/java.compiler/share/classes/javax/lang/model/util/AbstractTypeVisitor14.java ! src/java.compiler/share/classes/javax/lang/model/util/ElementKindVisitor14.java ! src/java.compiler/share/classes/javax/lang/model/util/ElementScanner14.java ! src/java.compiler/share/classes/javax/lang/model/util/SimpleAnnotationValueVisitor14.java ! src/java.compiler/share/classes/javax/lang/model/util/SimpleElementVisitor14.java ! src/java.compiler/share/classes/javax/lang/model/util/SimpleTypeVisitor14.java ! src/java.compiler/share/classes/javax/lang/model/util/TypeKindVisitor14.java ! src/jdk.compiler/share/classes/com/sun/tools/javac/code/Source.java ! src/jdk.compiler/share/classes/com/sun/tools/javac/jvm/ClassFile.java ! src/jdk.compiler/share/classes/com/sun/tools/javac/jvm/Target.java ! src/jdk.compiler/share/classes/com/sun/tools/javac/processing/PrintingProcessor.java ! test/hotspot/jtreg/runtime/CommandLine/VMDeprecatedOptions.java ! test/langtools/tools/javac/api/TestGetSourceVersions.java ! test/langtools/tools/javac/classfiles/ClassVersionChecker.java ! test/langtools/tools/javac/lib/JavacTestingAbstractProcessor.java ! test/langtools/tools/javac/preview/classReaderTest/Client.nopreview.out ! test/langtools/tools/javac/preview/classReaderTest/Client.preview.out ! test/langtools/tools/javac/versions/Versions.java Changeset: 01b30bfa Author: Martin Doerr <mdoerr(a)openjdk.org> Date: 2021-12-09 17:04:55 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/01b30bfa99e95cf1e9209c8de1f3… 8253860: PPC: Relocation::pd_set_data_value conflates compressed oops and klasses Reviewed-by: dlong, rrich ! src/hotspot/cpu/ppc/c1_LIRAssembler_ppc.cpp ! src/hotspot/cpu/ppc/macroAssembler_ppc.cpp ! src/hotspot/cpu/ppc/nativeInst_ppc.hpp ! src/hotspot/cpu/ppc/ppc.ad ! src/hotspot/cpu/ppc/relocInfo_ppc.cpp Changeset: bc31ccc9 Author: Michael McMahon <michaelm(a)openjdk.org> Date: 2021-12-09 17:38:49 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/bc31ccc95be9523cc6c64932f6d3… 8278312: Update SimpleSSLContext keystore to use SANs for localhost IP addresses Reviewed-by: dfuchs + test/jdk/com/sun/net/httpserver/SANTest.java ! test/jdk/java/net/httpclient/http2/server/Http2TestServer.java ! test/lib/jdk/test/lib/net/testkeys Changeset: abebde74 Author: Albert Mingkun Yang <ayang(a)openjdk.org> Date: 2021-12-09 18:15:53 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/abebde74409e87f4bab2e72ed27a… 8278421: G1: Remove unused HeapRegion::verify Reviewed-by: iwalulya, tschatzl ! src/hotspot/share/gc/g1/heapRegion.cpp ! src/hotspot/share/gc/g1/heapRegion.hpp Changeset: fcd67a52 Author: Toshio Nakamura <tnakamura(a)openjdk.org> Committer: Naoto Sato <naoto(a)openjdk.org> Date: 2021-12-09 19:05:31 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/fcd67a5242b7db06e4cb06b57497… 8278185: Custom JRE cannot find non-ASCII named module inside Reviewed-by: naoto ! src/java.base/share/classes/jdk/internal/jimage/ImageStringsReader.java + test/jdk/tools/jimage/JImageNonAsciiNameTest.java Changeset: 918b3505 Author: Jonathan Gibbons <jjg(a)openjdk.org> Date: 2021-12-09 21:47:50 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/918b3505e1cbbf5ac380cbcca43a… 8278516: Typos in snippet for java.compiler Reviewed-by: darcy ! src/java.compiler/share/classes/javax/tools/StandardJavaFileManager.java Changeset: d40e90b4 Author: Daniel D. Daugherty <dcubed(a)openjdk.org> Date: 2021-12-09 22:03:04 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/d40e90b4a1b55753a178d824c4c6… 8278521: ProblemList java/lang/management/ThreadMXBean/ThreadLists.java Reviewed-by: bpb ! test/jdk/ProblemList.txt Changeset: ec0a5ac8 Author: Jesper Wilhelmsson <jwilhelm(a)openjdk.org> Date: 2021-12-09 23:24:40 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/ec0a5ac8fec75c5c3eda3d9909d2… Merge Changeset: 066b348d Author: Kim Barrett <kbarrett(a)openjdk.org> Date: 2021-12-10 01:32:05 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/066b348dbc989cf90048b136a8b4… 8277807: Increase default initial concurrent refinement threshold Reviewed-by: tschatzl, sjohanss ! src/hotspot/share/gc/g1/g1ConcurrentRefine.cpp Changeset: f29745db Author: Fei Yang <felix.yang(a)huawei.com> Date: 2021-12-09 20:39:54 +0000 URL: https://git.openjdk.java.net/riscv-port/commit/f29745db46fecb5f5dcc93930bb1… Merge branch 'master' into riscv-port ! src/hotspot/cpu/ppc/c1_LIRAssembler_ppc.cpp ! src/hotspot/share/runtime/arguments.cpp ! src/hotspot/cpu/ppc/c1_LIRAssembler_ppc.cpp ! src/hotspot/share/runtime/arguments.cpp

1 0

[riscv-port] RFR: 8278322: riscv: Support RVC: compressed instructions
by Xiaolin Zheng 09 Dec '21

09 Dec '21

Hi team, This patch support RISC-V RVC extension. It can introduce: * 21% code size reduction in template interpreter generated code * 20%~25% code size reduction in C1 generated code, evaluated by a common SpringBoot program * 15%~20% code size reduction in C2 generated code, evaluated by a common SpringBoot program In my observation, the code size footprint could be reduced to nearly a level of the AArch64 back-end. About the performance, there seems a stable ~0.7% performance improvement on SPECjbb2015 on one HiFive Unleashed board, considering the code density increase. I think the performance aspect might be a speculative behavior on different hardware implementations because C910's performance might be better than that, but HiFive Unleashed may be more general. Things about this patch: * If an instruction is compressible, then we will implicitly emit a 16-bit compressed instruction instead of the 32-bit instruction in Assembler. * About the `_nc` postfix of some of Assembler instructions: we know a bunch of places should be reserved for patching, where we cannot change them into compressed instructions. `_nc` is short for `not compressible` - with this, those instructions should keep their origin 4-byte form and remain uncompressed. * There are things not easy to compress like MachBranchNodes. Please see the comments in the code - currently this patch does not support this. We will support this improvement in patches coming afterward. The macros after their expansion might be like: void andr(Register Rd, Register Rs1, Register Rs2) { { Register src = noreg; if (UseCExt && Rs1->is_compressed_valid() && Rs2->is_compressed_valid() && ((src = Rs1, Rs2 == Rd) || (src = Rs2, Rs1 == Rd))) { and_c(Rd, Rs2); return; } } unsigned insn = 0; patch((address)&insn, 6, 0, 0b0110011); patch((address)&insn, 14, 12, 0b111); patch((address)&insn, 31, 25, 0b0000000); patch_reg((address)&insn, 7, Rd); patch_reg((address)&insn, 15, Rs1); patch_reg((address)&insn, 20, Rs2); emit(insn); }; For further information, please see comments in `assembler_riscv_cext.hpp`. -- This patch may need some time to acquire a review. I have polished this patch for quite a long time and it might seem stable under a bunch of full-tier tests and some tier1 jdk/hotspot jtreg tests. But I think we might mark it `experimental` at first, though it is turned on by default. [The original patch](https://github.com/riscv-collab/riscv-openjdk/pull/7), just for reference, might be quite different from the current one. I am pleased to receive any suggestion. Thanks, Xiaolin ------------- Commit messages: - Support RVC: compressed instructions Changes: https://git.openjdk.java.net/riscv-port/pull/24/files Webrev: https://webrevs.openjdk.java.net/?repo=riscv-port&pr=24&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8278322 Stats: 1891 lines in 29 files changed: 1401 ins; 81 del; 409 mod Patch: https://git.openjdk.java.net/riscv-port/pull/24.diff Fetch: git fetch https://git.openjdk.java.net/riscv-port pull/24/head:pull/24 PR: https://git.openjdk.java.net/riscv-port/pull/24

3 16