Re: RFR: 8254162: Implementation of Foreign-Memory Access API (Third Incubator) [v26]

2020-11-10 Thread Alan Bateman
On Mon, 9 Nov 2020 16:07:13 GMT, Maurizio Cimadamore wrote: >> This patch contains the changes associated with the third incubation round >> of the foreign memory access API incubation (see JEP 393 [1]). This >> iteration focus on improving the usability of the API in 3 main ways: >> >> * fi

Re: RFR: 8254231: Implementation of Foreign Linker API (Incubator) [v17]

2020-11-10 Thread Maurizio Cimadamore
> This patch contains the changes associated with the first incubation round of > the foreign linker access API incubation > (see JEP 389 [1]). This work is meant to sit on top of the foreign memory > access support (see JEP 393 [2] and associated pull request [3]). > > The main goal of this API

Re: RFR: 8254231: Implementation of Foreign Linker API (Incubator) [v18]

2020-11-10 Thread Maurizio Cimadamore
> This patch contains the changes associated with the first incubation round of > the foreign linker access API incubation > (see JEP 389 [1]). This work is meant to sit on top of the foreign memory > access support (see JEP 393 [2] and associated pull request [3]). > > The main goal of this API

Re: RFR: 8254231: Implementation of Foreign Linker API (Incubator) [v15]

2020-11-10 Thread Jorn Vernee
On Mon, 9 Nov 2020 04:10:30 GMT, David Holmes wrote: >> Maurizio Cimadamore has updated the pull request with a new target base due >> to a merge or a rebase. The pull request now contains 64 commits: >> >> - Merge branch '8254162' into 8254231_linker >> - Fix post-merge issues caused by 8219

Re: RFR: 8255559: Leak File Descriptors Because of ResolverLocalFilesystem#engineResolveURI()

2020-11-10 Thread Weijun Wang
On Tue, 10 Nov 2020 11:37:04 GMT, Prajwal Kumaraswamy wrote: > …ineResolveURI(). > > Actual fix looks like this, due to git diff there are lot of changes( mostly > because of the spaces) being displayed. > > --- a/src/share/classes/org/jcp/xml/dsig/internal/dom/DOMReference.java > +++ b/src/

Re: RFR: 8255559: Leak File Descriptors Because of ResolverLocalFilesystem#engineResolveURI()

2020-11-10 Thread Prajwal Kumaraswamy
On Tue, 10 Nov 2020 13:26:01 GMT, Weijun Wang wrote: >> …ineResolveURI(). >> >> Actual fix looks like this, due to git diff there are lot of changes( >> mostly because of the spaces) being displayed. >> >> --- a/src/share/classes/org/jcp/xml/dsig/internal/dom/DOMReference.java >> +++ b/src/sh

RFR: 8255559: Leak File Descriptors Because of ResolverLocalFilesystem#engineResolveURI()

2020-11-10 Thread Prajwal Kumaraswamy
…ineResolveURI(). Actual fix looks like this, due to git diff there are lot of changes( mostly because of the spaces) being displayed. --- a/src/share/classes/org/jcp/xml/dsig/internal/dom/DOMReference.java +++ b/src/share/classes/org/jcp/xml/dsig/internal/dom/DOMReference.java @@ -500,6 +500,7

Re: RFR: 8255559: Leak File Descriptors Because of ResolverLocalFilesystem#engineResolveURI()

2020-11-10 Thread Weijun Wang
On Tue, 10 Nov 2020 11:37:04 GMT, Prajwal Kumaraswamy wrote: > …ineResolveURI(). > > Actual fix looks like this, due to git diff there are lot of changes( mostly > because of the spaces) being displayed. > > --- a/src/share/classes/org/jcp/xml/dsig/internal/dom/DOMReference.java > +++ b/src/

Re: RFR: 8255559: Leak File Descriptors Because of ResolverLocalFilesystem#engineResolveURI()

2020-11-10 Thread Weijun Wang
On Tue, 10 Nov 2020 17:14:42 GMT, Weijun Wang wrote: >> …ineResolveURI(). >> >> Actual fix looks like this, due to git diff there are lot of changes( >> mostly because of the spaces) being displayed. >> >> --- a/src/share/classes/org/jcp/xml/dsig/internal/dom/DOMReference.java >> +++ b/src/sh

Re: RFR: 8254231: Implementation of Foreign Linker API (Incubator) [v15]

2020-11-10 Thread Coleen Phillimore
On Mon, 9 Nov 2020 16:31:23 GMT, Jorn Vernee wrote: >> src/hotspot/cpu/aarch64/universalUpcallHandler_aarch64.cpp line 121: >> >>> 119: upcall_info.upcall_method.name, >>> upcall_info.upcall_method.sig, >>> 120: &args, thread); >>> 121: } >> >>

Re: RFR: 8254231: Implementation of Foreign Linker API (Incubator) [v18]

2020-11-10 Thread Coleen Phillimore
On Tue, 10 Nov 2020 14:16:22 GMT, Maurizio Cimadamore wrote: >> This patch contains the changes associated with the first incubation round >> of the foreign linker access API incubation >> (see JEP 389 [1]). This work is meant to sit on top of the foreign memory >> access support (see JEP 393

Re: RFR: 8254231: Implementation of Foreign Linker API (Incubator) [v18]

2020-11-10 Thread David Holmes
On Tue, 10 Nov 2020 14:16:22 GMT, Maurizio Cimadamore wrote: >> This patch contains the changes associated with the first incubation round >> of the foreign linker access API incubation >> (see JEP 389 [1]). This work is meant to sit on top of the foreign memory >> access support (see JEP 393