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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Apr 06, 2022 at 08:36:32AM -0700, Lucas De Marchi wrote:
> 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@xxxxxxxxx>
> > > Cc: Lucas De Marchi <lucas.demarchi@xxxxxxxxx>
> > > Cc: Dave Airlie <airlied@xxxxxxxxxx>
> > > Cc: Daniel Vetter <daniel.vetter@xxxxxxxxx>
> > > Cc: Jason Ekstrand <jason@xxxxxxxxxxxxxx>
> > > Cc: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx>
> > 
> > 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

Yes, please drop ADL*/RKL patch in core-for-CI. 

--
Zbigniew



[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux