X-Recipient: archive-cygwin AT delorie DOT com
DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org E878F3951866
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cygwin.com;
	s=default; t=1615212103;
	bh=qFsKvItN47HORnAJL78CpryJmAiVcD34N9nhr8QVnz0=;
	h=Date:To:Subject:References:In-Reply-To:List-Id:List-Unsubscribe:
	 List-Archive:List-Post:List-Help:List-Subscribe:From:Reply-To:Cc:
	 From;
	b=O/O0xSkpCg61L2n0tIbygBUnNUUnG0+VdJFZ7qoiKum/nWG4bQxLs5NJZboHSss03
	 496+nmgPxbLR0124sOt4Ffr6Gg1dw4/LHSrqqguIAKxtQ0AX8jZ7vt7A4EJ7gKIXuy
	 gSuR5Qe680d1BW1RYdT6G6ZKmONgPAD1IU6S2klY=
X-Original-To: cygwin AT cygwin DOT com
Delivered-To: cygwin AT cygwin DOT com
DMARC-Filter: OpenDMARC Filter v1.3.2 sourceware.org 78FBD3939C04
Date: Mon, 8 Mar 2021 15:01:34 +0100
To: cygwin AT cygwin DOT com
Subject: Re: segfault on 32bit cygwin snapshot
Message-ID: <YEYuPpgQ3WhoXwdn@calimero.vinschen.de>
Mail-Followup-To: cygwin AT cygwin DOT com
References: <20210304180534 DOT 7c3825e6ee989a2bf62f0652 AT nifty DOT ne DOT jp>
 <CAB8Xom-HwxH9LnbLL8cAoXh60tjU592T6XFEDJzvhv28Do-UsQ AT mail DOT gmail DOT com>
 <20210304205000 DOT 4ac34afe1a8b767ea2986291 AT nifty DOT ne DOT jp>
 <b7338aaa-c1e1-1816-35f8-666c2f6ac618 AT cornell DOT edu>
 <98df59b3-a139-03d2-c8c5-31ff4f8448ab AT gmail DOT com>
 <625cb9d2-b6cc-3ada-eea9-44a5e73da17c AT gmail DOT com>
 <72a86908-e150-2070-24f7-79ca82de9916 AT maxrnd DOT com>
 <YEJDaQQ+U/+j3uQb AT calimero DOT vinschen DOT de>
 <Pine DOT BSF DOT 4 DOT 63 DOT 2103051447400 DOT 41714 AT m0 DOT truegem DOT net>
 <7b34da67-7550-765b-3731-31b51b2a5df9 AT maxrnd DOT com>
MIME-Version: 1.0
Content-Disposition: inline
In-Reply-To: <7b34da67-7550-765b-3731-31b51b2a5df9@maxrnd.com>
X-Provags-ID: V03:K1:DCEpFJPn6TMGRBefaBJR/AVSiUwFTvQKgj0HGzJe4AGshz1X+Ym
 k2NlCIn6QkTtovenvpMGYvV6+a2Wz3hLPT2phqLr6pX4DdsfyWSqf0Ms4ytPcuZK/mgipqE
 LiywStKN8BytxgV3xhZsjWZOaYMTE70G+tlEJyBgruXxyoYMpINP9Lp25lFWrfnJtj6Zb5N
 X3ggPMWkiSlKX5gOalhSg==
X-UI-Out-Filterresults: notjunk:1;V03:K0:F0tvq6l1YrA=:MVAIS2a336sQhY7Zv4wmuw
 PXFh2t64ieaPdvWxh5+7aQXUKXzWprRDVJDiOKb5jiaFYYFKpwZL6LuxGp8PQaCb1dy3uYNgZ
 gvrj0khjcYFiMdZftErN8/NkSMtgWDFgMatQm+VPOFVuLWtO+i38XoGHmBKmMVMCNlCJeolV9
 UnONhmdMrHwjZ+tBOvCO+95GHozn7VC8MlbDiPMasJtUwFGmpvB9wr/hB4B4xnJcE2+IAxgq/
 QeEriYJluo8RlxZ8YbiTnvTwOneaSP5h+kWAudxHjY8vfwY9Qve9rvp0vvK32hO3SnZYHhdcU
 0u/U7qxOtON3nx/tUfKjgt0T9CpPh1Mpd+bu2Brq7oJZ12y7INRSVEY7hbrujaRZxHDeDXMWg
 sq0+iEdYA3cwAhaoaLaWJNJX/9qBx7GZULt2WwdaM6WXZ33ytToXpIkIHSUfuwLvDrlGAb8MJ
 XPZewpZEJg==
X-Spam-Status: No, score=-99.8 required=5.0 tests=BAYES_00, BODY_8BITS,
 GOOD_FROM_CORINNA_CYGWIN, KAM_DMARC_NONE, KAM_DMARC_STATUS, RCVD_IN_DNSWL_NONE,
 RCVD_IN_MSPIKE_H2, SPF_HELO_NONE, SPF_NEUTRAL,
 TXREP autolearn=ham autolearn_force=no version=3.4.2
X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on
 server2.sourceware.org
X-BeenThere: cygwin AT cygwin DOT com
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: General Cygwin discussions and problem reports <cygwin.cygwin.com>
List-Unsubscribe: <https://cygwin.com/mailman/options/cygwin>,
 <mailto:cygwin-request AT cygwin DOT com?subject=unsubscribe>
List-Archive: <https://cygwin.com/pipermail/cygwin/>
List-Post: <mailto:cygwin AT cygwin DOT com>
List-Help: <mailto:cygwin-request AT cygwin DOT com?subject=help>
List-Subscribe: <https://cygwin.com/mailman/listinfo/cygwin>,
 <mailto:cygwin-request AT cygwin DOT com?subject=subscribe>
