Looks good. On Jan 21, 2014, at 12:41 AM, Olivier Lagneau <olivier.lagn...@oracle.com> wrote:
> Please find the new webrev with copyright date fixed (changed to 2014). > > Webrev: http://cr.openjdk.java.net/~olagneau/8019389/webrev.01/ > > Olivier. > > Olivier Lagneau said on date 1/20/2014 5:50 PM: >> >> Oops, right ! >> >> Will fix that. >> >> Olivier. >> >> shanliang said on date 1/20/2014 4:13 PM: >>> >>> Olivier, >>> >>> Now it is 2014 :) >>> >>> >>> Olivier Lagneau wrote: >>>> >>>> Please review the following simple fix. >>>> >>>> Issue: https://bugs.openjdk.java.net/browse/JDK-8019389 >>>> Webrev: http://cr.openjdk.java.net/~olagneau/8019389/webrev.00/ >>>> >>>> The issue is due to the fact that _invokeHandle bytecode is generated by >>>> hotspot, >>>> but is not declared in agent code. Just declaring the new bytecode solves >>>> the assertion failure. >>>> >>>> However the tests reported in 8019389 (bootstrapOtherStratumInStackTrace, >>>> targetOtherStratumInStackTrace) >>>> suffer the problem from JDK-7016268 : Can't get strata information >>>> through SA-JDI >>>> Thus, the "stratum mismatch" related to JDK-7016268 will still be present >>>> after fix. >>>> This second problem has to be fixed through JDK-7016268. >>>> >>>> Thanks, >>>> Olivier. >>> >> >
