Re: [gomp4] OpenACC async re-work

2017-10-10 Thread Cesar Philippidis
On 10/10/2017 11:08 AM, Thomas Schwinge wrote: > Reported by Cesar for a test case similar to the one below, where we > observe: > > acc_prof-cuda-1.exe: [...]/libgomp/oacc-profiling.c:592: > goacc_profiling_dispatch_p: Assertion `thr->prof_info == NULL' failed. > > This is because of: > >

Re: [gomp4] OpenACC async re-work

2017-10-10 Thread Thomas Schwinge
Hi! Reported by Cesar for a test case similar to the one below, where we observe: acc_prof-cuda-1.exe: [...]/libgomp/oacc-profiling.c:592: goacc_profiling_dispatch_p: Assertion `thr->prof_info == NULL' failed. This is because of: On Tue, 25 Jul 2017 20:51:05 +0800, Chung-Lin Tang wrote:

Re: [gomp4] OpenACC async re-work

2017-07-25 Thread Chung-Lin Tang
On 2017/7/25 10:09 PM, Cesar Philippidis wrote: > On 07/25/2017 05:51 AM, Chung-Lin Tang wrote: >> On 2017/6/29 6:31 AM, Cesar Philippidis wrote: > >> Attached is the updated version of the patch, re-tested. >> >> Thomas, do you need some more time to look over it? Or should I commit it >> first?

Re: [gomp4] OpenACC async re-work

2017-07-25 Thread Cesar Philippidis
On 07/25/2017 05:51 AM, Chung-Lin Tang wrote: > On 2017/6/29 6:31 AM, Cesar Philippidis wrote: > Attached is the updated version of the patch, re-tested. > > Thomas, do you need some more time to look over it? Or should I commit it > first? I'm not too concerned about the profiling stuff becaus

Re: [gomp4] OpenACC async re-work

2017-07-25 Thread Chung-Lin Tang
On 2017/6/29 6:31 AM, Cesar Philippidis wrote: > On 06/27/2017 03:56 AM, Chung-Lin Tang wrote: >> On 2017/6/27 6:45 AM, Cesar Philippidis wrote: (1) Instead of essentially implementing the entire OpenACC async support inside the plugin, we now use an opaque 'goacc_asyncqueue' implemented

Re: [gomp4] OpenACC async re-work

2017-06-28 Thread Cesar Philippidis
On 06/27/2017 03:56 AM, Chung-Lin Tang wrote: > On 2017/6/27 6:45 AM, Cesar Philippidis wrote: >>> (1) Instead of essentially implementing the entire OpenACC async support >>> inside the plugin, we now use an opaque 'goacc_asyncqueue' implemented >>> by the plugin, along with core 'test', 'synchron

Re: [gomp4] OpenACC async re-work

2017-06-27 Thread Chung-Lin Tang
On 2017/6/27 6:45 AM, Cesar Philippidis wrote: >> (1) Instead of essentially implementing the entire OpenACC async support >> inside the plugin, we now use an opaque 'goacc_asyncqueue' implemented >> by the plugin, along with core 'test', 'synchronize', 'serialize', etc. >> plugin functions. Most o

Re: [gomp4] OpenACC async re-work

2017-06-26 Thread Cesar Philippidis
I still need more time to review this, but ... On 06/24/2017 12:54 AM, Chung-Lin Tang wrote: > Hi Cesar, Thomas, > This patch is the re-implementation of OpenACC async we talked about. > The changes are rather large, so I am putting it here for a few days before > actually committing them to gomp-