summaryrefslogtreecommitdiffstats
path: root/drivers/gpu/drm/i915/i915_gem_request.c
diff options
context:
space:
mode:
authorChris Wilson2017-02-23 08:44:22 +0100
committerChris Wilson2017-02-23 15:56:36 +0100
commit0f2f61d4a8d4611155e7d1633db271ead9057981 (patch)
tree8520c06cb4c63f8c1bb1252c8cca4cf31987a2bc /drivers/gpu/drm/i915/i915_gem_request.c
parentdrm/i915: Immediately process a reset before starting waiting (diff)
downloadkernel-qcow2-linux-0f2f61d4a8d4611155e7d1633db271ead9057981.tar.gz
kernel-qcow2-linux-0f2f61d4a8d4611155e7d1633db271ead9057981.tar.xz
kernel-qcow2-linux-0f2f61d4a8d4611155e7d1633db271ead9057981.zip
drm/i915: Remove one level of indention from wait-for-execute
Now that the code is getting simpler, we can reduce the indentation when waiting for the global_seqno. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com> Link: http://patchwork.freedesktop.org/patch/msgid/20170223074422.4125-17-chris@chris-wilson.co.uk
Diffstat (limited to 'drivers/gpu/drm/i915/i915_gem_request.c')
-rw-r--r--drivers/gpu/drm/i915/i915_gem_request.c39
1 files changed, 18 insertions, 21 deletions
diff --git a/drivers/gpu/drm/i915/i915_gem_request.c b/drivers/gpu/drm/i915/i915_gem_request.c
index 9669747d1822..fbfeb5f8d069 100644
--- a/drivers/gpu/drm/i915/i915_gem_request.c
+++ b/drivers/gpu/drm/i915/i915_gem_request.c
@@ -1086,32 +1086,29 @@ long i915_wait_request(struct drm_i915_gem_request *req,
intel_wait_init(&wait);
restart:
- if (!intel_wait_update_request(&wait, req)) {
- do {
- set_current_state(state);
-
- if (intel_wait_update_request(&wait, req))
- break;
+ do {
+ set_current_state(state);
+ if (intel_wait_update_request(&wait, req))
+ break;
- if (flags & I915_WAIT_LOCKED &&
- __i915_wait_request_check_and_reset(req))
- continue;
+ if (flags & I915_WAIT_LOCKED &&
+ __i915_wait_request_check_and_reset(req))
+ continue;
- if (signal_pending_state(state, current)) {
- timeout = -ERESTARTSYS;
- goto complete;
- }
+ if (signal_pending_state(state, current)) {
+ timeout = -ERESTARTSYS;
+ goto complete;
+ }
- if (!timeout) {
- timeout = -ETIME;
- goto complete;
- }
+ if (!timeout) {
+ timeout = -ETIME;
+ goto complete;
+ }
- timeout = io_schedule_timeout(timeout);
- } while (1);
+ timeout = io_schedule_timeout(timeout);
+ } while (1);
- GEM_BUG_ON(!intel_wait_has_seqno(&wait));
- }
+ GEM_BUG_ON(!intel_wait_has_seqno(&wait));
GEM_BUG_ON(!i915_sw_fence_signaled(&req->submit));
/* Optimistic short spin before touching IRQs */