On Fri, 6 Mar 2015 17:01:13 +0300 Ilya Verbin <iver...@gmail.com> wrote:
> On Thu, Feb 26, 2015 at 20:25:11 +0300, Ilya Verbin wrote: > > On Wed, Feb 25, 2015 at 10:36:08 +0100, Thomas Schwinge wrote: > > > > Julian Brown <jul...@codesourcery.com> wrote: > > > > This is a version of the previously-posted patch to rework > > > > initialisation and support the proposed load/unload hooks, > > > > merged to gomp4 branch and tested alongside the two patches > > > > (from > > > > Currently the 'struct gomp_memory_mapping' contains 'lock' and > > 'is_initialized'. Do you still need them? Or we can use > > gomp_device_descr::lock and is_initialized instead? If yes, then > > we can replace the gomp_memory_mapping structure with a splay_tree, > > as it was before the OpenACC merge. > > Ping? Apologies, I've been distracted with travel and other things. I suspect, as you suggest, that the gomp_memory_mapping lock/is_initialized fields may no longer be required. I haven't yet had time to address that nor all of Thomas's comments on the patch (mostly breakage with multiple devices), and I'm unlikely to have time this week either due to vacation... Thanks, Julian