From: Corinna Vinschen via Cygwin <cygwin AT cygwin DOT com>
Reply-To: cygwin AT cygwin DOT com
Cc: Corinna Vinschen <corinna-cygwin AT cygwin DOT com>
Content-Type: text/plain; charset="utf-8"
Errors-To: cygwin-bounces AT cygwin DOT com
Sender: "Cygwin" <cygwin-bounces AT cygwin DOT com>
Content-Transfer-Encoding: 8bit
X-MIME-Autoconverted: from base64 to 8bit by delorie.com id 128E1ldO014999

On Mar  8 03:07, Mark Geisert wrote:
> Following up to myself...
> 
> Mark Geisert wrote:
> > Hi Corinna,
> > 
> > On Fri, 5 Mar 2021, Corinna Vinschen wrote:
> > > On Mar  5 01:11, Mark Geisert wrote:
> > > > Marco Atzeri via Cygwin wrote:
> > > > > On 04.03.2021 21:17, Marco Atzeri wrote:
> > > > > > On 04.03.2021 16:17, Ken Brown via Cygwin wrote:
> > > > > > > On 3/4/2021 6:50 AM, Takashi Yano via Cygwin wrote:
> > > > > > > > On Thu, 4 Mar 2021 12:11:11 +0100
> > > > > > > > marco atzeri wrote:
> > > > > > > > > I have no problem to patch Python to solve the issue,
> > > > > > > > > but I have not seen evidence of the dlsym mechanism .
> > > > > > > > > But of course I an NOT and expert in this field.
> > > > > > > > > 
> > > > > > > > > If someone looking to the code can give me some hints,
> > > > > > > > > I will appreciate
> > > > > > > > 
> > > > > > > > I am also not sure where the dlsym() is used in python.
> > > > > > > > At least, os.uname() works in python 3.8.7 and 2.7.18 in my
> > > > > > > > environment even without that snippet. It seems that os.uname()
> > > > > > > > does not use dlsym(). Do I overlook something?
> > > > > > > 
> > > > > > > This all started because Mark reported a problem building python 3.8.3:
> > > > > > > 
> > > > > > >    https://cygwin.com/pipermail/cygwin-apps/2020-December/040765.html
> > > > > > > https://cygwin.com/pipermail/cygwin-developers/2020-December/012019.html
> > > > > > > 
> > > > > > > It's strange that Marco never bumped into the problem.
> > > > > > > 
> > > > > > > Ken
> > > > > > 
> > > > > > I never built python using cygwin snapshots as Mark was trying to do,
> > > > > > all my builds were using 3.1.7.
> > > > > > 
> > > > > > Let me set a separate enviroment for building on latest snapshot
> > > > > 
> > > > > I can not replicate with latest snapshot
> > > > > 
> > > > > $ uname -svr
> > > > > CYGWIN_NT-10.0-WOW 3.2.0s(0.340/5/3) 2021-03-01 15:42
> > > > > 
> > > > > nor in 64bit when building 3.8.8
> > > > > 
> > > > > For what I see the DLL is always using a proper import
> > > > > from cygwin1.dll
> > > > > 
> > > > > $ objdump -x libpython3.8.dll |grep uname
> > > > >          2b9de0   2170  uname
> > > > >          2b9de8   2171  uname_x
> > > > > 
> > > > > the only thing not standard on my build system is a case sensitive
> > > > > filesystem and mount
> > > > 
> > > > I had concerns that I had somehow corrupted my build environment, and it was
> > > > Marco's successes that convinced me to reinstall 3.1.7 to recover a
> > > > known-good environment.  Then seeing Marco go ahead and release the
> > > > different Python releases (yay!) I didn't investigate any further.
> > > > 
> > > > I'm now trying to locate the os.uname usage of dlopen/dlsym again just for
> > > > the record but am having some difficulty.  I'll reply again when I've got
> > > > it.
> > > 
> > > Guys,
> > > 
> > > if it turns out that we fixed a problem that doesn't actually is a
> > > real-world problem, I'm wondering if we shouldn't just revert the Cygwin
> > > patch we're talking about here (commit 532b91d24e9496) and be done with
> > > it.
> > > 
> > > Special casing dynamic loading of uname just to support some experimental
> > > bordercase doesn't make much sense.  In that case I'm all for "don't do
> > > that"!
> > 
> > That may well be the appropriate endpoint, but please let me dig a
> > little further into the recent Python versions.  The fact that they had
> > an explicit dlopen/dlsym to get at uname(), but now they don't, troubles
> > me. I want to be sure us Cygwin folk aren't in an inadvertent "arms
> > race" with the Python devs over the uname API change.  Dunno why this
> > didn't occur 15+ years ago, but here we are.
> > 
> > I think it was in Python's Modules/posixmodule.c.  They're certainly
> > using uname() directly in their most recent builds.  But I believe that
> > wasn't always the case, even just a few months ago.  Let me dig for a
> > day or two.
> 
> Not to put too fine a point on it, but I now believe I hit the issue I
> reported due to my own corrupted build environment.  Marco and Ken have both
> built recent Python packages on Cygwin 3.1.7 and proto-3.2.0 respectively,
> Ken having reverted the commit you're talking about (not sure about Marco).
> Both had success.
> 
> I believe reverting that commit 532b91d24e9496 is the correct action to take.

Did so this morning.


Thanks,
Corinna
--
Problem reports:      https://cygwin.com/problems.html
FAQ:                  https://cygwin.com/faq/
Documentation:        https://cygwin.com/docs.html
Unsubscribe info:     https://cygwin.com/ml/#unsubscribe-simple