git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Ævar Arnfjörð Bjarmason" <avarab@gmail.com>
To: rsbecker@nexbridge.com
Cc: git@vger.kernel.org, Jeff Hostetler <jeffhost@microsoft.com>
Subject: Re: [BUG} t2201.7+ Failure to Create Thread in 2.39.0-rc0
Date: Thu, 24 Nov 2022 19:40:45 +0100	[thread overview]
Message-ID: <221124.86o7swnqc1.gmgdl@evledraar.gmail.com> (raw)
In-Reply-To: <010201d9002e$8e2f9940$aa8ecbc0$@nexbridge.com>


[CC-ing Jeff, this is from the "stopwatch timers" topic]

On Thu, Nov 24 2022, rsbecker@nexbridge.com wrote:

> While running t2201.7 (and subsequent) for 2.39.0-rc0, I hit a condition
> where:

Don't you mean t0211-trace2-perf.sh?

> <snip>
> + test_when_finished rm trace.perf actual
> + test_config_global trace2.perfBrief 1
> + pwd
> + test_config_global trace2.perfTarget
> /home/ituglib/randall/jenkins/.jenkins/workspace/Git_Pipeline/t/trash
> directory.t0211-trace2-perf/trace.perf
> + test-tool trace2 101timer 5 10 3
> fatal: failed to create thread[0]
>
> was encountered, causing the test to fail. Pointers on resolving this would
> be helpful.

Apparently we really could use a NO_PTHREADS=Y CI setup, because that's
all that's needed to reproduce this on linux & elsewhere. Running the
test with that:
	
	+ test-tool trace2 101timer 5 10 3
	fatal: failed to create thread[0]
	error: last command exited with $?=128

For that all that's needed is this:
	
	diff --git a/t/t0211-trace2-perf.sh b/t/t0211-trace2-perf.sh
	index 0b3436e8cac..cfba6861322 100755
	--- a/t/t0211-trace2-perf.sh
	+++ b/t/t0211-trace2-perf.sh
	@@ -203,7 +203,7 @@ test_expect_success 'stopwatch timer test/test1' '
	 	have_timer_event "main" "timer" "test" "test1" 5 actual
	 '
	 
	-test_expect_success 'stopwatch timer test/test2' '
	+test_expect_success PTHREADS 'stopwatch timer test/test2' '
	 	test_when_finished "rm trace.perf actual" &&
	 	test_config_global trace2.perfBrief 1 &&
	 	test_config_global trace2.perfTarget "$(pwd)/trace.perf" &&
	@@ -249,7 +249,7 @@ test_expect_success 'global counter test/test1' '
	 	have_counter_event "main" "counter" "test" "test1" 15 actual
	 '
	 
	-test_expect_success 'global counter test/test2' '
	+test_expect_success PTHREADS 'global counter test/test2' '
	 	test_when_finished "rm trace.perf actual" &&
	 	test_config_global trace2.perfBrief 1 &&
	 	test_config_global trace2.perfTarget "$(pwd)/trace.perf" &&

I think that's the correct fix here, and it should go into rc1.

  reply	other threads:[~2022-11-24 18:47 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-11-24 17:59 [BUG} t2201.7+ Failure to Create Thread in 2.39.0-rc0 rsbecker
2022-11-24 18:40 ` Ævar Arnfjörð Bjarmason [this message]
2022-11-24 19:18   ` rsbecker
2022-11-24 21:35     ` Ævar Arnfjörð Bjarmason
2022-11-24 21:51   ` Ævar Arnfjörð Bjarmason
2022-11-24 22:22     ` rsbecker

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=221124.86o7swnqc1.gmgdl@evledraar.gmail.com \
    --to=avarab@gmail.com \
    --cc=git@vger.kernel.org \
    --cc=jeffhost@microsoft.com \
    --cc=rsbecker@nexbridge.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).