[jdk11u-dev] RFR: 8261916: gtest/GTestWrapper.java vmErrorTest.unimplemented1_vm_assert failed

Thomas Stuefe stuefe at openjdk.java.net
Fri Nov 5 13:44:32 UTC 2021


Hi,

I'd like to backport "JDK-8263728: gtest/GTestWrapper.java vmErrorTest.unimplemented1_vm_assert failed", since it prevents sporadic gtest errors.

Original commit: https://github.com/openjdk/jdk/commit/8c1112a6

This one is not a clean backport:
- `vmError.cpp` changed slightly, since https://bugs.openjdk.java.net/browse/JDK-8258479 did some code cleanups for JDK17
- more importantly, this patch touches `os::abort()`. That function had been unified into `os_posix.cpp` for all posix platforms with https://bugs.openjdk.java.net/browse/JDK-8263564 for JDK 17. I briefly considered backporting JDK-8263564, but decided against it since it is too invasive and unnecessary. Instead, I changed the various os::abort() implementations for all individual OSes, including Solaris.

Tests: Nightlies at SAP, GHAs.

Thanks, Thomas

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

Commit messages:
 - backport JDK-8263728

Changes: https://git.openjdk.java.net/jdk11u-dev/pull/590/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk11u-dev&pr=590&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8261916
  Stats: 31 lines in 6 files changed: 16 ins; 4 del; 11 mod
  Patch: https://git.openjdk.java.net/jdk11u-dev/pull/590.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk11u-dev pull/590/head:pull/590

PR: https://git.openjdk.java.net/jdk11u-dev/pull/590


More information about the jdk-updates-dev mailing list