Re: RFR: 8255822: Zero: improve build-time JVMTI handling [v3]

2020-11-09 Thread Aleksey Shipilev
> Current Zero interpreter has the optimization for JVMTI support. It > recognizes that JVMTI is disabled most of the time, and that JVMTI checks in > the interpreter code slows it down considerably. (I measured it myself when > working on this patch: removing this optimization yields about 20%

Re: RFR: 8256048: Incomplete gitignore setting for netbeans project [v3]

2020-11-09 Thread Erik Joelsson
On Mon, 9 Nov 2020 23:31:09 GMT, Jie Fu wrote: >> Hi all, >> >> The gitignore setting for netbeans project seems to be incomplete. >> Only nbproject/private/ [1] is ignored. >> >> But there are also other files under nbproject/, which should be ignored too. >> >>

Re: RFR: 8256048: Incomplete gitignore setting for netbeans project [v3]

2020-11-09 Thread Jie Fu
On Mon, 9 Nov 2020 18:03:31 GMT, Erik Joelsson wrote: >> You misunderstand me. Adding the following line is fine and will make git >> ignore any nbproject dir in the root of the repository, which is what I >> assume you want: >> >> >> But removing the following existing line is not, as this

Re: RFR: 8256048: Incomplete gitignore setting for netbeans project [v3]

2020-11-09 Thread Jie Fu
> Hi all, > > The gitignore setting for netbeans project seems to be incomplete. > Only nbproject/private/ [1] is ignored. > > But there are also other files under nbproject/, which should be ignored too. > > $ tree -L 1 nbproject/ > nbproject/ > ├──

Re: RFR: 8212879: Make JVMTI TagMap table concurrent [v6]

2020-11-09 Thread Coleen Phillimore
On Thu, 5 Nov 2020 12:42:40 GMT, Coleen Phillimore wrote: >> Thank you for the update, Coleen! >> I leave it for you to decide to refactor the gc_notification or not. >> Thanks, >> Serguei > > Thanks @sspitsyn . I'm going to leave the gc_notification code because > structurally the two sides

Re: RFR: 8256048: Incomplete gitignore setting for netbeans project [v2]

2020-11-09 Thread Erik Joelsson
On Mon, 9 Nov 2020 15:25:28 GMT, Jie Fu wrote: >> .gitignore line 5: >> >>> 3: /.idea/ >>> 4: /.vscode/ >>> 5: /nbproject/ >> >> I believe the existing nbproject/private/ is meant to catch any private >> files created by someone opening any of the various nbproject directories >> that exist

Re: RFR: 8256048: Incomplete gitignore setting for netbeans project [v2]

2020-11-09 Thread Erik Joelsson
On Mon, 9 Nov 2020 18:02:32 GMT, Erik Joelsson wrote: >> Thanks @erikj79 for your review. >> Updated. >> >> Any comments? >> Thanks. > > You misunderstand me. Adding the following line is fine and will make git > ignore any nbproject dir in the root of the repository, which is what I > assume

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

2020-11-09 Thread Maurizio Cimadamore
> 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: > > * first, by providing a way to obtain truly *shared* segments, which

Re: RFR: 8256048: Incomplete gitignore setting for netbeans project [v2]

2020-11-09 Thread Jie Fu
On Mon, 9 Nov 2020 13:53:35 GMT, Erik Joelsson wrote: >> Jie Fu has updated the pull request incrementally with one additional commit >> since the last revision: >> >> Don't remove nbproject/private/ > > .gitignore line 5: > >> 3: /.idea/ >> 4: /.vscode/ >> 5: /nbproject/ > > I believe the

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

2020-11-09 Thread Maurizio Cimadamore
> 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: > > * first, by providing a way to obtain truly *shared* segments, which

Re: RFR: 8256048: Incomplete gitignore setting for netbeans project [v2]

2020-11-09 Thread Jie Fu
> Hi all, > > The gitignore setting for netbeans project seems to be incomplete. > Only nbproject/private/ [1] is ignored. > > But there are also other files under nbproject/, which should be ignored too. > > $ tree -L 1 nbproject/ > nbproject/ > ├──

Re: RFR: 8256048: Incomplete gitignore setting for netbeans project

2020-11-09 Thread Erik Joelsson
On Mon, 9 Nov 2020 12:07:37 GMT, Jie Fu wrote: > Hi all, > > The gitignore setting for netbeans project seems to be incomplete. > Only nbproject/private/ [1] is ignored. > > But there are also other files under nbproject/, which should be ignored too. > > $ tree

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

2020-11-09 Thread Maurizio Cimadamore
> 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: > > * first, by providing a way to obtain truly *shared* segments, which

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

2020-11-09 Thread Maurizio Cimadamore
> 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: > > * first, by providing a way to obtain truly *shared* segments, which

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

2020-11-09 Thread Maurizio Cimadamore
On Sun, 8 Nov 2020 16:28:41 GMT, Alan Bateman 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 29 commits: >> >> - Fix post-merge issues caused by 8219014 >> - Merge branch 'master' into 8254162 >>

RFR: 8256048: Incomplete gitignore setting for netbeans project

2020-11-09 Thread Jie Fu
Hi all, The gitignore setting for netbeans project seems to be incomplete. Only nbproject/private/ [1] is ignored. But there are also other files under nbproject/, which should be ignored too. $ tree -L 1 nbproject/ nbproject/ ├── configurations.xml ├── private └──