[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