On pe, 2015-08-14 at 13:49 +0100, Chris Wilson wrote: > On Fri, Aug 14, 2015 at 03:38:55PM +0300, Imre Deak wrote: > > This is a v2 of [1]. Since v1 the HW team confirmed that there is an > > HW issue in A steppings with the GPU/CPU snoop logic, which explains why > > we need this workaround. > > I've been testing this extensively, and I do believe we can use clflush > for all gen - it is a measurable improvement over using the > heavyweight/locked read of ACTHD, and none of the coherency tests have > thrown any warnings (over a period of a couple of months now). Ok. Do you mean only places where there is already the ACTHD w/a, or to extend it also to other platforms? Imo doing this as a follow-up to this patchset would make sense, to keep platform specific changes separate. --Imre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
