On ke, 2016-07-27 at 12:14 +0100, Chris Wilson wrote: > > /* Delay flushing when rings are still busy.*/ > - mutex_lock(&dev_priv->fb_tracking.lock); > + spin_lock(&dev_priv->fb_tracking.lock); > frontbuffer_bits &= ~dev_priv->fb_tracking.busy_bits; > - mutex_unlock(&dev_priv->fb_tracking.lock); > + spin_unlock(&dev_priv->fb_tracking.lock); > > - if (!frontbuffer_bits) > + if (frontbuffer_bits == 0) Ugh, there are two styles within same patch already, maybe go with ! here too. That unified; Reviewed-by: Joonas Lahtien <joonas.lahtinen@xxxxxxxxxxxxxxx> Regards, Joonas -- Joonas Lahtinen Open Source Technology Center Intel Corporation _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
