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

[Thomas Stuefe]

On Fri, 5 Nov 2021 10:56:20 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:

> 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

This pull request has now been integrated.

Changeset: 46d85236
Author:    Thomas Stuefe <stuefe at openjdk.org>
URL:       https://git.openjdk.java.net/jdk11u-dev/commit/46d85236888e85bd22d7bd08adbc984fe1971bbc
Stats:     30 lines in 5 files changed: 16 ins; 3 del; 11 mod

8261916: gtest/GTestWrapper.java vmErrorTest.unimplemented1_vm_assert failed

Reviewed-by: clanger
Backport-of: 8c1112a690480340bb9d69bfaa463b5619829808

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

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