| 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
| ||
|
Message-ID: <f28634bf-b2bb-438f-bfff-98d3b0bcfe15@collabora.com>
Date: Tue, 23 Jul 2024 22:47:22 -0300
From: Helen Koike <helen.koike@...labora.com>
To: Vignesh Raman <vignesh.raman@...labora.com>,
dri-devel@...ts.freedesktop.org
Cc: daniels@...labora.com, airlied@...il.com, daniel@...ll.ch,
guilherme.gallo@...labora.com, sergi.blanch.torne@...labora.com,
deborah.brouwer@...labora.com, robdclark@...il.com,
linux-mediatek@...ts.infradead.org, linux-rockchip@...ts.infradead.org,
linux-amlogic@...ts.infradead.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v7 2/5] drm/ci: mediatek: add tests for mediatek display
driver
On 12/07/2024 06:15, Vignesh Raman wrote:
> For mediatek mt8183, the display driver is mediatek, while the
> gpu driver is panfrost. Currently, in drm-ci for mt8183, only
> the gpu driver is tested. Refactor the existing mediatek jobs
> and add support in drm-ci to test both display and gpu driver
> for mt8183 and update xfails.
>
> Since the correct driver name is passed from the job to test gpu
> and display driver, remove the check to set IGT_FORCE_DRIVER
> based on driver name for mediatek jobs.
>
> Update the MAINTAINERS file to include xfails for panfrost driver.
>
> Signed-off-by: Vignesh Raman <vignesh.raman@...labora.com>
lgtm
Acked-by: Helen Koike <helen.koike@...labora.com>
Thanks
Helen
> ---
>
> v2:
> - Refactor the patch to rename job to indicate display driver testing,
> rename the existing xfail files, and remove IGT_FORCE_DRIVER from the
> script since it's now set by the job.
>
> v3:
> - Add the job name in GPU_VERSION and use it for xfail file names instead
> of using DRIVER_NAME. Also update xfails.
>
> v4:
> - Remove the display suffix in job and rename xfails accordingly.
> Remove the change adding job name in GPU_VERSION.
>
> v5:
> - Add mediatek-display job.
>
> v6:
> - Squash commits for display and gpu driver testing. Reword the commit message.
>
>
> v7:
> - Rebase with recent drm-ci fixes and retest with latest IGT.
>
> ---
> MAINTAINERS | 1 +
> drivers/gpu/drm/ci/gitlab-ci.yml | 2 +-
> drivers/gpu/drm/ci/igt_runner.sh | 7 ----
> drivers/gpu/drm/ci/test.yml | 38 +++++++++++++++----
> .../drm/ci/xfails/mediatek-mt8183-fails.txt | 28 +++++++++++---
> .../drm/ci/xfails/mediatek-mt8183-flakes.txt | 7 ++++
> .../drm/ci/xfails/mediatek-mt8183-skips.txt | 4 +-
> .../drm/ci/xfails/panfrost-mt8183-fails.txt | 11 ++++++
> .../drm/ci/xfails/panfrost-mt8183-skips.txt | 18 +++++++++
> 9 files changed, 92 insertions(+), 24 deletions(-)
> create mode 100644 drivers/gpu/drm/ci/xfails/mediatek-mt8183-flakes.txt
> create mode 100644 drivers/gpu/drm/ci/xfails/panfrost-mt8183-fails.txt
> create mode 100644 drivers/gpu/drm/ci/xfails/panfrost-mt8183-skips.txt
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index cb8355f20b36..f3783f2269ff 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -1741,6 +1741,7 @@ L: dri-devel@...ts.freedesktop.org
> S: Supported
> T: git https://gitlab.freedesktop.org/drm/misc/kernel.git
> F: Documentation/gpu/panfrost.rst
> +F: drivers/gpu/drm/ci/xfails/panfrost*
> F: drivers/gpu/drm/panfrost/
> F: include/uapi/drm/panfrost_drm.h
>
> diff --git a/drivers/gpu/drm/ci/gitlab-ci.yml b/drivers/gpu/drm/ci/gitlab-ci.yml
> index b09976c3d2c2..9d800f629f08 100644
> --- a/drivers/gpu/drm/ci/gitlab-ci.yml
> +++ b/drivers/gpu/drm/ci/gitlab-ci.yml
> @@ -121,8 +121,8 @@ stages:
> - mediatek
> - meson
> - msm
> + - panfrost
> - rockchip
> - - virtio-gpu
> - software-driver
>
> # YAML anchors for rule conditions
> diff --git a/drivers/gpu/drm/ci/igt_runner.sh b/drivers/gpu/drm/ci/igt_runner.sh
> index 37c724dba8cb..6193865f23a3 100755
> --- a/drivers/gpu/drm/ci/igt_runner.sh
> +++ b/drivers/gpu/drm/ci/igt_runner.sh
> @@ -23,13 +23,6 @@ case "$DRIVER_NAME" in
> rockchip|meson)
> export IGT_FORCE_DRIVER="panfrost"
> ;;
> - mediatek)
> - if [ "$GPU_VERSION" = "mt8173" ]; then
> - export IGT_FORCE_DRIVER=${DRIVER_NAME}
> - elif [ "$GPU_VERSION" = "mt8183" ]; then
> - export IGT_FORCE_DRIVER="panfrost"
> - fi
> - ;;
> amdgpu|vkms)
> # Cannot use HWCI_KERNEL_MODULES as at that point we don't have the module in /lib
> mv /install/modules/lib/modules/* /lib/modules/. || true
> diff --git a/drivers/gpu/drm/ci/test.yml b/drivers/gpu/drm/ci/test.yml
> index ee908b66aad2..219eca5ef24c 100644
> --- a/drivers/gpu/drm/ci/test.yml
> +++ b/drivers/gpu/drm/ci/test.yml
> @@ -280,38 +280,62 @@ amdgpu:stoney:
> GPU_VERSION: stoney
> RUNNER_TAG: mesa-ci-x86-64-lava-hp-11A-G6-EE-grunt
>
> -.mediatek:
> +.mediatek-device:
> extends:
> - .lava-igt:arm64
> stage: mediatek
> variables:
> - DRIVER_NAME: mediatek
> DTB: ${DEVICE_TYPE}
> BOOT_METHOD: depthcharge
> KERNEL_IMAGE_TYPE: ""
>
> -mediatek:mt8173:
> +.mediatek-display:
> + stage: mediatek
> + variables:
> + DRIVER_NAME: mediatek
> +
> +.panfrost-gpu:
> + stage: panfrost
> + variables:
> + DRIVER_NAME: panfrost
> +
> +.mt8173:
> extends:
> - - .mediatek
> + - .mediatek-device
> parallel: 4
> variables:
> DEVICE_TYPE: mt8173-elm-hana
> GPU_VERSION: mt8173
> RUNNER_TAG: mesa-ci-x86-64-lava-mt8173-elm-hana
>
> -mediatek:mt8183:
> +.mt8183:
> extends:
> - - .mediatek
> + - .mediatek-device
> parallel: 3
> variables:
> DEVICE_TYPE: mt8183-kukui-jacuzzi-juniper-sku16
> GPU_VERSION: mt8183
> RUNNER_TAG: mesa-ci-x86-64-lava-mt8183-kukui-jacuzzi-juniper-sku16
>
> +mediatek:mt8173:
> + extends:
> + - .mt8173
> + - .mediatek-display
> +
> +mediatek:mt8183:
> + extends:
> + - .mt8183
> + - .mediatek-display
> +
> +panfrost:mt8183:
> + extends:
> + - .mt8183
> + - .panfrost-gpu
> +
> # drm-mtk doesn't even probe yet in mainline for mt8192
> .mediatek:mt8192:
> extends:
> - - .mediatek
> + - .mediatek-device
> parallel: 3
> variables:
> DEVICE_TYPE: mt8192-asurada-spherion-r0
> diff --git a/drivers/gpu/drm/ci/xfails/mediatek-mt8183-fails.txt b/drivers/gpu/drm/ci/xfails/mediatek-mt8183-fails.txt
> index 5b7d623f404b..cf3a747f7cec 100644
> --- a/drivers/gpu/drm/ci/xfails/mediatek-mt8183-fails.txt
> +++ b/drivers/gpu/drm/ci/xfails/mediatek-mt8183-fails.txt
> @@ -1,8 +1,24 @@
> -dumb_buffer@...ate-clear,Fail
> -dumb_buffer@...ate-valid-dumb,Fail
> +core_setmaster@...ter-drop-set-shared-fd,Fail
> +device_reset@...d-reset-bound,Fail
> +device_reset@...et-bound,Fail
> +device_reset@...ind-cold-reset-rebind,Fail
> +device_reset@...ind-reset-rebind,Fail
> +dumb_buffer@...ate-clear,Crash
> dumb_buffer@...alid-bpp,Fail
> -dumb_buffer@...-invalid-size,Fail
> -dumb_buffer@...-uaf,Fail
> -dumb_buffer@...-valid,Fail
> -panfrost/panfrost_prime@...-prime-import,Fail
> +fbdev@eof,Fail
> +fbdev@pan,Fail
> +fbdev@...d,Fail
> +fbdev@...ligned-read,Fail
> +kms_bw@...nected-linear-tiling-1-displays-1920x1080p,Fail
> +kms_bw@...nected-linear-tiling-1-displays-2160x1440p,Fail
> +kms_bw@...nected-linear-tiling-1-displays-2560x1440p,Fail
> +kms_bw@...ear-tiling-1-displays-1920x1080p,Fail
> +kms_bw@...ear-tiling-1-displays-2560x1440p,Fail
> +kms_bw@...ear-tiling-1-displays-3840x2160p,Fail
> +kms_color@...alid-gamma-lut-sizes,Fail
> +kms_flip@...p-vs-panning-vs-hang,Fail
> +kms_flip@...p-vs-suspend,Fail
> +kms_lease@...se-uevent,Fail
> +kms_properties@...ne-properties-atomic,Fail
> +kms_rmfb@...se-fd,Fail
> tools_test@...ls_test,Fail
> diff --git a/drivers/gpu/drm/ci/xfails/mediatek-mt8183-flakes.txt b/drivers/gpu/drm/ci/xfails/mediatek-mt8183-flakes.txt
> new file mode 100644
> index 000000000000..2d1f24e56083
> --- /dev/null
> +++ b/drivers/gpu/drm/ci/xfails/mediatek-mt8183-flakes.txt
> @@ -0,0 +1,7 @@
> +# Board Name: mt8183-kukui-jacuzzi-juniper-sku16
> +# Bug Report: https://lore.kernel.org/linux-mediatek/0b2a1899-15dd-42fa-8f63-ea0ca28dbb17@collabora.com/T/#u
> +# Failure Rate: 100
> +# IGT Version: 1.28-gf13702b8e
> +# Linux Version: 6.10.0-rc5
> +kms_bw@...ear-tiling-1-displays-2560x1440p
> +kms_cursor_legacy@...sor-vs-flip-atomic-transitions
> diff --git a/drivers/gpu/drm/ci/xfails/mediatek-mt8183-skips.txt b/drivers/gpu/drm/ci/xfails/mediatek-mt8183-skips.txt
> index 03eefa518afa..562662eff802 100644
> --- a/drivers/gpu/drm/ci/xfails/mediatek-mt8183-skips.txt
> +++ b/drivers/gpu/drm/ci/xfails/mediatek-mt8183-skips.txt
> @@ -2,6 +2,7 @@
> ^amdgpu.*
> ^msm.*
> nouveau_.*
> +^panfrost.*
> ^v3d.*
> ^vc4.*
> ^vmwgfx*
> @@ -10,9 +11,6 @@ nouveau_.*
> gem_.*
> i915_.*
>
> -# Panfrost is not a KMS driver, so skip the KMS tests
> -kms_.*
> -
> # Currently fails and causes coverage loss for other tests
> # since core_getversion also fails.
> core_hotunplug.*
> diff --git a/drivers/gpu/drm/ci/xfails/panfrost-mt8183-fails.txt b/drivers/gpu/drm/ci/xfails/panfrost-mt8183-fails.txt
> new file mode 100644
> index 000000000000..fd6155dd99ef
> --- /dev/null
> +++ b/drivers/gpu/drm/ci/xfails/panfrost-mt8183-fails.txt
> @@ -0,0 +1,11 @@
> +dumb_buffer@...ate-clear,Fail
> +dumb_buffer@...ate-valid-dumb,Fail
> +dumb_buffer@...alid-bpp,Fail
> +dumb_buffer@...-invalid-size,Fail
> +dumb_buffer@...-uaf,Fail
> +dumb_buffer@...-valid,Fail
> +fbdev@eof,Fail
> +fbdev@...d,Fail
> +fbdev@...ligned-read,Fail
> +panfrost/panfrost_prime@...-prime-import,Fail
> +tools_test@...ls_test,Fail
> diff --git a/drivers/gpu/drm/ci/xfails/panfrost-mt8183-skips.txt b/drivers/gpu/drm/ci/xfails/panfrost-mt8183-skips.txt
> new file mode 100644
> index 000000000000..03eefa518afa
> --- /dev/null
> +++ b/drivers/gpu/drm/ci/xfails/panfrost-mt8183-skips.txt
> @@ -0,0 +1,18 @@
> +# Skip driver specific tests
> +^amdgpu.*
> +^msm.*
> +nouveau_.*
> +^v3d.*
> +^vc4.*
> +^vmwgfx*
> +
> +# Skip intel specific tests
> +gem_.*
> +i915_.*
> +
> +# Panfrost is not a KMS driver, so skip the KMS tests
> +kms_.*
> +
> +# Currently fails and causes coverage loss for other tests
> +# since core_getversion also fails.
> +core_hotunplug.*
Powered by blists - more mailing lists