Re: RFR: JDK-8065773: JDI: UOE is not thrown, when redefineClasses changes a class modifier

2019-02-09 Thread David Holmes
On 9/02/2019 11:29 pm, gary.ad...@oracle.com wrote: I'm willing to do the leg work on cleaning up the tests, but I'll need some help identifying where the redefine operations are missing the inner_class_flags checks. src/hotspot/share/prims/jvmtiRedefineClasses.cpp This code needs expanding:

Re: RFR 8218705: Test sun/tools/jcmd/TestJcmdDefaults.java fails on Linux

2019-02-09 Thread Daniil Titov
Hi David, Thank you for reviewing this change. You are right, these tests should be run before pushing JDK-8218705. I apologize for that. I run the usual set (tier1,tier2,tier3 and serviceability tests) but it was not enough in this case. Best regards, Daniil On 2/9/19, 4:31 AM, "David Holme

Re: RFR: JDK-8065773: JDI: UOE is not thrown, when redefineClasses changes a class modifier

2019-02-09 Thread gary.ad...@oracle.com
I'm willing to do the leg work on cleaning up the tests, but I'll need some help identifying where the redefine operations are missing the inner_class_flags checks. Is spec change or a CSR required when we change the current behavior to be more strict? On 2/8/19 9:24 PM, David Holmes wrote:

Re: RFR 8218705: Test sun/tools/jcmd/TestJcmdDefaults.java fails on Linux

2019-02-09 Thread David Holmes
Hi Daniil, The fix seems reasonable. These tests should all have been run before pushing JDK-8205654. Thanks, David -- On 9/02/2019 3:37 pm, Daniil Titov wrote: Please review the change that fix the test failure. The problem here is that the code in sun.tools.common.ProcessArgumentMatch

Re: [RFR]8215622: Add dump to file support for jmap histo

2019-02-09 Thread 臧琳
Thanks All for reviewing the change and CSR, I am still OOTO now and I will update the webrev later and ask your help for review again. Happy Chinese New Year! Cheers, Lin From: serguei.spit...@oracle.com Sent: Wednesday, February 6, 2019 6:44:43 AM To: