lkml.org 
[lkml]   [2022]   [May]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [RFC v3] drm/msm: Add initial ci/ subdirectory
From
Date
On 5/11/22 3:20 PM, Rob Clark wrote:
> On Tue, May 10, 2022 at 11:15 PM Tomeu Vizoso
> <tomeu.vizoso@collabora.com> wrote:
>>
>> And use it to store expectations about what the drm/msm driver is
>> supposed to pass in the IGT test suite.
>>
>> Also include a configuration file that points to the out-of-tree CI
>> scripts.
>>
>> By storing the test expectations along the code we can make sure both
>> stay in sync with each other, and so we can know when a code change
>> breaks those expectations.
>>
>> This will allow all contributors to drm/msm to reuse the infrastructure
>> already in gitlab.freedesktop.org to test the driver on several
>> generations of the hardware.
>>
>> v2:
>> - Fix names of result expectation files to match SoC
>> - Don't execute tests that are going to skip on all boards
>>
>> v3:
>> - Remove tracking of dmesg output during test execution
>>
>> Signed-off-by: Tomeu Vizoso <tomeu.vizoso@collabora.com>
>> ---
>> Documentation/gpu/msm_automated_testing.rst | 70 +++++++++
>> drivers/gpu/drm/msm/ci/gitlab-ci.yml | 11 ++
>> drivers/gpu/drm/msm/ci/msm.testlist | 144 ++++++++++++++++++
>> .../gpu/drm/msm/ci/msm_apq8016_results.txt | 141 +++++++++++++++++
>> .../gpu/drm/msm/ci/msm_apq8096_results.txt | 141 +++++++++++++++++
>> drivers/gpu/drm/msm/ci/msm_sc7180_results.txt | 142 +++++++++++++++++
>> drivers/gpu/drm/msm/ci/msm_sdm845_results.txt | 142 +++++++++++++++++
>> 7 files changed, 791 insertions(+)
>> create mode 100644 Documentation/gpu/msm_automated_testing.rst
>> create mode 100644 drivers/gpu/drm/msm/ci/gitlab-ci.yml
>> create mode 100644 drivers/gpu/drm/msm/ci/msm.testlist
>> create mode 100644 drivers/gpu/drm/msm/ci/msm_apq8016_results.txt
>> create mode 100644 drivers/gpu/drm/msm/ci/msm_apq8096_results.txt
>> create mode 100644 drivers/gpu/drm/msm/ci/msm_sc7180_results.txt
>> create mode 100644 drivers/gpu/drm/msm/ci/msm_sdm845_results.txt
>>
>
> [snip]
>
>> diff --git a/drivers/gpu/drm/msm/ci/gitlab-ci.yml b/drivers/gpu/drm/msm/ci/gitlab-ci.yml
>> new file mode 100644
>> index 000000000000..9b7caa7fcab2
>> --- /dev/null
>> +++ b/drivers/gpu/drm/msm/ci/gitlab-ci.yml
>> @@ -0,0 +1,11 @@
>> +variables:
>> + # Change this to use your fork of drm-ci
>> + DRM_CI_PROJECT_PATH: &drm-ci-project-path gfx-ci/drm-ci
>> + DRM_CI_COMMIT_SHA: &drm-ci-commit-sha msm
>
> btw, shouldn't we list an exact commit sha (or tag) for drm-ci, or am
> I overlooking something? We want to be able to uprev the drm-ci
> version (and therefore igt version) in sync with any necessary updates
> to the expectations

You are right, that was me being lazy and using a branch name to not
have to update this every time.

But this should better be a commit sha or tag. Will do it on v4.

Cheers,

Tomeu

\
 
 \ /
  Last update: 2022-05-11 16:05    [W:0.192 / U:0.128 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site