Hi Yasumasa, Thank you for the quick fix! It looks good. I'll sponsor your fix after we get at least one more review.
Thanks, Serguei On 10/9/17 19:03, Yasumasa Suenaga wrote:
Hi all, The following serviceability/sa tests are failed after 8187403: serviceability/sa/TestHeapDumpForInvokeDynamic.java serviceability/sa/TestHeapDumpForLargeArray.java serviceability/sa/jmap-hprof/JMapHProfLargeHeapTest.java These failures are caused by the address of HeapRegion. The address which is passed to c'tor of HeapRegion might not be OopHandle. So we have to switch the method of address calculation. I uploaded webrev for this issue. Could you review it? http://cr.openjdk.java.net/~ysuenaga/JDK-8189069/webrev.00/ I cannot access JPRT. So I need a sponsor. Thanks, Yasumasa
