[Intel-gfx] [PATCH 4/5] drm/i915: Always flush tiling changes before accessing through the GTT
chris at chris-wilson.co.uk
Sat Apr 14 14:59:25 PDT 2012
On Sat, 14 Apr 2012 23:54:52 +0200, Daniel Vetter <daniel at ffwll.ch> wrote:
> On Sat, Apr 14, 2012 at 09:55:50AM +0100, Chris Wilson wrote:
> > As we defer updating the fence register from set-tiling to the point of
> > use, we need to declare every access through the GTT as either fenced or
> > unfenced.
> > This patches fixes up a couple of freshly introduced GTT accesses which
> > missed the fence flush.
> > Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> Actually no, this set_to_gtt is actually just a flush+wait_rendering
> because the shmem pwrite/pread afters does clever in-line clflushing.
> Abuse of interfaces, I know ...
Probably best to leave a note, so the next time I do a grep looking for
use of set-to-gtt without a following fence declaration, I don't submit
another patch. :-)
Chris Wilson, Intel Open Source Technology Centre
More information about the Intel-gfx