[PATCH v2 12/14] drm/i915: Refactor direct GPU reset from request waiters

Chris Wilson
2017-02-14T09:54:30Z

Combine the common code for the pair of waiters into a single function.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
---
drivers/gpu/drm/i915/i915_gem_request.c | 21 ++++++++++++---------
1 file changed, 12 insertions(+), 9 deletions(-)

diff --git a/drivers/gpu/drm/i915/i915_gem_request.c b/drivers/gpu/drm/i915/i915_gem_request.c
index bd921c4b0100..6f7beaeda3ff 100644
--- a/drivers/gpu/drm/i915/i915_gem_request.c
+++ b/drivers/gpu/drm/i915/i915_gem_request.c
@@ -1006,6 +1006,16 @@ bool __i915_spin_request(const struct drm_i915_gem_request *req,
return false;
}

+static bool __i915_reset_request(struct drm_i915_gem_request *request)
+{
+ if (likely(!i915_reset_in_progress(&request->i915->gpu_error)))
+ return false;
+
+ __set_current_state(TASK_RUNNING);
+ i915_reset(request->i915);
+ return true;
+}
+
/**
* i915_wait_request - wait until execution of request has finished
* @req: the request to wait upon
@@ -1069,11 +1079,8 @@ long i915_wait_request(struct drm_i915_gem_request *req,
break;

if (flags & I915_WAIT_LOCKED &&
- i915_reset_in_progress(&req->i915->gpu_error)) {
- __set_current_state(TASK_RUNNING);
- i915_reset(req->i915);
+ __i915_reset_request(req))
continue;
- }

if (signal_pending_state(state, current)) {
timeout = -ERESTARTSYS;
@@ -1142,12 +1149,8 @@ long i915_wait_request(struct drm_i915_gem_request *req,
* to come along and update the breadcrumb (either directly
* itself, or indirectly by recovering the GPU).
*/
- if (flags & I915_WAIT_LOCKED &&
- i915_reset_in_progress(&req->i915->gpu_error)) {
- __set_current_state(TASK_RUNNING);
- i915_reset(req->i915);
+ if (flags & I915_WAIT_LOCKED && __i915_reset_request(req))
continue;
- }

/* Only spin if we know the GPU is processing this request */
if (i915_spin_request(req, state, 2))
--
2.11.0