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

[Christoph Langer]

On Thu, 11 Nov 2021 14:32:01 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
>
> Thomas Stuefe has updated the pull request incrementally with one additional commit since the last revision:
> 
>   remove stray edit to os_posix.cpp

Marked as reviewed by clanger (Reviewer).

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

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