[igt-dev] [PATCH i-g-t] tests/amdgpu: Add test for AMD DP DSC

[Petri Latvala]

On Fri, Nov 26, 2021 at 11:43:52AM +0200, Petri Latvala wrote:
> On Thu, Nov 25, 2021 at 01:58:14PM -0500, Rodrigo Siqueira Jordao wrote:
> > 
> > 
> > On 2021-11-25 10:57 a.m., Rodrigo Siqueira Jordao wrote:
> > > 
> > > 
> > > On 2021-11-25 2:11 a.m., Petri Latvala wrote:
> > > > On Thu, Nov 25, 2021 at 01:01:04AM +0200, Vudum, Lakshminarayana wrote:
> > > > > @Rodrigo Siqueira I don't have the possibility to re-run the
> > > > > series. @Latvala, Petri Can you help here?
> > > > > 
> > > > > Lakshmi.
> > > > 
> > > > Unfortunately patchwork got confused by your emails and thinks you've
> > > > sent a patch, so the re-run button just reattempts the incorrect
> > > > patch.
> > > > 
> > > > Rodrigo, please re-send this patch.
> > > > 
> > > 
> > > 
> > > Hi Petri/Lakshmi,
> > > 
> > > Thanks a lot for your help.
> > > 
> > > I submitted a V2:
> > > https://patchwork.freedesktop.org/series/97108/
> > > 
> > > Thanks.
> > 
> > Lakshmi/Petri,
> > 
> > For some reason, it looks like that the new version is still assigned to
> > Lakshmi in the patchwork
> 
> Is the patch data itself correct?
> 
> > , and it has some weird failures (see below). Any
> > idea? Do I need to resubmit the patch with a different subject?
> > 
> > Thanks
> > 
> > Gitlab.Pipeline:
> > 
> > Pipeline status: FAILED.
> > 
> > see https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/pipelines/453561 for
> > the overview.
> > 
> > containers:igt has failed
> > (https://gitlab.freedesktop.org/gfx-ci/igt-ci-tags/-/jobs/16168063):
> >   STEP 2: COPY installdir/opt/igt /opt/igt
> >   1895d20f5469210af8281acac7769fa51efbde5ca3931bd457af2c2834e1f7ee
> >   STEP 3: COPY .gitlab-ci/docker-help.sh /usr/local/bin/docker-help.sh
> >   b743d155d30d20100edf9b2df89664275b2ea33d402f9df69cf5a1824a781b01
> >   STEP 4: ENV PATH="/opt/igt/bin:${PATH}"
> >   5eb0330fdfca164d1ef769d62f1d3c1a539a09a81da3ca01273c4e7afe2ee974
> >   STEP 5: ENV
> > LD_LIBRARY_PATH="/opt/igt/lib:/opt/igt/lib64:${LD_LIBRARY_PATH}"
> >   b4ea9d4538270ad267ff798de51f4b7bed3e1ae27f9d3df8cea163e78acc9d3e
> >   STEP 6: ENV IGT_TEST_ROOT="/opt/igt/libexec/igt-gpu-tools"
> >   ae060122c9eeab50d82bbafa097e8cd72bccdc8e8cb00b42de1b38531854d099
> >   STEP 7: CMD docker-help.sh
> >   STEP 8: COMMIT registry.freedesktop.org/gfx-ci/igt-ci-tags/igt:commit-dcfa579ba0a38131881dfd441d558b54d4cd3183
> >   fcc29304314f515e7c8612f480ef56419de32ac5b5917000b434768cf94b6793
> >   Getting image source signatures
> >   Copying blob
> > sha256:508381dd82aeb851da958db0528f2893285a9392bd0787020d3569a61fcdd513
> >   Copying blob
> > sha256:3504a8337d3b7eaabfd338d6a971d02441d28ebef8bd81f730d87f936ed58079
> >   Copying blob
> > sha256:f3f4119da36f8525b923a310c4b1f15579ca311a753c9973ef966cdb2a71a4e8
> >   section_end:1637863318:step_script
> >   ERROR: Job failed: execution took longer than 1h0m0s seconds
> 
> You can ignore this error. gitlab had issues talking to its docker
> registry.
> 
> 
> -- 
> Petri Latvala
> 
> 
> 
> 
> > 
> > CI.BAT
> > 
> > IGT changes
> > Possible regressions
> > igt at kms_addfb_basic@addfb25-bad-modifier:
> > 
> > fi-rkl-11600: PASS -> FAIL +36 similar issues
> > igt at kms_addfb_basic@basic-x-tiled-legacy:
> > 
> > fi-rkl-guc: PASS -> FAIL +36 similar issues
> > igt at kms_addfb_basic@bo-too-small:
> > 
> > fi-tgl-1115g4: PASS -> FAIL +37 similar issues
> > igt at kms_addfb_basic@bo-too-small-due-to-tiling:
> > 
> > fi-icl-u2: PASS -> FAIL +46 similar issues
> > igt at kms_busy@basic:
> > 
> > fi-icl-u2: NOTRUN -> FAIL
> > igt at kms_pipe_crc_basic@nonblocking-crc-pipe-a:
> > 
> > fi-tgl-u2: NOTRUN -> FAIL +53 similar issues
> > igt at prime_vgem@basic-fence-flip:
> > 
> > fi-icl-u2: PASS -> CRASH +2 similar issues
> > 
> > fi-tgl-u2: NOTRUN -> CRASH
> > 
> > fi-rkl-11600: PASS -> CRASH
> > 
> > fi-rkl-guc: PASS -> CRASH
> > 
> > fi-tgl-1115g4: PASS -> CRASH
> > 


As for these failures, they are caused by the change to
igt_atomic_properties array in igt_kms.h. You added a prop there but
didn't change any code that loops through the whole array. Look for
IGT_NUM_CRTC_PROPS.


-- 
Petri Latvala