X-Recipient: archive-cygwin AT delorie DOT com
X-Spam-Check-By: sourceware.org
Date: Mon, 6 Aug 2012 17:43:07 +0200
From: Corinna Vinschen <corinna-cygwin AT cygwin DOT com>
To: cygwin AT cygwin DOT com
Subject: Re: Race condition that leads to random crashes in cygwin-based builds.
Message-ID: <20120806154307.GB31162@calimero.vinschen.de>
Reply-To: cygwin AT cygwin DOT com
Mail-Followup-To: cygwin AT cygwin DOT com
References: <CACgPhYpV-M477Mfugp3DjdWCEvbJ-8LhyH12y=iGP_MuK6VTAQ AT mail DOT gmail DOT com> <20120724135718 DOT GA29107 AT calimero DOT vinschen DOT de> <20120806085817 DOT GA21876 AT calimero DOT vinschen DOT de> <CACgPhYru7WdRfuTtyzXcq0Q5=wUZeiUn+n1a4UOdq7psASUjFw AT mail DOT gmail DOT com>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
In-Reply-To: <CACgPhYru7WdRfuTtyzXcq0Q5=wUZeiUn+n1a4UOdq7psASUjFw@mail.gmail.com>
User-Agent: Mutt/1.5.21 (2010-09-15)
Mailing-List: contact cygwin-help AT cygwin DOT com; run by ezmlm
Precedence: bulk
List-Id: <cygwin.cygwin.com>
List-Unsubscribe: <mailto:cygwin-unsubscribe-archive-cygwin=delorie DOT com AT cygwin DOT 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



Please, don't http://cygwin.com/acronyms/#TOFU


On Aug  6 19:31, Andrey Khalyavin wrote:
> 2012/8/6 Corinna Vinschen wrote:
> > Andrey?
> >
> > On Jul 24 15:57, Corinna Vinschen wrote:
> >> On Jul 24 17:25, Andrey Khalyavin wrote:
> >> > Hi, we have build bots that crash randomly on Windows XP and rarely on
> >> > Windows 7.
> >> > [...]
> >> > 5. Task switching happens and first process proceeds to execute
> >> > GetVersionEx with
> >> > version cleared by memset and so not having its size set.
> >> > 6. GetVersionEx returns error and first process fails to start.
> >> >
> >> > If there is no easy way to add synchronization to wincapc::init, I
> >> > suggest to make
> >> > wincap a regular (not shared) variable.
> >>
> >> There's another way, afaics.  The idea here was that wincap is only
> >> ever set once, and even *if* the information is written twice, the
> >> content will be identical.
> >>
> >> So, afaics, the above problem is a result of using memset at all.  At
> >> startup, wincap is all 0 anyway, so the memset is not required and
> >> apparently it even hurts.  Weird that nobody saw this problem before.
> >>
> >> I applied a patch which should fix this problem.  Please give the
> >> next developer snapshot from http://cygwin.com/snapshots/ a try,
> >> or build yourself from CVS.
> >
> > Ping?  Any feedback?  Did you ever try a snapshot?
> 
> I updated our cygwin with core libraries from 20120725 snapshot. There
> are still crashes in our build, I'm investigating them. Haven't got a
> crash dump yet. This time I have to catch them on the bots instead of
> local computer.

Please use the *latest* snapshot from CVS, 20120803.  There were other
potential reasons for crashes which only have been (tried to) fix in
the latest snapshot.


Thanks,
Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Project Co-Leader          cygwin AT cygwin DOT com
Red Hat

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