Re: [PATCH v2 0/2] drm/msm/gen_header: allow skipping the validation

2024-05-10 Thread Helen Koike
On 03/05/2024 17:23, Dmitry Baryshkov wrote: On Fri, 3 May 2024 at 21:15, Dmitry Baryshkov wrote: In order to remove pointless messages regarding missing lxml, skip validation of MSM register files against the schema. Only the driver developers really care and/or can fix the files. Keep

Re: [PATCH v2 2/2] drm/ci: validate drm/msm XML register files against schema

2024-05-10 Thread Helen Koike
answered my second question actually. That basically I didnt also want EXPERT to find its way into actual running kernels. Hence, I am fine with this change now Reviewed-by: Abhinav Kumar But, I will wait to hear from helen, vignesh about what they think of this. lgfm Acked-by: Helen Koike Thanks Helen

Re: [PATCH v1 5/5] drm/ci: update xfails for the new testlist

2024-04-30 Thread Helen Koike
On 30/04/2024 06:11, Vignesh Raman wrote: Now the testlist is used from IGT build, so update xfails with the new testlist. Signed-off-by: Vignesh Raman --- .../gpu/drm/ci/xfails/amdgpu-stoney-fails.txt | 47 +++ .../drm/ci/xfails/amdgpu-stoney-flakes.txt| 8 +-

Re: [PATCH v1 4/5] drm/ci: skip driver specific tests

2024-04-30 Thread Helen Koike
On 30/04/2024 06:11, Vignesh Raman wrote: Skip driver specific tests and skip kms tests for panfrost driver since it is not a kms driver. Signed-off-by: Vignesh Raman --- .../gpu/drm/ci/xfails/amdgpu-stoney-skips.txt | 14 +- drivers/gpu/drm/ci/xfails/i915-amly-skips.txt

Re: [PATCH v1 3/5] drm/ci: build virtual GPU driver as module

2024-04-30 Thread Helen Koike
On 30/04/2024 06:11, Vignesh Raman wrote: With latest IGT, the tests tries to load the module and it fails. So build the virtual GPU driver for virtio as module. Signed-off-by: Vignesh Raman Acked-by: Helen Koike --- drivers/gpu/drm/ci/build.sh | 1 - drivers/gpu/drm/ci

Re: [PATCH v1 1/5] drm/ci: uprev mesa version

2024-04-30 Thread Helen Koike
version which includes a fix for this issue. https://gitlab.freedesktop.org/mesa/mesa/-/commit/908f444e Also copy helper scripts to install, so that the ci jobs can use these scripts for logging. Signed-off-by: Vignesh Raman Acked-by: Helen Koike --- drivers/gpu/drm/ci/build.sh | 1

Re: [PATCH] drm: ci: fix the xfails for apq8016

2024-04-29 Thread Helen Koike
On 25/04/2024 11:01, Helen Koike wrote: On 08/04/2024 14:04, Abhinav Kumar wrote: Hi Helen Gentle reminder on this. If you are okay, we can land it via msm-next tree... Thanks Abhinav On 4/1/2024 1:48 PM, Abhinav Kumar wrote: After IGT migrating to dynamic sub-tests, the pipe

Re: drm-misc migration to Gitlab server

2024-04-29 Thread Helen Koike
On 29/04/2024 14:56, Jani Nikula wrote: On Mon, 29 Apr 2024, Helen Koike wrote: On 01/04/2024 13:49, Tvrtko Ursulin wrote: No issues accessing the repo just a slight confusion and how to handle the workflow. More specifically, if I have a patch which wants to be merged to drm-misc-next

Re: drm-misc migration to Gitlab server

2024-04-29 Thread Helen Koike
On 01/04/2024 13:49, Tvrtko Ursulin wrote: On 12/03/2024 13:56, Maxime Ripard wrote: Hi, On Tue, Feb 20, 2024 at 09:49:25AM +0100, Maxime Ripard wrote: ## Changing the default location repo Dim gets its repos list in the drm-rerere nightly.conf file. We will need to change that file to

Re: [PATCH] drm: ci: fix the xfails for apq8016

2024-04-25 Thread Helen Koike
correctly work and was resulting in those failures not being seen. Here is the passing pipeline for apq8016 with this change: https://gitlab.freedesktop.org/drm/msm/-/jobs/57050562 Signed-off-by: Abhinav Kumar I'm sorry about my delay. Acked-by: Helen Koike I'm also merging it to msm-misc-next

Re: [PATCH v5 10/10] drm/ci: add tests on vkms

2024-04-01 Thread Helen Koike
On 01/04/2024 03:12, Vignesh Raman wrote: Add job that runs igt on top of vkms. Signed-off-by: Vignesh Raman Acked-by: Jessica Zhang Tested-by: Jessica Zhang Acked-by: Maxime Ripard Signed-off-by: Helen Koike Acked-by: Helen Koike Thanks Helen --- v4: - New patch in the series

Re: [PATCH v5 09/10] drm/ci: rockchip: Add job to test panfrost GPU driver

2024-04-01 Thread Helen Koike
. Signed-off-by: Vignesh Raman Btw, just a nit, I would squash this with the previous commit, so we don't have a point in history with panfrost tests, than without them, than with them again. But with or without this change: Acked-by: Helen Koike Thanks Helen --- v2: - Add panfrost GPU

Re: [PATCH v5 08/10] drm/ci: rockchip: Refactor existing rockchip jobs

