On Wed, Jul 17, 2013 at 08:34:48AM -0700, Jesse Barnes wrote:
> On Wed, 17 Jul 2013 14:51:28 +0200
> Daniel Vetter <[email protected]> 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
--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to