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

[Vignesh Raman <vignesh.raman@collabora.com>]

Hi Maíra,

On 19/02/24 14:22, Vignesh Raman wrote:
> Hi Maíra,
> 
> On 10/02/24 23:50, 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 on those hardware.
>>>>
>>>> Signed-off-by: Vignesh Raman <vignesh.raman@collabora.com>
>>>> ---
>>>>
>>>> v3:
>>>>    - New patch in series to uprev IGT and update testlist.
>>>>
>>>> ---
>>>>   drivers/gpu/drm/ci/gitlab-ci.yml              |   2 +-
>>>>   drivers/gpu/drm/ci/igt_runner.sh              |  12 +-
>>>>   drivers/gpu/drm/ci/testlist-amdgpu.txt        | 151 
>>>> ++++++++++++++++++
>>>>   drivers/gpu/drm/ci/testlist-msm.txt           |  50 ++++++
>>>>   drivers/gpu/drm/ci/testlist-panfrost.txt      |  17 ++
>>>>   drivers/gpu/drm/ci/testlist-v3d.txt           |  73 +++++++++
>>>>   drivers/gpu/drm/ci/testlist-vc4.txt           |  49 ++++++
>>>>   drivers/gpu/drm/ci/testlist.txt               | 100 ++++--------
>>>>   .../gpu/drm/ci/xfails/amdgpu-stoney-fails.txt |  24 ++-
>>>>   .../drm/ci/xfails/amdgpu-stoney-flakes.txt    |   9 +-
>>>>   .../gpu/drm/ci/xfails/amdgpu-stoney-skips.txt |  10 +-
>>>>   11 files changed, 427 insertions(+), 70 deletions(-)
>>>>   create mode 100644 drivers/gpu/drm/ci/testlist-amdgpu.txt
>>>>   create mode 100644 drivers/gpu/drm/ci/testlist-msm.txt
>>>>   create mode 100644 drivers/gpu/drm/ci/testlist-panfrost.txt
>>>>   create mode 100644 drivers/gpu/drm/ci/testlist-v3d.txt
>>>>   create mode 100644 drivers/gpu/drm/ci/testlist-vc4.txt
>>>>
>>>> diff --git a/drivers/gpu/drm/ci/gitlab-ci.yml 
>>>> b/drivers/gpu/drm/ci/gitlab-ci.yml
>>>> index bc8cb3420476..e2b021616a8e 100644
>>>> --- a/drivers/gpu/drm/ci/gitlab-ci.yml
>>>> +++ b/drivers/gpu/drm/ci/gitlab-ci.yml
>>>> @@ -5,7 +5,7 @@ variables:
>>>>     UPSTREAM_REPO: git://anongit.freedesktop.org/drm/drm
>>>>     TARGET_BRANCH: drm-next
>>>> -  IGT_VERSION: d2af13d9f5be5ce23d996e4afd3e45990f5ab977
>>>> +  IGT_VERSION: b0cc8160ebdc87ce08b7fd83bb3c99ff7a4d8610
>>>>     DEQP_RUNNER_GIT_URL: 
>>>> https://gitlab.freedesktop.org/anholt/deqp-runner.git
>>>>     DEQP_RUNNER_GIT_TAG: v0.15.0
>>>> diff --git a/drivers/gpu/drm/ci/igt_runner.sh 
>>>> b/drivers/gpu/drm/ci/igt_runner.sh
>>>> index f001e015d135..2fd09b9b7cf6 100755
>>>> --- a/drivers/gpu/drm/ci/igt_runner.sh
>>>> +++ b/drivers/gpu/drm/ci/igt_runner.sh
>>>> @@ -64,10 +64,20 @@ if ! grep -q "core_getversion" 
>>>> /install/testlist.txt; then
>>>>   fi
>>>>   set +e
>>>> +if [ "$DRIVER_NAME" = "amdgpu" ]; then
>>>> +    TEST_LIST="/install/testlist-amdgpu.txt"
>>>> +elif [ "$DRIVER_NAME" = "msm" ]; then
>>>> +    TEST_LIST="/install/testlist-msm.txt"
>>>> +elif [ "$DRIVER_NAME" = "panfrost" ]; then
>>>> +    TEST_LIST="/install/testlist-panfrost.txt"
>>>> +else
>>>> +    TEST_LIST="/install/testlist.txt"
>>>> +fi
>>>> +
>>>
>>> Isn't V3D and VC4 testlists missing?
> 
> Yes. We need to add ci jobs to test v3d/vc4. The initial idea was just 
> to split the testlist per driver and add vc4/v3d tests so that it can be 
> used in future. I will add the jobs as part of v4.

To include RPi jobs, we need to tweak mesa-ci to pass kernel and dtb and 
update mesa-ci in drm-ci. So will send this as a seperate patch/series.

Regards,
Vignesh