2024-04-01 Thread Helen Koike
/test.yml +++ b/drivers/gpu/drm/ci/test.yml @@ -155,33 +155,47 @@ msm:sdm845: script: - ./install/bare-metal/cros-servo.sh -rockchip:rk3288: - extends: -- .lava-igt:arm32 +.rockchip: same here, s/.rockchip/.rockchip-device With or without this change: Acked-by: Helen Koike Thanks

Re: [PATCH v5 07/10] drm/ci: meson: Add job to test panfrost GPU driver

2024-04-01 Thread Helen Koike
-off-by: Vignesh Raman Nit: Maybe squash this with previous commit. With or without this change: Acked-by: Helen Koike Thanks Helen --- v2: - Add panfrost GPU jobs for amlogic meson SOC with new xfails. v3: - Skip KMS tests for panfrost driver since it is not a not a KMS driver

Re: [PATCH v5 06/10] drm/ci: meson: Refactor existing meson jobs

2024-04-01 Thread Helen Koike
with the driver name, what do you think? With or without this change: Acked-by: Helen Koike Thanks Helen extends: - .lava-igt:arm64 - stage: meson variables: -DRIVER_NAME: meson DTB: ${DEVICE_TYPE} BOOT_METHOD: u-boot KERNEL_IMAGE_TYPE: "image" -meson:g12b

Re: [PATCH v5 05/10] drm/ci: mediatek: Add job to test panfrost and powervr GPU driver

2024-04-01 Thread Helen Koike
a/MAINTAINERS b/MAINTAINERS index f7d0040a6c21..333704ceefb6 100644 --- a/MAINTAINERS +++ b/MAINTAINERS Note: We'll also need an entry in MAINTAINERS file for powervr (but it is fine to only add it once we have tests running on it). Acked-by: Helen Koike Thanks Helen @@ -1666,6 +1666,7 @@ S

Re: [PATCH v5 04/10] drm/ci: mediatek: Refactor existing mediatek jobs

2024-04-01 Thread Helen Koike
On 01/04/2024 03:12, Vignesh Raman wrote: For mediatek mt8173 and mt8183, the display driver is mediatek. Currently, in drm-ci for mediatek, only the display driver is tested. Refactor the existing mediatek jobs so that gpu driver testing jobs can be added later and update xfails accordingly.

Re: [PATCH v5 03/10] drm/ci: uprev IGT and update testlist

2024-04-01 Thread Helen Koike
On 01/04/2024 03:12, Vignesh Raman wrote: Uprev IGT and add amd, v3d, vc4 and vgem specific tests to testlist and skip driver-specific tests. Also add testlist to the MAINTAINERS file and update xfails. Signed-off-by: Vignesh Raman lgtm Acked-by: Helen Koike Thanks --- v3: - New

Re: [PATCH v4 09/11] drm/ci: rockchip: Refactor existing rockchip jobs

2024-03-15 Thread Helen Koike
On 15/03/2024 08:18, Vignesh Raman wrote: Hi Helen, On 07/03/24 19:32, Helen Koike wrote: On 06/03/2024 00:06, Vignesh Raman wrote: For rockchip rk3288 and rk3399, the display driver is rockchip. Currently, in drm-ci for rockchip, only the display driver is tested. Refactor the existing

Re: [PATCH v4 03/11] drm/ci: uprev IGT and update testlist

2024-03-15 Thread Helen Koike
On 15/03/2024 08:12, Vignesh Raman wrote: Hi Helen, On 07/03/24 19:05, Helen Koike wrote: On 06/03/2024 00:06, Vignesh Raman wrote: Uprev IGT and add amd, v3d, vc4 and vgem specific tests to testlist. Have testlist.txt per driver and include a base testlist so that the driver specific

Re: [PATCH v3] drm/ci: update device type for volteer devices

2024-03-07 Thread Helen Koike
On 07/03/2024 10:21, Helen Koike wrote: On 06/03/2024 23:18, Vignesh Raman wrote: Volteer devices in the collabora lab are categorized under the asus-cx9400-volteer device type. The majority of these units has an Intel Core i5-1130G7 CPU, while some of them have a Intel Core i7-1160G7 CPU

Re: [PATCH v4 00/11] drm/ci: Add support for GPU and display testing

2024-03-07 Thread Helen Koike
On 06/03/2024 00:06, Vignesh Raman wrote: Some ARM SOCs have a separate display controller and GPU, each with different drivers. For mediatek mt8173, the GPU driver is powervr, and the display driver is mediatek. In the case of mediatek mt8183, the GPU driver is panfrost, and the display

Re: [PATCH v4 09/11] drm/ci: rockchip: Refactor existing rockchip jobs

2024-03-07 Thread Helen Koike
On 06/03/2024 00:06, Vignesh Raman wrote: For rockchip rk3288 and rk3399, the display driver is rockchip. Currently, in drm-ci for rockchip, only the display driver is tested. Refactor the existing rockchip jobs so that gpu driver testing jobs can be added later and update xfails accordingly.

Re: [PATCH v4 07/11] drm/ci: meson: Refactor existing meson jobs

2024-03-07 Thread Helen Koike
On 06/03/2024 00:06, Vignesh Raman wrote: For Amlogic Meson SOC the display driver is meson. Currently, in drm-ci for meson, only the display driver is tested. Refactor the existing meson jobs so that gpu driver testing jobs can be added later and update xfails accordingly. Signed-off-by:

Re: [PATCH v4 05/11] drm/ci: mediatek: Refactor existing mediatek jobs

2024-03-07 Thread Helen Koike
On 06/03/2024 00:06, Vignesh Raman wrote: For mediatek mt8173 and mt8183, the display driver is mediatek. Currently, in drm-ci for mediatek, only the display driver is tested. Refactor the existing mediatek jobs so that gpu driver testing jobs can be added later and update xfails accordingly.

