Integrated: 8322538: remove fatal from JVM_VirtualThread functions for !INCLUDE_JVMTI

[Serguei Spitsyn]

On Thu, 21 Dec 2023 02:29:33 GMT, Serguei Spitsyn <sspitsyn at openjdk.org> wrote:

> The macro `#else` branch conditions of `#if INCLUDE_JVMTI` in the `JVM_VirtualThread*` methods/functions (see `jvm.cpp`) are incorrect and have to be removed.
> For example, the lines 4022-4023 have to be removed from the fragment below:
> 
>   4013	JVM_ENTRY(void, JVM_VirtualThreadDisableSuspend(JNIEnv* env, jobject vthread, jboolean enter))
>   4014	#if INCLUDE_JVMTI
>   4015	  if (!DoJVMTIVirtualThreadTransitions) {
>   4016	    assert(!JvmtiExport::can_support_virtual_threads(), "sanity check");
>   4017	    return;
>   4018	  }
>   4019	  assert(thread->is_disable_suspend() != (bool)enter,
>   4020	         "nested or unbalanced monitor enter/exit is not allowed");
>   4021	  thread->toggle_is_disable_suspend();
>   4022	#else
>   4023	  fatal("Should only be called with JVMTI enabled");
>   4024	#endif
>   4025	JVM_END

This pull request has now been integrated.

Changeset: aff659aa
Author:    Serguei Spitsyn <sspitsyn at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/aff659aaf7c73ff8eb903fd3e426e1b42ea6d95a
Stats:     12 lines in 1 file changed: 0 ins; 12 del; 0 mod

8322538: remove fatal from JVM_VirtualThread functions for !INCLUDE_JVMTI

Reviewed-by: dholmes, alanb

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

PR: https://git.openjdk.org/jdk/pull/17174