X-Recipient:	archive-cygwin AT delorie DOT com
DomainKey-Signature:	a=rsa-sha1; c=nofws; d=sourceware.org; h=list-id
	:list-unsubscribe:list-subscribe:list-archive:list-post
	:list-help:sender:date:from:to:cc:subject:message-id:reply-to
	:references:mime-version:content-type:in-reply-to; q=dns; s=
	default; b=KAvgNBpz8N8JbW3Z2Pr+vVrNbkz5yFHkuJicQwfCKCiojgv+25gmd
	CAE6f81+e2dwoGUx8XWlc8FEbuh61E/aG/JV6XbfW8aTzNN31/f9dAHsj6i+1FN7
	sndSJimw7GQlg68sIGUZ3KeNi4rbO3HSYL/YtiJ4XMqd8Oz48Ph5MM=
DKIM-Signature:	v=1; a=rsa-sha1; c=relaxed; d=sourceware.org; h=list-id
	:list-unsubscribe:list-subscribe:list-archive:list-post
	:list-help:sender:date:from:to:cc:subject:message-id:reply-to
	:references:mime-version:content-type:in-reply-to; s=default;
	bh=8qpub50GlmigxM+DtphyBNXlvBU=; b=mXntMGDGm4W7tnqwv3BO4VwxMqwV
	uln8Z93PbI1taSi4JSlJfALZXBoeL7BmZkz7yURCNQFq7oJF4j63ON1m/qDyrYrc
	F0Am8zTCf55sY9/Bng1QdBEDOTAWMA1T1LdUaEXcXOXyAGGSa44FhoWVRBZz5Cpk
	idrPx9R/zo/lXco=
Mailing-List:	contact cygwin-help AT cygwin DOT com; run by ezmlm
List-Id:	<cygwin.cygwin.com>
List-Subscribe:	<mailto:cygwin-subscribe AT cygwin DOT com>
List-Archive:	<http://sourceware.org/ml/cygwin/>
List-Post:	<mailto:cygwin AT cygwin DOT com>
List-Help:	<mailto:cygwin-help AT cygwin DOT com>, <http://sourceware.org/ml/#faqs>
Sender:	cygwin-owner AT cygwin DOT com
Mail-Followup-To:	cygwin AT cygwin DOT com
Delivered-To:	mailing list cygwin AT cygwin DOT com
Authentication-Results:	sourceware.org; auth=none
X-Virus-Found:	No
X-Spam-SWARE-Status:	No, score=-5.9 required=5.0 tests=AWL,BAYES_00 autolearn=ham version=3.3.2
X-HELO:	calimero.vinschen.de
Date:	Tue, 22 Jul 2014 15:20:01 +0200
From:	Corinna Vinschen <corinna-cygwin AT cygwin DOT com>
To:	cygwin AT cygwin DOT com
Cc:	DJ Delorie <dj AT redhat DOT com>, Kai Tietz <ktietz AT redhat DOT com>
Subject:	Re: Core dump on 32-bit Cygwin if program calls dlopen
Message-ID:	<20140722132001.GD27005@calimero.vinschen.de>
Reply-To:	cygwin AT cygwin DOT com
Mail-Followup-To:	cygwin AT cygwin DOT com, DJ Delorie <dj AT redhat DOT com>, Kai Tietz <ktietz AT redhat DOT com>
References:	<20140715130829 DOT GK10401 AT calimero DOT vinschen DOT de> <53C532D9 DOT 7050506 AT gmail DOT com> <20140715143915 DOT GA8330 AT calimero DOT vinschen DOT de> <20140716070231 DOT GA25020 AT calimero DOT vinschen DOT de> <53C6F68A DOT 9000801 AT gmail DOT com> <20140717073734 DOT GA15332 AT calimero DOT vinschen DOT de> <53C7EC48 DOT 5060903 AT dronecode DOT org DOT uk> <20140717182429 DOT GS15332 AT calimero DOT vinschen DOT de> <20140722082711 DOT GA22860 AT calimero DOT vinschen DOT de> <53CE604D DOT 1070309 AT dronecode DOT org DOT uk>
MIME-Version:	1.0
In-Reply-To:	<53CE604D.1070309@dronecode.org.uk>
User-Agent:	Mutt/1.5.23 (2014-03-12)

