On Tue, Feb 02, 2016 at 11:06:27AM +0000, Tvrtko Ursulin wrote: > From: Tvrtko Ursulin <[email protected]> > > 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 <[email protected]> > Cc: Daniel Vetter <[email protected]>
Makes sense, Reviewed-by: Chris Wilson <[email protected]> -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/intel-gfx
