On Tue, Feb 02, 2016 at 11:06:27AM +0000, Tvrtko Ursulin wrote: > From: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx> > > Code does read-modify-write but the read was outside the lock. > > It is fine since the caller holds struct mutex, but if we > correct this we open up the opportunity for decreasing the > mutex duration time since the call to ironlake_enable_drps > does not need it any longer since it is covered by the > mchdev_lock lock. > > Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx> > Cc: Daniel Vetter <daniel.vetter@xxxxxxxx> Makes sense, Reviewed-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
