On Tue, 13 Jul 2021 10:13:39 GMT, Anton Kozlov <akoz...@openjdk.org> wrote:
>> The change adds W^X transition in RawMonitor family of functions, fixing the >> crash. RawMonitor functions are treated specially, so W^X transition is not >> inserted automatically. A better fix would be in .xml description for >> entries generation, but for now it is too risky. I hope to get this fixed in >> 17 in the simplest way and do re-work in the 18. This change is still 100% >> correct for the reported bug. >> >> Verified by: >> >> jtreg -vmoption:-XX:+AssertWXAtThreadSync >> test/hotspot/jtreg/vmTestbase/nsk/jvmti/scenarios/bcinstr/BI04/bi04t002 > > Anton Kozlov has updated the pull request incrementally with one additional > commit since the last revision: > > Subset of RawMonitor; comments added Thanks for the reviews. It's my fault I have not explained how W^X is supposed to work. I still does not consider this as the smell, but since someone thinks so, it's my bad. I'm going to create a page on cr.openjdk.java.net that will be a memo of the current state. ------------- PR: https://git.openjdk.java.net/jdk17/pull/244
