Re: RFR: 7169257: Fix for 6424631 only partially ported to JDK 7 & JDK 8

2012-05-22 Thread Seán Coffey
Ok, so I went off today to set up the initial manual unit testcase supplied with 6424631 report. It turns out that 6424631 should no longer be reproducible in 7u or later due to CR 7093090 changes (Reduce synchronization in java.security.Policy.getPolicyNoCheck) The 7093090 changes which went

Re: RFR: 7169257: Fix for 6424631 only partially ported to JDK 7 & JDK 8

2012-05-21 Thread Seán Coffey
Thanks Sean, Further testing has thrown up a testcase failure. (recursion in Policy setup) - This one skipped by me prior to logging RFR. Let me look into it and get back with an updated webrev. Regards, Sean. On 21/05/12 13:37, Sean Mullan wrote: The changes look good. --Sean On 5/21/12

Re: RFR: 7169257: Fix for 6424631 only partially ported to JDK 7 & JDK 8

2012-05-21 Thread Sean Mullan
The changes look good. --Sean On 5/21/12 6:25 AM, Seán Coffey wrote: > > Recently noticed that the 6424631 fix was not fully ported to the JDK 7 > workspace. Porting the changes (to 8 & 7) so that no regression is seen > for users migrating to latest JDK families. > > bug report : http://bugs

RFR: 7169257: Fix for 6424631 only partially ported to JDK 7 & JDK 8

2012-05-21 Thread Seán Coffey
Recently noticed that the 6424631 fix was not fully ported to the JDK 7 workspace. Porting the changes (to 8 & 7) so that no regression is seen for users migrating to latest JDK families. bug report : http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7169257 webrev : http://cr.openjdk.java.n