[Intel-gfx] [PATCH] [topic/core-for-CI] Revert "drm/i915/dg2: Add relocation exception"

Lucas De Marchi lucas.demarchi at intel.com
Wed Apr 6 15:36:32 UTC 2022


On Thu, Mar 31, 2022 at 10:16:02AM -0700, Lucas De Marchi wrote:
>On Wed, Mar 23, 2022 at 06:46:38PM +0100, Zbigniew Kempczyński wrote:
>>This reverts commit 904ebf2ba89edaeba5c7c10540e43dba63541dc6.
>>
>>Failures on dg2 tests were caused by invalid alignment when local memory
>>was in use. Changes which adopt alignment according to gen were already
>>merged in IGT so lets revert relocation temporary enabler for dg2. Keeping
>>it is a little bit problematic for IGT because on premerge we would see
>>results with kernel which supports relocation. To see no-relocation
>>results we need to send disabler (like this revert), point IGT with
>>"Test-with" tag what is cumbersome and time consuming so lets do this
>>permanently. If we will see some failures they need to be fixed instead
>>of keeping relocation enabler.
>>
>>Signed-off-by: Zbigniew Kempczyński <zbigniew.kempczynski at intel.com>
>>Cc: Lucas De Marchi <lucas.demarchi at intel.com>
>>Cc: Dave Airlie <airlied at redhat.com>
>>Cc: Daniel Vetter <daniel.vetter at intel.com>
>>Cc: Jason Ekstrand <jason at jlekstrand.net>
>>Cc: Rodrigo Vivi <rodrigo.vivi at intel.com>
>
>thanks. I double checked BAT and things seem to be equivalent now
>without that hack. I removed it from topic/core-for-CI.
>
>How are we with the igt tests executed in full run?

ping?

Also, we still have this exception for ADL-P, ADL-S and RKL. Should we
go ahead and try removing them?

Lucas De Marchi


More information about the Intel-gfx mailing list