[Intel-gfx] ✗ Fi.CI.BAT: failure for Revert "drm/i915/dg2: Add relocation exception" (rev2)

[Lucas De Marchi]

On Thu, Mar 24, 2022 at 09:00:52AM +0100, Zbigniew Kempczyński wrote:
>On Wed, Mar 23, 2022 at 06:51:17PM +0000, Patchwork wrote:
>>    Patch Details
>>
>>    Series:  Revert "drm/i915/dg2: Add relocation exception" (rev2)
>>    URL:     https://patchwork.freedesktop.org/series/101669/
>>    State:   failure
>>    Details: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22660/index.html
>>
>>            CI Bug Log - changes from CI_DRM_11398 -> Patchwork_22660
>>
>> Summary
>>
>>    FAILURE
>>
>>    Serious unknown changes coming with Patchwork_22660 absolutely need to be
>>    verified manually.
>>
>>    If you think the reported changes have nothing to do with the changes
>>    introduced in Patchwork_22660, please notify your bug team to allow them
>>    to document this new failure mode, which will reduce false positives in
>>    CI.
>>
>>    External URL:
>>    https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_22660/index.html
>>
>> Participating hosts (45 -> 42)
>>
>>    Additional (5): bat-dg2-8 bat-dg2-9 fi-kbl-8809g bat-rpls-1 bat-jsl-1
>>    Missing (8): fi-kbl-soraka shard-tglu fi-hsw-4200u bat-adlm-1 fi-bsw-cyan
>>    fi-ctg-p8600 shard-rkl fi-bdw-samus
>>
>> Possible new issues
>>
>>    Here are the unknown changes that may have been introduced in
>>    Patchwork_22660:
>>
>>   IGT changes
>>
>>     Possible regressions
>>
>>      * igt at i915_selftest@live at hangcheck:
>>           * fi-hsw-4770: PASS -> INCOMPLETE
>
>Unrelated to the change.
>
>>
>>     Suppressed
>>
>>    The following results come from untrusted machines, tests, or statuses.
>>    They do not affect the overall result.
>>
>>      * igt at gem_busy@busy at all:
>>
>>           * {bat-dg2-8}: NOTRUN -> INCOMPLETE
>>      * igt at gem_exec_gttfill@basic:
>>
>>           * {bat-dg2-9}: NOTRUN -> SKIP
>>      * igt at gem_exec_suspend@basic-s0 at smem:
>>
>>           * {bat-dg2-9}: NOTRUN -> FAIL +7 similar issues
>
>For this one I got success on no-reloc code:
>
>./gem_exec_suspend --run basic-S0
>IGT-Version: 1.26-NO-GIT (x86_64) (Linux: 5.17.0+ x86_64)
>Starting subtest: basic-S0
>Starting dynamic subtest: smem
>[cmd] rtcwake: assuming RTC uses UTC ...
>rtcwake: wakeup from "freeze" using /dev/rtc0 at Thu Mar 24 07:56:03 2022
>Dynamic subtest smem: SUCCESS (2.817s)
>Starting dynamic subtest: lmem0
>[cmd] rtcwake: assuming RTC uses UTC ...
>rtcwake: wakeup from "freeze" using /dev/rtc0 at Thu Mar 24 07:56:20 2022
>Dynamic subtest lmem0: SUCCESS (3.184s)
>Subtest basic-S0: SUCCESS (6.001s)
>
>@Lucas - please consider reverting relocations for dg2. This will speed
>up our work and I've fixed yesterday all potential problems with no-reloc
>for BAT on dg2.

Please trigger a re-test on this to compare to a recent drm-tip.
It would be bad if this then blocks the execution of other tests and we
drop the pass and execution rates.

Lucas De Marchi