On Tue, Feb 07, 2017 at 04:24:25PM +0200, Joonas Lahtinen wrote: > On pe, 2017-02-03 at 12:57 +0000, Chris Wilson wrote: > > @@ -2024,8 +2024,16 @@ void i915_gem_runtime_suspend(struct drm_i915_private *dev_priv) > > for (i = 0; i < dev_priv->num_fence_regs; i++) { > > struct drm_i915_fence_reg *reg = &dev_priv->fence_regs[i]; > > > > - if (WARN_ON(reg->pin_count)) > > - continue; > > + /* Ideally we want to assert that the fence register is not > > + * live at this point (i.e. that no piece of code will is > > s/will is/will be/ > > With that, > > Reviewed-by: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx> Fixed up and applied, thanks. -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx