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] [thread-next>] [day] [month] [year] [list]
Message-ID: <9a968260-e7a2-45fb-9a3b-1a8e15d49018@collabora.com>
Date: Mon, 15 Jan 2024 12:24:52 -0300
From: Helen Koike <helen.koike@...labora.com>
To: Vignesh Raman <vignesh.raman@...labora.com>,
 linux-arm-msm@...r.kernel.org, dri-devel@...ts.freedesktop.org,
 freedreno@...ts.freedesktop.org
Cc: daniels@...labora.com, sean@...rly.run, guilherme.gallo@...labora.com,
 sergi.blanch.torne@...labora.com, robdclark@...il.com,
 quic_abhinavk@...cinc.com, david.heidelberg@...labora.com,
 dmitry.baryshkov@...aro.org, marijn.suijten@...ainline.org,
 linux-kernel@...r.kernel.org
Subject: Re: [PATCH v1] drm/ci: add sc7180-trogdor-kingoftown



On 04/01/2024 05:14, Vignesh Raman wrote:
> Add job that executes the IGT test suite for sc7180-trogdor-kingoftown.
> This commit also updates xfails for sc7180-trogdor-lazor-limozeen.
> 
> Signed-off-by: Vignesh Raman <vignesh.raman@...labora.com>

lgtm

Acked-by: Helen Koike <helen.koike@...labora.com>

Thanks
Helen

