Re: [Intel-gfx] [PATCH 7/7] drm/i915: Stop parking the signaler around reset

2018-05-16 Thread Chris Wilson
Quoting Tvrtko Ursulin (2018-05-16 10:49:34) > > On 16/05/2018 10:25, Chris Wilson wrote: > > Quoting Tvrtko Ursulin (2018-05-16 10:15:34) > >> > >> On 16/05/2018 07:47, Chris Wilson wrote: > >>> We cannot call kthread_park() from softirq context, so let's avoid it > >>> entirely during the reset.

Re: [Intel-gfx] [PATCH 7/7] drm/i915: Stop parking the signaler around reset

2018-05-16 Thread Tvrtko Ursulin
On 16/05/2018 10:25, Chris Wilson wrote: Quoting Tvrtko Ursulin (2018-05-16 10:15:34) On 16/05/2018 07:47, Chris Wilson wrote: We cannot call kthread_park() from softirq context, so let's avoid it entirely during the reset. We wanted to suspend the signaler so that it would not mark a request

Re: [Intel-gfx] [PATCH 7/7] drm/i915: Stop parking the signaler around reset

2018-05-16 Thread Chris Wilson
Quoting Tvrtko Ursulin (2018-05-16 10:15:34) > > On 16/05/2018 07:47, Chris Wilson wrote: > > We cannot call kthread_park() from softirq context, so let's avoid it > > entirely during the reset. We wanted to suspend the signaler so that it > > would not mark a request as complete at the same time

Re: [Intel-gfx] [PATCH 7/7] drm/i915: Stop parking the signaler around reset

2018-05-16 Thread Tvrtko Ursulin
On 16/05/2018 07:47, Chris Wilson wrote: We cannot call kthread_park() from softirq context, so let's avoid it entirely during the reset. We wanted to suspend the signaler so that it would not mark a request as complete at the same time as we marked it as being in error. Instead of parking the s