Looks good. Best regards, Vladimir Ivanov On 12/22/17 12:35 PM, Rahul Raghavan wrote:
Hi,
Request help to review the following fix proposal.
<webrev> - http://cr.openjdk.java.net/~rraghavan/8193699/webrev.01/
Related <jbs> links - -- https://bugs.openjdk.java.net/browse/JDK-8193699 -- https://bugs.openjdk.java.net/browse/JDK-8167372 - 'Add code to check for getting oops while thread is in native' -- https://bugs.openjdk.java.net/browse/JDK-8191227 - 'Unsafe handle resolution in ConstantOopWriteValue::write_on() / print_on() and LIR_Assembler::jobject2reg()'
Found root cause of 8193699 issue, as missing changes for aarch64 in 'macroAssembler_aarch64.cpp', similar to changes done for 8191227 before pushing 8167372 support. So now fixed the issue of unsafe handle resolution in 'macroAssembler_aarch64.cpp' with the usage of ThreadInVMfromUnknown. Stuart Monteith confirmed build failure fixed with above proposed changeset and no new issues with testing.
Thanks, Rahul