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

[coleen.phillimore at oracle.com]

On 1/6/20 11:09 PM, serguei.spitsyn at oracle.com wrote:
> Hi Coleen,
>
> Thank you for reviewing it!
> I've sent another trivial patch as suggested by Chris.
> It is for better consistency.

Yes, Chris's suggestion is better.
thanks!
Coleen
>
> Thanks,
> Seguei
>
>
> On 1/6/20 6:29 PM, coleen.phillimore at oracle.com wrote:
>> 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
>>
>