> ---
> 
> See pipeline: https://gitlab.freedesktop.org/vigneshraman/linux/-/pipelines/1069934
> 
> ---
>   drivers/gpu/drm/ci/build.sh                   |  1 +
>   drivers/gpu/drm/ci/test.yml                   | 25 ++++++++++++----
>   .../gpu/drm/ci/xfails/msm-sc7180-fails.txt    | 30 -------------------
>   .../gpu/drm/ci/xfails/msm-sc7180-flakes.txt   | 17 -----------
>   .../gpu/drm/ci/xfails/msm-sc7180-skips.txt    |  7 -----
>   .../msm-sc7180-trogdor-kingoftown-fails.txt   | 17 +++++++++++
>   ...sm-sc7180-trogdor-lazor-limozeen-fails.txt | 17 +++++++++++
>   7 files changed, 55 insertions(+), 59 deletions(-)
>   delete mode 100644 drivers/gpu/drm/ci/xfails/msm-sc7180-fails.txt
>   delete mode 100644 drivers/gpu/drm/ci/xfails/msm-sc7180-flakes.txt
>   delete mode 100644 drivers/gpu/drm/ci/xfails/msm-sc7180-skips.txt
>   create mode 100644 drivers/gpu/drm/ci/xfails/msm-sc7180-trogdor-kingoftown-fails.txt
>   create mode 100644 drivers/gpu/drm/ci/xfails/msm-sc7180-trogdor-lazor-limozeen-fails.txt
> 
> diff --git a/drivers/gpu/drm/ci/build.sh b/drivers/gpu/drm/ci/build.sh
> index 97ff43759b91..331a61e0d25a 100644
> --- a/drivers/gpu/drm/ci/build.sh
> +++ b/drivers/gpu/drm/ci/build.sh
> @@ -26,6 +26,7 @@ if [[ "$KERNEL_ARCH" = "arm64" ]]; then
>       DEVICE_TREES+=" arch/arm64/boot/dts/mediatek/mt8183-kukui-jacuzzi-juniper-sku16.dtb"
>       DEVICE_TREES+=" arch/arm64/boot/dts/mediatek/mt8192-asurada-spherion-r0.dtb"
>       DEVICE_TREES+=" arch/arm64/boot/dts/qcom/sc7180-trogdor-lazor-limozeen-nots-r5.dtb"
> +    DEVICE_TREES+=" arch/arm64/boot/dts/qcom/sc7180-trogdor-kingoftown.dtb"
>   elif [[ "$KERNEL_ARCH" = "arm" ]]; then
>       GCC_ARCH="arm-linux-gnueabihf"
>       DEBIAN_ARCH="armhf"
> diff --git a/drivers/gpu/drm/ci/test.yml b/drivers/gpu/drm/ci/test.yml
> index a079f3632a95..0c7920c0254d 100644
> --- a/drivers/gpu/drm/ci/test.yml
> +++ b/drivers/gpu/drm/ci/test.yml
> @@ -82,20 +82,35 @@
>     tags:
>       - $RUNNER_TAG
>   
> -msm:sc7180:
> +.msm-sc7180:
>     extends:
>       - .lava-igt:arm64
>     stage: msm
> -  parallel: 4
>     variables:
>       DRIVER_NAME: msm
> -    DEVICE_TYPE: sc7180-trogdor-lazor-limozeen
> -    DTB: sc7180-trogdor-lazor-limozeen-nots-r5
>       BOOT_METHOD: depthcharge
>       KERNEL_IMAGE_TYPE: ""
> -    GPU_VERSION: sc7180
> +
> +msm:sc7180-trogdor-lazor-limozeen:
> +  extends:
> +    - .msm-sc7180
> +  parallel: 4
> +  variables:
> +    DEVICE_TYPE: sc7180-trogdor-lazor-limozeen
> +    DTB: sc7180-trogdor-lazor-limozeen-nots-r5
> +    GPU_VERSION: ${DEVICE_TYPE}
>       RUNNER_TAG: mesa-ci-x86-64-lava-sc7180-trogdor-lazor-limozeen
>   
> +msm:sc7180-trogdor-kingoftown:
> +  extends:
> +    - .msm-sc7180
> +  parallel: 6
> +  variables:
> +    DEVICE_TYPE: sc7180-trogdor-kingoftown
> +    DTB: sc7180-trogdor-kingoftown
> +    GPU_VERSION: ${DEVICE_TYPE}
> +    RUNNER_TAG: mesa-ci-x86-64-lava-sc7180-trogdor-kingoftown
> +
>   msm:apq8016:
>     extends:
>       - .baremetal-igt-arm64
> diff --git a/drivers/gpu/drm/ci/xfails/msm-sc7180-fails.txt b/drivers/gpu/drm/ci/xfails/msm-sc7180-fails.txt
> deleted file mode 100644
> index f71166a57731..000000000000
> --- a/drivers/gpu/drm/ci/xfails/msm-sc7180-fails.txt
> +++ /dev/null
> @@ -1,30 +0,0 @@
> -kms_color@...-0-25,Fail
> -kms_color@...-0-50,Fail
> -kms_color@...-0-75,Fail
> -kms_color@...-blue-to-red,Fail
> -kms_color@...-green-to-red,Fail
> -kms_color@...-negative,Fail
> -kms_color@...-red-to-blue,Fail
> -kms_color@...-signed,Fail
> -kms_cursor_legacy@...sor-vs-flip-toggle,Fail
> -kms_cursor_legacy@...sor-vs-flip-varying-size,Fail
> -kms_cursor_legacy@...sorA-vs-flipA-atomic-transitions,Crash
> -kms_flip@...p-vs-modeset-vs-hang,Fail
> -kms_flip@...p-vs-panning-vs-hang,Fail
> -kms_pipe_crc_basic@...pare-crc-sanitycheck-nv12,Fail
> -kms_plane_alpha_blend@...ha-7efc,Fail
> -kms_plane_alpha_blend@...erage-7efc,Fail
> -kms_plane_alpha_blend@...erage-vs-premult-vs-constant,Fail
> -kms_plane_alpha_blend@...e-A-alpha-7efc,Fail
> -kms_plane_alpha_blend@...e-A-coverage-7efc,Fail
> -kms_plane_alpha_blend@...e-A-coverage-vs-premult-vs-constant,Fail
> -kms_plane_alpha_blend@...e-B-alpha-7efc,Fail
> -kms_plane_alpha_blend@...e-B-alpha-basic,Fail
> -kms_plane_alpha_blend@...e-B-alpha-opaque-fb,Fail
> -kms_plane_alpha_blend@...e-B-constant-alpha-max,Fail
> -kms_plane_alpha_blend@...e-B-constant-alpha-mid,Fail
> -kms_plane_alpha_blend@...e-B-coverage-7efc,Fail
> -kms_plane_alpha_blend@...e-B-coverage-vs-premult-vs-constant,Fail
> -kms_rmfb@...se-fd,Fail
> -kms_universal_plane@...able-primary-vs-flip-pipe-b,Fail
> -kms_universal_plane@...versal-plane-pipe-B-sanity,Fail
> diff --git a/drivers/gpu/drm/ci/xfails/msm-sc7180-flakes.txt b/drivers/gpu/drm/ci/xfails/msm-sc7180-flakes.txt
> deleted file mode 100644
> index 04730044ed12..000000000000
> --- a/drivers/gpu/drm/ci/xfails/msm-sc7180-flakes.txt
> +++ /dev/null
> @@ -1,17 +0,0 @@
> -kms_color@...-0-25
> -kms_color@...-0-50
> -kms_color@...-0-75
> -kms_color@...-blue-to-red
> -kms_color@...-green-to-red
> -kms_color@...-negative
> -kms_color@...-red-to-blue
> -kms_color@...-signed
> -kms_flip@...p-vs-modeset-vs-hang
> -kms_flip@...p-vs-panning-vs-hang
> -kms_plane@...el-format
> -kms_plane@...el-format-source-clamping
> -kms_plane@...ne-position-covered
> -kms_plane@...ne-position-hole
> -kms_plane@...ne-position-hole-dpms
> -kms_writeback@...teback-fb-id
> -kms_writeback@...teback-invalid-parameters
> diff --git a/drivers/gpu/drm/ci/xfails/msm-sc7180-skips.txt b/drivers/gpu/drm/ci/xfails/msm-sc7180-skips.txt
> deleted file mode 100644
> index e59a2fddfde0..000000000000
> --- a/drivers/gpu/drm/ci/xfails/msm-sc7180-skips.txt
> +++ /dev/null
> @@ -1,7 +0,0 @@
> -# Suspend to RAM seems to be broken on this machine
> -.*suspend.*
> -
> -# Test incorrectly assumes that CTM support implies gamma/degamma
> -# LUT support.  None of the subtests handle the case of only having
> -# CTM support
> -#kms_color.*
> diff --git a/drivers/gpu/drm/ci/xfails/msm-sc7180-trogdor-kingoftown-fails.txt b/drivers/gpu/drm/ci/xfails/msm-sc7180-trogdor-kingoftown-fails.txt
> new file mode 100644
> index 000000000000..7e4d8744fcc6
> --- /dev/null
> +++ b/drivers/gpu/drm/ci/xfails/msm-sc7180-trogdor-kingoftown-fails.txt
> @@ -0,0 +1,17 @@
> +kms_color@...-0-25,Fail
> +kms_color@...-0-50,Fail
> +kms_color@...-0-75,Fail
> +kms_color@...-blue-to-red,Fail
> +kms_color@...-green-to-red,Fail
> +kms_color@...-negative,Fail
> +kms_color@...-red-to-blue,Fail
> +kms_color@...-signed,Fail
> +kms_cursor_legacy@...sor-vs-flip-toggle,Fail
> +kms_cursor_legacy@...sor-vs-flip-varying-size,Fail
> +kms_flip@...p-vs-modeset-vs-hang,Fail
> +kms_flip@...p-vs-panning-vs-hang,Fail
> +kms_pipe_crc_basic@...pare-crc-sanitycheck-nv12,Fail
> +kms_plane_alpha_blend@...ha-7efc,Fail
> +kms_plane_alpha_blend@...erage-7efc,Fail
> +kms_plane_alpha_blend@...erage-vs-premult-vs-constant,Fail
> +kms_rmfb@...se-fd,Fail
> diff --git a/drivers/gpu/drm/ci/xfails/msm-sc7180-trogdor-lazor-limozeen-fails.txt b/drivers/gpu/drm/ci/xfails/msm-sc7180-trogdor-lazor-limozeen-fails.txt
> new file mode 100644
> index 000000000000..7e4d8744fcc6
> --- /dev/null
> +++ b/drivers/gpu/drm/ci/xfails/msm-sc7180-trogdor-lazor-limozeen-fails.txt
> @@ -0,0 +1,17 @@
> +kms_color@...-0-25,Fail
> +kms_color@...-0-50,Fail
> +kms_color@...-0-75,Fail
> +kms_color@...-blue-to-red,Fail
> +kms_color@...-green-to-red,Fail
> +kms_color@...-negative,Fail
> +kms_color@...-red-to-blue,Fail
> +kms_color@...-signed,Fail
> +kms_cursor_legacy@...sor-vs-flip-toggle,Fail
> +kms_cursor_legacy@...sor-vs-flip-varying-size,Fail
> +kms_flip@...p-vs-modeset-vs-hang,Fail
> +kms_flip@...p-vs-panning-vs-hang,Fail
> +kms_pipe_crc_basic@...pare-crc-sanitycheck-nv12,Fail
> +kms_plane_alpha_blend@...ha-7efc,Fail
> +kms_plane_alpha_blend@...erage-7efc,Fail
> +kms_plane_alpha_blend@...erage-vs-premult-vs-constant,Fail
> +kms_rmfb@...se-fd,Fail

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