darcs

Issue 741 darcs optimize --reorder-patches segfaults

Title darcs optimize --reorder-patches segfaults
Priority bug Status resolved
Milestone Resolved in
Superseder Nosy List SamB, darcs-devel, dmitry.kurochkin, kowey, simonmar, thorkilnaur, tommy
Assigned To
Topics

Created on 2008-03-09.03:12:09 by SamB, last changed 2009-10-23.23:36:47 by admin.

Files
File name Uploaded Type Edit Remove
optimize-reorder-segfault.txt SamB, 2008-03-09.03:12:02 text/plain
Messages
msg3849 (view) Author: SamB Date: 2008-03-09.03:12:02
Ran command:
valgrind darcs optimize --reorder-patches --debug --timing >
optimize-reorder-segfault.txt 2>&1

With repository lazily based on: http://darcs.haskell.org/ghc-hashedrepo

Output is attached.
Attachments
msg3853 (view) Author: SamB Date: 2008-03-10.10:44:23
To clarify, this happens even without valgrind. I had been vainly hoping that
valgrind might actually be able to trace the problem down.

Obviously, it doesn't have any knowledge of GHC's heap, so I'm not sure why I
was thinking it might help. (Perhaps I hadn't remembered that black holes are
heap objects?)
msg3854 (view) Author: SamB Date: 2008-03-10.10:53:44
Oh, you might possibly want to use
http://naesten.dyndns.org:8080/repos/ghc-hashedrepo/ as a source, if d.h.o is
still crawling like it has been...
msg3878 (view) Author: simonmar Date: 2008-03-12.23:10:54
What version of GHC was your darcs compiled with?  Do you have a different
version of GHC to try, and if so, does the error go away?

What size is the darcs process when it dies?
msg3879 (view) Author: SamB Date: 2008-03-13.11:50:48
I'm using the debian package ghc6 6.8.2-1.

Running under gdb to keep it from dying when it segfaults...
top says:

  PID USER      PR  NI  VIRT  RES  SHR S %CPU %MEM    TIME+  COMMAND           
                              
30512 naesten   23   5 34256  27m 3496 T  0.0  5.5   0:03.59 darcs
msg4078 (view) Author: droundy Date: 2008-03-28.19:46:45
Just to be clear, was the darcs that showed this behavior compiled with
bytestring support enabled?
msg4087 (view) Author: SamB Date: 2008-03-28.21:41:07
On 3/28/08, David Roundy <bugs@darcs.net> wrote:

> Just to be clear, was the darcs that showed this behavior compiled with
> bytestring support enabled?

I hadn't enabled it, so unless it was the default... no.
msg4089 (view) Author: droundy Date: 2008-03-28.22:44:25
On Fri, Mar 28, 2008 at 09:41:09PM -0000, Samuel Bronson wrote:
> On 3/28/08, David Roundy <bugs@darcs.net> wrote:
> > Just to be clear, was the darcs that showed this behavior compiled with
> > bytestring support enabled?
> 
> I hadn't enabled it, so unless it was the default... no.

Hmmm.  For a while, it was the default.  Do you have the --exact-version
you compiled with?
-- 
David Roundy
Department of Physics
Oregon State University
msg4091 (view) Author: SamB Date: 2008-03-28.23:27:46
On 3/28/08, David Roundy <bugs@darcs.net> wrote:
>
> On Fri, Mar 28, 2008 at 09:41:09PM -0000, Samuel Bronson wrote:
> > On 3/28/08, David Roundy <bugs@darcs.net> wrote:
> > > Just to be clear, was the darcs that showed this behavior compiled with
> > > bytestring support enabled?
> >
> > I hadn't enabled it, so unless it was the default... no.
>
> Hmmm.  For a while, it was the default.  Do you have the --exact-version
> you compiled with?

% darcs --exact-version
darcs compiled on Mar  9 2008, at 17:29:06
unknown

