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? -- Jesse Barnes, Intel Open Source Technology Center -- To unsubscribe from this list: send the line "unsubscribe stable" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html