[15u] RFR (S) 8249192: MonitorInfo stores raw oops across safepoints

[Harold Seigel]

Hi Aleksey,

Since this fix was backported to JDK15-CPU, it will be in the 15.0.2 
release.  You do not need to do a separate backport to JDK-15u.

Thanks, Harold

On 9/15/2020 4:28 AM, Aleksey Shipilev wrote:
> On 9/15/20 3:29 AM, David Holmes wrote:
>> On 14/09/2020 5:55 pm, Aleksey Shipilev wrote:
>>> On 9/14/20 9:51 AM, David Holmes wrote:
>>>> Hi Aleksey,
>>>>
>>>> On 14/09/2020 5:29 pm, Aleksey Shipilev wrote:
>>>>> On 9/9/20 9:10 AM, Aleksey Shipilev wrote:
>>>>>> Original bug:
>>>>>>       https://bugs.openjdk.java.net/browse/JDK-8249192
>>>>>> https://hg.openjdk.java.net/jdk/jdk/rev/61f9028f360d
>>>>>>
>>>>>> 15u variant:
>>>>>> https://cr.openjdk.java.net/~shade/8249192/webrev.15u.01/
>>>>>>
>>>>>> It differs from jdk/jdk variant by a conflict in jvmtiEnvBase.cpp,
>>>>>> where removed "ResourceMark rm"
>>>>>> does not have "current_thread" argument yet. It was added later with
>>>>>> JDK-8247729:
>>>>>> https://hg.openjdk.java.net/jdk/jdk/rev/f8a9be0f9e1a#l2.82
>>>>>>
>>>>>> Testing: tier{1,2} with and without Shenandoah [which seems to find
>>>>>> some of these bugs]
>>>>>
>>>>> Um.
>>>>>
>>>>> I see there is a "backport" to 15.0.2 already:
>>>>>      https://bugs.openjdk.java.net/browse/JDK-8251485
>>>>>
>>>>> ...yet there is no changeset at all:
>>>>> https://hg.openjdk.java.net/jdk-updates/jdk15u/log?rev=8251485
>>>>>
>>>>> What's up with that? Rob, Thomas?
>>>>
>>>> I think an oversight on our (Oracle) end. It went into the 15u CPU 
>>>> repo
>>>> instead of the open repo.
>>>
>>> Ah. Well, I can push my versions of 8249192 (once reviewed here) and
>>> 8251118 to jdk-updates/jdk15u then. But it would be nice if someone 
>>> from
>>> Oracle would compare the 8249192 changeset above with what is pushed to
>>> 15u CPU repo.
>>
>> They are the same.
>
> Thanks, I'll count that as the backport review then.
>
> Added the fix-request with the synopsis of all this mess :)
>