Re: Adding binutils to the GNU Toolchain buildbot on sourceware

Mark Wielaard
2022-04-28T14:20:46Z

[-- Attachment #1: Type: text/plain, Size: 1399 bytes --]

Hi Luis (added Tom ad Dan to CC for debian-ppc64/fedora-ppc64le),

On Thu, Apr 28, 2022 at 01:23:44PM +0100, Luis Machado wrote:
> It would make sense to build-test gdb if any of the following changes:

Thanks, updated the gdb_files list. Also added fedora-x86_64,
debian-armhf, debian-arm64 and fedora-s390 builders. See attached.

Tom, would it be OK to add a debian-ppc64 builder for gdb? As you can
see in the master.cfg it only does a build of gdb and gdbserver for
now, no tests are run. So it just would make sure that things
build. We'll might add tests later, if we can figure out a sane subset
that should always be green (and won't take more than a couple of
minutes to run). BTW. I am also working on adding more builders for
binutils (gas, ld, binutils) but the current testsuite doesn't seem
clean on anything than latest fedora-x86_64.

Dan, the fedora-ppc64le worker currently doesn't have enough diskspace
to add binutils and gdb builders. There is only 4GB left. I was
wondering if we should deprecate the fedora-ppc64 builder now that we
have Tom's debian-ppc64 which is much faster and has a more modern GNU
Toolchain installed. The fedora-ppc64 setup is no longer supported and
I believe we are the only users. So if we deprecate it maybe we can
repurpose some of the resources (disk space) to the fedora-ppc64le VM,
so we can add builders for binutils and gdb.

Thanks,

Mark

[-- Attachment #2: 0001-Update-gdb-files-and-fedora-x86_64-debian-armhf-arm6.patch --]
[-- Type: text/plain, Size: 4563 bytes --]

From 75a51ffbf1e47c36dd53d2e0bc42b8ec100baa8d Mon Sep 17 00:00:00 2001
From: Mark Wielaard <mark@klomp.org>
Date: Thu, 28 Apr 2022 15:58:06 +0200
Subject: [PATCH] Update gdb files and fedora-x86_64,
debian-armhf/arm64,fedora-s390x builders

---
builder/master.cfg | 51 ++++++++++++++++++++++++++++++++++++++++++----
htdocs/index.html | 10 +++++++++
2 files changed, 57 insertions(+), 4 deletions(-)

diff --git a/builder/master.cfg b/builder/master.cfg
index 0c39365..75b2801 100644
--- a/builder/master.cfg
+++ b/builder/master.cfg
@@ -379,9 +379,20 @@ c['schedulers'].append(binutils_scheduler)

# Only trigger scheduler for changes to gdb (or deps)
gdb_files = ["bfd/",
- "gdb/", "gdbserver/", "gdbsupport/",
- "gnulib/", "sim/",
- "include/", "libiberty/", "opcodes/",
+ "config/",
+ "gdb/",
+ "gdbserver/",
+ "gdbsupport/",
+ "include/",
+ "intl/",
+ "libbacktrace/",
+ "libctf/",
+ "libdecnumber/",
+ "libiberty/",
+ "opcodes/",
+ "readline/",
+ "sim/",
+ "zlib/",
"configure", "Makefile.in"]

def gdbImportant(change):
@@ -397,7 +408,11 @@ gdb_scheduler = schedulers.SingleBranchScheduler(
change_filter=util.ChangeFilter(project="binutils-gdb",
branch="master"),
fileIsImportant=gdbImportant,
- builderNames=["gdb-centos-x86_64"])
+ builderNames=["gdb-centos-x86_64",
+ "gdb-fedora-x86_64",
+ "gdb-debian-armhf",
+ "gdb-debian-arm64",
+ "gdb-fedora-s390x"])
c['schedulers'].append(gdb_scheduler)

# A scheduler for everything binutils-gdb without filters
@@ -1189,6 +1204,34 @@ gdb_centos_x86_64_builder = util.BuilderConfig(
factory=gdb_factory)
c['builders'].append(gdb_centos_x86_64_builder)

+gdb_fedora_x86_64_builder = util.BuilderConfig(
+ name="gdb-fedora-x86_64",
+ workernames=["fedora-x86_64"],
+ tags=["gdb", "fedora", "x86_64"],
+ factory=gdb_factory)
+c['builders'].append(gdb_fedora_x86_64_builder)
+
+gdb_debian_armhf_builder = util.BuilderConfig(
+ name="gdb-debian-armhf",
+ workernames=["debian-armhf"],
+ tags=["gdb", "debian", "armhf"],
+ factory=gdb_factory)
+c['builders'].append(gdb_debian_armhf_builder)
+
+gdb_debian_arm64_builder = util.BuilderConfig(
+ name="gdb-debian-arm64",
+ workernames=["debian-arm64"],
+ tags=["gdb", "debian", "arm64"],
+ factory=gdb_factory)
+c['builders'].append(gdb_debian_arm64_builder)
+
+gdb_fedora_s390x_builder = util.BuilderConfig(
+ name="gdb-fedora-s390x",
+ workernames=["fedora-s390x"],
+ tags=["gdb", "fedora", "s390x"],
+ factory=gdb_factory)
+c['builders'].append(gdb_fedora_s390x_builder)
+

# binutils-gdb build steps, factory and builders
# just a native build
diff --git a/htdocs/index.html b/htdocs/index.html
index b08a17b..b98b00a 100644
--- a/htdocs/index.html
+++ b/htdocs/index.html
@@ -129,8 +129,10 @@


gccrust
+<!--
debian-arm64

+-->
fedora-x86_64

debian-i386
@@ -148,6 +150,14 @@
gdb
centos-x86_64

+ fedora-x86_64
+
+ debian-armhf
+
+ debian-arm64
+
+ fedora-s390x
+


libabigail
--
2.27.0