lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <ad62555b-92ab-45d6-a334-8de8cb43b8da@igalia.com>
Date: Tue, 6 May 2025 15:22:51 -0300
From: Helen Koike <koike@...lia.com>
To: Vignesh Raman <vignesh.raman@...labora.com>,
 dri-devel@...ts.freedesktop.org
Cc: daniels@...labora.com, daniel@...ishbar.org, helen.fornazier@...il.com,
 airlied@...il.com, simona.vetter@...ll.ch, robdclark@...il.com,
 guilherme.gallo@...labora.com, sergi.blanch.torne@...labora.com,
 valentine.burley@...labora.com, lumag@...nel.org,
 dmitry.baryshkov@....qualcomm.com, quic_abhinavk@...cinc.com,
 mripard@...nel.org, maarten.lankhorst@...ux.intel.com, tzimmermann@...e.de,
 linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2 2/2] drm/ci: uprev mesa and ci-templates



On 05/05/2025 11:11, Vignesh Raman wrote:
> The current s3cp stopped working after the migration. Update to the
> latest mesa and ci-templates to get s3cp working again and adapt to
> recent changes in mesa-ci.
> 
> Signed-off-by: Vignesh Raman <vignesh.raman@...labora.com>

Acked-by: Helen Koike <helen.fornazier@...il.com>

