This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: gomp_target_fini
- From: Alexander Monakov <amonakov at ispras dot ru>
- To: Jakub Jelinek <jakub at redhat dot com>
- Cc: Thomas Schwinge <thomas at codesourcery dot com>, Bernd Schmidt <bschmidt at redhat dot com>, Ilya Verbin <iverbin at gmail dot com>, Chung-Lin Tang <cltang at codesourcery dot com>, James Norris <jnorris at codesourcery dot com>, gcc-patches at gcc dot gnu dot org, Kirill Yukhin <kirill dot yukhin at gmail dot com>
- Date: Thu, 21 Apr 2016 16:43:22 +0300 (MSK)
- Subject: Re: gomp_target_fini
- Authentication-results: sourceware.org; auth=none
- References: <20151201131559 dot GU5675 at tucnak dot redhat dot com> <20151201172927 dot GA7692 at msticlxl57 dot ims dot intel dot com> <20151201190504 dot GY5675 at tucnak dot redhat dot com> <20151208144559 dot GB14238 at msticlxl57 dot ims dot intel dot com> <20151211172713 dot GF5675 at tucnak dot redhat dot com> <20151214164736 dot GA63018 at msticlxl57 dot ims dot intel dot com> <87r3impode dot fsf at kepler dot schwinge dot homeip dot net> <56A0F83E dot 3040808 at redhat dot com> <20160122101607 dot GN3017 at tucnak dot redhat dot com> <87d1pl3dgd dot fsf at hertz dot schwinge dot homeip dot net> <20160419140410 dot GR2920 at laptop dot zalov dot cz>
On Tue, 19 Apr 2016, Jakub Jelinek wrote:
> On Tue, Apr 19, 2016 at 04:01:06PM +0200, Thomas Schwinge wrote:
> > Two other solutions have been proposed in the past months: Chung-Lin's
> > patches with subject: "Adjust offload plugin interface for avoiding
> > deadlock on exit", later: "Resolve libgomp plugin deadlock on exit",
> > later: "Resolve deadlock on plugin exit" (still pending review/approval),
> > and Alexander's much smaller patch with subject: "libgomp plugin: make
> > cuMemFreeHost error non-fatal",
> > <http://news.gmane.org/find-root.php?message_id=%3C1458323327-9908-4-git-send-email-amonakov%40ispras.ru%3E>.
> > (Both of which I have not reviewed in detail.) Assuming that Chung-Lin's
> > patches are considered too invasive for gcc-6-branch, can we at least get
> > Alexander's patch committed to gcc-6-branch as well as on trunk, please?
>
> Yeah, Alex' patch is IMHO fine, even for gcc-6-branch.
Applied to both.
Thanks.
Alexander