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

[Chris Cole]

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