RFR: 8325821: [REDO] use "dmb.ishst+dmb.ishld" for release barrier [v5]

kuaiwei duke at openjdk.org
Thu May 30 08:30:06 UTC 2024


On Wed, 29 May 2024 11:34:09 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

>> kuaiwei has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Remove tailing white space
>
> Note that current jcstress run would likely fail due to [JDK-8332670](https://bugs.openjdk.org/browse/JDK-8332670).

> > This looks ready to me. I think we need jcstress with C1 and C2, and we should be done. @shipilev , do you agree?
> 
> Yes. Just run jcstress with defaults, maybe limiting the time budget to about 24 hours, and we are done. Default configuration would work through different combinations of C1/C2 compilations for all actors, which is what we want to check for this change: that we don't mess up the barrier emitting scheme in different compilers/interpreters.

I can run the jcstress test. I will run fastdebug build with 
`java -jar jcstress-latest.jar -tb 24h`
Is it the correct command ?

-------------

PR Comment: https://git.openjdk.org/jdk/pull/19278#issuecomment-2138995483


More information about the hotspot-dev mailing list