January 2021 - core-libs-dev

RFR: 8258588: MD5 MessageDigest in java.util.UUID should be cached
by PROgrm_JARvis 19 May '21

19 May '21

Please review this change moving lookup of MD5 digest in `java.lang.UUID` to an internal holder class. ------------- Commit messages: - 8258588: MD5 MessageDigest in java.util.UUID should be cached Changes: https://git.openjdk.java.net/jdk/pull/1821/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=1821&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8258588 Stats: 33 lines in 1 file changed: 26 ins; 6 del; 1 mod Patch: https://git.openjdk.java.net/jdk/pull/1821.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/1821/head:pull/1821 PR: https://git.openjdk.java.net/jdk/pull/1821

7 28

RFR: 4926314: Optimize Reader.read(CharBuffer)
by Philippe Marschall 26 Apr '21

26 Apr '21

Implement three optimiztations for Reader.read(CharBuffer) * Add a code path for heap buffers in Reader#read to use the backing array instead of allocating a new one. * Change the code path for direct buffers in Reader#read to limit the intermediate allocation to `TRANSFER_BUFFER_SIZE`. * Implement `InputStreamReader#read(CharBuffer)` and delegate to `StreamDecoder`. * Implement `StreamDecoder#read(CharBuffer)` and avoid buffer allocation. ------------- Commit messages: - 4926314: Optimize Reader.read(CharBuffer) Changes: https://git.openjdk.java.net/jdk/pull/1915/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=1915&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-4926314 Stats: 101 lines in 3 files changed: 68 ins; 8 del; 25 mod Patch: https://git.openjdk.java.net/jdk/pull/1915.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/1915/head:pull/1915 PR: https://git.openjdk.java.net/jdk/pull/1915

7 47

RFR: 8253795: Implementation of JEP 391: macOS/AArch64 Port
by Anton Kozlov 25 Mar '21

25 Mar '21

Please review the implementation of JEP 391: macOS/AArch64 Port. It's heavily based on existing ports to linux/aarch64, macos/x86_64, and windows/aarch64. Major changes are in: * src/hotspot/cpu/aarch64: support of the new calling convention (subtasks JDK-8253817, JDK-8253818) * src/hotspot/os_cpu/bsd_aarch64: copy of os_cpu/linux_aarch64 with necessary adjustments (JDK-8253819) * src/hotspot/share, test/hotspot/gtest: support of write-xor-execute (W^X), required on macOS/AArch64 platform. It's implemented with pthread_jit_write_protect_np provided by Apple. The W^X mode is local to a thread, so W^X mode change relates to the java thread state change (for java threads). In most cases, JVM executes in write-only mode, except when calling a generated stub like SafeFetch, which requires a temporary switch to execute-only mode. The same execute-only mode is enabled when a java thread executes in java or native states. This approach of managing W^X mode turned out to be simple and efficient enough. * src/jdk.hotspot.agent: serviceability agent implementation (JDK-8254941) ------------- Commit messages: - Fix build - JDK-8253816: Update after recent changes - Rework gtests to always have wx_write - Revert gtest changes - Fix gtests in debug - Merge remote-tracking branch 'upstream/master' into jdk-macos - Fix windows_aarch64 - Use r18_tls instead of r18_reserved - JDK-8253742: bsd_aarch64 part - JDK-8257882: bsd_aarch64 part - ... and 40 more: https://git.openjdk.java.net/jdk/compare/82adfb32...3d4f4c23 Changes: https://git.openjdk.java.net/jdk/pull/2200/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=2200&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8253795 Stats: 3335 lines in 117 files changed: 3230 ins; 41 del; 64 mod Patch: https://git.openjdk.java.net/jdk/pull/2200.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/2200/head:pull/2200 PR: https://git.openjdk.java.net/jdk/pull/2200

26 231

RFR: 8260605: Various java.lang.invoke cleanups
by Claes Redestad 18 Mar '21

18 Mar '21

- Remove unused code - Inline and simplify the bootstrap method invocation code (remove pointless reboxing checks etc) - Apply pattern matching to make some code more readable ------------- Commit messages: - Avoid unnecessary reboxing checks, inline and simplify class/mt invokes - j.l.invoke cleanups Changes: https://git.openjdk.java.net/jdk/pull/2300/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=2300&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8260605 Stats: 269 lines in 14 files changed: 11 ins; 161 del; 97 mod Patch: https://git.openjdk.java.net/jdk/pull/2300.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/2300/head:pull/2300 PR: https://git.openjdk.java.net/jdk/pull/2300

2 14

RFR: 8259925: [Vector API] Unreasonable IndexOutOfBoundsException message when length < vlen
by Jie Fu 17 Mar '21

17 Mar '21

Hi all, For this reproducer: import jdk.incubator.vector.ByteVector; import jdk.incubator.vector.VectorSpecies; public class Test { static final VectorSpecies<Byte> SPECIES_128 = ByteVector.SPECIES_128; static byte[] a = new byte[8]; static byte[] b = new byte[8]; public static void main(String[] args) { ByteVector av = ByteVector.fromArray(SPECIES_128, a, 0); av.intoArray(b, 0); System.out.println("b: " + b[0]); } } The following IndexOutOfBoundsException message (length -7) is unreasonable. Exception in thread "main" java.lang.IndexOutOfBoundsException: Index 0 out of bounds for length -7 It might be better to fix it like this. Exception in thread "main" java.lang.IndexOutOfBoundsException: Index 0 out of bounds for length 0 Thanks. Best regards, Jie ------------- Commit messages: - 8259925: [Vector API] Unreasonable IndexOutOfBoundsException message when length < vlen Changes: https://git.openjdk.java.net/jdk/pull/2127/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=2127&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8259925 Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod Patch: https://git.openjdk.java.net/jdk/pull/2127.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/2127/head:pull/2127 PR: https://git.openjdk.java.net/jdk/pull/2127

