[8u] RFR 8155951 & 8151066

Zhengyu Gu zgu at redhat.com
Tue Aug 20 16:43:58 UTC 2019


Hi Aleksey,

On 8/20/19 12:10 PM, Aleksey Shipilev wrote:
> On 8/13/19 8:13 PM, Zhengyu Gu wrote:
>> The patch does not apply cleanly. Other than a few minor conflicts, e.g. copyright years, breaking
>> single line long parameter list to multi-lines and line shifts, jdk8u lock declaration  (def macro
>> in mutexLocker.cpp) does not take 5th argument.
> 
> So, now that we acquire the lock with MutexLocker, it always checks for safepoint? Wanted to
> understand what makes dropping the 5th argument safe.

The 5th parameter is Monitor::_safepoint_check_always, so I think it is 
safe to drop it.

Thanks,

-Zhengyu



> 
>> 8u backport: http://cr.openjdk.java.net/~zgu/JDK-8155951-8u/webrev.00/index.html
> 
> Otherwise backport looks good to me.
> 


More information about the jdk8u-dev mailing list