[PATCH 2/5] CI: Update ppc64 to use Debian Trixie

Andrew Cooper posted 5 patches 2 months, 3 weeks ago
There is a newer version of this series
[PATCH 2/5] CI: Update ppc64 to use Debian Trixie
Posted by Andrew Cooper 2 months, 3 weeks ago
Everything works fine with Debian 13.  Provide two new build jobs (for a total
of 6), and update the test job.

Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
---
CC: Anthony PERARD <anthony.perard@vates.tech>
CC: Michal Orzel <michal.orzel@amd.com>
CC: Jan Beulich <jbeulich@suse.com>
CC: Julien Grall <julien@xen.org>
CC: Roger Pau Monné <roger.pau@citrix.com>
CC: Stefano Stabellini <sstabellini@kernel.org>
CC: Shawn Anastasio <sanastasio@raptorengineering.com>
CC: Oleksii Kurochko <oleksii.kurochko@gmail.com>
CC: Doug Goldstein <cardoe@cardoe.com>
CC: Marek Marczykowski-Górecki <marmarek@invisiblethingslab.com>
CC: Victor Lira <victorm.lira@amd.com>

https://gitlab.com/xen-project/hardware/xen-staging/-/pipelines/1975927256
---
 automation/gitlab-ci/build.yaml | 18 ++++++++++++++++--
 automation/gitlab-ci/test.yaml  |  2 +-
 2 files changed, 17 insertions(+), 3 deletions(-)

diff --git a/automation/gitlab-ci/build.yaml b/automation/gitlab-ci/build.yaml
index ab5211f77e5e..3fe539dc5683 100644
--- a/automation/gitlab-ci/build.yaml
+++ b/automation/gitlab-ci/build.yaml
@@ -319,10 +319,10 @@ debian-12-x86_64-clang-debug:
   variables:
     CONTAINER: debian:12-x86_64
 
-debian-12-ppc64le-gcc-debug:
+debian-13-ppc64le-gcc-debug:
   extends: .gcc-ppc64le-cross-build-debug
   variables:
-    CONTAINER: debian:12-ppc64le
+    CONTAINER: debian:13-ppc64le
     KBUILD_DEFCONFIG: ppc64_defconfig
     HYPERVISOR_ONLY: y
     EXTRA_XEN_CONFIG: |
@@ -705,6 +705,20 @@ debian-12-ppc64le-gcc:
     KBUILD_DEFCONFIG: ppc64_defconfig
     HYPERVISOR_ONLY: y
 
+debian-12-ppc64le-gcc-debug:
+  extends: .gcc-ppc64le-cross-build-debug
+  variables:
+    CONTAINER: debian:12-ppc64le
+    KBUILD_DEFCONFIG: ppc64_defconfig
+    HYPERVISOR_ONLY: y
+
+debian-13-ppc64le-gcc:
+  extends: .gcc-ppc64le-cross-build
+  variables:
+    CONTAINER: debian:13-ppc64le
+    KBUILD_DEFCONFIG: ppc64_defconfig
+    HYPERVISOR_ONLY: y
+
 # RISC-V 64 cross-build
 debian-12-riscv64-gcc:
   extends: .gcc-riscv64-cross-build
diff --git a/automation/gitlab-ci/test.yaml b/automation/gitlab-ci/test.yaml
index 1f0b27b2378a..2f6f3affa637 100644
--- a/automation/gitlab-ci/test.yaml
+++ b/automation/gitlab-ci/test.yaml
@@ -712,4 +712,4 @@ qemu-smoke-ppc64le-powernv9-gcc:
   script:
     - ./automation/scripts/qemu-smoke-ppc64le.sh powernv9 2>&1 | tee ${LOGFILE}
   needs:
-    - debian-12-ppc64le-gcc-debug
+    - debian-13-ppc64le-gcc-debug
-- 
2.39.5


Re: [PATCH 2/5] CI: Update ppc64 to use Debian Trixie
Posted by Anthony PERARD 2 months, 2 weeks ago
On Sat, Aug 09, 2025 at 11:12:03PM +0100, Andrew Cooper wrote:
> diff --git a/automation/gitlab-ci/build.yaml b/automation/gitlab-ci/build.yaml
> index ab5211f77e5e..3fe539dc5683 100644
> --- a/automation/gitlab-ci/build.yaml
> +++ b/automation/gitlab-ci/build.yaml
> @@ -319,10 +319,10 @@ debian-12-x86_64-clang-debug:
>    variables:
>      CONTAINER: debian:12-x86_64
>  
> -debian-12-ppc64le-gcc-debug:
> +debian-13-ppc64le-gcc-debug:
>    extends: .gcc-ppc64le-cross-build-debug
>    variables:
> -    CONTAINER: debian:12-ppc64le
> +    CONTAINER: debian:13-ppc64le
>      KBUILD_DEFCONFIG: ppc64_defconfig
>      HYPERVISOR_ONLY: y
>      EXTRA_XEN_CONFIG: |
> @@ -705,6 +705,20 @@ debian-12-ppc64le-gcc:
>      KBUILD_DEFCONFIG: ppc64_defconfig
>      HYPERVISOR_ONLY: y
>  
> +debian-12-ppc64le-gcc-debug:
> +  extends: .gcc-ppc64le-cross-build-debug
> +  variables:
> +    CONTAINER: debian:12-ppc64le
> +    KBUILD_DEFCONFIG: ppc64_defconfig
> +    HYPERVISOR_ONLY: y
> +

Why did you remove the EXTRA_XEN_CONFIG from this job? Currently, the
job is setup as:

    debian-12-ppc64le-gcc-debug:
      extends: .gcc-ppc64le-cross-build-debug
      variables:
        CONTAINER: debian:12-ppc64le
        KBUILD_DEFCONFIG: ppc64_defconfig
        HYPERVISOR_ONLY: y
        EXTRA_XEN_CONFIG: |
          CONFIG_UBSAN=y
          CONFIG_UBSAN_FATAL=y

Thanks,

-- 
Anthony PERARD
Re: [PATCH 2/5] CI: Update ppc64 to use Debian Trixie
Posted by Andrew Cooper 1 month, 2 weeks ago
On 13/08/2025 4:16 pm, Anthony PERARD wrote:
> On Sat, Aug 09, 2025 at 11:12:03PM +0100, Andrew Cooper wrote:
>> diff --git a/automation/gitlab-ci/build.yaml b/automation/gitlab-ci/build.yaml
>> index ab5211f77e5e..3fe539dc5683 100644
>> --- a/automation/gitlab-ci/build.yaml
>> +++ b/automation/gitlab-ci/build.yaml
>> @@ -319,10 +319,10 @@ debian-12-x86_64-clang-debug:
>>    variables:
>>      CONTAINER: debian:12-x86_64
>>  
>> -debian-12-ppc64le-gcc-debug:
>> +debian-13-ppc64le-gcc-debug:
>>    extends: .gcc-ppc64le-cross-build-debug
>>    variables:
>> -    CONTAINER: debian:12-ppc64le
>> +    CONTAINER: debian:13-ppc64le
>>      KBUILD_DEFCONFIG: ppc64_defconfig
>>      HYPERVISOR_ONLY: y
>>      EXTRA_XEN_CONFIG: |
>> @@ -705,6 +705,20 @@ debian-12-ppc64le-gcc:
>>      KBUILD_DEFCONFIG: ppc64_defconfig
>>      HYPERVISOR_ONLY: y
>>  
>> +debian-12-ppc64le-gcc-debug:
>> +  extends: .gcc-ppc64le-cross-build-debug
>> +  variables:
>> +    CONTAINER: debian:12-ppc64le
>> +    KBUILD_DEFCONFIG: ppc64_defconfig
>> +    HYPERVISOR_ONLY: y
>> +
> Why did you remove the EXTRA_XEN_CONFIG from this job? Currently, the
> job is setup as:
>
>     debian-12-ppc64le-gcc-debug:
>       extends: .gcc-ppc64le-cross-build-debug
>       variables:
>         CONTAINER: debian:12-ppc64le
>         KBUILD_DEFCONFIG: ppc64_defconfig
>         HYPERVISOR_ONLY: y
>         EXTRA_XEN_CONFIG: |
>           CONFIG_UBSAN=y
>           CONFIG_UBSAN_FATAL=y

Because the build run under QEMU changes too.  (See the hunk updating
test.yaml)

We use UBSAN/FATAL for builds where we boot the resulting hypervisor,
and not for the plain build tests.  (In farness, UBSAN/FATAL is newer
than the last time I shuffled test baselines).

If we're going to start having config like this on multiple builds,
we're going to need to find a better way to handle it, because we cannot
be duplicating it between different jobs.

~Andrew