Thanks Tobias.

dl

On 12/18/18 4:08 AM, Tobias Hartmann wrote:
Hi Dean,

this looks good to me.

Best regards,
Tobias

On 17.12.18 23:49, dean.l...@oracle.com wrote:
https://bugs.openjdk.java.net/browse/JDK-8214329
http://cr.openjdk.java.net/~dlong/8214329/webrev/

In "8212605: Pure-Java implementation of AccessController.doPrivileged", the 
stackwalk in
JVM_GetStackAccessControlContext was changed from using a vframeStream to using 
a javaVFrame, so
that it could access locals. However this caused a regression because 
javaVFrame has more
overhead.   JDK-8215205 fixes the problem by allowing the use of javaVFrame 
only when needed.

This change reverts JVM_GetStackAccessControlContext back to using 
vframeStream, and uses the new
API in JDK-8215205 to get a javaVFrame lazily.  It also removes two unnecessary 
calls to java_sender().

dl

Reply via email to