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 BCCEA3858C53; Tue, 26 Apr 2022 22:59:37 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org BCCEA3858C53 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 7FE50302BBED; Wed, 27 Apr 2022 00:59:36 +0200 (CEST) Received: by reform (Postfix, from userid 1000) id 6CB762E82D76; Wed, 27 Apr 2022 00:59:35 +0200 (CEST) Date: Wed, 27 Apr 2022 00:59:35 +0200 From: Mark Wielaard To: Overseers mailing list Cc: Luis Machado , "Frank Ch. Eigler" , "gdb@sourceware.org" , binutils@sourceware.org Subject: Re: Adding binutils to the GNU Toolchain buildbot on sourceware Message-ID: References: <5c1f217a-109c-2973-6c69-abf412133dee@arm.com> <524b04b7-a78c-7aae-4605-b40f61e6830c@arm.com> MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="E8GZsnjTRQIQ/3Pq" Content-Disposition: inline In-Reply-To: X-Spam-Status: No, score=-9.8 required=5.0 tests=BAYES_00, GIT_PATCH_0, 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: gdb@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gdb mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 26 Apr 2022 22:59:40 -0000 --E8GZsnjTRQIQ/3Pq Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Hi Frank, On Tue, Apr 26, 2022 at 05:40:25AM -0400, Frank Ch. Eigler via Overseers wrote: > > Unfortunately gdb's testsuite is not too reliable. It's been improved over > > the years, but still gives quite a bit of non-deterministic results based on > > distro version/compiler version etc. So I'd leave those out in favor of just > > making sure things build properly. > > This problem is why we're building out a gadget called bunsen, which > is a tool to absorb histories of testsuites, and draw statistical > conclusions. Still early days, but noisy testsuites are not a > problem. Right. And because Keith is very involved with this having builders that feed bunsen lots of gdb testsuite results is important. Sorry if I made it sound like "large checks" aren't important for the builder. They are! But I do think we should try to split the "heavy test everything builders" from the "quick sanity check builders" a bit. e.g. maybe we can use the fedrawhide-x86_64 worker only for the full gdb-binutils builder and use another worker for the "quick" binutils only builder. That way a "quick" builder doesn't get behind a couple of "heavy" builds (which means you might get your quick result only after a couple of hours). How about using the other fedora-x86_64 worker (as attached)? I tried a couple of other workers, but some were too slow, or didn't generate a clean make check-ld results. Cheers, Mark --E8GZsnjTRQIQ/3Pq Content-Type: text/x-diff; charset=us-ascii Content-Disposition: attachment; filename="binutils-fedora-x86_64.patch" diff --git a/builder/master.cfg b/builder/master.cfg index 340904f..b706401 100644 --- a/builder/master.cfg +++ b/builder/master.cfg @@ -123,7 +123,7 @@ debian_ppc64_worker = worker.Worker("debian-ppc64", 'fitzsim@fitzsim.org']); c['workers'].append(debian_ppc64_worker) -# Power VM run by Tom on a Talos workstation +# Frank's Fedora Rawhide builder fedrawhide_x86_64_worker = worker.Worker("fedrawhide-x86_64", getpw("fedrawhide-x86_64"), max_builds=1, @@ -374,7 +374,8 @@ binutils_scheduler = schedulers.SingleBranchScheduler( change_filter=util.ChangeFilter(project="binutils-gdb", branch="master"), fileIsImportant=binutilsImportant, - builderNames=["binutils-debian-amd64", "binutils-fedrawhide-x86_64"]) + builderNames=["binutils-debian-amd64", + "binutils-fedora-x86_64"]) c['schedulers'].append(binutils_scheduler) # Only trigger scheduler for changes to gdb (or deps) @@ -1146,12 +1147,12 @@ binutils_debian_amd64_builder = util.BuilderConfig( factory=binutils_factory) c['builders'].append(binutils_debian_amd64_builder) -binutils_fedrawhide_x86_64_builder = util.BuilderConfig( - name="binutils-fedrawhide-x86_64", - workernames=["fedrawhide-x86_64"], +binutils_fedora_x86_64_builder = util.BuilderConfig( + name="binutils-fedora-x86_64", + workernames=["fedora-x86_64"], tags=["binutils", "fedora", "x86_64"], factory=binutils_factory) -c['builders'].append(binutils_fedrawhide_x86_64_builder) +c['builders'].append(binutils_fedora_x86_64_builder) # gdb build steps, factory and builders diff --git a/htdocs/index.html b/htdocs/index.html index b08a17b..d732d5d 100644 --- a/htdocs/index.html +++ b/htdocs/index.html @@ -34,6 +34,8 @@ binutils debian-amd64
 + fedora-x86_64
+ bzip2 --E8GZsnjTRQIQ/3Pq--