On Thu, 2015-07-30 at 08:54 +0200, Maarten Lankhorst wrote: > Op 29-07-15 om 15:26 schreef Ander Conselvan De Oliveira: > > On Mon, 2015-07-27 at 14:35 +0200, Maarten Lankhorst wrote: > > > Now that everything's atomic, checking encoder->base.crtc is enough. > > Don't you need to check encoder->base.crtc->state->active too? > Not sure, I think stealing a encoder being bound to any crtc is probably good enough reason to > warn. > We probably don't hold the right lock to dereference crtc->state. Fair enough. Add that to the commit message and you can add Reviewed-by: Ander Conselvan de Oliveira <conselvan2@xxxxxxxxx> _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
