RFR(XXS): 8236124: Minimal VM slowdebug build failed after JDK-8212160

coleen.phillimore at oracle.com coleen.phillimore at oracle.com
Tue Jan 7 02:29:36 UTC 2020


This looks trivial.  Thank you for fixing it.
Coleen

On 1/6/20 9:18 PM, serguei.spitsyn at oracle.com wrote:
> Please, review a trivial fix for bug:
>   https://bugs.openjdk.java.net/browse/JDK-8236124
>
> Patch suggested by A. Shipilev:
>
> diff --git a/src/hotspot/share/code/nmethod.cpp 
> b/src/hotspot/share/code/nmethod.cpp
> --- a/src/hotspot/share/code/nmethod.cpp
> +++ b/src/hotspot/share/code/nmethod.cpp
> @@ -1601,7 +1601,7 @@
>        ServiceThread::enqueue_deferred_event(&event);
>      } else {
>        // This enters the nmethod barrier outside in the caller.
> -      state->enqueue_event(&event);
> +      JVMTI_ONLY(state->enqueue_event(&event));
>      }
>    }
>  }
>
>
> Summary:
>   The slowdebug build was broken by the fix of JDK-8212160 which 
> introduced new function: enqueue_event().
>   The fix is to call it only if the JVM TI is enabled.
>
> Testing:
>   Ran slowdebug build locally.
>
> Thanks,
> Serguei



More information about the serviceability-dev mailing list