8267042: bug in monitor locking/unlocking on ARM32 C1 due to uninitialized BasicObjectLock::_displaced_header

[Boris Ulasevich]

Hi Chris,

Good. Thank you! Here is the link on pull request:
https://github.com/openjdk/jdk/pull/4218

regards,
Boris

On 27.05.2021 07:37, Chris Cole wrote:
> Hi Boris,
>
> Here is an updated test case at
> https://github.com/chrishcole/jdk/blob/arm32_c1_monitor_bug/test/hotspot/jtreg/compiler/c1/Test8267042.java.
>
> Thanks,
> Chris
>
> On Mon, May 24, 2021 at 11:56 AM Chris Cole <chris at sageembedded.com> wrote:
>> Hi Boris,
>>
>> On Sun, May 23, 2021 at 1:38 PM Boris Ulasevich
>> <boris.ulasevich at bell-sw.com> wrote:
>>> Hi Chris,
>>>
>>> I checked that the fix works fine and does not bring any regression.
>>> Let me create pull request and sponsor the change.
>>> Here is the fix and reproducer I prepared for review:
>>> https://github.com/openjdk/jdk/compare/master...bulasevich:jdk-8267042-arm32-c1-lock-01
>> Sounds great. Looks good to me.
>>
>>> (I think it is worth simplifying the MonitorBugTest)
>> Yes, I agree. If you want I could try and simplify this, but won't
>> have time to work on this until tomorrow.
>>
>> Thanks,
>> Chris