Re: [Intel-gfx] [PATCH] drm/i915/selftests: Hold rpm for unparking

2018-08-09 Thread Mika Kuoppala
Chris Wilson writes: > The call to i915_gem_unpark() checks that we hold a rpm wakeref before > taking a long term wakeref for i915->gt.awake. We should therefore make > sure we do hold the wakeref when directly calling unpark to disable > the retire worker. > > Fixes: 932cac10c8fb ("drm/i915/sel

Re: [Intel-gfx] [PATCH] drm/i915/selftests: Hold rpm for unparking

2018-08-09 Thread Chris Wilson
Quoting Mika Kuoppala (2018-08-09 11:40:35) > Chris Wilson writes: > > > The call to i915_gem_unpark() checks that we hold a rpm wakeref before > > taking a long term wakeref for i915->gt.awake. We should therefore make > > sure we do hold the wakeref when directly calling unpark to disable > > t

Re: [Intel-gfx] [PATCH] drm/i915/selftests: Hold rpm for unparking

2018-08-09 Thread Mika Kuoppala
Chris Wilson writes: > The call to i915_gem_unpark() checks that we hold a rpm wakeref before > taking a long term wakeref for i915->gt.awake. We should therefore make > sure we do hold the wakeref when directly calling unpark to disable > the retire worker. We as a caller need to hold the rpm a

[Intel-gfx] [PATCH] drm/i915/selftests: Hold rpm for unparking

2018-08-08 Thread Chris Wilson
The call to i915_gem_unpark() checks that we hold a rpm wakeref before taking a long term wakeref for i915->gt.awake. We should therefore make sure we do hold the wakeref when directly calling unpark to disable the retire worker. Fixes: 932cac10c8fb ("drm/i915/selftests: Prevent background reaping