That's all I have. The tree was fairly up-to-date at the time...
msg4097 (view) Author: droundy Date: 2008-03-29.11:56:12
On Fri, Mar 28, 2008 at 11:27:47PM -0000, Samuel Bronson wrote:
> On 3/28/08, David Roundy <bugs@darcs.net> wrote:
> >
> > On Fri, Mar 28, 2008 at 09:41:09PM -0000, Samuel Bronson wrote:
> > > On 3/28/08, David Roundy <bugs@darcs.net> wrote:
> > > > Just to be clear, was the darcs that showed this behavior compiled with
> > > > bytestring support enabled?
> > >
> > > I hadn't enabled it, so unless it was the default... no.
> >
> > Hmmm.  For a while, it was the default.  Do you have the --exact-version
> > you compiled with?
> 
> % darcs --exact-version
> darcs compiled on Mar  9 2008, at 17:29:06
> unknown
> 
> That's all I have. The tree was fairly up-to-date at the time...

Bytestring was disabled as a default option on March 4.  Between the
possibility that it wasn't immediately pushed, and that "fairly up-to-date"
included a few days of lag, my guess is that your darcs wasn't compiled
with that fix, and I think I'll close this bug for now.
-- 
David Roundy
Department of Physics
Oregon State University
msg4192 (view) Author: SamB Date: 2008-04-05.19:44:19
Hmm, it does seem to be gone now.
History
Date User Action Args
2008-03-09 03:12:09SamBcreate
2008-03-10 10:44:25SamBsetstatus: unread -> unknown
nosy: droundy, tommy, beschmi, kowey, SamB
messages: + msg3853
2008-03-10 10:53:46SamBsetnosy: droundy, tommy, beschmi, kowey, SamB
messages: + msg3854
2008-03-12 23:10:55simonmarsetnosy: + simonmar
messages: + msg3878
2008-03-13 11:50:50SamBsetnosy: droundy, tommy, beschmi, kowey, SamB, simonmar
messages: + msg3879
2008-03-28 19:46:46droundysetnosy: droundy, tommy, beschmi, kowey, SamB, simonmar
messages: + msg4078
2008-03-28 21:41:09SamBsetnosy: droundy, tommy, beschmi, kowey, SamB, simonmar
messages: + msg4087
2008-03-28 22:44:26droundysetnosy: droundy, tommy, beschmi, kowey, SamB, simonmar
messages: + msg4089
2008-03-28 23:27:47SamBsetnosy: droundy, tommy, beschmi, kowey, SamB, simonmar
messages: + msg4091
2008-03-29 11:56:14droundysetnosy: droundy, tommy, beschmi, kowey, SamB, simonmar
messages: + msg4097
2008-03-29 12:02:06droundysetstatus: unknown -> resolved
nosy: droundy, tommy, beschmi, kowey, SamB, simonmar
2008-04-05 19:44:20SamBsetstatus: resolved -> unknown
nosy: droundy, tommy, beschmi, kowey, SamB, simonmar
messages: + msg4192
2008-04-05 20:53:34droundysetstatus: unknown -> resolved
nosy: droundy, tommy, beschmi, kowey, SamB, simonmar
2009-08-06 17:56:18adminsetnosy: + markstos, jast, Serware, dmitry.kurochkin, darcs-devel, zooko, dagit, mornfall, simon, thorkilnaur, - droundy, SamB, simonmar
2009-08-06 21:00:17adminsetnosy: - beschmi
2009-08-10 22:17:44adminsetnosy: + SamB, simonmar, - markstos, darcs-devel, zooko, jast, dagit, Serware, mornfall
2009-08-25 18:07:30adminsetnosy: + darcs-devel, - simon
2009-08-27 13:57:30adminsetnosy: tommy, kowey, darcs-devel, SamB, simonmar, thorkilnaur, dmitry.kurochkin
2009-10-23 22:38:17adminsetnosy: + marlowsd, - simonmar
2009-10-23 23:36:47adminsetnosy: + simonmar, - marlowsd