Commit aba73826 authored by Chris Wilson's avatar Chris Wilson
Browse files

drm/i915/gt: Do not suspend bonded requests if one hangs 



Treat the dependency between bonded requests as weak and leave the
remainder of the pair on the GPU if one hangs.

Signed-off-by: default avatarChris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: default avatarMika Kuoppala <mika.kuoppala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20210120121718.26435-1-chris@chris-wilson.co.uk
parent 14ca83ee

drivers/gpu/drm/i915/gt/intel_execlists_submission.c

+6 −0
Original line number Original line Diff line number Diff line
@@ -1990,6 +1990,9 @@ static void __execlists_hold(struct i915_request *rq) 
			struct i915_request *w =

			struct i915_request *w =

				container_of(p->waiter, typeof(*w), sched);

				container_of(p->waiter, typeof(*w), sched);





			if (p->flags & I915_DEPENDENCY_WEAK)

				continue;



			/* Leave semaphores spinning on the other engines */

			/* Leave semaphores spinning on the other engines */

			if (w->engine != rq->engine)

			if (w->engine != rq->engine)

				continue;

				continue;
@@ -2088,6 +2091,9 @@ static void __execlists_unhold(struct i915_request *rq) 
			struct i915_request *w =

			struct i915_request *w =

				container_of(p->waiter, typeof(*w), sched);

				container_of(p->waiter, typeof(*w), sched);





			if (p->flags & I915_DEPENDENCY_WEAK)

				continue;



			/* Propagate any change in error status */

			/* Propagate any change in error status */

			if (rq->fence.error)

			if (rq->fence.error)

				i915_request_set_error_once(w, rq->fence.error);

				i915_request_set_error_once(w, rq->fence.error);