public inbox for buildbot@sourceware.org
 help / color / mirror / Atom feed
* binutils builder.sourceware.org status
@ 2022-07-03 16:28 Mark Wielaard
  2022-07-04  6:21 ` Jan Beulich
  0 siblings, 1 reply; 2+ messages in thread
From: Mark Wielaard @ 2022-07-03 16:28 UTC (permalink / raw)
  To: binutils; +Cc: buildbot

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

Hi,

A quick overview of the status of the buildbot binutils status (and
enabling of libctf for those builders that use GCC12 now).

https://builder.sourceware.org/buildbot/#/builders?tags=binutils
We have the following fully green CI builders:

binutils-debian-amd64
binutils-debian-ppc64
binutils-fedora-ppc64le
binutils-fedora-x86_64
binutils-opensuseleap-x86_64
binutils-opensusetw-x86_64
binutils-rawhide-x86_64

And the following red CI builders:

binutils-debian-arm64
binutils-debian-armhf
binutils-debian-i386
binutils-fedora-s390x

All 4 fail some ld tests. For now I disabled the ld tests on arm64 and
armhf to turn them green and also because they are both running
debian-old stable and are somewhat slow.

binutils-debian-i386 fails:

FAIL: Run p_align-1b with PIE
FAIL: Run p_align-1d with -Wl,-z,max-page-size=0x1000 with PIE
FAIL: Run pr19031
FAIL: Run got1
FAIL: Undefined weak symbol (-fPIE -no-pie)
FAIL: Undefined weak symbol (-fPIE -pie)

binutils-fedora-s390x fails:

FAIL: pr26580-3
FAIL: Run pr19719 fun undefined

The are also two "full" binutils-gdb builders:

binutils-gdb-fedrawhide-x86_64
binutils-gdb-clang-fedrawhide-x86_64

Which are currently in a somewhat limbo state since the gdb testsuite
seems to crash the kernel... Test results (when the builder do
succeed) are in bunsen though. Which is interesting to compare to see
what difference a non-gcc compiler makes.
https://builder.sourceware.org/testruns/?has_keyvalue_like_k=testrun.git_describe&has_keyvalue_like_v=%25binutils-gdb%25

There are try-builders for all of the above, minus the full
binutils-gdb builder. The try builders aren't used that much yet. And
one complaint was that since several CI builders have existing
failures it isn't immediately clear whether a patch caused something
or not. So it would be nice if we could resolve (or xfail) the above
tests.

We are seeing some git update failures on s390x and ppc64le, it isn't
entirely sure why this happens only on these arches, but it is
annoying since it is seen as a new failure and so sents warning
emails. Still investigating how to work around this.

The fedora-x86_64, fedora-s390x fedora-ppc64le, opensuse
tumbleweed-x86_64 and rawhide-x86_64 have been upgraded and use GCC12
now. For these builder (and the try builders) libctf is now explicitly
enabled, compiled and tested. libctf logs are now also uploaded to
bunsed. Please double check the master.cfg patch to see if it does the
right thing now.

Thanks,

Mark

[-- Attachment #2: 0001-Don-t-run-ld-checks-on-arm64-and-armhf.patch --]
[-- Type: text/x-diff, Size: 3579 bytes --]

From d2b7178a1338d63a80ba6176988a6e7b450c2687 Mon Sep 17 00:00:00 2001
From: Mark Wielaard <mark@klomp.org>
Date: Sun, 3 Jul 2022 13:47:36 +0200
Subject: [PATCH] Don't run ld checks on arm64 and armhf

They fail because running on debian-oldstable and they boards are a
little slow. We can enable them when we get new arm hardware and a
newer distro installed.
---
 builder/master.cfg | 29 +++++++++++++++++++++++++++--
 1 file changed, 27 insertions(+), 2 deletions(-)

diff --git a/builder/master.cfg b/builder/master.cfg
index f29afa9..63d80d8 100644
--- a/builder/master.cfg
+++ b/builder/master.cfg
@@ -1609,6 +1609,22 @@ binutils_steps_bunsen = bunsen_logfile_upload_steps([
         (["../binutils-build/gas/config.log"], "gas"),
         (["../binutils-build/binutils/binutils.sum", "../binutils-build/binutils/binutils.log", "../binutils-build/binutils/config.log"], "binutils")])
 
+binutils_step_check_gas_binutils = steps.Test(
+        workdir='binutils-build',
+        command=['make',
+                 util.Interpolate('-j%(prop:ncpus)s'),
+                 'check-gas', 'check-binutils'],
+        name='make check',
+        logfiles={ "gas.sum": "gas/testsuite/gas.sum",
+                   "gas.log": "gas/testsuite/gas.log",
+                   "binutils.sum": "binutils/binutils.sum",
+                   "binutils.log": "binutils/binutils.log" },
+        haltOnFailure=False, flunkOnFailure=True)
+binutils_steps_bunsen_gas_binutils = bunsen_logfile_upload_steps([
+        (["../binutils-build/gas/testsuite/gas.sum", "../binutils-build/gas/testsuite/gas.log"], "gas/testsuite"),
+        (["../binutils-build/gas/config.log"], "gas"),
+        (["../binutils-build/binutils/binutils.sum", "../binutils-build/binutils/binutils.log", "../binutils-build/binutils/config.log"], "binutils")])
+
 binutils_factory = util.BuildFactory()
 binutils_factory.addStep(binutils_step_git)
 binutils_factory.addStep(binutils_step_rm_build_dir)
@@ -1625,6 +1641,15 @@ binutils_factory_target_all.addStep(binutils_step_compile)
 binutils_factory_target_all.addStep(binutils_step_check)
 binutils_factory_target_all.addSteps(binutils_steps_bunsen)
 
+# Builds everything, but checks just gas and binutils, not ld.
+binutils_factory_gas_binutils = util.BuildFactory()
+binutils_factory_gas_binutils.addStep(binutils_step_git)
+binutils_factory_gas_binutils.addStep(binutils_step_rm_build_dir)
+binutils_factory_gas_binutils.addStep(binutils_step_configure)
+binutils_factory_gas_binutils.addStep(binutils_step_compile)
+binutils_factory_gas_binutils.addStep(binutils_step_check_gas_binutils)
+binutils_factory_gas_binutils.addSteps(binutils_steps_bunsen_gas_binutils)
+
 binutils_debian_amd64_builder = util.BuilderConfig(
 	name="binutils-debian-amd64",
         properties={'container-file':
@@ -1689,7 +1714,7 @@ binutils_debian_armhf_builder = util.BuilderConfig(
         workernames=["debian-armhf"],
         tags=["binutils", "debian", "armhf"],
         collapseRequests=True,
-        factory=binutils_factory)
+        factory=binutils_factory_gas_binutils)
 c['builders'].append(binutils_debian_armhf_builder)
 
 binutils_debian_arm64_builder = util.BuilderConfig(
@@ -1697,7 +1722,7 @@ binutils_debian_arm64_builder = util.BuilderConfig(
         workernames=["debian-arm64"],
         tags=["binutils", "debian", "arm64"],
         collapseRequests=True,
-        factory=binutils_factory)
+        factory=binutils_factory_gas_binutils)
 c['builders'].append(binutils_debian_arm64_builder)
 
 binutils_debian_i386_builder = util.BuilderConfig(
-- 
2.30.2


[-- Attachment #3: 0001-Enable-building-testing-and-log-uploading-for-binuti.patch --]
[-- Type: text/x-diff, Size: 9762 bytes --]

From e253f7c708a12567439aac7e28f43a6fe93eeaee Mon Sep 17 00:00:00 2001
From: Mark Wielaard <mark@klomp.org>
Date: Sun, 3 Jul 2022 18:26:11 +0200
Subject: [PATCH] Enable building, testing and log uploading for binutils
 libctf

The fedora-x86_64, fedora-s390x fedora-ppc64le, opensuse
tumbleweed-x86_64 and rawhide-x86_64 have been upgraded and use GCC12
and can build and test libctf now.
---
 builder/master.cfg | 83 ++++++++++++++++++++++++++++++++++++++++------
 1 file changed, 73 insertions(+), 10 deletions(-)

diff --git a/builder/master.cfg b/builder/master.cfg
index 63d80d8..92a6981 100644
--- a/builder/master.cfg
+++ b/builder/master.cfg
@@ -1575,6 +1575,14 @@ binutils_step_configure = steps.Configure(
                  '--enable-shared'],
         name='configure',
         haltOnFailure=True)
+binutils_step_configure_libctf = steps.Configure(
+        workdir='binutils-build',
+        command=['../binutils-gdb/configure',
+                 '--enable-gold',
+                 '--enable-libctf',
+                 '--enable-shared'],
+        name='configure',
+        haltOnFailure=True)
 binutils_step_configure_target_all = steps.Configure(
         workdir='binutils-build',
         command=['../binutils-gdb/configure',
@@ -1583,6 +1591,15 @@ binutils_step_configure_target_all = steps.Configure(
                  '--enable-targets=all'],
         name='configure',
         haltOnFailure=True)
+binutils_step_configure_target_all_libctf = steps.Configure(
+        workdir='binutils-build',
+        command=['../binutils-gdb/configure',
+                 '--enable-gold',
+                 '--enable-shared',
+                 '--enable-targets=all',
+		 '--enable-libctf'],
+        name='configure',
+        haltOnFailure=True)
 binutils_step_compile = steps.Compile(
         workdir='binutils-build',
         command=['make',
@@ -1590,6 +1607,14 @@ binutils_step_compile = steps.Compile(
                  'all-gas', 'all-ld', 'all-binutils', 'all-gold'],
         name='make',
         haltOnFailure=True)
+binutils_step_compile_libctf = steps.Compile(
+        workdir='binutils-build',
+        command=['make',
+                 util.Interpolate('-j%(prop:ncpus)s'),
+                 'all-gas', 'all-ld', 'all-binutils', 'all-gold',
+                 'all-libctf'],
+        name='make',
+        haltOnFailure=True)
 binutils_step_check = steps.Test(
         workdir='binutils-build',
         command=['make',
@@ -1609,6 +1634,28 @@ binutils_steps_bunsen = bunsen_logfile_upload_steps([
         (["../binutils-build/gas/config.log"], "gas"),
         (["../binutils-build/binutils/binutils.sum", "../binutils-build/binutils/binutils.log", "../binutils-build/binutils/config.log"], "binutils")])
 
+binutils_step_check_libctf = steps.Test(
+        workdir='binutils-build',
+        command=['make',
+                 util.Interpolate('-j%(prop:ncpus)s'),
+                 'check-ld', 'check-gas', 'check-binutils', 'check-libctf'],
+        name='make check',
+        logfiles={ "ld.sum": "ld/ld.sum",
+                   "ld.log": "ld/ld.log",
+                   "gas.sum": "gas/testsuite/gas.sum",
+                   "gas.log": "gas/testsuite/gas.log",
+                   "binutils.sum": "binutils/binutils.sum",
+                   "binutils.log": "binutils/binutils.log",
+                   "libctf.sum": "libctf/libctf.sum",
+                   "libctf.log": "libctf/libctf.log" },
+        haltOnFailure=False, flunkOnFailure=True)
+binutils_steps_bunsen_libctf = bunsen_logfile_upload_steps([
+        (["../binutils-build/ld/ld.sum", "../binutils-build/ld/ld.log", "../binutils-build/ld/config.log"], "ld"),
+        (["../binutils-build/gas/testsuite/gas.sum", "../binutils-build/gas/testsuite/gas.log"], "gas/testsuite"),        
+        (["../binutils-build/gas/config.log"], "gas"),
+        (["../binutils-build/binutils/binutils.sum", "../binutils-build/binutils/binutils.log", "../binutils-build/binutils/config.log"], "binutils"),
+        (["../binutils-build/libctf/libctf.sum", "../binutils-build/libctf/libctf.log", "../libctf-build/libctf/config.log"], "libctf")])
+
 binutils_step_check_gas_binutils = steps.Test(
         workdir='binutils-build',
         command=['make',
@@ -1633,6 +1680,14 @@ binutils_factory.addStep(binutils_step_compile)
 binutils_factory.addStep(binutils_step_check)
 binutils_factory.addSteps(binutils_steps_bunsen)
 
+binutils_factory_libctf = util.BuildFactory()
+binutils_factory_libctf.addStep(binutils_step_git)
+binutils_factory_libctf.addStep(binutils_step_rm_build_dir)
+binutils_factory_libctf.addStep(binutils_step_configure_libctf)
+binutils_factory_libctf.addStep(binutils_step_compile_libctf)
+binutils_factory_libctf.addStep(binutils_step_check_libctf)
+binutils_factory_libctf.addSteps(binutils_steps_bunsen_libctf)
+
 binutils_factory_target_all = util.BuildFactory()
 binutils_factory_target_all.addStep(binutils_step_git)
 binutils_factory_target_all.addStep(binutils_step_rm_build_dir)
@@ -1641,6 +1696,14 @@ binutils_factory_target_all.addStep(binutils_step_compile)
 binutils_factory_target_all.addStep(binutils_step_check)
 binutils_factory_target_all.addSteps(binutils_steps_bunsen)
 
+binutils_factory_target_all_libctf = util.BuildFactory()
+binutils_factory_target_all_libctf.addStep(binutils_step_git)
+binutils_factory_target_all_libctf.addStep(binutils_step_rm_build_dir)
+binutils_factory_target_all_libctf.addStep(binutils_step_configure_target_all_libctf)
+binutils_factory_target_all_libctf.addStep(binutils_step_compile_libctf)
+binutils_factory_target_all_libctf.addStep(binutils_step_check_libctf)
+binutils_factory_target_all_libctf.addSteps(binutils_steps_bunsen_libctf)
+
 # Builds everything, but checks just gas and binutils, not ld.
 binutils_factory_gas_binutils = util.BuildFactory()
 binutils_factory_gas_binutils.addStep(binutils_step_git)
@@ -1675,7 +1738,7 @@ binutils_fedora_x86_64_builder = util.BuilderConfig(
 		    readContainerFile('fedora-latest')},
 	workernames=vm_workers,
         tags=["binutils", "fedora", "x86_64"],
-        factory=binutils_factory)
+        factory=binutils_factory_libctf)
 c['builders'].append(binutils_fedora_x86_64_builder)
 
 binutils_try_fedora_x86_64_builder = util.BuilderConfig(
@@ -1685,21 +1748,21 @@ binutils_try_fedora_x86_64_builder = util.BuilderConfig(
 		    readContainerFile('fedora-latest')},
 	workernames=vm_workers,
         tags=["binutils-try", "fedora", "x86_64"],
-        factory=binutils_factory)
+        factory=binutils_factory_libctf)
 c['builders'].append(binutils_try_fedora_x86_64_builder)
 
 binutils_fedora_s390x_builder = util.BuilderConfig(
 	name="binutils-fedora-s390x",
         workernames=["fedora-s390x"],
         tags=["binutils", "fedora", "s390x"],
-        factory=binutils_factory)
+        factory=binutils_factory_libctf)
 c['builders'].append(binutils_fedora_s390x_builder)
 
 binutils_try_fedora_s390x_builder = util.BuilderConfig(
 	name="binutils-try-fedora-s390x",
         workernames=["fedora-s390x"],
         tags=["binutils-try", "fedora", "s390x"],
-        factory=binutils_factory)
+        factory=binutils_factory_libctf)
 c['builders'].append(binutils_try_fedora_s390x_builder)
 
 binutils_debian_ppc64_builder = util.BuilderConfig(
@@ -1744,7 +1807,7 @@ binutils_fedora_ppc64le_builder = util.BuilderConfig(
 	name="binutils-fedora-ppc64le",
         workernames=["fedora-ppc64le"],
         tags=["binutils", "fedora", "ppc64le"],
-        factory=binutils_factory)
+        factory=binutils_factory_libctf)
 c['builders'].append(binutils_fedora_ppc64le_builder)
 
 binutils_try_fedora_ppc64le_builder = util.BuilderConfig(
@@ -1752,7 +1815,7 @@ binutils_try_fedora_ppc64le_builder = util.BuilderConfig(
         collapseRequests=True,
         workernames=["fedora-ppc64le"],
         tags=["binutils-try", "fedora", "ppc64le"],
-        factory=binutils_factory)
+        factory=binutils_factory_libctf)
 c['builders'].append(binutils_try_fedora_ppc64le_builder)
 
 binutils_opensusetw_x86_64_builder = util.BuilderConfig(
@@ -1761,7 +1824,7 @@ binutils_opensusetw_x86_64_builder = util.BuilderConfig(
 		    readContainerFile('opensuse-tumbleweed')},
         workernames=vm_workers,
         tags=["binutils", "opensusetw", "x86_64"],
-        factory=binutils_factory_target_all)
+        factory=binutils_factory_target_all_libctf)
 c['builders'].append(binutils_opensusetw_x86_64_builder)
 
 binutils_try_opensusetw_x86_64_builder = util.BuilderConfig(
@@ -1771,7 +1834,7 @@ binutils_try_opensusetw_x86_64_builder = util.BuilderConfig(
 		    readContainerFile('opensuse-tumbleweed')},
         workernames=vm_workers,
         tags=["binutils-try", "opensusetw", "x86_64"],
-        factory=binutils_factory_target_all)
+        factory=binutils_factory_target_all_libctf)
 c['builders'].append(binutils_try_opensusetw_x86_64_builder)
 
 binutils_opensuseleap_x86_64_builder = util.BuilderConfig(
@@ -1789,7 +1852,7 @@ binutils_rawhide_x86_64_builder = util.BuilderConfig(
 		    readContainerFile('fedora-rawhide')},
         workernames=vm_workers,
         tags=["binutils", "fedora", "x86_64"],
-        factory=binutils_factory)
+        factory=binutils_factory_libctf)
 c['builders'].append(binutils_rawhide_x86_64_builder)
 
 binutils_try_opensuseleap_x86_64_builder = util.BuilderConfig(
@@ -1809,7 +1872,7 @@ binutils_try_rawhide_x86_64_builder = util.BuilderConfig(
 		    readContainerFile('fedora-rawhide')},
         workernames=vm_workers,
         tags=["binutils-try", "fedora", "x86_64"],
-        factory=binutils_factory)
+        factory=binutils_factory_libctf)
 c['builders'].append(binutils_try_rawhide_x86_64_builder)
 
 # gdb build steps, factory and builders
-- 
2.30.2


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2022-07-04  6:22 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-07-03 16:28 binutils builder.sourceware.org status Mark Wielaard
2022-07-04  6:21 ` Jan Beulich

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 read-only IMAP folder(s) and NNTP newsgroup(s).