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 D90FC3858CDB for ; Wed, 17 Aug 2022 13:01:01 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org D90FC3858CDB 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 (unknown [178.228.156.55]) (using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by gnu.wildebeest.org (Postfix) with ESMTPSA id 53298300070C; Wed, 17 Aug 2022 15:01:00 +0200 (CEST) Received: by reform (Postfix, from userid 1000) id E5A572E80362; Wed, 17 Aug 2022 15:00:58 +0200 (CEST) Date: Wed, 17 Aug 2022 15:00:58 +0200 From: Mark Wielaard To: Christophe Lyon Cc: buildbot@sourceware.org Subject: Re: Arm GCC buildbot workers Message-ID: References: <84fe482f-0b5e-face-9a7a-bb04ac162686@arm.com> <20220811203956.GE5520@gnu.wildebeest.org> <20220812164415.GI5520@gnu.wildebeest.org> <45bd3411-6b54-c49e-3b6d-5cbf85a5dd10@arm.com> MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="psUAf5pfL8bGC5qW" Content-Disposition: inline In-Reply-To: <45bd3411-6b54-c49e-3b6d-5cbf85a5dd10@arm.com> X-Spam-Status: No, score=-11.0 required=5.0 tests=BAYES_00, GIT_PATCH_0, JMQ_SPF_NEUTRAL, KAM_DMARC_STATUS, SPF_HELO_NONE, SPF_PASS, TXREP, T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org X-BeenThere: buildbot@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: "The https://builder.sourceware.org/ buildbot" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 17 Aug 2022 13:01:03 -0000 --psUAf5pfL8bGC5qW Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Hi Christophe, On Tue, Aug 16, 2022 at 01:54:00PM +0200, Christophe Lyon wrote: > On 8/12/22 18:44, Mark Wielaard wrote: > > The ubuntu22_04-arm64 worker seems to happily do full gcc bootstrap > > and check builds: > > https://builder.sourceware.org/buildbot/#/workers/34 > > > > It does find a couple of failing testcases, which you might want to > > look at, but most seem expected. All results are recorded in bunsen to > > be compared. > > Great, thanks! I guess the errors are the same as observed on > gcc-full-debian-arm64? I assume so, but don't have internet access at the moment to check. You can use the bunsen results database to compare different builds: https://builder.sourceware.org/testruns/ > Anyway I don't think any target has a clean 'make check' in GCC :-( > > This will increase the need for a regression-detection mechanism in the > buildbot, otherwise we'll only have red bullets which people will ignore.... Yes. I am hoping we can use the bunsen results database to check for regressions. We have the current build number of the builder, so we should be able to pull the bunsen results of the previous build. Then we should be able to compare the dejagnu summary. > > The ubuntu22_04-armhf worker however fails to do a full gcc build: > > https://builder.sourceware.org/buildbot/#/workers/33 > > > > In file included from ../../../gcc/libgcc/../gcc/tsystem.h:87, > > from ../../../gcc/libgcc/libgcc2.c:27: > > /usr/include/stdio.h:27:10: fatal error: bits/libc-header-start.h: No such file or directory > > 27 | #include > > | ^~~~~~~~~~~~~~~~~~~~~~~~~~ > > compilation terminated. > > > > It looks like that happens while building libcc for > > armv8l-unknown-linux-gnueabihf. Maybe missing devel header files? Or > > does the build needs to be configured to exclude this target? > > So... I dug a bit and... you need to add --with-float=hard on GCC's > configure line. For some reason GCC's configure is not able to infer > that from the trailing 'hf' in the target name. Can you add it for this > target only or is the configure line shared by all GCC workers? Currently the configure line is shared by all gcc-full builders. But that can be changed. As in the attached patch (apologies for my python skills). I'll push it once I have internet access again. Cheers, Mark --psUAf5pfL8bGC5qW Content-Type: text/x-diff; charset=us-ascii Content-Disposition: attachment; filename="0001-Add-gcc_full_build_factory_gen-function.patch" >From dc366c58f2f63cc5aa4955f60ba2c337203c05d0 Mon Sep 17 00:00:00 2001 From: Mark Wielaard Date: Wed, 17 Aug 2022 14:58:40 +0200 Subject: [PATCH] Add gcc_full_build_factory_gen function Call it with extra_configure_arg="--with-float=hard" for the gcc_full_ubuntu_armhf_builder. --- builder/master.cfg | 62 +++++++++++++++++++++++++--------------------- 1 file changed, 34 insertions(+), 28 deletions(-) diff --git a/builder/master.cfg b/builder/master.cfg index f945b14..dc51fd0 100644 --- a/builder/master.cfg +++ b/builder/master.cfg @@ -2987,31 +2987,36 @@ gcc_debian_amd64_builder = util.BuilderConfig( factory=gcc_build_factory) c['builders'].append(gcc_debian_amd64_builder) -gcc_full_build_factory = util.BuildFactory() -gcc_full_build_factory.addStep(gcc_build_git_step) -gcc_full_build_factory.addStep(gcc_rm_build_step) -gcc_full_build_configure_step = steps.Configure( - workdir='gcc-build', - command=['../gcc/configure',], - name='configure', - haltOnFailure=True) -gcc_full_build_factory.addStep(gcc_full_build_configure_step) -gcc_full_build_factory.addStep(steps.Compile( - workdir='gcc-build', - command=['make', util.Interpolate('-j%(prop:maxcpus)s')], - name='make', - haltOnFailure=True)) -# We want parallelism to get through this as quickly as possible. -# Even if that means bunsen gets some parallel/duplicate log files -gcc_full_build_factory.addStep(steps.Test( - workdir='gcc-build', - command=['make', 'check', util.Interpolate('-j%(prop:maxcpus)s')], - name='make check', - haltOnFailure=False, flunkOnFailure=True)) -gcc_full_build_factory.addSteps(bunsen_logfile_upload_cpio_steps( - ["*.log", "*.sum"], - workdir='gcc-build')) -gcc_full_build_factory.addStep(gcc_rm_build_step) +def gcc_full_build_factory_gen(extra_configure_arg=None): + gcc_full_build_factory = util.BuildFactory() + gcc_full_build_factory.addStep(gcc_build_git_step) + gcc_full_build_factory.addStep(gcc_rm_build_step) + configure_command = ['../gcc/configure'] + if extra_configure_arg: + configure_command.append(extra_configure_arg) + gcc_full_build_configure_step = steps.Configure( + workdir='gcc-build', + command=configure_command, + name='configure', + haltOnFailure=True) + gcc_full_build_factory.addStep(gcc_full_build_configure_step) + gcc_full_build_factory.addStep(steps.Compile( + workdir='gcc-build', + command=['make', util.Interpolate('-j%(prop:maxcpus)s')], + name='make', + haltOnFailure=True)) + # We want parallelism to get through this as quickly as possible. + # Even if that means bunsen gets some parallel/duplicate log files + gcc_full_build_factory.addStep(steps.Test( + workdir='gcc-build', + command=['make', 'check', util.Interpolate('-j%(prop:maxcpus)s')], + name='make check', + haltOnFailure=False, flunkOnFailure=True)) + gcc_full_build_factory.addSteps(bunsen_logfile_upload_cpio_steps( + ["*.log", "*.sum"], + workdir='gcc-build')) + gcc_full_build_factory.addStep(gcc_rm_build_step) + return gcc_full_build_factory gcc_full_debian_amd64_builder = util.BuilderConfig( name="gcc-full-debian-amd64", @@ -3020,7 +3025,7 @@ gcc_full_debian_amd64_builder = util.BuilderConfig( readContainerFile('debian-stable')}, workernames=big_vm_workers, tags=["gcc-full", "debian", "x86_64"], - factory=gcc_full_build_factory) + factory=gcc_full_build_factory_gen()) c['builders'].append(gcc_full_debian_amd64_builder) gcc_full_ubuntu_armhf_builder = util.BuilderConfig( @@ -3028,7 +3033,8 @@ gcc_full_ubuntu_armhf_builder = util.BuilderConfig( collapseRequests=True, workernames=["ubuntu22_04-armhf"], tags=["gcc-full", "ubuntu", "armhf"], - factory=gcc_full_build_factory) + factory=gcc_full_build_factory_gen( + extra_configure_arg="--with-float=hard")) c['builders'].append(gcc_full_ubuntu_armhf_builder) gcc_full_ubuntu_arm64_builder = util.BuilderConfig( @@ -3036,7 +3042,7 @@ gcc_full_ubuntu_arm64_builder = util.BuilderConfig( collapseRequests=True, workernames=["ubuntu22_04-arm64"], tags=["gcc-full", "ubuntu", "arm64"], - factory=gcc_full_build_factory) + factory=gcc_full_build_factory_gen()) c['builders'].append(gcc_full_ubuntu_arm64_builder) -- 2.30.2 --psUAf5pfL8bGC5qW--