Re: [PATCH v4 03/11] drm/ci: uprev IGT and update testlist

2024-03-07 Thread Helen Koike
On 06/03/2024 00:06, Vignesh Raman wrote: Uprev IGT and add amd, v3d, vc4 and vgem specific tests to testlist. Have testlist.txt per driver and include a base testlist so that the driver specific tests will run only on those hardware. Also add testlists to the MAINTAINERS file.

Re: [PATCH v4 02/11] drm/ci: uprev mesa version

2024-03-07 Thread Helen Koike
includes a fix for this issue. https://gitlab.freedesktop.org/mesa/mesa/-/commit/908f444e Signed-off-by: Vignesh Raman Acked-by: Helen Koike Thanks Helen --- v3: - New patch in series to uprev mesa. v4: - Fix checkpatch warning. --- drivers/gpu/drm/ci/container.yml | 6

Re: [PATCH v3] drm/ci: update device type for volteer devices

2024-03-07 Thread Helen Koike
ctory") Reviewed-by: David Heidelberg Signed-off-by: Vignesh Raman Acked-by: Helen Koike Thanks Helen --- v2: - Add fixes tag so change gets propagated to stable. https://gitlab.freedesktop.org/vigneshraman/linux/-/pipelines/1119672 v3: - Fix checkpatch warning. Please use cor

Re: [PATCH 1/3] kci-gitlab: Introducing GitLab-CI Pipeline for Kernel Testing

2024-03-04 Thread Helen Koike
skip having a default .gitlab-ci.yml in the root directory and instead include clear instructions in our documentation for using these templates. Thanks, Helen Koike (And then perhaps have a 'gitlab' directory under that. I'm not sure whether - and how much - commonality there might be between

[PATCH 2/3] kci-gitlab: Add documentation

2024-02-28 Thread Helen Koike
Add documentation of kci-gitlab. Signed-off-by: Helen Koike --- Documentation/ci/gitlab-ci/gitlab-ci.rst | 404 +++ Documentation/index.rst | 7 + MAINTAINERS | 1 + 3 files changed, 412 insertions(+) create mode 100644

[PATCH 1/3] kci-gitlab: Introducing GitLab-CI Pipeline for Kernel Testing

2024-02-28 Thread Helen Koike
. It leverages an integrated cache for efficient build times and introduces a flexible 'scenarios' mechanism for subsystem-specific extensions. [ci: add prerequisites to run check-patch on MRs] Co-developed-by: Tales Aparecida Signed-off-by: Tales Aparecida Signed-off-by: Helen Koike --- Hey all, You

[PATCH 0/3] kci-gitlab: Introducing GitLab-CI Pipeline for Kernel Testing

2024-02-28 Thread Helen Koike
ghts and suggestions on this initiative. Also, if you want to see this initiave move faster, we are happy to discuss funding options. Best regards, Helen Koike Helen Koike (3): kci-gitlab: Introducing GitLab-CI Pipeline for Kernel Testing kci-gitlab: Add documentation kci-gitlab: docs: Add ima

Re: [PATCH v4 3/8] drm: ci: arm64.config: update ONBOARD_USB_HUB to ONBOARD_USB_DEV

2024-02-20 Thread Helen Koike
On 20/02/2024 11:05, Javier Carrasco wrote: The onboard_usb_hub driver has been updated to support non-hub devices, which has led to some renaming. Update to the new name (ONBOARD_USB_DEV) accordingly. Signed-off-by: Javier Carrasco Acked-by: Helen Koike Regards, Helen --- drivers

Re: [PATCH] drm: ci: uprev IGT

2024-02-20 Thread Helen Koike
On 20/02/2024 09:17, Dmitry Baryshkov wrote: Bump IGT revision to pick up Rob Clark's fixes for the msm driver: - msm_submit@invalid-duplicate-bo-submit,Fail Signed-off-by: Dmitry Baryshkov Do you have a gitlab pipeline link I can check? Thanks Helen ---

Re: [PATCH 3/3] drm/ci: mark universal-plane-sanity as failing on SC7180

2024-02-19 Thread Helen Koike
On 19/02/2024 17:25, Helen Koike wrote: On 18/02/2024 01:12, Dmitry Baryshkov wrote: The test kms_universal_plane@universal-plane-sanity fails on both SC7180 platforms. The drm/msm returns -ERANGE as it can not handle passet scaling range, however the test is not ready to handle that. Mark

Re: [PATCH 2/3] drm/ci: update msm-apq8096-fails list

2024-02-19 Thread Helen Koike
On 19/02/2024 17:26, Helen Koike wrote: On 18/02/2024 01:12, Dmitry Baryshkov wrote: Mark two tests as passing on the APQ8096 / db820c platform. Signed-off-by: Dmitry Baryshkov Acked-by: Helen Koike Applied to drm-misc/drm-misc-next Thanks Helen ---   drivers/gpu/drm/ci/xfails

Re: [PATCH 1/3] drm/ci: update IGT test names

2024-02-19 Thread Helen Koike
On 19/02/2024 17:25, Helen Koike wrote: On 19/02/2024 14:42, Dmitry Baryshkov wrote: On Mon, 19 Feb 2024 at 18:48, Helen Koike wrote: On 19/02/2024 11:33, Dmitry Baryshkov wrote: On Mon, 19 Feb 2024 at 15:21, Helen Koike wrote: Hi Dmitry, On 18/02/2024 01:12, Dmitry Baryshkov

Re: [PATCH 2/3] drm/ci: update msm-apq8096-fails list

2024-02-19 Thread Helen Koike
On 18/02/2024 01:12, Dmitry Baryshkov wrote: Mark two tests as passing on the APQ8096 / db820c platform. Signed-off-by: Dmitry Baryshkov Acked-by: Helen Koike --- drivers/gpu/drm/ci/xfails/msm-apq8096-fails.txt | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/ci

Re: [PATCH 3/3] drm/ci: mark universal-plane-sanity as failing on SC7180

2024-02-19 Thread Helen Koike
merical result out of range Signed-off-by: Dmitry Baryshkov Acked-by: Helen Koike --- .../gpu/drm/ci/xfails/msm-sc7180-trogdor-kingoftown-fails.txt| 1 + .../drm/ci/xfails/msm-sc7180-trogdor-lazor-limozeen-fails.txt| 1 + 2 files changed, 2 insertions(+) diff --git a/drivers/gpu/drm/c

Re: [PATCH 1/3] drm/ci: update IGT test names

2024-02-19 Thread Helen Koike
On 19/02/2024 14:42, Dmitry Baryshkov wrote: On Mon, 19 Feb 2024 at 18:48, Helen Koike wrote: On 19/02/2024 11:33, Dmitry Baryshkov wrote: On Mon, 19 Feb 2024 at 15:21, Helen Koike wrote: Hi Dmitry, On 18/02/2024 01:12, Dmitry Baryshkov wrote: Since the addition of testlist.txt

Re: [PATCH] drm/ci: skip suspend tests for both msm-sc7180 machines

2024-02-19 Thread Helen Koike
On 19/02/2024 11:47, Vignesh Raman wrote: On 17/02/24 02:26, Dmitry Baryshkov wrote: The commit ea489a3d983b ("drm/ci: add sc7180-trogdor-kingoftown") dropped the msm-sc7180-skips.txt file, which disabled suspend-to-RAM tests. However testing shows that STR tests still can fail. Restore the

Re: [PATCH 1/3] drm/ci: update IGT test names

2024-02-19 Thread Helen Koike
On 19/02/2024 11:33, Dmitry Baryshkov wrote: On Mon, 19 Feb 2024 at 15:21, Helen Koike wrote: Hi Dmitry, On 18/02/2024 01:12, Dmitry Baryshkov wrote: Since the addition of testlist.txt the IGT has changed some of test names. Some test names were changed to use '-' instead

Re: [PATCH 1/3] drm/ci: update IGT test names

2024-02-19 Thread Helen Koike
Hi Dmitry, On 18/02/2024 01:12, Dmitry Baryshkov wrote: Since the addition of testlist.txt the IGT has changed some of test names. Some test names were changed to use '-' instead of '_'. In other cases tests were just renamed. Follow those changes. Signed-off-by: Dmitry Baryshkov Thanks

Re: [PATCH v3 3/9] drm/ci: mediatek: Add job to test panfrost and powervr GPU driver

2024-02-19 Thread Helen Koike
On 19/02/2024 06:39, Vignesh Raman wrote: Hi Helen, On 09/02/24 23:51, Helen Koike wrote: On 30/01/2024 12:03, Vignesh Raman wrote: For mediatek mt8173, the GPU driver is powervr and for mediatek mt8183, the GPU driver is panfrost. So add support in drm-ci to test panfrost and powervr

Re: [PATCH v2] drm: ci: use clk_ignore_unused for apq8016

2024-02-14 Thread Helen Koike
On 14/02/2024 10:30, Helen Koike wrote: On 14/02/2024 05:37, Dmitry Baryshkov wrote: If the ADV7511 bridge driver is compiled as a module, while DRM_MSM is built-in, the clk_disable_unused congests with the runtime PM handling of the DSI PHY for the clk_prepare_lock(). This causes apq8016

Re: [PATCH v3 9/9] drm/ci: uprev IGT and update testlist

2024-02-14 Thread Helen Koike
On 10/02/2024 15:20, Maíra Canal wrote: On 2/10/24 15:17, Maíra Canal wrote: On 1/30/24 12:03, Vignesh Raman wrote: Uprev IGT and add amd, v3d, vc4 and vgem specific tests to testlist. Have testlist.txt per driver and include a base testlist so that the driver specific tests will run only

Re: [PATCH v2] drm: ci: use clk_ignore_unused for apq8016

2024-02-14 Thread Helen Koike
ctory") Reviewed-by: Javier Martinez Canillas Signed-off-by: Dmitry Baryshkov Acked-by: Helen Koike Thanks Helen --- Changes in v2: - Added a comment, describing the issue and a way to reproduce it (Javier) --- drivers/gpu/drm/ci/test.yml | 5 - 1 file changed, 4 insertions(+),

Re: [PATCH v3 1/9] drm/ci: arm64.config: Enable CONFIG_DRM_ANALOGIX_ANX7625

2024-02-09 Thread Helen Koike
CONFIG_DRM_ANALOGIX_ANX7625=y in CI arm64 config. Couldn't you load the module as it is done on https://cgit.freedesktop.org/drm/drm-misc/tree/drivers/gpu/drm/ci/igt_runner.sh#n35 ? This is not a blocker, in any case Acked-by: Helen Koike Thanks Helen Signed-off-by: Vignesh Raman --- v2

Re: [PATCH v3 3/9] drm/ci: mediatek: Add job to test panfrost and powervr GPU driver

2024-02-09 Thread Helen Koike
On 30/01/2024 12:03, Vignesh Raman wrote: For mediatek mt8173, the GPU driver is powervr and for mediatek mt8183, the GPU driver is panfrost. So add support in drm-ci to test panfrost and powervr GPU driver for mediatek SOCs and update xfails. Powervr driver was merged in linux kernel, but

Re: [PATCH v4] drm/ci: add tests on vkms

2024-02-09 Thread Helen Koike
On 01/02/2024 03:53, Vignesh Raman wrote: Add job that runs igt on top of vkms. Signed-off-by: Vignesh Raman Acked-by: Jessica Zhang Tested-by: Jessica Zhang Acked-by: Maxime Ripard Signed-off-by: Helen Koike --- v2: - do not mv modules to /lib/modules in the job definition, leave

Re: [PATCH v1] drm/ci: add sc7180-trogdor-kingoftown

2024-01-15 Thread Helen Koike
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 Applied to drm-misc-next. Thanks Helen --- See pipeline:

Re: [PATCH v2] drm/ci: uprev mesa version: fix kdl commit fetch

2024-01-15 Thread Helen Koike
fetch issue. Uprev mesa in drm-ci to fix this. This commit updates the kernel tag and adds .never-post-merge-rules due to the mesa uprev. It also fixes an issue where the virtio-gpu pipeline was not getting created with the mesa uprev. Reviewed-by: David Heidelberg Acked-by: Helen Koike Reviewed

Re: [PATCH v1] drm/ci: Update xfails for newly added msm tests

2024-01-15 Thread Helen Koike
On 15/01/2024 07:17, Vignesh Raman wrote: msm tests are added to testlist.txt, so update the xfails for jobs in msm stage. Signed-off-by: Vignesh Raman Applied to drm-misc-next. Thanks Helen --- Depends on

Re: [PATCH] drm/ci: Add msm tests

2024-01-15 Thread Helen Koike
On 08/01/2024 16:50, Rob Clark wrote: From: Rob Clark The msm tests should skip on non-msm hw, so I think it should be safe to enable everywhere. Signed-off-by: Rob Clark Applied to drm-misc-next. Thanks Helen --- drivers/gpu/drm/ci/testlist.txt | 49

Re: [PATCH] drm/ci: Add msm tests

2024-01-15 Thread Helen Koike
-msm hw, so I think it should be safe to enable everywhere. Signed-off-by: Rob Clark Acked-by: Helen Koike ---    drivers/gpu/drm/ci/testlist.txt | 49 +    1 file changed, 49 insertions(+) diff --git a/drivers/gpu/drm/ci/testlist.txt b/drivers/gpu/drm/ci

Re: [PATCH v1] drm/ci: Update xfails for newly added msm tests

2024-01-15 Thread Helen Koike
/gfx-ci/linux/-/commit/4b49f902ec6f2bb382cbbf489870573f4b43371e +# https://gitlab.freedesktop.org/gfx-ci/linux/-/commit/38cdf4c5559771e2474ae0fecef8469f65147bc1 Any chance we could make an upstream patch to solve this? +msm_mapping@* In any case: Acked-by: Helen Koike Thanks Helen

Re: [PATCH v1] drm/ci: add sc7180-trogdor-kingoftown

2024-01-15 Thread Helen Koike
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 lgtm Acked-by: Helen Koike Thanks Helen --- See pipeline: https

Re: [PATCH v3] drm/ci: add tests on vkms

2024-01-15 Thread Helen Koike
Hi, Thanks for picking this up. On 02/01/2024 06:41, Vignesh Raman wrote: From: Helen Koike Add job that runs igt on top of vkms. Signed-off-by: Vignesh Raman Acked-by: Jessica Zhang Tested-by: Jessica Zhang Acked-by: Maxime Ripard Signed-off-by: Helen Koike --- v2: - do not mv

Re: Automatically update drm CI dependencies?

2024-01-08 Thread Helen Koike
On 20/12/2023 08:11, Bagas Sanjaya wrote: On 12/19/23 23:43, Helen Koike wrote: Hi, On 14/12/2023 06:38, Bagas Sanjaya wrote: Hi all, I'm referring to dependabot PR on torvalds.git GitHub mirror [1]. I know that PRs submitted there are not accepted (the repo is essentially read-only

Re: Automatically update drm CI dependencies?

2023-12-19 Thread Helen Koike
Hi, On 14/12/2023 06:38, Bagas Sanjaya wrote: Hi all, I'm referring to dependabot PR on torvalds.git GitHub mirror [1]. I know that PRs submitted there are not accepted (the repo is essentially read-only mirror), hence this mail question. In summary, dependabot submitted automated PR that

Re: [PATCH] drm/ci: uprev mesa version: fix kdl commit fetch

2023-12-14 Thread Helen Koike
-by: Helen Koike Thanks Helen --- drivers/gpu/drm/ci/gitlab-ci.yml | 14 -- 1 file changed, 12 insertions(+), 2 deletions(-)

Re: [PATCH v7 00/10] drm: ci: fixes

2023-12-13 Thread Helen Koike
On 07/12/2023 06:18, Vignesh Raman wrote: The patch series contains improvements, enabling new ci jobs which enables testing for Mediatek MT8173, Qualcomm APQ 8016 and VirtIO GPU, fixing issues with the ci jobs and updating the expectation files. v2: - Use fdtoverlay command to merge

Re: [PATCH v6 09/10] drm/doc: ci: Add IGT version details for flaky tests

2023-12-05 Thread Helen Koike
On 29/11/2023 09:18, Vignesh Raman wrote: Document the IGT version in the flaky tests reporting template. Nice! Signed-off-by: Vignesh Raman Acked-by: Helen Koike Thanks Helen --- v6: - Added a new patch in the series to update drm ci documentation --- Documentation/gpu

Re: [PATCH v6 08/10] drm: ci: uprev IGT

2023-12-05 Thread Helen Koike
which fixes this issue. Signed-off-by: Vignesh Raman Acked-by: Helen Koike Thanks Helen --- v6: - Added a new patch in the series to uprev IGT to fix memory corruption --- drivers/gpu/drm/ci/gitlab-ci.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu

Re: [PATCH v6 07/10] drm: ci: virtio: Make artifacts available

2023-12-05 Thread Helen Koike
On 29/11/2023 09:18, Vignesh Raman wrote: There were no artifacts available for virtio job. So make the artifacts available in the pipeline job. Signed-off-by: Vignesh Raman Acked-by: Helen Koike Thanks Helen --- v5: - Added a new patch in the series to make artifacts available

Re: [PATCH v6 06/10] drm: ci: mediatek: Set IGT_FORCE_DRIVER for mt8173

2023-12-05 Thread Helen Koike
VER=${DRIVER_NAME} +fi Just a nit, could you add this in the first case statement? Just to avoid confusing about double assignment? -> but feel free to ignore this comment. With or without this: Acked-by: Helen Koike Thanks Helen + if [ -e "/install/xfails/$DRIVER_NAME-$GPU_VERSION-s

Re: [PATCH] drm/ci: remove rebase-merge directory

2023-12-05 Thread Helen Koike
. Suggested-by: David Heidelberg Signed-off-by: Dmitry Baryshkov Acked-by: Helen Koike Thanks! Helen --- drivers/gpu/drm/ci/build.sh | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/gpu/drm/ci/build.sh b/drivers/gpu/drm/ci/build.sh index e5c5dcedd108..ca2923ed2290 100644

Re: github version complaints about the gitlab CI requirements.txt

2023-11-13 Thread Helen Koike
thought I should ask the people who actually set this up. Comments? I just tested your attached patch and looks fine, the scripts with those requirements worked as expected, so please go ahead with your patch with Tested-by: Helen Koike Now I'm thinking how to prevent those warnings in the future

Re: [PATCH 0/3] drm/ci: Add support for SM8250 Gitlab Runner

2023-11-09 Thread Helen Koike
On 07/11/2023 21:10, Jessica Zhang wrote: On 11/6/2023 9:35 AM, Jessica Zhang wrote: On 11/4/2023 6:02 AM, Helen Koike wrote: Hi Jessica, On 10/10/2023 19:25, Jessica Zhang wrote: Recently, we've registered a Gitlab runner for a Qualcomm RB5 device that will be hosted and maintained

Re: [PATCH 3/3] drm/ci: Add skips, fails and flakes for SM8250

2023-11-04 Thread Helen Koike
On 10/10/2023 19:25, Jessica Zhang wrote: Add skips, fails and flakes for the SM8250 test. Generated using update-xfails.py [1] [1] https://patchwork.freedesktop.org/patch/561453/?series=124793=1 Signed-off-by: Abhinav Kumar Signed-off-by: Jessica Zhang ---

Re: [PATCH 0/3] drm/ci: Add support for SM8250 Gitlab Runner

2023-11-04 Thread Helen Koike
Hi Jessica, On 10/10/2023 19:25, Jessica Zhang wrote: Recently, we've registered a Gitlab runner for a Qualcomm RB5 device that will be hosted and maintained in Qualcomm labs. This series will add a corresponding CI job for testing SM8250 devices and add the skip/fails/flakes list. We were

Re: [PATCH Resend] Fix line Length

2023-10-31 Thread Helen Koike
On 30/10/2023 08:36, Bagas Sanjaya wrote: On 30/10/2023 13:12, Julia Lawall wrote: On Mon, 30 Oct 2023, Bagas Sanjaya wrote: Hi Julia, The submitter touched one of CI scripts for the DRM subsystem. To test this patch, there must be a way to run these scripts locally (which may requires

Re: (subset) [PATCH v2] drm/doc: ci: Require more context for flaky tests

2023-10-27 Thread Helen Koike
On 26/10/2023 12:31, Maxime Ripard wrote: Hi, On Thu, Oct 26, 2023 at 11:27:18AM -0300, Helen Koike wrote: On 26/10/2023 10:27, Maxime Ripard wrote: On Thu, Oct 26, 2023 at 09:08:03AM -0300, Helen Koike wrote: On 26/10/2023 09:01, Helen Koike wrote: On 26/10/2023 07:58, Maxime Ripard

Re: (subset) [PATCH v2] drm/doc: ci: Require more context for flaky tests

2023-10-26 Thread Helen Koike
On 26/10/2023 10:27, Maxime Ripard wrote: On Thu, Oct 26, 2023 at 09:08:03AM -0300, Helen Koike wrote: On 26/10/2023 09:01, Helen Koike wrote: On 26/10/2023 07:58, Maxime Ripard wrote: On Wed, 25 Oct 2023 16:24:41 +0200, Maxime Ripard wrote: Flaky tests can be very difficult

Re: (subset) [PATCH v2] drm/doc: ci: Require more context for flaky tests

2023-10-26 Thread Helen Koike
On 26/10/2023 09:01, Helen Koike wrote: On 26/10/2023 07:58, Maxime Ripard wrote: On Wed, 25 Oct 2023 16:24:41 +0200, Maxime Ripard wrote: Flaky tests can be very difficult to reproduce after the facts, which will make it even harder to ever fix. Let's document the metadata we agreed

Re: (subset) [PATCH v2] drm/doc: ci: Require more context for flaky tests

2023-10-26 Thread Helen Koike
On 26/10/2023 07:58, Maxime Ripard wrote: On Wed, 25 Oct 2023 16:24:41 +0200, Maxime Ripard wrote: Flaky tests can be very difficult to reproduce after the facts, which will make it even harder to ever fix. Let's document the metadata we agreed on to provide more context to anyone trying to

Re: [PATCH v2] drm/doc: ci: Require more context for flaky tests

2023-10-25 Thread Helen Koike
-devel/CAPj87rPbJ1V1-R7WMTHkDat2A4nwSd61Df9mdGH2PR=ZzxaU=q...@mail.gmail.com/ Acked-by: Daniel Vetter Signed-off-by: Maxime Ripard Acked-by: Helen Koike --- Cc: Helen Koike Changes from v1: - Rephrase the sentence a bit to make it clearer we expect a bug report by mail

Re: [PATCH] drm/doc: ci: Require more context for flaky tests

2023-10-25 Thread Helen Koike
On 23/10/2023 12:09, Maxime Ripard wrote: On Fri, Oct 20, 2023 at 01:33:59AM -0300, Helen Koike wrote: On 19/10/2023 13:51, Helen Koike wrote: On 19/10/2023 06:46, Maxime Ripard wrote: Flaky tests can be very difficult to reproduce after the facts, which will make it even harder to ever

[PATCH v3 10/10] drm/ci: docs: add step about how to request privileges

2023-10-23 Thread Helen Koike
Clarify the procedure developer must follow to request privileges to run tests on Freedesktop gitlab CI. This measure was added to avoid untrusted people to misuse the infrastructure. Signed-off-by: Helen Koike Reviewed-by: David Heidelberg --- v2: - fix typo in commit message v3: -no changes

[PATCH v3 09/10] drm/ci: do not automatically retry on error

2023-10-23 Thread Helen Koike
Since the kernel doesn't use a bot like Mesa that requires tests to pass in order to merge the patches, leave it to developers and/or maintainers to manually retry. Suggested-by: Rob Clark Signed-off-by: Helen Koike Reviewed-by: David Heidelberg --- v2: - no changes v3: -no changes

[PATCH v3 08/10] drm/ci: export kernel config

2023-10-23 Thread Helen Koike
Export the resultant kernel config, making it easier to verify if the resultant config was correctly generated. Suggested-by: Rob Clark Signed-off-by: Helen Koike Acked-by: Dmitry Baryshkov Reviewed-by: David Heidelberg --- v2: - no changes v3: -no changes --- drivers/gpu/drm/ci/build.sh

[PATCH v3 07/10] drm/ci: increase i915 job timeout to 1h30m

2023-10-23 Thread Helen Koike
(1h6m39s)  job i915:apl 2/3 has new status: success (1h4m45s)  job i915:apl 1/3 has new status: success (1h7m38s) (generated with ci_run_n_monitor.py script) The longest job is 1h19m58s, so adjust the timeout. Signed-off-by: Helen Koike --- v3: - new patch --- drivers/gpu/drm/ci/test.yml | 5

[PATCH v3 06/10] drm/ci: add subset-1-gfx to LAVA_TAGS and adjust shards

2023-10-23 Thread Helen Koike
developers from getting their patches merged. Signed-off-by: Helen Koike Reviewed-by: David Heidelberg --- v2: - add subset-1-gfx tag to LAVA_TAGS - update commit message v3: no changes --- drivers/gpu/drm/ci/gitlab-ci.yml | 2 +- drivers/gpu/drm/ci/test.yml | 23 ++- 2

[PATCH v3 05/10] drm/ci: clean up xfails (specially flakes list)

2023-10-23 Thread Helen Koike
Since the script that collected the list of the expectation files was bogus and placing test to the flakes list incorrectly, restart the expectation files with the correct script. This reduces a lot the number of tests in the flakes list. Signed-off-by: Helen Koike Reviewed-by: David Heidelberg

[PATCH v3 04/10] drm/ci: uprev IGT and make sure core_getversion is run

2023-10-23 Thread Helen Koike
was ok. Uprev IGT to include this modification and include core_getversion test in all the shards. Signed-off-by: Helen Koike Reviewed-by: David Heidelberg --- v2: - fixed typo in the commit message v3: - no changes --- drivers/gpu/drm/ci/gitlab-ci.yml | 2 +- drivers/gpu/drm/ci

[PATCH v3 03/10] drm/ci: add helper script update-xfails.py

2023-10-23 Thread Helen Koike
with the same name or if they are flakes) and execute: update-xfails.py https://gitlab.freedesktop.org/helen.fornazier/linux/-/pipelines/970661 git diff should show you that it updated files in xfails folder. Signed-off-by: Helen Koike Tested-by: Vignesh Raman Reviewed-by: David Heidelberg --- Hello

[PATCH v3 02/10] drm/ci: fix DEBIAN_ARCH and get amdgpu probing

2023-10-23 Thread Helen Koike
amdgpu driver wasn't loading because amdgpu firmware wasn't being installed in the rootfs due to the wrong DEBIAN_ARCH variable. rename ARCH to DEBIAN_ARCH also, so we don't have the confusing DEBIAN_ARCH, KERNEL_ARCH and ARCH. Signed-off-by: Helen Koike Reviewed-by: David Heidelberg --- v2

[PATCH v3 00/10] drm/ci: fixes and improvements

2023-10-23 Thread Helen Koike
flakes - update to latest version of mesa to get the patch that disables bare-metal retries - fix typos Helen Koike (10): drm/ci: uprev mesa version: fix container build & crosvm drm/ci: fix DEBIAN_ARCH and get amdgpu probing drm/ci: add helper script update-xfails.py drm/ci: uprev

[PATCH v3 01/10] drm/ci: uprev mesa version: fix container build & crosvm

2023-10-23 Thread Helen Koike
-by: Helen Koike [crosvm mesa update] Co-Developed-by: Vignesh Raman Signed-off-by: Vignesh Raman [v1 container build uprev] Tested-by: Jessica Zhang Acked-by: Jessica Zhang Reviewed-by: David Heidelberg --- v2: - update to an even newer version of mesa to integrate crosvm uprev v3: - update

Re: [PATCH] drm/doc: ci: Require more context for flaky tests

2023-10-19 Thread Helen Koike
On 19/10/2023 13:51, Helen Koike wrote: On 19/10/2023 06:46, Maxime Ripard wrote: Flaky tests can be very difficult to reproduce after the facts, which will make it even harder to ever fix. Let's document the metadata we agreed on to provide more context to anyone trying to address

[PATCH v2 9/9] drm/ci: docs: add step about how to request privileges

2023-10-19 Thread Helen Koike
Clarify the procedure developer must follow to request privileges to run tests on Freedesktop gitlab CI. This measure was added to avoid untrusted people to misuse the infrastructure. Signed-off-by: Helen Koike Reviewed-by: David Heidelberg --- v2: - fix typo in commit message

[PATCH v2 8/9] drm/ci: do not automatically retry on error

2023-10-19 Thread Helen Koike
Since the kernel doesn't use a bot like Mesa that requires tests to pass in order to merge the patches, leave it to developers and/or maintainers to manually retry. Suggested-by: Rob Clark Signed-off-by: Helen Koike Reviewed-by: David Heidelberg --- v2: - no changes --- drivers/gpu/drm/ci

[PATCH v2 7/9] drm/ci: export kernel config

2023-10-19 Thread Helen Koike
Export the resultant kernel config, making it easier to verify if the resultant config was correctly generated. Suggested-by: Rob Clark Signed-off-by: Helen Koike Acked-by: Dmitry Baryshkov Reviewed-by: David Heidelberg --- v2: - no changes --- drivers/gpu/drm/ci/build.sh | 1

[PATCH v2 6/9] drm/ci: add subset-1-gfx to LAVA_TAGS and adjust shards

2023-10-19 Thread Helen Koike
developers from getting their patches merged. Signed-off-by: Helen Koike Reviewed-by: David Heidelberg --- v2: - add subset-1-gfx tag to LAVA_TAGS - update commit message --- drivers/gpu/drm/ci/gitlab-ci.yml | 2 +- drivers/gpu/drm/ci/test.yml | 23 ++- 2 files changed, 15

[PATCH v2 5/9] drm/ci: clean up xfails (specially flakes list)

2023-10-19 Thread Helen Koike
Since the script that collected the list of the expectation files was bogus and placing test to the flakes list incorrectly, restart the expectation files with the correct script. This reduces a lot the number of tests in the flakes list. Signed-off-by: Helen Koike Reviewed-by: David Heidelberg

[PATCH v2 4/9] drm/ci: uprev IGT and make sure core_getversion is run

2023-10-19 Thread Helen Koike
was ok. Uprev IGT to include this modification and include core_getversion test in all the shards. Signed-off-by: Helen Koike Reviewed-by: David Heidelberg --- v2: - fixed typo in the commit message --- drivers/gpu/drm/ci/gitlab-ci.yml | 2 +- drivers/gpu/drm/ci/igt_runner.sh | 31

[PATCH v2 3/9] drm/ci: add helper script update-xfails.py

2023-10-19 Thread Helen Koike
with the same name or if they are flakes) and execute: update-xfails.py https://gitlab.freedesktop.org/helen.fornazier/linux/-/pipelines/970661 git diff should show you that it updated files in xfails folder. Signed-off-by: Helen Koike Tested-by: Vignesh Raman Reviewed-by: David Heidelberg --- Hello

[PATCH v2 2/9] drm/ci: fix DEBIAN_ARCH and get amdgpu probing

2023-10-19 Thread Helen Koike
amdgpu driver wasn't loading because amdgpu firmware wasn't being installed in the rootfs due to the wrong DEBIAN_ARCH variable. rename ARCH to DEBIAN_ARCH also, so we don't have the confusing DEBIAN_ARCH, KERNEL_ARCH and ARCH. Signed-off-by: Helen Koike Reviewed-by: David Heidelberg --- v2

[PATCH v2 1/9] drm/ci: uprev mesa version: fix container build & crosvm

2023-10-19 Thread Helen Koike
-by: Helen Koike [crosvm mesa update] Co-Developed-by: Vignesh Raman Signed-off-by: Vignesh Raman [v1 container build uprev] Tested-by: Jessica Zhang Acked-by: Jessica Zhang Reviewed-by: David Heidelberg --- v2: - update to an even newer version of mesa to integrate crosvm uprev --- drivers/gpu

[PATCH v2 0/9] drm/ci: fixes and improvements

2023-10-19 Thread Helen Koike
...@linaro.org/ [PATCH] drm/ci: Enable CONFIG_BACKLIGHT_CLASS_DEVICE https://patchwork.kernel.org/project/dri-devel/patch/20231002164715.157298-1-robdcl...@gmail.com/ Helen Koike (9): drm/ci: uprev mesa version: fix container build & crosvm drm/ci: fix DEBIAN_ARCH and get amdgpu probing drm/ci:

  1   2   3   >