--p2kqVDKq5asng8Dg
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Jul 22 13:59, Jon TURNEY wrote:
> On 22/07/2014 09:27, Corinna Vinschen wrote:
> >On Jul 17 20:24, Corinna Vinschen wrote:
> >>On Jul 17 16:31, Jon TURNEY wrote:
> >>>On 17/07/2014 08:37, Corinna Vinschen wrote:
> >>>>It's the libgcc DLL which gives us grief, so a new libgcc package is
> >>>>sufficent, afaics.  We should check if this DLL fixes the problem and
> >>>>then make it "curr" soon, I think.
> >>>
> >>>I briefly tested this other patch with my test case from the mail abov=
e, and
> >>>it doesn't seem to help.
> >>>[...]
> >>I asked DJ to take another look, but I guess ultimately we need the
> >>attention of one of the GCC Windows maintainers.  Kai Tietz seems to be
> >>unavailable right now, unfortunately.
> >
> >Looks like I totally misunderstood DJ's patch.  The patch does *not*
> >change libgcc, it changes cygmin-crtbegin.c, thus the crtbegin.o file
> >which is statically linked into the executable.
> >
> >That means, to fix the issue, you don't have to replace libgcc, you
> >have to recompile the executable against the new crtbegin.o.
> >
> >DJ still claims his patch is the correct one.  The simple testcase
> >dlopen'ing cyggs-9.dll works fine with the new crtbegin.o, according to
> >him.
>=20
> Sorry, I hadn't tested it correctly.

No worries, this was my fault.  Talking about building a new libgcc
rather than building a new crtbegin.o was bound to lead everyone
off track :|

> Building my test with an updated crtbegin.o as well, my test case is fixe=
d.

Nice!

> I agree this patch seems better than my suggested one, as it makes crtbeg=
in
> do the right thing in the face of unbalanced libgcc load/unload, rather t=
han
> attempting to balance the libgcc load/unload as mine does.

Ok.  I CC'ed DJ so he knows all is good, and then we probably need a
windows GCC maintainer to approve the patch I guess.  Kai?  Ping?


Corinna

--=20
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Maintainer                 cygwin AT cygwin DOT com
Red Hat

--p2kqVDKq5asng8Dg
Content-Type: application/pgp-signature

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1

iQIcBAEBAgAGBQJTzmUBAAoJEPU2Bp2uRE+g0ZgP/ict/Q8awLpbh6mCvzfNK629
KuoPZlOJLYnhmnt/1C1J2KB+/IymvORRyD5KHtz9WMuz8aZi0takM2G5b9w1Lxe1
vUBxOrqH7E308lh822abYIq6O+61FUqoKQipCc6HIxsQQzQa/hp3JbntPNP1qTzL
Z/k2x5wv+9be77tG4XjDryJ1aSvGFoCXtT7AO9KqFlCeNFWeTl1kNrYm8FfaqkGC
x3FfhlH2heG45FkCBWckPNnPsrXXBLkkLMRc76mMRLZA3ZO/foqLeOa1L2WOZKCS
xKHRD8mHWlHusSXLETlYhOsScSVq1LJKfXtPJM42RuDhKAE2NsQXDf+yQsT1vyx+
aJ797L1NXMxQDrT+wiXeTH29HIMR04JQR/ZRkmwkXiccfS0j3aMoy21UUReoEZO7
GG3x3l73NHii8HbjUJF6JcWodiqtiovJs1D3x9FQJQIW41RzFErwnUKJkTnl0Zh8
t5qZ2cYNVbVBDkRnmdt88OD4YCe689hzwoeICxgmrBURqruhgqurHvE4rs/jmPf6
ZchDO8WnOpIisIuHtJszyRgH8VxBv6W21Ur8MwuV9RhgPFU5dTtjrziqo4w3U1pq
nVbYSFxaPoaumlsZHKgwrQ8EF0mXi9fP3GkKOlts0IeN/8Ge+cp7wq8oYe/mdNHb
59v8eFSNoya8V55D433j
=7AUh
-----END PGP SIGNATURE-----

--p2kqVDKq5asng8Dg--

- Raw text -