From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from gnu.wildebeest.org (gnu.wildebeest.org [45.83.234.184]) by sourceware.org (Postfix) with ESMTPS id C48B13858D28 for ; Sun, 24 Apr 2022 10:22:53 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org C48B13858D28 Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=klomp.org Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=klomp.org Received: from reform (deer0x0b.wildebeest.org [172.31.17.141]) (using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by gnu.wildebeest.org (Postfix) with ESMTPSA id 0EFC4302BBED; Sun, 24 Apr 2022 12:22:50 +0200 (CEST) Received: by reform (Postfix, from userid 1000) id 813052E804F0; Sun, 24 Apr 2022 12:22:50 +0200 (CEST) Date: Sun, 24 Apr 2022 12:22:50 +0200 From: Mark Wielaard To: Thomas Fitzsimmons Cc: "Frank Ch. Eigler" , elfutils-devel@sourceware.org Subject: Re: run-debuginfod-webapi-concurrency.sh Message-ID: References: <20220423011953.1D7713858430@sourceware.org> <20220423214525.GJ23969@redhat.com> MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="CMBJVWNHX0NMNaom" Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: X-Spam-Status: No, score=-9.5 required=5.0 tests=BAYES_00, GIT_PATCH_0, JMQ_SPF_NEUTRAL, KAM_DMARC_STATUS, SPF_HELO_NONE, SPF_PASS, TXREP autolearn=ham autolearn_force=no version=3.4.4 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on server2.sourceware.org X-BeenThere: elfutils-devel@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Elfutils-devel mailing list List-Unsubscribe: , List-Archive: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 24 Apr 2022 10:22:56 -0000 --CMBJVWNHX0NMNaom Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Hi, On Sat, Apr 23, 2022 at 10:56:01PM -0400, Thomas Fitzsimmons wrote: > "Frank Ch. Eigler" writes: > >> But there is another way to prevent the "Server reached connection > >> limit. Closing inbound connection." Pass the MHD_USE_ITC flag to > >> MHD_start_daemon: > > > > Yeah, that looked promising to me too. When I was last working on > > this, that would have been my next thing to try. I can't think of > > a relevant downside, so let's try it. (Add a #ifdef guard around > > that macro, for older libmicrohttpd, like rhel7 methinks.) > > On debian-ppc64, with and without the MHD_USE_ITC patch, I ran the test > 20 times in a shell loop. With MHD_USE_ITC, I got 20 passes, without > it, 9 passes and 11 failures. > > With the patch applied, a full "make check" succeeded. Nice. Thanks for the feedback and testing. I pushed the attached. Cheers, Mark --CMBJVWNHX0NMNaom Content-Type: text/x-diff; charset=utf-8 Content-Disposition: attachment; filename="0001-debuginfod-Use-MHD_USE_ITC-in-MHD_start_daemon-flags.patch" Content-Transfer-Encoding: 8bit >From 3bcf887340fd47d0d8a3671cc45abe2989d1fd6c Mon Sep 17 00:00:00 2001 From: Mark Wielaard Date: Sun, 24 Apr 2022 12:16:58 +0200 Subject: [PATCH] debuginfod: Use MHD_USE_ITC in MHD_start_daemon flags MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit This prevents the "Server reached connection limit. Closing inbound connection." issue we have been seeing in the run-debuginfod-webapi-concurrency.sh testcase. From the manual: If the connection limit is reached, MHD’s behavior depends a bit on other options. If MHD_USE_ITC was given, MHD will stop accepting connections on the listen socket. This will cause the operating system to queue connections (up to the listen() limit) above the connection limit. Those connections will be held until MHD is done processing at least one of the active connections. If MHD_USE_ITC is not set, then MHD will continue to accept() and immediately close() these connections. https://sourceware.org/bugzilla/show_bug.cgi?id=28708 Signed-off-by: Mark Wielaard --- debuginfod/ChangeLog | 4 ++++ debuginfod/debuginfod.cxx | 3 +++ tests/ChangeLog | 4 ++++ tests/run-debuginfod-webapi-concurrency.sh | 4 +++- 4 files changed, 14 insertions(+), 1 deletion(-) diff --git a/debuginfod/ChangeLog b/debuginfod/ChangeLog index d6f7b282..0f1bca6f 100644 --- a/debuginfod/ChangeLog +++ b/debuginfod/ChangeLog @@ -1,3 +1,7 @@ +2022-04-24 Mark Wielaard + + * debuginfod.cxx (main): Add MHD_USE_ITC to MHD_start_daemon flags. + 2022-04-13 Aaron Merey * debuginfod-client.c (debuginfod_query_server): diff --git a/debuginfod/debuginfod.cxx b/debuginfod/debuginfod.cxx index 9c0217f6..adca8208 100644 --- a/debuginfod/debuginfod.cxx +++ b/debuginfod/debuginfod.cxx @@ -3910,6 +3910,9 @@ main (int argc, char *argv[]) | MHD_USE_EPOLL #endif | MHD_USE_DUAL_STACK +#if MHD_VERSION >= 0x00095200 + | MHD_USE_ITC +#endif | MHD_USE_DEBUG, /* report errors to stderr */ http_port, NULL, NULL, /* default accept policy */ diff --git a/tests/ChangeLog b/tests/ChangeLog index 2286f53f..44b8df88 100644 --- a/tests/ChangeLog +++ b/tests/ChangeLog @@ -1,3 +1,7 @@ +2022-04-24 Mark Wielaard + + * run-debuginfod-webapi-concurrency.sh: Fix PR number in xfail. + 2022-04-23 Mark Wielaard * run-debuginfod-webapi-concurrency.sh: Lower parallel lookups. diff --git a/tests/run-debuginfod-webapi-concurrency.sh b/tests/run-debuginfod-webapi-concurrency.sh index 4928f6d0..47dcadcc 100755 --- a/tests/run-debuginfod-webapi-concurrency.sh +++ b/tests/run-debuginfod-webapi-concurrency.sh @@ -62,6 +62,8 @@ do PID1=0 done -xfail "grep Server.reached.connection vlog$PORT1" # PR18661 +# Note this xfail comes too late, the above wait_ready for +# http_responses_transfer_bytes_count will have failed. +xfail "grep Server.reached.connection vlog$PORT1" # PR28661 exit 0 -- 2.30.2 --CMBJVWNHX0NMNaom--