RFR(M): 8227680: FastJNIAccessors: Check for JVMTI field access event requests at runtime
erik.osterlund at oracle.com
Wed Jul 24 08:57:41 UTC 2019
Looks good for me.
Thanks for cleaning up this code!
> On 23 Jul 2019, at 19:50, Doerr, Martin <martin.doerr at sap.com> wrote:
> Hi Erik,
> adding Andrew and Aleksey.
>> The new webrev looks good.
>> Note though the following though... it looks like the AArch64 code
>> doesn't do appropriate fencing if the field is volatile.
> I agree. I was not aware of JDK-8179954 (https://bugs.openjdk.java.net/browse/JDK-8179954).
> My new aarch64 proposal:
> Note: All platforms were tested except arm (32 bit). I could also return (address)-1 if JvmtiExport::can_post_field_access() in case nobody wants this for arm.
> Best regards,
More information about the hotspot-runtime-dev