RFR: 8257997: sun/security/ssl/SSLSocketImpl/SSLSocketLeak.java again reports leaks after JDK-8257884

2020-12-10 Thread Christoph Langer
The fix for [JDK-8257884](https://bugs.openjdk.java.net/browse/JDK-8257884) had a flaw which made the test fail even more often on Windows than before. Here is the correction. - Commit messages: - 8257997: sun/security/ssl/SSLSocketImpl/SSLSocketLeak.java again reports leaks after

Re: RFR: 8257997: sun/security/ssl/SSLSocketImpl/SSLSocketLeak.java again reports leaks after JDK-8257884

2020-12-10 Thread Matthias Baesken
On Thu, 10 Dec 2020 08:34:31 GMT, Christoph Langer wrote: > The fix for [JDK-8257884](https://bugs.openjdk.java.net/browse/JDK-8257884) > had a flaw which made the test fail even more often on Windows than before. > Here is the correction. Marked as reviewed by mbaesken (Reviewer). --

Integrated: 8257997: sun/security/ssl/SSLSocketImpl/SSLSocketLeak.java again reports leaks after JDK-8257884

2020-12-10 Thread Christoph Langer
On Thu, 10 Dec 2020 08:34:31 GMT, Christoph Langer wrote: > The fix for [JDK-8257884](https://bugs.openjdk.java.net/browse/JDK-8257884) > had a flaw which made the test fail even more often on Windows than before. > Here is the correction. This pull request has now been integrated. Changeset:

Re: RFR: 8257733: Move module-specific data from make to respective module [v2]

2020-12-10 Thread Naoto Sato
On Mon, 7 Dec 2020 14:27:45 GMT, Magnus Ihse Bursie wrote: >> A lot (but not all) of the data in make/data is tied to a specific module. >> For instance, the publicsuffixlist is used by java.base, and fontconfig by >> java.desktop. (A few directories, like mainmanifest, is *actually* used by >

Re: RFR: 8255917: runtime/cds/SharedBaseAddress.java failed "assert(reserved_rgn != 0LL) failed: No reserved region" [v5]

2020-12-10 Thread Yumin Qi
> Hi, Please review > Windows mapping for file into memory could not happen to reserved memory. > In mapping CDS archive we first reserve enough memory then before mapping, > release them. For cds archive and using class space, need split the whole > space into two, that is, release the whole

Withdrawn: 8255917: runtime/cds/SharedBaseAddress.java failed "assert(reserved_rgn != 0LL) failed: No reserved region"

2020-12-10 Thread Yumin Qi
On Mon, 7 Dec 2020 05:01:27 GMT, Yumin Qi wrote: > Hi, Please review > Windows mapping for file into memory could not happen to reserved memory. > In mapping CDS archive we first reserve enough memory then before mapping, > release them. For cds archive and using class space, need split the w

Re: RFR: 8255917: runtime/cds/SharedBaseAddress.java failed "assert(reserved_rgn != 0LL) failed: No reserved region" [v5]

2020-12-10 Thread Yumin Qi
On Tue, 8 Dec 2020 06:12:36 GMT, Yumin Qi wrote: >> Changes requested by iklam (Reviewer). > > Please check 03. 02 is generated when merge with most current and remote head > not updated correctly. After set remote head correct, 03 is regenerated and > is correct one for review. Thanks This br