Re: [PATCH] OpenACC reference count overhaul

Hi! On 2019-12-11T18:22:00+0100, I wrote: > On 2019-10-29T12:15:01+, Julian Brown wrote: >> I've removed the special-case handling >> of pointers in the enter/exit data code, and combined the >> gomp_acc_remove_pointer code (which now iterated over mappings >> one-at-a-time anyway) with the

Re: 'find_group_last' (was: [PATCH] OpenACC reference count overhaul)

On Wed, 18 Dec 2019 10:18:14 +0100 Thomas Schwinge wrote: > Hi Julian! > > Thanks for walking me through this. > > On 2019-12-14T00:19:04+, Julian Brown > wrote: > > On Fri, 13 Dec 2019 16:25:25 +0100 > > Thomas Schwinge wrote: > >> On 2019-10-29T12:15:01+, Julian Brown > >>

Re: 'find_group_last' (was: [PATCH] OpenACC reference count overhaul)

Hi Julian! Thanks for walking me through this. On 2019-12-14T00:19:04+, Julian Brown wrote: > On Fri, 13 Dec 2019 16:25:25 +0100 > Thomas Schwinge wrote: >> On 2019-10-29T12:15:01+, Julian Brown >> wrote: >> > static int >> > -find_pointer (int pos, size_t mapnum, unsigned short

Re: 'find_group_last' (was: [PATCH] OpenACC reference count overhaul)

On Sat, 14 Dec 2019 00:19:04 + Julian Brown wrote: > On Fri, 13 Dec 2019 16:25:25 +0100 > Thomas Schwinge wrote: > > > Hi Julian! > > > > On 2019-10-29T12:15:01+, Julian Brown > > wrote: > > > static int > > > -find_pointer (int pos, size_t mapnum, unsigned short *kinds) > > >

Re: 'find_group_last' (was: [PATCH] OpenACC reference count overhaul)

On Fri, 13 Dec 2019 16:25:25 +0100 Thomas Schwinge wrote: > Hi Julian! > > On 2019-10-29T12:15:01+, Julian Brown > wrote: > > static int > > -find_pointer (int pos, size_t mapnum, unsigned short *kinds) > > +find_group_last (int pos, size_t mapnum, unsigned short *kinds) > > { > > - if

'find_group_last' (was: [PATCH] OpenACC reference count overhaul)

Hi Julian! On 2019-10-29T12:15:01+, Julian Brown wrote: > On Mon, 21 Oct 2019 16:14:11 +0200 > Thomas Schwinge wrote: > >> On 2019-10-03T09:35:04-0700, Julian Brown >> wrote: >> > --- a/libgomp/oacc-parallel.c >> > +++ b/libgomp/oacc-parallel.c >> > @@ -56,12 +56,29 @@ find_pointer (int

Re: [PATCH] OpenACC reference count overhaul

Hi! On 2019-10-29T12:15:01+, Julian Brown wrote: > On Mon, 21 Oct 2019 16:14:11 +0200 > Thomas Schwinge wrote: >> On 2019-10-03T09:35:04-0700, Julian Brown >> wrote: >> > void >> > -gomp_acc_remove_pointer (void *h, size_t s, bool force_copyfrom, >> > int async, >> > -

Re: [PATCH] OpenACC reference count overhaul

On 2019/12/10 12:04 AM, Julian Brown wrote: I'm citing below the changes introducing 'gomp_remove_var_async', modelled similar to the existing 'gomp_unmap_vars_async'. Also for both these, do I understand correctly, that it's actually not the 'gomp_unref_tgt' that needs to be "delayed" via

PR92881 "[OpenACC] In async context, need to use 'gomp_remove_var_async' instead of 'gomp_remove_var'" (was: [PATCH] OpenACC reference count overhaul)

Hi Julian! On 2019-12-09T15:04:15+, Julian Brown wrote: > On Mon, 9 Dec 2019 15:44:25 +0100 > Thomas Schwinge wrote: >> On 2019-10-03T09:35:04-0700, Julian Brown >> wrote: >> > --- a/libgomp/oacc-mem.c >> > +++ b/libgomp/oacc-mem.c >> >> > @@ -715,48 +684,34 @@ delete_copyout (unsigned

[PR92116, PR92877] [OpenACC] Replace 'openacc.data_environ' by standard libgomp mechanics (was: [PATCH] OpenACC reference count overhaul)

Hi! \o/ Yay for the first split-out piece of the big "OpenACC reference count overhaul" going in: On 2019-10-29T12:15:01+, Julian Brown wrote: > On Mon, 21 Oct 2019 16:14:11 +0200 > Thomas Schwinge wrote: >> Remeber to look into "Potential null >> pointer

Re: [PATCH] OpenACC reference count overhaul

On Mon, 9 Dec 2019 15:44:25 +0100 Thomas Schwinge wrote: > Hi Julian! > > On 2019-10-03T09:35:04-0700, Julian Brown > wrote: > > --- a/libgomp/oacc-mem.c > > +++ b/libgomp/oacc-mem.c > > > @@ -715,48 +684,34 @@ delete_copyout (unsigned f, void *h, size_t > > s, int async, const char

Re: [PATCH] OpenACC reference count overhaul

Hi Julian! On 2019-10-03T09:35:04-0700, Julian Brown wrote: > --- a/libgomp/oacc-mem.c > +++ b/libgomp/oacc-mem.c > @@ -715,48 +684,34 @@ delete_copyout (unsigned f, void *h, size_t s, int > async, const char *libfnname) >if (f & FLAG_COPYOUT) > [...] > gomp_copy_dev2host

Re: [PATCH] OpenACC reference count overhaul

On Sat, 9 Nov 2019 01:28:51 + Julian Brown wrote: > On Thu, 31 Oct 2019 19:11:57 +0100 > Thomas Schwinge wrote: > > > So that's not related to reference counting, needs to be discussed > > separately. > > > > ..., and while I do agree that the current code is a bit "strange" > >

Re: [PATCH] OpenACC reference count overhaul

On Thu, 31 Oct 2019 19:11:57 +0100 Thomas Schwinge wrote: > Hi Julian! > > On 2019-10-29T12:15:01+, Julian Brown > wrote: > > This is a new version of the patch which hopefully addresses all > > review comments. Further commentary below. > > Thanks, great, looking into that one -- I see

Re: [PATCH] OpenACC reference count overhaul

Hi Julian! On 2019-10-29T12:15:01+, Julian Brown wrote: > This is a new version of the patch which hopefully addresses all review > comments. Further commentary below. Thanks, great, looking into that one -- I see you're removing more and more special-case, strange code, replacing it with

Re: [PATCH] OpenACC reference count overhaul

Hi! This is a new version of the patch which hopefully addresses all review comments. Further commentary below. On Mon, 21 Oct 2019 16:14:11 +0200 Thomas Schwinge wrote: > On 2019-10-03T09:35:04-0700, Julian Brown > wrote: > > This patch has been broken out of the patch supporting OpenACC 2.6

Re: [PATCH] OpenACC reference count overhaul

Hi Julian! On 2019-10-21T16:14:11+0200, I wrote: > On 2019-10-03T09:35:04-0700, Julian Brown wrote: >> This patch has been broken out of the patch supporting OpenACC 2.6 manual >> deep copy last posted here: >> >> https://gcc.gnu.org/ml/gcc-patches/2018-12/msg01084.html > > Thanks. I

Re: [PATCH] OpenACC reference count overhaul

Hi! On 2019-10-03T09:35:04-0700, Julian Brown wrote: > This patch has been broken out of the patch supporting OpenACC 2.6 manual > deep copy last posted here: > > https://gcc.gnu.org/ml/gcc-patches/2018-12/msg01084.html Thanks. Remeber to look into "Potential

Re: [PATCH] OpenACC reference count overhaul

regressing tests no longer regress, but I haven't yet made any changes to VREFCOUNT_LINK_KEY, etc. as suggested in the review of the attach/detach patch: https://gcc.gnu.org/ml/gcc-patches/2019-10/msg01374.html OK? (ChangeLog as before.) Julian >From e7b21dfe343f1ca556652cc54793e0656eb95685 Mon Sep 17 00:00:00 2001

Re: [PATCH] OpenACC reference count overhaul

Hi Julian! On 2019-10-03T09:35:04-0700, Julian Brown wrote: > This patch has been broken out of the patch supporting OpenACC 2.6 manual > deep copy last posted here: > > https://gcc.gnu.org/ml/gcc-patches/2018-12/msg01084.html Thanks. > a couple of > tests need fixing also Let's look at

[PATCH] OpenACC reference count overhaul

This patch has been broken out of the patch supporting OpenACC 2.6 manual deep copy last posted here: https://gcc.gnu.org/ml/gcc-patches/2018-12/msg01084.html As part of developing that patch, libgomp's OpenACC reference counting implementation proved to be somewhat inconsistent, especially