On Tue, Feb 02, 2016 at 11:06:27AM +0000, Tvrtko Ursulin wrote: > From: Tvrtko Ursulin <tvrtko.ursu...@intel.com> > > 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.ursu...@intel.com> > Cc: Daniel Vetter <daniel.vet...@ffwll.ch>
Makes sense, Reviewed-by: Chris Wilson <ch...@chris-wilson.co.uk> -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx
