summaryrefslogtreecommitdiff
path: root/Documentation/githooks.txt
diff options
context:
space:
mode:
authorLibravatar Jeff King <peff@peff.net>2013-04-16 15:50:07 -0400
committerLibravatar Junio C Hamano <gitster@pobox.com>2013-04-16 15:02:48 -0700
commit1ece66bc9e5433dff008786daba9918f3e2a6525 (patch)
tree5b67788dbf4e6e819cd562c0eb2414bd39c51287 /Documentation/githooks.txt
parentusage: allow pluggable die-recursion checks (diff)
downloadtgif-1ece66bc9e5433dff008786daba9918f3e2a6525.tar.xz
run-command: use thread-aware die_is_recursing routine
If we die from an async thread, we do not actually exit the program, but just kill the thread. This confuses the static counter in usage.c's default die_is_recursing function; it updates the counter once for the thread death, and then when the main program calls die() itself, it erroneously thinks we are recursing. The end result is that we print "recursion detected in die handler" instead of the real error in such a case (the easiest way to trigger this is having a remote connection hang up while running a sideband demultiplexer). This patch solves it by using a per-thread counter when the async_die function is installed; we detect recursion in each thread (including the main one), but they do not step on each other's toes. Other threaded code does not need to worry about this, as they do not install specialized die handlers; they just let a die() from a sub-thread take down the whole program. Since we are overriding the default recursion-check function, there is an interesting corner case that is not a problem, but bears some explanation. Imagine the main thread calls die(), and then in the die_routine starts an async call. We will switch to using thread-local storage, which starts at 0, for the main thread's counter, even though the original counter was actually at 1. That's OK, though, for two reasons: 1. It would miss only the first level of recursion, and would still find recursive failures inside the async helper. 2. We do not currently and are not likely to start doing anything as heavyweight as starting an async routine from within a die routine or helper function. Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'Documentation/githooks.txt')
0 files changed, 0 insertions, 0 deletions