Re: [Intel-gfx] [PATCH 1/2] drm/i915: Break busywaiting for requests on pending signals

2015-11-16 Thread Tvrtko Ursulin
On 16/11/15 11:22, Chris Wilson wrote: On Mon, Nov 16, 2015 at 09:54:10AM +, Tvrtko Ursulin wrote: Hi, On 15/11/15 13:32, Chris Wilson wrote: The busywait in __i915_spin_request() does not respect pending signals and so may consume the entire timeslice for the task instead of returning t

Re: [Intel-gfx] [PATCH 1/2] drm/i915: Break busywaiting for requests on pending signals

2015-11-16 Thread Chris Wilson
On Mon, Nov 16, 2015 at 09:54:10AM +, Tvrtko Ursulin wrote: > > Hi, > > On 15/11/15 13:32, Chris Wilson wrote: > >The busywait in __i915_spin_request() does not respect pending signals > >and so may consume the entire timeslice for the task instead of > >returning to userspace to handle the s

Re: [Intel-gfx] [PATCH 1/2] drm/i915: Break busywaiting for requests on pending signals

2015-11-16 Thread Tvrtko Ursulin
Hi, On 15/11/15 13:32, Chris Wilson wrote: The busywait in __i915_spin_request() does not respect pending signals and so may consume the entire timeslice for the task instead of returning to userspace to handle the signal. Obviously correct to break the spin, but if spending a jiffie to react

[Intel-gfx] [PATCH 1/2] drm/i915: Break busywaiting for requests on pending signals

2015-11-15 Thread Chris Wilson
The busywait in __i915_spin_request() does not respect pending signals and so may consume the entire timeslice for the task instead of returning to userspace to handle the signal. Fixes regression from commit 2def4ad99befa25775dd2f714fdd4d92faec6e34 [v4.2] Author: Chris Wilson Date: Tue Apr 7 1