Re: [Intel-gfx] [PATCH v3 08/12] drm/i915: Fix struct mutex vs. RPS lock inversion

2016-02-11 Thread Chris Wilson
On Tue, Feb 02, 2016 at 02:46:58PM +, Tvrtko Ursulin wrote: > From: Tvrtko Ursulin > > RPS lock must be taken before the struct_mutex to avoid > locking inversion. So stop grabbing it for the whole > powersave initialization and instead only take it during > the sections which need it. > > A

[Intel-gfx] [PATCH v3 08/12] drm/i915: Fix struct mutex vs. RPS lock inversion

2016-02-02 Thread Tvrtko Ursulin
From: Tvrtko Ursulin RPS lock must be taken before the struct_mutex to avoid locking inversion. So stop grabbing it for the whole powersave initialization and instead only take it during the sections which need it. Also, struct_mutex is not needed any more since dedicated RPS lock was added in: