Re: [Intel-gfx] [PATCH 3/3] drm/i915: Remove superfluous locking around userfault_list

2016-10-13 Thread Daniel Vetter
On Tue, Oct 11, 2016 at 03:37:59PM +0100, Chris Wilson wrote: > Now that we have reduced the access to the list to either (a) under the > struct_mutex whilst holding the RPM wakeref (so that concurrent writers to > the list are serialised by struct_mutex) and (b) under the atomic > runtime suspend

[Intel-gfx] [PATCH 3/3] drm/i915: Remove superfluous locking around userfault_list

2016-10-11 Thread Chris Wilson
Now that we have reduced the access to the list to either (a) under the struct_mutex whilst holding the RPM wakeref (so that concurrent writers to the list are serialised by struct_mutex) and (b) under the atomic runtime suspend (which cannot run concurrently with any other accessor due to the atom