On Wed, Dec 30, 2009 at 04:55:37PM +0100, Petr Rockai wrote:
> Nathan Gray <[email protected]> writes:
> > In another terminal I investigated a little bit.  It looks like
> > 'darcs unpull' is hanging:
> >
> >   $ date
> >   Tue Dec 29 16:29:33 EST 2009
> >
> >   $ ps ax|grep darcs
> >   5311 pts/0    S+     0:17 darcs-benchmark /home/kolibrie/bin/darcs-2.3.1 
> > /home/kolibrie/bin/darcs
> >   6136 pts/0    Sl+    0:21 /home/kolibrie/bin/darcs-2.3.1 unpull --last 
> > 1000 --all +RTS -sdarcs-stats -RTS
> >
> >   $ uptime
> >   16:29:31 up  1:05,  3 users,  load average: 0.10, 0.32, 0.47
>
> I have seen this happen (actually, it happens on buildbot as well, and
> it doesn't happen on my laptop...), too. The question is what's going
> wrong... On the buildbot, I haven't noticed any hanging darcs
> processes. I am starting to suspect, that darcs is waiting for some user
> input in that case. It could be the "this will make unrevert impossible"
> one, but I don't see anything in the benchmark code that could produce
> one and there is none in the tarballs. Also, it should be reproducible
> that way.
> 
> > Any ideas what I should do, besides kill it?
>
> If you could find out what that hanging darcs process is doing (strace
> -p is your friend), that'd be great. Also, running that command manually
> could help to find out: go to _playground/repo and run darcs-2.3.1
> unpull --last 1000 --all.

Strace repeats the same output forever for the 'unpull' process:

  $ strace -p 6136
  Process 6136 attached - interrupt to quit
  futex(0x91bcbc0, FUTEX_WAIT_PRIVATE, 7, NULL) = ? ERESTARTSYS (To be 
restarted)
  --- SIGVTALRM (Virtual timer expired) @ 0 (0) ---
  sigreturn()                             = ? (mask now [INT])
  futex(0x91bcbc0, FUTEX_WAIT_PRIVATE, 7, NULL) = ? ERESTARTSYS (To be 
restarted)
  --- SIGVTALRM (Virtual timer expired) @ 0 (0) ---
  sigreturn()                             = ? (mask now [INT])
  ...

Strace on the 'darcs-benchmark' process has much less output, but
appears to repeat as well:

  $ strace -p 5311
  Process 5311 attached - interrupt to quit
  select(10, [9], [], NULL, {9, 508000})  = 0 (Timeout)
  gettimeofday({1262616419, 519725}, NULL) = 0
  select(10, [9], [], NULL, {134, 217727}) = 0 (Timeout)
  gettimeofday({1262616553, 739716}, NULL) = 0
  select(10, [9], [], NULL, {134, 217727}...

After killing darcs-benchmark, running unpull manually yields a
prompt:

  $ darcs-2.3.1 unpull --last 1000 --all
  This operation will make unrevert impossible!
  Proceed? [yn]

This is strange because I have made no changes to the repo:

  $ darcs w -s
  No changes!

Nor has the benchmark, so far as I can tell:

  darcs-2.3.1 get (full) [darcs]: 1!..2!..11.0s, 11.0M
  darcs-2.3.1 get (lazy, x10) [darcs]: 1!..2!..10.3s, 3.0M
  darcs-2.3.1 pull 100 [darcs]: 1!..2...  3.9s, 21.0M
  darcs-2.3.1 annotate [darcs]: 1...2...  23.7s, 186.0M
  darcs-2.3.1 wh x50 [darcs]: 1...2...    0.8s, 0.0M
  darcs-2.3.1 wh mod x50 [darcs]: 1...2...5.8s, 2.0M
  darcs-2.3.1 wh -l x20 [darcs]: 1...2... 3.7s, 4.0M
  darcs-2.3.1 check [darcs]: 1...2...     21.9s, 43.0M
  darcs-2.3.1 repair [darcs]: 1...2...    22.2s, 43.0M
  darcs-2.3.1 pull 1000 [darcs]: 1...Terminated

Running darcs unrevert asks about (see full output further down):

  hunk ./.authorspellings 1
  rmfile ./.authorspellings
  hunk ./.latex2html-init 1
  rmfile ./.latex2html-init
  
Full output from unrevert:

  $ darcs unrevert
  hunk ./.authorspellings 1
  --- Canonical author spellings can be defined in this file, to clean up
  --- the output of show authors. Each line contains one or more
  --- comma-separated fields: the canonical name and email address in
  --- angle brackets, optionally followed by additional regular
  --- expression patterns. Blank lines and lines beginning with -- are
  --- ignored. An author string which contains the canonical email
  --- address or any of the patterns will be replaced by the full
  --- canonical form.  All matching is case-insensitive. To match the
  --- whole author string use ^ and $.
  -
  -Aaron Denney <[email protected]>
  -Aaron Kaplan <[email protected]>
  -Adam Megacz <[email protected]>
  -Aggelos Economopoulos <[email protected]>
  -Anders Hockersten <[email protected]>
  -Andres Loeh <[email protected]>, [email protected], [email protected]
  -Andrew J. Kroll <[email protected]>
  -Andrew L Johnson <[email protected]>
  -Andrew Pimlott <[email protected]>
  -Anthony Towns <[email protected]>
  -Benedikt Schmidt <[email protected]>, [email protected]
  -Benjamin Franksen <[email protected]>, [email protected]
  -Brian Alliet <[email protected]>
  -Bryce Wilcox-O'Hearn <[email protected]>
  -Christian Egli <[email protected]>
  -Christian Kellermann <[email protected]>
  -Daniel Buenzli <[email protected]>
  -Daniel Gorin <[email protected]>
  -David Roundy <[email protected]>, [email protected], 
[email protected], ^David$
  -Don Bindner <[email protected]>
  -Don Stewart <[email protected]>, [email protected]
  -Edwin Thomson <[email protected]>, 
[email protected]
  -Era Eriksson <[email protected]>
  -Eric Gaudet <[email protected]>
  -Eric Kow <[email protected]>, [email protected], [email protected], 
[email protected], [email protected]
  -Erik Schnetter <[email protected]>, [email protected], 
[email protected]
  -Esa Ilari Vuokko <[email protected]>
  -Florent Becker <[email protected]>
  -Florian Weimer <[email protected]>
  -Gaetan Lehmann <[email protected]>
  -Gwern Branwen <[email protected]>, ^$
  -Henning Glawe <[email protected]>
  -Ian Lynagh <[email protected]>
  -Isaac Jones <[email protected]>
  -Jan Braun <[email protected]>
  -Jan Scheffczyk <[email protected]>
  -Jani Monoses <[email protected]>
  -Jens Petersson <[email protected]>
  -Jeremy Fincher <[email protected]>
  -Joe Edmonds <[email protected]>
  -Joeri van Ruth <[email protected]>, jvr\[email protected]
  -Jonathon Mah <[email protected]>
  -Juliusz Chroboczek <[email protected]>
  -Kannan Goundan <[email protected]>
  -Ketil Malde <[email protected]>
  -Kirill Smelkov <[email protected]>, [email protected]
  -Kirsten Chevalier <[email protected]>
  -Lele Gaifax <[email protected]>, [email protected], 
[email protected]
  -Lennart Kolmodin <[email protected]>, [email protected]
  -Marco T\c3\balio Gontijo e Silva <[email protected]>, [email protected]
  -Mark Stosberg <[email protected]>, testerALL --ignore-times
  -Marnix Klooster <[email protected]>, marnix[._]klooster, 
[email protected]
  -Martin Schaffner <[email protected]>
  -Matthias Kilian <[email protected]>
  -Michael G Schwern <[email protected]>
  -Mirian Crzig Lennox <[email protected]>, mirian@
  -Nathaniel Filardo <[email protected]>
  -Nicolas Pouillard <[email protected]>
  -Nigel Rowe <[email protected]>
  -Nils Decker <[email protected]>
  -Pekka Pessi <[email protected]>, Pekka Pessi <[email protected]>, 
[email protected], [email protected]
  -Peter Hercek <[email protected]>
  -Peter Maxwell <[email protected]>
  -Peter Simons <[email protected]>
  -Peter Strand <[email protected]>
  -Petr Rockai <[email protected]>
  -Ralph Corderoy <[email protected]>
  -Randy Roesler <[email protected]>
  -Reinier Lamers <[email protected]>
  -Richard Smith <[email protected]>
  -Samuel Bronson <[email protected]>
  -Sean Robinson <[email protected]>
  -Sean Russell <[email protected]>
  -Shae Erisson <[email protected]>
  -Simon Marlow <[email protected]>, [email protected]
  -Simon Michael <[email protected]>
  -Spencer Janssen <[email protected]>, [email protected]
  -Stephen Reindl <[email protected]>
  -Thies C. Arntzen <[email protected]>
  -Thomas L. Bevan <[email protected]>
  -Thomas Zander <[email protected]>, [email protected]
  -Thorkil Naur <[email protected]>
  -Tomasz Zielonka <[email protected]>, 
[email protected], [email protected]
  -Tommy Pettersson <[email protected]>
  -Trent W. Buck <[email protected]>, [email protected]
  -Vaclav Haisman <[email protected]>
  -Vadim Zaliva <[email protected]>
  -Victor Hugo Borja Rodriguez <[email protected]>
  -Vladimir Vysotsky <[email protected]>
  -Zack Brown <[email protected]>
  Shall I unrevert this change? (1/4)  [ynWsfvplxdaqjk], or ? for help: y
  rmfile ./.authorspellings
  Shall I unrevert this change? (2/4)  [ynWsfvplxdaqjk], or ? for help: y
  hunk ./.latex2html-init 1
  -
  -# This file is used by latex2html to appropriately reference our custom icons
  -
  -$ICONSERVER = ".";
  -
  -%iconsizes = (
  -    'up' ,'WIDTH="22" HEIGHT="22" title="Up" ',
  -    'next','WIDTH="22" HEIGHT="22" title="Next" ',
  -    'previous','WIDTH="22" HEIGHT="22" title="Previous" ',
  -    'next_group' ,'WIDTH="22" HEIGHT="22" title="Next Group" ',
  -    'next_inactive' ,'WIDTH="22" HEIGHT="22" title="Next" ',
  -    'previous_group','WIDTH="107" HEIGHT="22" title="Previous Group" ',
  -    'change_begin','WIDTH="104" HEIGHT="22" ',
  -    'change_begin_right','WIDTH="104" HEIGHT="22" ALIGN="RIGHT"',
  -    'change_end','WIDTH="104" HEIGHT="22"',
  -    'change_end_right','WIDTH="104" HEIGHT="22" ALIGN="RIGHT"',
  -    'change_delete','WIDTH="109" HEIGHT="22"',
  -    'change_delete_right','WIDTH="109" HEIGHT="22" ALIGN="RIGHT"',
  -    'contents','WIDTH="22" HEIGHT="22" title="Contents" ',
  -    'index','WIDTH="22" HEIGHT="22" title="Index" ',
  -    'image','WIDTH="22" HEIGHT="22" title="Image" '
  -);
  -
  -1;
  Shall I unrevert this change? (3/4)  [ynWsfvplxdaqjk], or ? for help: y
  rmfile ./.latex2html-init
  Shall I unrevert this change? (4/4)  [ynWsfvplxdaqjk], or ? for help: y

Any guidance what I should do?

-kolibrie

Attachment: signature.asc
Description: Digital signature

_______________________________________________
darcs-users mailing list
[email protected]
http://lists.osuosl.org/mailman/listinfo/darcs-users

Reply via email to