2 12

RFR: 8080272 Refactor I/O stream copying to use java.io.InputStream.transferTo
by Andrey Turbanov 16 Mar '21

16 Mar '21

8080272 Refactor I/O stream copying to use java.io.InputStream.transferTo ------------- Commit messages: - 8080272: Refactor I/O stream copying to use java.io.InputStream.transferTo - 8080272: Refactor I/O stream copying to use java.io.InputStream.transferTo Changes: https://git.openjdk.java.net/jdk/pull/1853/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=1853&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8080272 Stats: 319 lines in 26 files changed: 1 ins; 248 del; 70 mod Patch: https://git.openjdk.java.net/jdk/pull/1853.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/1853/head:pull/1853 PR: https://git.openjdk.java.net/jdk/pull/1853

10 57

RFR: 8248862: Implement Enhanced Pseudo-Random Number Generators
by Jim Laskey 15 Mar '21

15 Mar '21

This PR is to introduce a new random number API for the JDK. The primary API is found in RandomGenerator and RandomGeneratorFactory. Further description can be found in the JEP https://openjdk.java.net/jeps/356 . ------------- Commit messages: - 8248862: Implement Enhanced Pseudo-Random Number Generators - Merge branch 'master' into 8248862 - Merge branch 'master' into 8248862 - 8248862: Implement Enhanced Pseudo-Random Number Generators - Merge branch 'master' into 8248862 - 8248862: Implement Enhanced Pseudo-Random Number Generators - 8248862; Implement Enhanced Pseudo-Random Number Generators - Merge branch 'master' into 8248862 - 8248862: Implement Enhanced Pseudo-Random Number Generators - Merge branch 'master' into 8248862 - ... and 27 more: https://git.openjdk.java.net/jdk/compare/9efbb463...7469ca58 Changes: https://git.openjdk.java.net/jdk/pull/1273/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=1273&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8248862 Stats: 14875 lines in 31 files changed: 11094 ins; 3704 del; 77 mod Patch: https://git.openjdk.java.net/jdk/pull/1273.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/1273/head:pull/1273 PR: https://git.openjdk.java.net/jdk/pull/1273

9 24

RFR: 8166026: Refactor java/lang shell tests to java
by Ivan Šipka 03 Mar '21

03 Mar '21

Refactor `test/jdk/java/lang/annotation/loaderLeak/LoaderLeak.sh` as java test. ------------- Commit messages: - 8166026: Refactor java/lang shell tests to java Changes: https://git.openjdk.java.net/jdk/pull/1577/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=1577&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8166026 Stats: 501 lines in 6 files changed: 196 ins; 305 del; 0 mod Patch: https://git.openjdk.java.net/jdk/pull/1577.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/1577/head:pull/1577 PR: https://git.openjdk.java.net/jdk/pull/1577

4 31

RFR: 8254001: [Metrics] Enhance parsing of cgroup interface files for version detection
by Severin Gehwolf 24 Feb '21

24 Feb '21

This is an enhancement which solves two issues: 1. Multiple reads of relevant cgroup interface files. Now interface files are only read once per file (just like Hotspot). 2. Proxies creation of the impl specific subsystem via `determineType()` as before, but now reads all relevant interface files: `/proc/cgroups`, `/proc/self/mountinfo` and `/proc/self/cgroup`. Once read it passes the parsed information to the impl specific subsystem classes for instantiation. This allows for more flexibility of testing as interface files can be mocked and, thus, more cases can be tested that way without having access to these specific systems. For example, proper regression tests for JDK-8217766 and JDK-8253435 have been added now with this in place. * [x] Tested on Linux x86_64 on cgroups v1 and cgroups v2. Container tests pass. ------------- Commit messages: - 8254001: [Metrics] Enhance parsing of cgroup interface files for version detection Changes: https://git.openjdk.java.net/jdk/pull/1393/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=1393&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8254001 Stats: 537 lines in 5 files changed: 268 ins; 193 del; 76 mod Patch: https://git.openjdk.java.net/jdk/pull/1393.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/1393/head:pull/1393 PR: https://git.openjdk.java.net/jdk/pull/1393

6 26

RFR: 8247918: Clarify Reader.skip behavior for end of stream
by Brian Burkhalter 19 Feb '21

19 Feb '21

Please review this clarification of the specification of the method `skip(long)` in `java.io.Reader` and its subclasses. Specifically, the behavior of the method is made clear for the case when the `Reader` is already at the end of its stream when the method is invoked. A corresponding CSR will be filed. Also, the change includes an update to an existing test in order to verify that the specification change reflects actual behavior. ------------- Commit messages: - 8247918: Clarify Reader.skip behavior for end of stream Changes: https://git.openjdk.java.net/jdk/pull/2274/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=2274&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8247918 Stats: 132 lines in 8 files changed: 62 ins; 38 del; 32 mod Patch: https://git.openjdk.java.net/jdk/pull/2274.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/2274/head:pull/2274 PR: https://git.openjdk.java.net/jdk/pull/2274

5 14