On Wed, Feb 13, 2013 at 09:56:05PM +0200, Imre Deak wrote: > As explained by Chris Wilson gem objects in stolen memory are always > coherent with the GPU so we don't need to ever flush the CPU caches for > these. > > This fixes a breakage - at least with the compact sg patches applied - > during the resume/restore gtt mappings path, when we tried to clflush an > FB object in stolen memory, but since stolen objects don't have backing > pages we passed an invalid page pointer to drm_clflush_page(). > > Signed-off-by: Imre Deak <[email protected]>
To the best of my knowledge, this is correct: Reviewed-by: Chris Wilson <[email protected]> Though the stolen framework landed in 3.8, tough call as to whether this should be in 3.8 as well given the backported clflush fix... I guess we are simply too late, so drm-intel-next + Cc: [email protected] Imre do you mind digging up the sha of both the introduction of stolen and the clflush of unbounded upon resume? -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/intel-gfx
