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

Reply via email to