> ---
> 
> v2:
>    - Uprev mesa and ci-templates to the latest version to get s3cp
>      working again.
> 
> ---
>   drivers/gpu/drm/ci/build.yml      | 10 ++++++--
>   drivers/gpu/drm/ci/container.yml  | 30 ++++++----------------
>   drivers/gpu/drm/ci/gitlab-ci.yml  | 41 +++++++++++++++++++++----------
>   drivers/gpu/drm/ci/igt_runner.sh  |  5 +++-
>   drivers/gpu/drm/ci/image-tags.yml | 20 ++++++---------
>   drivers/gpu/drm/ci/lava-submit.sh |  4 ++-
>   drivers/gpu/drm/ci/test.yml       | 31 +++++++++++------------
>   7 files changed, 72 insertions(+), 69 deletions(-)
> 
> diff --git a/drivers/gpu/drm/ci/build.yml b/drivers/gpu/drm/ci/build.yml
> index 8eb56ebcf4aa..af27ff5de369 100644
> --- a/drivers/gpu/drm/ci/build.yml
> +++ b/drivers/gpu/drm/ci/build.yml
> @@ -13,7 +13,7 @@
>       - .build
>       - .use-debian/arm64_build
>     tags:
> -    - aarch64
> +    - $FDO_RUNNER_JOB_PRIORITY_TAG_AARCH64
>     variables:
>       DEFCONFIG: "arch/arm/configs/multi_v7_defconfig"
>       KERNEL_IMAGE_NAME: "zImage"
> @@ -24,7 +24,7 @@
>       - .build
>       - .use-debian/arm64_build
>     tags:
> -    - aarch64
> +    - $FDO_RUNNER_JOB_PRIORITY_TAG_AARCH64
>     variables:
>       DEFCONFIG: "arch/arm64/configs/defconfig"
>       KERNEL_IMAGE_NAME: "Image"
> @@ -44,16 +44,22 @@
>   
>   igt:arm32:
>     extends: .build:arm32
> +  variables:
> +    GIT_DEPTH: 10
>     script:
>       - FDO_CI_CONCURRENT=${FDO_CI_CONCURRENT} bash drivers/gpu/drm/ci/build-igt.sh
>   
>   igt:arm64:
>     extends: .build:arm64
> +  variables:
> +    GIT_DEPTH: 10
>     script:
>       - FDO_CI_CONCURRENT=${FDO_CI_CONCURRENT} bash drivers/gpu/drm/ci/build-igt.sh
>   
>   igt:x86_64:
>     extends: .build:x86_64
> +  variables:
> +    GIT_DEPTH: 10
>     script:
>       - FDO_CI_CONCURRENT=${FDO_CI_CONCURRENT} bash drivers/gpu/drm/ci/build-igt.sh
>   
> diff --git a/drivers/gpu/drm/ci/container.yml b/drivers/gpu/drm/ci/container.yml
> index 56c95c2f91ae..5f90508578a3 100644
> --- a/drivers/gpu/drm/ci/container.yml
> +++ b/drivers/gpu/drm/ci/container.yml
> @@ -20,31 +20,15 @@ debian/arm64_build:
>       EXTRA_LOCAL_PACKAGES: "jq libasound2 libcairo2 libdw1 libglib2.0-0 libjson-c5"
>   
>   # Disable container jobs that we won't use
> -alpine/x86_64_build:
> -  rules:
> -    - when: never
> -
> -debian/arm32_test-base:
> -  rules:
> -    - when: never
> -
> -debian/arm32_test-gl:
> -  rules:
> -    - when: never
> -
> -debian/arm32_test-vk:
> -  rules:
> -    - when: never
> -
> -debian/arm64_test-gl:
> +debian/arm64_test-vk:
>     rules:
>       - when: never
>   
> -debian/arm64_test-vk:
> +debian/baremetal_arm32_test-gl:
>     rules:
>       - when: never
>   
> -debian/baremetal_arm32_test:
> +debian/baremetal_arm64_test-vk:
>     rules:
>       - when: never
>   
> @@ -64,19 +48,19 @@ debian/x86_64_test-android:
>     rules:
>       - when: never
>   
> -debian/x86_64_test-vk:
> +debian/x86_64_test-video:
>     rules:
>       - when: never
>   
> -fedora/x86_64_build:
> +debian/x86_64_test-vk:
>     rules:
>       - when: never
>   
> -debian/android_build:
> +fedora/x86_64_build:
>     rules:
>       - when: never
>   
> -.debian/x86_64_test-android:
> +debian/android_build:
>     rules:
>       - when: never
>   
> diff --git a/drivers/gpu/drm/ci/gitlab-ci.yml b/drivers/gpu/drm/ci/gitlab-ci.yml
> index 193f67121328..d9e1042fc140 100644
> --- a/drivers/gpu/drm/ci/gitlab-ci.yml
> +++ b/drivers/gpu/drm/ci/gitlab-ci.yml
> @@ -1,6 +1,6 @@
>   variables:
>     DRM_CI_PROJECT_PATH: &drm-ci-project-path mesa/mesa
> -  DRM_CI_COMMIT_SHA: &drm-ci-commit-sha 82ab58f6c6f94fa80ca7e1615146f08356e3ba69
> +  DRM_CI_COMMIT_SHA: &drm-ci-commit-sha 02337aec715c25dae7ff2479d986f831c77fe536
>   
>     UPSTREAM_REPO: https://gitlab.freedesktop.org/drm/kernel.git
>     TARGET_BRANCH: drm-next
> @@ -11,7 +11,7 @@ variables:
>     DEQP_RUNNER_GIT_TAG: v0.20.0
>   
>     FDO_UPSTREAM_REPO: helen.fornazier/linux   # The repo where the git-archive daily runs
> -  MESA_TEMPLATES_COMMIT: &ci-templates-commit d5aa3941aa03c2f716595116354fb81eb8012acb
> +  MESA_TEMPLATES_COMMIT: &ci-templates-commit c6aeb16f86e32525fa630fb99c66c4f3e62fc3cb
>     DRM_CI_PROJECT_URL: https://gitlab.freedesktop.org/${DRM_CI_PROJECT_PATH}
>     CI_PRE_CLONE_SCRIPT: |-
>             set -o xtrace
> @@ -36,7 +36,11 @@ variables:
>     JOB_ARTIFACTS_BASE: ${PIPELINE_ARTIFACTS_BASE}/${CI_JOB_ID}
>     KERNEL_IMAGE_BASE: https://${S3_HOST}/${S3_KERNEL_BUCKET}/${KERNEL_REPO}/${KERNEL_TAG}
>     LAVA_TAGS: subset-1-gfx
> -  LAVA_JOB_PRIORITY: 30
> +  # Default priority for non-merge pipelines
> +  FDO_RUNNER_JOB_PRIORITY_TAG_X86_64: ""  # Empty tags are ignored by gitlab
> +  FDO_RUNNER_JOB_PRIORITY_TAG_X86_64_KVM: kvm
> +  FDO_RUNNER_JOB_PRIORITY_TAG_AARCH64: aarch64
> +  JOB_PRIORITY: 30
>     ARTIFACTS_BASE_URL: https://${CI_PROJECT_ROOT_NAMESPACE}.${CI_PAGES_DOMAIN}/-/${CI_PROJECT_NAME}/-/jobs/${CI_JOB_ID}/artifacts
>     # Python scripts for structured logger
>     PYTHONPATH: "$PYTHONPATH:$CI_PROJECT_DIR/install"
> @@ -70,16 +74,13 @@ default:
>   
>   
>   include:
> -  - project: 'freedesktop/ci-templates'
> -    ref: 16bc29078de5e0a067ff84a1a199a3760d3b3811
> -    file:
> -      - '/templates/ci-fairy.yml'
>     - project: 'freedesktop/ci-templates'
>       ref: *ci-templates-commit
>       file:
>         - '/templates/alpine.yml'
>         - '/templates/debian.yml'
>         - '/templates/fedora.yml'
> +      - '/templates/ci-fairy.yml'
>     - project: *drm-ci-project-path
>       ref: *drm-ci-commit-sha
>       file:
> @@ -103,6 +104,7 @@ include:
>         - '/src/microsoft/ci/gitlab-ci-inc.yml'
>         - '/src/nouveau/ci/gitlab-ci-inc.yml'
>         - '/src/virtio/ci/gitlab-ci-inc.yml'
> +      - 'docs/gitlab-ci.yml'
>     - drivers/gpu/drm/ci/image-tags.yml
>     - drivers/gpu/drm/ci/container.yml
>     - drivers/gpu/drm/ci/static-checks.yml
> @@ -114,6 +116,7 @@ include:
>   stages:
>     - sanity
>     - container
> +  - deploy
>     - git-archive
>     - build-for-tests
>     - build-only
> @@ -236,15 +239,14 @@ python-artifacts:
>   
>   
>   # Git archive
> -make git archive:
> +make-git-archive:
>     extends:
>       - .fdo.ci-fairy
>     stage: git-archive
>     rules:
>       - !reference [.scheduled_pipeline-rules, rules]
> -  # ensure we are running on packet
>     tags:
> -    - packet.net
> +    - $FDO_RUNNER_JOB_PRIORITY_TAG_X86_64
>     script:
>       # Remove drm-ci files we just added
>       - rm -rf .gitlab-ci.*
> @@ -264,6 +266,8 @@ sanity:
>     extends:
>       - .fdo.ci-fairy
>     stage: sanity
> +  tags:
> +    - $FDO_RUNNER_JOB_PRIORITY_TAG_X86_64
>     rules:
>       - if: *is-pre-merge
>         when: on_success
> @@ -282,7 +286,6 @@ sanity:
>           DEBIAN_BUILD_TAG
>           DEBIAN_PYUTILS_TAG
>           DEBIAN_TEST_GL_TAG
> -        KERNEL_ROOTFS_TAG
>           KERNEL_TAG
>           PKG_REPO_REV
>         )
> @@ -298,14 +301,14 @@ sanity:
>       when: on_failure
>       reports:
>         junit: check-*.xml
> -  tags:
> -    - placeholder-job
>   
>   
>   mr-label-maker-test:
>     extends:
>       - .fdo.ci-fairy
>     stage: sanity
> +  tags:
> +    - $FDO_RUNNER_JOB_PRIORITY_TAG_X86_64
>     rules:
>       - !reference [.mr-label-maker-rules, rules]
>     variables:
> @@ -328,3 +331,15 @@ mr-label-maker-test:
>         optional: true
>       - job: toml-lint
>         optional: true
> +
> +deploy-docs:
> +   rules:
> +    - when: never
> +
> +linkcheck-docs:
> +   rules:
> +    - when: never
> +
> +test-docs:
> +   rules:
> +    - when: never
> diff --git a/drivers/gpu/drm/ci/igt_runner.sh b/drivers/gpu/drm/ci/igt_runner.sh
> index 2a0599f12c58..602decf09ea0 100755
> --- a/drivers/gpu/drm/ci/igt_runner.sh
> +++ b/drivers/gpu/drm/ci/igt_runner.sh
> @@ -19,6 +19,7 @@ set +e
>   cat /sys/kernel/debug/dri/*/state
>   set -e
>   
> +mkdir -p /lib/modules
>   case "$DRIVER_NAME" in
>       amdgpu|vkms)
>           # Cannot use HWCI_KERNEL_MODULES as at that point we don't have the module in /lib
> @@ -47,7 +48,9 @@ else
>       ARCH="x86_64"
>   fi
>   
> -curl -L --retry 4 -f --retry-all-errors --retry-delay 60 -s $PIPELINE_ARTIFACTS_BASE/$ARCH/igt.tar.gz | tar --zstd -v -x -C /
> +curl -L --retry 4 -f --retry-all-errors --retry-delay 60 -s https://${PIPELINE_ARTIFACTS_BASE}/${ARCH}/igt.tar.gz -o igt.tar.gz
> +tar --zstd -xvf igt.tar.gz -C /
> +rm -f igt.tar.gz
>   
>   TESTLIST="/igt/libexec/igt-gpu-tools/ci-testlist.txt"
>   
> diff --git a/drivers/gpu/drm/ci/image-tags.yml b/drivers/gpu/drm/ci/image-tags.yml
> index c04ba0e69935..7acc2e2a8eaa 100644
> --- a/drivers/gpu/drm/ci/image-tags.yml
> +++ b/drivers/gpu/drm/ci/image-tags.yml
> @@ -1,24 +1,18 @@
>   variables:
> -   CONTAINER_TAG: "20250307-mesa-uprev"
> -   DEBIAN_X86_64_BUILD_BASE_IMAGE: "debian/x86_64_build-base"
> +   CONTAINER_TAG: "20250502-mesa-uprev"
>      DEBIAN_BASE_TAG: "${CONTAINER_TAG}"
> -
> -   DEBIAN_X86_64_BUILD_IMAGE_PATH: "debian/x86_64_build"
>      DEBIAN_BUILD_TAG: "${CONTAINER_TAG}"
>   
> -   KERNEL_ROOTFS_TAG: "${CONTAINER_TAG}"
> +   DEBIAN_TEST_GL_TAG: "${CONTAINER_TAG}"
>      # default kernel for rootfs before injecting the current kernel tree
> -   KERNEL_TAG: "v6.13-rc4-mesa-5e77"
> +   KERNEL_TAG: "v6.14-mesa-0bdd"
>      KERNEL_REPO: "gfx-ci/linux"
> -   PKG_REPO_REV: "bca9635d"
> -
> -   DEBIAN_X86_64_TEST_BASE_IMAGE: "debian/x86_64_test-base"
> -   DEBIAN_X86_64_TEST_IMAGE_GL_PATH: "debian/x86_64_test-gl"
> -   DEBIAN_TEST_GL_TAG: "${CONTAINER_TAG}"
> +   PKG_REPO_REV: "95bf62c"
>   
> -   DEBIAN_PYUTILS_IMAGE: "debian/x86_64_pyutils"
>      DEBIAN_PYUTILS_TAG: "${CONTAINER_TAG}"
>   
> +   ALPINE_X86_64_BUILD_TAG: "${CONTAINER_TAG}"
>      ALPINE_X86_64_LAVA_SSH_TAG: "${CONTAINER_TAG}"
>   
> -   CONDITIONAL_BUILD_ANGLE_TAG: fec96cc945650c5fe9f7188cabe80d8a
> +   CONDITIONAL_BUILD_ANGLE_TAG: 384145a4023315dae658259bee07c43a
> +   CONDITIONAL_BUILD_PIGLIT_TAG: a19e424b8a3f020dbf1b9dd29f220a4f
> diff --git a/drivers/gpu/drm/ci/lava-submit.sh b/drivers/gpu/drm/ci/lava-submit.sh
> index f22720359b33..a295102c3468 100755
> --- a/drivers/gpu/drm/ci/lava-submit.sh
> +++ b/drivers/gpu/drm/ci/lava-submit.sh
> @@ -41,7 +41,6 @@ section_start prepare_rootfs "Preparing root filesystem"
>   
>   set -ex
>   
> -section_switch rootfs "Assembling root filesystem"
>   ROOTFS_URL="$(get_path_to_artifact lava-rootfs.tar.zst)"
>   [ $? != 1 ] || exit 1
>   
> @@ -64,6 +63,9 @@ section_switch lava_submit "Submitting job for scheduling"
>   
>   touch results/lava.log
>   tail -f results/lava.log &
> +# Ensure that we are printing the commands that are being executed,
> +# making it easier to debug the job in case it fails.
> +set -x
>   PYTHONPATH=artifacts/ artifacts/lava/lava_job_submitter.py \
>   	--farm "${FARM}" \
>   	--device-type "${DEVICE_TYPE}" \
> diff --git a/drivers/gpu/drm/ci/test.yml b/drivers/gpu/drm/ci/test.yml
> index 84a25f0e783b..795a2631833b 100644
> --- a/drivers/gpu/drm/ci/test.yml
> +++ b/drivers/gpu/drm/ci/test.yml
> @@ -24,7 +24,7 @@
>   
>   .lava-igt:arm32:
>     extends:
> -    - .lava-test:arm32
> +    - .lava-arm32-test-gl
>     variables:
>       HWCI_TEST_SCRIPT: "/install/igt_runner.sh"
>       DEBIAN_ARCH: "armhf"
> @@ -33,15 +33,14 @@
>       - testing:arm32
>     needs:
>       - alpine/x86_64_lava_ssh_client
> -    - kernel+rootfs_arm32
> -    - debian/x86_64_build
> +    - debian/arm32_test-gl
>       - python-artifacts
>       - testing:arm32
>       - igt:arm32
>   
>   .lava-igt:arm64:
>     extends:
> -    - .lava-test:arm64
> +    - .lava-arm64-test-gl
>     variables:
>       HWCI_TEST_SCRIPT: "/install/igt_runner.sh"
>       DEBIAN_ARCH: "arm64"
> @@ -50,15 +49,14 @@
>       - testing:arm64
>     needs:
>       - alpine/x86_64_lava_ssh_client
> -    - kernel+rootfs_arm64
> -    - debian/x86_64_build
> +    - debian/arm64_test-gl
>       - python-artifacts
>       - testing:arm64
>       - igt:arm64
>   
>   .lava-igt:x86_64:
>     extends:
> -    - .lava-test:x86_64
> +    - .lava-x86_64-test-gl
>     variables:
>       HWCI_TEST_SCRIPT: "/install/igt_runner.sh"
>       DEBIAN_ARCH: "amd64"
> @@ -67,16 +65,15 @@
>       - testing:x86_64
>     needs:
>       - alpine/x86_64_lava_ssh_client
> -    - kernel+rootfs_x86_64
> -    - debian/x86_64_build
> +    - debian/x86_64_test-gl
>       - python-artifacts
>       - testing:x86_64
>       - igt:x86_64
>   
>   .baremetal-igt-arm64:
>     extends:
> -    - .baremetal-test-arm64
> -    - .use-debian/baremetal_arm64_test
> +    - .baremetal-test-arm64-gl
> +    - .use-debian/baremetal_arm64_test-gl
>       - .allow_failure_lockdep
>     timeout: "1h30m"
>     rules:
> @@ -91,7 +88,7 @@
>       BM_CMDLINE: "ip=dhcp console=ttyMSM0,115200n8 $BM_KERNEL_EXTRA_ARGS root=/dev/nfs rw nfsrootdebug nfsroot=,tcp,nfsvers=4.2 init=/init $BM_KERNELARGS"
>       FARM: google
>     needs:
> -    - debian/baremetal_arm64_test
> +    - debian/baremetal_arm64_test-gl
>       - job: testing:arm64
>         artifacts: false
>       - igt:arm64
> @@ -101,19 +98,21 @@
>   .software-driver:
>     stage: software-driver
>     extends:
> +    - .test-gl
>       - .allow_failure_lockdep
>     timeout: "1h30m"
>     rules:
>       - !reference [.scheduled_pipeline-rules, rules]
>       - when: on_success
> -  extends:
> -    - .test-gl
>     tags:
> -    - kvm
> +    - $FDO_RUNNER_JOB_PRIORITY_TAG_X86_64_KVM
> +  before_script:
> +    - !reference [default, before_script]
> +    - rm -rf install
> +    - tar -xf artifacts/install.tar
>     script:
>       - ln -sf $CI_PROJECT_DIR/install /install
>       - mv install/bzImage /kernel/bzImage
> -    - mkdir -p /lib/modules
>       - install/crosvm-runner.sh install/igt_runner.sh
>     needs:
>       - debian/x86_64_test-gl


Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