Re: RFR: 8283689: Update the foreign linker VM implementation [v10]

2022-05-12 Thread Jorn Vernee
On Thu, 12 May 2022 13:55:20 GMT, Nick Gasson wrote: >> That's fine, just needed to check. But as these are now being committed to >> mainline the year does need to change to 2022 - at least in Oracle >> copyright. @nick-arm will have to advise what he thinks should be done with >> the ARM

Re: RFR: 8283689: Update the foreign linker VM implementation [v10]

2022-05-12 Thread Jorn Vernee
On Thu, 12 May 2022 13:55:20 GMT, Nick Gasson wrote: >> That's fine, just needed to check. But as these are now being committed to >> mainline the year does need to change to 2022 - at least in Oracle >> copyright. @nick-arm will have to advise what he thinks should be done with >> the ARM

Re: RFR: 8283689: Update the foreign linker VM implementation [v10]

2022-05-12 Thread Nick Gasson
On Thu, 12 May 2022 13:07:24 GMT, David Holmes wrote: >> I think the Arm Ltd one was probably changed by me in one of the PRs in the >> Panama repo. > > That's fine, just needed to check. But as these are now being committed to > mainline the year does need to change to 2022 - at least in

Re: RFR: 8283689: Update the foreign linker VM implementation [v10]

2022-05-12 Thread David Holmes
On Thu, 12 May 2022 08:23:11 GMT, Nick Gasson wrote: >> src/hotspot/cpu/aarch64/foreign_globals_aarch64.cpp line 3: >> >>> 1: /* >>> 2: * Copyright (c) 2020, 2021, Oracle and/or its affiliates. All rights >>> reserved. >>> 3: * Copyright (c) 2019, 2021, Arm Limited. All rights reserved. >>

Re: RFR: 8283689: Update the foreign linker VM implementation [v10]

2022-05-12 Thread Jorn Vernee
On Thu, 12 May 2022 08:23:11 GMT, Nick Gasson wrote: >> src/hotspot/cpu/aarch64/foreign_globals_aarch64.cpp line 3: >> >>> 1: /* >>> 2: * Copyright (c) 2020, 2021, Oracle and/or its affiliates. All rights >>> reserved. >>> 3: * Copyright (c) 2019, 2021, Arm Limited. All rights reserved. >>

Re: RFR: 8283689: Update the foreign linker VM implementation [v10]

2022-05-12 Thread Nick Gasson
On Thu, 12 May 2022 03:34:19 GMT, David Holmes wrote: >> Jorn Vernee has updated the pull request with a new target base due to a >> merge or a rebase. The pull request now contains 26 commits: >> >> - Block async exceptions during upcalls >> - Merge branch 'foreign-preview-m' into

Re: RFR: 8283689: Update the foreign linker VM implementation [v10]

2022-05-11 Thread David Holmes
On Wed, 11 May 2022 17:51:31 GMT, Jorn Vernee wrote: >> Hi, >> >> This PR updates the VM implementation of the foreign linker, by bringing >> over commits from the panama-foreign repo. >> >> This is split off from the main JEP integration for 19, since we have >> limited resources to handle

Re: RFR: 8283689: Update the foreign linker VM implementation [v10]

2022-05-11 Thread Jorn Vernee
> Hi, > > This PR updates the VM implementation of the foreign linker, by bringing over > commits from the panama-foreign repo. > > This is split off from the main JEP integration for 19, since we have limited > resources to handle this. As such, this PR might fall over to 20, but it > would