On Wed, Jul 17, 2013 at 08:34:48AM -0700, Jesse Barnes wrote: > On Wed, 17 Jul 2013 14:51:28 +0200 > Daniel Vetter <daniel.vetter@xxxxxxxx> wrote: > > > + if (reg->obj) { > > + i915_gem_object_update_fence(reg->obj, reg, > > + reg->obj->tiling_mode); > > + } else { > > + i915_gem_write_fence(dev, i, NULL); > > + } > > Why do we look at reg->obj here? Can it be NULL? Or would > reg->obj->tiling_mode != I915_TILING_NONE do the same thing? Yes. This is the array of fence registers, irrespective of whether an object is using one. -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx