RFR: 8325821: [REDO] use "dmb.ishst+dmb.ishld" for release barrier [v7]
kuaiwei
duke at openjdk.org
Tue May 14 02:28:06 UTC 2024
On Fri, 10 May 2024 08:02:30 GMT, Andrew Haley <aph at openjdk.org> wrote:
> > @theRealAph Could you help review this PR? Thanks.
>
> I think we should go with your original simple patch for now. Trying to make the Assembler do the optimal thing has not turned out to be very easy, and I'm worried it's too much of a maintenance burden. Simply emitting `dmb st; dmb ld` for releasing stores is enough for now.
>
> Thank you for trying to make this work. I still have in my mind that there might be an easy way to do it, but it's looking unlikely.
I also think it has risk to pending instruction. This PR has much discussion on state machine. I will submit a new PR with my origin patch.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/18467#issuecomment-2109155105
More information about the hotspot-dev
mailing list